Added the comparison code between new and old value at setters on TableView

Change-Id: I3403e36af52e78eda1524e868b1d174b2876ea6c
Signed-off-by: youseong.ji <youseong.ji@samsung.com>

diff --git a/src/ui/controls/FUiCtrl_TableView.cpp b/src/ui/controls/FUiCtrl_TableView.cpp
index ab68dbd..1d7b2bc 100644 (file)
--- a/src/ui/controls/FUiCtrl_TableView.cpp
+++ b/src/ui/controls/FUiCtrl_TableView.cpp
@@ -670,6 +670,11 @@ _TableView::GetTableViewStyle(void) const
 void
 _TableView::SetTableViewStyle(TableViewStyle style)
 {
+       if(__tableViewStyle == style)
+       {
+               return;
+       }
+
        __tableViewStyle = style;
 
        if (__tableViewStyle == TABLE_VIEW_STYLE_SECTION)
@@ -832,6 +837,11 @@ _TableView::GetItemDividerColor(void) const
 result
 _TableView::SetPropertyItemDividerColor(const Variant& itemDividerColor)
 {
+       if(__itemDividerColor == itemDividerColor.ToColor())
+       {
+               return E_SUCCESS;
+       }
+
        __itemDividerColor = itemDividerColor.ToColor();
 
        return __pTableViewPresenter->SetStatusChanged(true);

diff --git a/src/ui/controls/FUiCtrl_TableViewItem.cpp b/src/ui/controls/FUiCtrl_TableViewItem.cpp
index 8703f5c..80e64f4 100644 (file)
--- a/src/ui/controls/FUiCtrl_TableViewItem.cpp
+++ b/src/ui/controls/FUiCtrl_TableViewItem.cpp
@@ -859,6 +859,11 @@ _TableViewItem::GetItemBackgroundBitmap(ListItemState itemState) const
 void
 _TableViewItem::SetItemBackgroundColor(ListItemState itemState, const Color bgColor)
 {
+       if(__colorItemBg[itemState] == bgColor)
+       {
+               return;
+       }
+
        __colorItemBg[itemState] = bgColor;
 
        SetItemChanged(true);
@@ -910,6 +915,11 @@ _TableViewItem::SetItemHeight(int itemHeight)
 void
 _TableViewItem::SetItemHeight(float itemHeight)
 {
+       if(_FloatCompare(GetSizeF().height, itemHeight))
+       {
+               return;
+       }
+
        SetSize(FloatDimension(GetSizeF().width, itemHeight));
 }
 
@@ -935,6 +945,11 @@ _TableViewItem::IsChecked(void) const
 void
 _TableViewItem::SetCheckedAnimationEnabled(bool enabled)
 {
+       if(__isCheckedAnimationEnabled == enabled)
+       {
+               return;
+       }
+
        __isCheckedAnimationEnabled = enabled;
 
        if (__isCheckedAnimationEnabled == false)
@@ -971,6 +986,11 @@ _TableViewItem::IsItemEnabled(void) const
 void
 _TableViewItem::SetItemType(const TableViewItemType itemType)
 {
+       if(__itemType == itemType)
+       {
+               return;
+       }
+
        __itemType = itemType;
 
        if (__pContextItem != null)
@@ -1996,18 +2016,15 @@ _TableViewItem::SetDrawingProperty(_ItemDrawingProperty* pDrawingProperty)
 
        __pDrawingProperty = pDrawingProperty;
 
-       if(__pDrawingProperty != null)
+       if (__pDrawingProperty->sectionStyleEnabled == true &&
+               !IsContextItem())
        {
-               if (__pDrawingProperty->sectionStyleEnabled == true &&
-                       !IsContextItem())
-               {
-                       CreateItemCover();
-               }
+               CreateItemCover();
+       }
 
-               if(__pDrawingProperty->itemDividerEnabled)
-               {
-                       CreateItemDivider();
-               }
+       if(__pDrawingProperty->itemDividerEnabled)
+       {
+               CreateItemDivider();
        }
 
        CreateAnnexStyle();
@@ -3589,6 +3606,11 @@ _TableViewItem::GetSimpleItemTextColor(TableViewItemDrawingStatus status) const
 result
 _TableViewItem::SetSimpleItemTextSize(int size)
 {
+       if(__simpleItemTextSize == size)
+       {
+               return E_SUCCESS;
+       }
+
        __simpleItemTextSize = size;
 
        if (__pSimpleItemText != null)

diff --git a/src/ui/controls/FUiCtrl_TableViewItemProviderAdaptor.cpp b/src/ui/controls/FUiCtrl_TableViewItemProviderAdaptor.cpp
index 28b0c2a..6223711 100644 (file)
--- a/src/ui/controls/FUiCtrl_TableViewItemProviderAdaptor.cpp
+++ b/src/ui/controls/FUiCtrl_TableViewItemProviderAdaptor.cpp
@@ -69,6 +69,12 @@ _TableViewItemProviderAdaptor::SetItemProvider(_TableViewItemProvider* pProvider
        __pTableViewProvider = pProvider;
 }
 
+_TableViewItemProvider*
+_TableViewItemProviderAdaptor::GetItemProvider() const
+{
+       return __pTableViewProvider;
+}
+
 bool
 _TableViewItemProviderAdaptor::IsItemProvider() const
 {

diff --git a/src/ui/controls/FUiCtrl_TableViewPresenter.cpp b/src/ui/controls/FUiCtrl_TableViewPresenter.cpp
index 5ee77e6..4a32a4d 100644 (file)
--- a/src/ui/controls/FUiCtrl_TableViewPresenter.cpp
+++ b/src/ui/controls/FUiCtrl_TableViewPresenter.cpp
@@ -209,8 +209,15 @@ _TableViewPresenter::SetItemProvider(const _TableViewItemProvider* pProvider)
 {
        result r = E_SUCCESS;
 
-       _TableViewItemProviderAdaptor* pProviderAdaptor = null;
-       pProviderAdaptor = static_cast <_TableViewItemProviderAdaptor*>(__pListModel->GetItemProviderAdaptor());
+       _TableViewItemProviderAdaptor* pProviderAdaptor = static_cast <_TableViewItemProviderAdaptor*>(__pListModel->GetItemProviderAdaptor());
+       if(pProviderAdaptor != null)
+       {
+               _TableViewItemProvider* provider = pProviderAdaptor->GetItemProvider();
+               if(provider != null && provider == pProvider)
+               {
+                       return r;
+               }
+       }
 
        if (pProviderAdaptor == null)
        {
@@ -1207,13 +1214,11 @@ _TableViewPresenter::SetItemChecked(int groupIndex, int itemIndex, bool checked)
                return E_INVALID_STATE;
        }
 
-       TableViewItemTag itemPos = {groupIndex, itemIndex};
-
-       bool enabled = __pListModel->IsItemEnabled(itemPos.groupIndex, itemPos.itemIndex);
+       bool enabled = __pListModel->IsItemEnabled(groupIndex, itemIndex);
        SysTryReturn(NID_UI_CTRL, (enabled == true), E_INVALID_OPERATION, E_INVALID_OPERATION,
                        "[E_INVALID_OPERATION] The item is disabled.");
 
-       if (__pListModel->IsItemChecked(itemPos.groupIndex, itemPos.itemIndex) == checked)
+       if (__pListModel->IsItemChecked(groupIndex, itemIndex) == checked)
        {
                return E_SUCCESS;
        }
@@ -1225,7 +1230,7 @@ _TableViewPresenter::SetItemChecked(int groupIndex, int itemIndex, bool checked)
                pItem->SetCheckedAnimationEnabled(checked);
        }
 
-       result r = __pListModel->SetItemChecked(itemPos.groupIndex, itemPos.itemIndex, checked);
+       result r = __pListModel->SetItemChecked(groupIndex, itemIndex, checked);
 
        return r;
 }
@@ -1249,14 +1254,12 @@ _TableViewPresenter::SetItemEnabled(int groupIndex, int itemIndex, bool enabled)
                return E_OUT_OF_RANGE;
        }
 
-       TableViewItemTag itemPos = {groupIndex, itemIndex};
-
-       if (__pListModel->IsItemEnabled(itemPos.groupIndex, itemPos.itemIndex) == enabled)
+       if (__pListModel->IsItemEnabled(groupIndex, itemIndex) == enabled)
        {
                return E_SUCCESS;
        }
 
-       result r = __pListModel->SetItemEnabled(itemPos.groupIndex, itemPos.itemIndex, enabled);
+       result r = __pListModel->SetItemEnabled(groupIndex, itemIndex, enabled);
 
        return r;
 }
@@ -3975,6 +3978,11 @@ _TableViewPresenter::SetReorderMode(bool enabled)
 
                if (pItem != null)
                {
+                       if(pItem->IsReorderMode() == enabled)
+                       {
+                               break;
+                       }
+
                        pItem->SetReorderMode(enabled);
 
                        if (enabled && itemPos.itemIndex != -1)

diff --git a/src/ui/inc/FUiCtrl_TableViewItemProviderAdaptor.h b/src/ui/inc/FUiCtrl_TableViewItemProviderAdaptor.h
index 766a6ec..47a18ba 100644 (file)
--- a/src/ui/inc/FUiCtrl_TableViewItemProviderAdaptor.h
+++ b/src/ui/inc/FUiCtrl_TableViewItemProviderAdaptor.h
@@ -50,6 +50,7 @@ public:
 
 // Operation
        void SetItemProvider(_TableViewItemProvider* pProvider);
+       _TableViewItemProvider* GetItemProvider() const;
        virtual bool IsItemProvider() const;
        virtual int GetGroupCount() const;
        virtual int GetItemCount(int groupIndex = 0) const;