Reorganize wxDataViewMainWindow::ItemAdded() for better readability.

Reuse common code. No real changes.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@68792 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/src/generic/datavgen.cpp b/src/generic/datavgen.cpp
index d2f8fe01bf5269793f54fe3e3d9a08925212f74f..233b83f5b38d6ab28abaf4b3a05c0f8ee21932b7 100644 (file)
--- a/src/generic/datavgen.cpp
+++ b/src/generic/datavgen.cpp
@@ -1976,33 +1976,34 @@ bool wxDataViewMainWindow::ItemAdded(const wxDataViewItem & parent, const wxData
         wxDataViewVirtualListModel *list_model =
             (wxDataViewVirtualListModel*) GetOwner()->GetModel();
         m_count = list_model->GetCount();
         wxDataViewVirtualListModel *list_model =
             (wxDataViewVirtualListModel*) GetOwner()->GetModel();
         m_count = list_model->GetCount();

-        UpdateDisplay();
-        return true;

     }
     }

+    else
+    {
+        SortPrepare();

 
 

-    SortPrepare();
+        wxDataViewTreeNode * node;
+        node = FindNode(parent);
+
+        if( node == NULL )
+            return false;

 
 

-    wxDataViewTreeNode * node;
-    node = FindNode(parent);
+        node->SetHasChildren( true );

 
 

-    if( node == NULL )
-        return false;
+        if( g_model->IsContainer( item ) )
+        {
+            wxDataViewTreeNode * newnode = new wxDataViewTreeNode( node );
+            newnode->SetItem(item);
+            newnode->SetHasChildren( true );
+            node->AddNode( newnode);
+        }
+        else
+            node->AddLeaf( item.GetID() );

 
 

-    node->SetHasChildren( true );
+        node->ChangeSubTreeCount(1);

 
 

-    if( g_model->IsContainer( item ) )
-    {
-        wxDataViewTreeNode * newnode = new wxDataViewTreeNode( node );
-        newnode->SetItem(item);
-        newnode->SetHasChildren( true );
-        node->AddNode( newnode);
+        m_count = -1;

     }
     }

-    else
-        node->AddLeaf( item.GetID() );

 
 

-    node->ChangeSubTreeCount(1);
-
-    m_count = -1;

     UpdateDisplay();
 
     return true;
     UpdateDisplay();
 
     return true;