]> git.saurik.com Git - wxWidgets.git/blobdiff - src/common/datavcmn.cpp
don't crash when loading images with verbose==false (patch 1449823)
[wxWidgets.git] / src / common / datavcmn.cpp
index 2168ba3072b5a510b95a18970cb49294612d606a..56966186a5af2244dfbf1db9b700fbfe13132695 100644 (file)
     #pragma hdrstop
 #endif
 
-#include "wx/defs.h"
-
 #if wxUSE_DATAVIEWCTRL
 
-#include "wx/object.h"
-#include "wx/dataview.h"
-#include "wx/log.h"
+#ifndef WX_PRECOMP
+    #include "wx/object.h"
+    #include "wx/log.h"
+#endif
+
 #include "wx/image.h"
+#include "wx/dataview.h"
 
-// --------------------------------------------------------- 
+// ---------------------------------------------------------
 // wxDataViewModel
-// --------------------------------------------------------- 
+// ---------------------------------------------------------
 
 IMPLEMENT_ABSTRACT_CLASS(wxDataViewModel, wxObject)
 
-// --------------------------------------------------------- 
+// ---------------------------------------------------------
 // wxDataViewListModel
-// --------------------------------------------------------- 
-
-class wxDataViewViewingColumn: public wxObject
-{
-public:
-    wxDataViewViewingColumn( wxDataViewColumn *view_column, size_t model_column )
-    {
-        m_viewColumn = view_column;
-        m_modelColumn = model_column;
-    }
-    
-    wxDataViewColumn   *m_viewColumn;
-    size_t              m_modelColumn;
-};
-
+// ---------------------------------------------------------
 
 IMPLEMENT_ABSTRACT_CLASS(wxDataViewListModel, wxDataViewModel)
 
 wxDataViewListModel::wxDataViewListModel()
 {
-    m_notifier = NULL;
     m_viewingColumns.DeleteContents( true );
+    m_notifiers.DeleteContents( true );
 }
 
 wxDataViewListModel::~wxDataViewListModel()
 {
-    if (m_notifier)
-        delete m_notifier;
 }
 
 bool wxDataViewListModel::RowAppended()
 {
-    if (m_notifier)
-        return m_notifier->RowAppended();
-        
-    return false;
+    bool ret = true;
+
+    wxNode *node = m_notifiers.GetFirst();
+    while (node)
+    {
+        wxDataViewListModelNotifier* notifier = (wxDataViewListModelNotifier*) node->GetData();
+        if (!notifier->RowAppended())
+            ret = false;
+        node = node->GetNext();
+    }
+
+    return ret;
 }
 
 bool wxDataViewListModel::RowPrepended()
 {
-    if (m_notifier)
-        return m_notifier->RowPrepended();
-        
-    return false;
+    bool ret = true;
+
+    wxNode *node = m_notifiers.GetFirst();
+    while (node)
+    {
+        wxDataViewListModelNotifier* notifier = (wxDataViewListModelNotifier*) node->GetData();
+        if (!notifier->RowPrepended())
+            ret = false;
+        node = node->GetNext();
+    }
+
+    return ret;
 }
 
 bool wxDataViewListModel::RowInserted( size_t before )
 {
-    if (m_notifier)
-        return m_notifier->RowInserted( before );
-        
-    return false;
+    bool ret = true;
+
+    wxNode *node = m_notifiers.GetFirst();
+    while (node)
+    {
+        wxDataViewListModelNotifier* notifier = (wxDataViewListModelNotifier*) node->GetData();
+        if (!notifier->RowInserted(before))
+            ret = false;
+        node = node->GetNext();
+    }
+
+    return ret;
 }
 
 bool wxDataViewListModel::RowDeleted( size_t row )
 {
-    if (m_notifier)
-        return m_notifier->RowDeleted( row );
-        
-    return false;
+    bool ret = true;
+
+    wxNode *node = m_notifiers.GetFirst();
+    while (node)
+    {
+        wxDataViewListModelNotifier* notifier = (wxDataViewListModelNotifier*) node->GetData();
+        if (!notifier->RowDeleted( row ))
+            ret = false;
+        node = node->GetNext();
+    }
+
+    return ret;
 }
 
 bool wxDataViewListModel::RowChanged( size_t row )
 {
-    if (m_notifier)
-        return m_notifier->RowChanged( row );
-        
-    return false;
+    bool ret = true;
+
+    wxNode *node = m_notifiers.GetFirst();
+    while (node)
+    {
+        wxDataViewListModelNotifier* notifier = (wxDataViewListModelNotifier*) node->GetData();
+        if (!notifier->RowChanged( row ))
+            ret = false;
+        node = node->GetNext();
+    }
+
+    return ret;
 }
 
 bool wxDataViewListModel::ValueChanged( size_t col, size_t row )
 {
-    bool ret = false;
+    bool ret = true;
 
-    if (m_notifier)
-        ret = m_notifier->ValueChanged( col, row );
-        
-    wxNode *node = m_viewingColumns.GetFirst();
+    wxNode *node = m_notifiers.GetFirst();
     while (node)
     {
-        wxDataViewViewingColumn* tmp = (wxDataViewViewingColumn*) node->GetData();
-        
-        if (tmp->m_modelColumn == col)
-            m_notifier->ValueChanged( tmp->m_viewColumn, col, row );
-    
+        wxDataViewListModelNotifier* notifier = (wxDataViewListModelNotifier*) node->GetData();
+        if (!notifier->ValueChanged( col, row ))
+            ret = false;
         node = node->GetNext();
     }
-    
+
+    return ret;
+}
+
+bool wxDataViewListModel::RowsReordered( size_t *new_order )
+{
+    bool ret = true;
+
+    wxNode *node = m_notifiers.GetFirst();
+    while (node)
+    {
+        wxDataViewListModelNotifier* notifier = (wxDataViewListModelNotifier*) node->GetData();
+        if (!notifier->RowsReordered( new_order ))
+            ret = false;
+        node = node->GetNext();
+    }
+
     return ret;
 }
 
 bool wxDataViewListModel::Cleared()
 {
-    if (m_notifier)
-        return m_notifier->Cleared();
-        
-    return false;
+    bool ret = true;
+
+    wxNode *node = m_notifiers.GetFirst();
+    while (node)
+    {
+        wxDataViewListModelNotifier* notifier = (wxDataViewListModelNotifier*) node->GetData();
+        if (!notifier->Cleared())
+            ret = false;
+        node = node->GetNext();
+    }
+
+    return ret;
 }
 
 void wxDataViewListModel::AddViewingColumn( wxDataViewColumn *view_column, size_t model_column )
@@ -142,39 +186,61 @@ void wxDataViewListModel::RemoveViewingColumn( wxDataViewColumn *column )
     while (node)
     {
         wxDataViewViewingColumn* tmp = (wxDataViewViewingColumn*) node->GetData();
-        
+
         if (tmp->m_viewColumn == column)
         {
             m_viewingColumns.DeleteObject( tmp );
             return;
         }
-    
+
         node = node->GetNext();
     }
 }
 
-void wxDataViewListModel::SetNotifier( wxDataViewListModelNotifier *notifier )
+void wxDataViewListModel::AddNotifier( wxDataViewListModelNotifier *notifier )
 {
-    if (m_notifier)
-        delete m_notifier;
-        
-   m_notifier = notifier;
+    m_notifiers.Append( notifier );
+    notifier->SetOwner( this );
 }
 
-wxDataViewListModelNotifier* wxDataViewListModel::GetNotifier()
+void wxDataViewListModel::RemoveNotifier( wxDataViewListModelNotifier *notifier )
 {
-    return m_notifier;
+    m_notifiers.DeleteObject( notifier );
 }
 
-// --------------------------------------------------------- 
-// wxDataViewSortedListModel
-// --------------------------------------------------------- 
+// ---------------------------------------------------------
+// wxDataViewSortedListModelNotifier
+// ---------------------------------------------------------
+
+class wxDataViewSortedListModelNotifier: public wxDataViewListModelNotifier
+{
+public:
+    wxDataViewSortedListModelNotifier( wxDataViewSortedListModel *model )
+    { m_model = model; }
+
+    virtual bool RowAppended() { return true; }
+    virtual bool RowPrepended()  { return true; }
+    virtual bool RowInserted( size_t WXUNUSED(before) )  { return true; }
+    virtual bool RowDeleted( size_t WXUNUSED(row) ) { return true; }
+    virtual bool RowChanged( size_t WXUNUSED(row) ) { return true; }
+    virtual bool ValueChanged( size_t col, size_t row )
+         { return m_model->ChildValueChanged( col, row); }
+    virtual bool RowsReordered( size_t *WXUNUSED(new_order) ) { return true; }
+    virtual bool Cleared() { return true; }
+
+    wxDataViewSortedListModel *m_model;
+};
+
+// ---------------------------------------------------------
+// wxDataViewSortedListModel compare function
+// ---------------------------------------------------------
 
 int wxCALLBACK wxDataViewListModelSortedDefaultCompare
       (size_t row1, size_t row2, size_t col, wxDataViewListModel* model )
 {
-    wxVariant value1 = model->GetValue( col, row1 );
-    wxVariant value2 = model->GetValue( col, row2 );
+    wxVariant value1,value2;
+    model->GetValue( value1, col, row1 );
+    model->GetValue( value2, col, row2 );
     if (value1.GetType() == wxT("string"))
     {
         wxString str1 = value1.GetString();
@@ -216,6 +282,9 @@ int LINKAGEMODE wxDataViewIntermediateCmp( size_t row1, size_t row2 )
     return s_CmpFunc( row1, row2, s_CmpCol, s_CmpModel );
 }
 
+// ---------------------------------------------------------
+// wxDataViewSortedListModel
+// ---------------------------------------------------------
 
 IMPLEMENT_ABSTRACT_CLASS(wxDataViewSortedListModel, wxDataViewListModel)
 
@@ -226,15 +295,100 @@ wxDataViewSortedListModel::wxDataViewSortedListModel( wxDataViewListModel *child
     s_CmpCol = 0;
     s_CmpModel = child;
     s_CmpFunc = wxDataViewListModelSortedDefaultCompare;
+
+    m_notifierOnChild = new wxDataViewSortedListModelNotifier( this );
+    m_child->AddNotifier( m_notifierOnChild );
+
+    Resort();
 }
 
 wxDataViewSortedListModel::~wxDataViewSortedListModel()
 {
+    m_child->RemoveNotifier( m_notifierOnChild );
+}
+
+void wxDataViewSortedListModel::Resort()
+{
+    m_array.Clear();
+    size_t n = m_child->GetNumberOfRows();
+    size_t i;
+    for (i = 0; i < n; i++)
+        m_array.Add( i );
+}
+
+#if 0
+static void Dump( wxDataViewListModel *model, size_t col )
+{
+    size_t n = model->GetNumberOfRows();
+    size_t i;
+    for (i = 0; i < n; i++)
+    {
+        wxVariant variant;
+        model->GetValue( variant, col, i );
+        wxString tmp;
+        tmp = variant.GetString();
+        wxPrintf( wxT("%d: %s\n"), (int) i, tmp.c_str() );
+    }
+}
+#endif
+
+bool wxDataViewSortedListModel::ChildValueChanged( size_t col, size_t row )
+{
+    size_t i;
+    size_t len = m_array.GetCount();
+
+    // Remove and readd sorted. Find out at which
+    // position it was and where it ended.
+    size_t start_pos = 0,end_pos = 0;
+    for (i = 0; i < len; i++)
+        if (m_array[i] == row)
+        {
+            start_pos = i;
+            break;
+        }
+    m_array.RemoveAt( start_pos );
+    m_array.Add( row );
+
+    for (i = 0; i < len; i++)
+        if (m_array[i] == row)
+        {
+            end_pos = i;
+            break;
+        }
+
+    if (end_pos == start_pos)
+        return wxDataViewListModel::ValueChanged( col, start_pos );
+
+    // Create an array where order[old] -> new_pos, so that
+    // if nothing changed order[0] -> 0 etc.
+    size_t *order = new size_t[ len ];
+    // Fill up initial values.
+    for (i = 0; i < len; i++)
+        order[i] = i;
+
+    if (start_pos < end_pos)
+    {
+        for (i = start_pos; i < end_pos; i++)
+            order[i] = order[i+1];
+        order[end_pos] = start_pos;
+    }
+    else
+    {
+        for (i = end_pos; i > start_pos; i--)
+            order[i] = order[i-1];
+        order[start_pos] = end_pos;
+    }
+
+    wxDataViewListModel::RowsReordered( order );
+
+    delete [] order;
+
+    return true;
 }
 
 size_t wxDataViewSortedListModel::GetNumberOfRows()
 {
-    return m_child->GetNumberOfRows();
+    return m_array.GetCount();
 }
 
 size_t wxDataViewSortedListModel::GetNumberOfCols()
@@ -247,17 +401,19 @@ wxString wxDataViewSortedListModel::GetColType( size_t col )
     return m_child->GetColType( col );
 }
 
-wxVariant wxDataViewSortedListModel::GetValue( size_t col, size_t row )
+void wxDataViewSortedListModel::GetValue( wxVariant &variant, size_t col, size_t row )
 {
     size_t child_row = m_array[row];
-    return m_child->GetValue( col, child_row );
+    m_child->GetValue( variant, col, child_row );
 }
 
 bool wxDataViewSortedListModel::SetValue( wxVariant &variant, size_t col, size_t row )
 {
     size_t child_row = m_array[row];
     bool ret = m_child->SetValue( variant, col, child_row );
-    // resort in ::ValueChanged()
+
+    // Resort in ::ChildValueChanged() which gets reported back.
+
     return ret;
 }
 
@@ -265,9 +421,9 @@ bool wxDataViewSortedListModel::RowAppended()
 {
     // you can only append
     bool ret = m_child->RowAppended();
-    
+
     // report RowInsrted
-    
+
     return ret;
 }
 
@@ -275,30 +431,31 @@ bool wxDataViewSortedListModel::RowPrepended()
 {
     // you can only append
     bool ret = m_child->RowAppended();
-    
+
     // report RowInsrted
-    
+
     return ret;
 }
 
-bool wxDataViewSortedListModel::RowInserted( size_t before )
+bool wxDataViewSortedListModel::RowInserted( size_t WXUNUSED(before) )
 {
     // you can only append
     bool ret = m_child->RowAppended();
-    
+
     // report different RowInsrted
-    
+
     return ret;
 }
 
 bool wxDataViewSortedListModel::RowDeleted( size_t row )
 {
     size_t child_row = m_array[row];
-    
+
     bool ret = m_child->RowDeleted( child_row );
-    
-    wxDataViewListModel::RowDeleted( row );
-    
+
+    // Do nothing here as the change in the
+    // child model will be reported back.
+
     return ret;
 }
 
@@ -306,9 +463,10 @@ bool wxDataViewSortedListModel::RowChanged( size_t row )
 {
     size_t child_row = m_array[row];
     bool ret = m_child->RowChanged( child_row );
-    
-    // report delete old pos, inserted new pos
-    
+
+    // Do nothing here as the change in the
+    // child model will be reported back.
+
     return ret;
 }
 
@@ -316,35 +474,32 @@ bool wxDataViewSortedListModel::ValueChanged( size_t col, size_t row )
 {
     size_t child_row = m_array[row];
     bool ret = m_child->ValueChanged( col, child_row );
-    
-    // Do nothing if not the sorted col..
-    // report delete old pos, inserted new pos
-    
-    return ret;
-}
 
-bool wxDataViewSortedListModel::Cleared()
-{
-    bool ret = m_child->Cleared();
-    
-    wxDataViewListModel::Cleared();
-    
+    // Do nothing here as the change in the
+    // child model will be reported back.
+
     return ret;
 }
 
-void wxDataViewSortedListModel::SetNotifier( wxDataViewListModelNotifier *notifier )
+bool wxDataViewSortedListModel::RowsReordered( size_t *WXUNUSED(new_order) )
 {
-    wxDataViewListModel::SetNotifier( notifier );
+    // We sort them ourselves.
+
+    return false;
 }
 
-wxDataViewListModelNotifier* wxDataViewSortedListModel::GetNotifier()
+bool wxDataViewSortedListModel::Cleared()
 {
-    return wxDataViewListModel::GetNotifier();
+    bool ret = m_child->Cleared();
+
+    wxDataViewListModel::Cleared();
+
+    return ret;
 }
 
-// --------------------------------------------------------- 
+// ---------------------------------------------------------
 // wxDataViewCellBase
-// --------------------------------------------------------- 
+// ---------------------------------------------------------
 
 IMPLEMENT_ABSTRACT_CLASS(wxDataViewCellBase, wxObject)
 
@@ -353,10 +508,10 @@ wxDataViewCellBase::wxDataViewCellBase( const wxString &varianttype, wxDataViewC
     m_variantType = varianttype;
     m_mode = mode;
 }
-    
-// --------------------------------------------------------- 
+
+// ---------------------------------------------------------
 // wxDataViewColumnBase
-// --------------------------------------------------------- 
+// ---------------------------------------------------------
 
 IMPLEMENT_ABSTRACT_CLASS(wxDataViewColumnBase, wxObject)
 
@@ -374,7 +529,7 @@ wxDataViewColumnBase::~wxDataViewColumnBase()
 {
     if (m_cell)
         delete m_cell;
-        
+
     if (GetOwner())
     {
         GetOwner()->GetModel()->RemoveViewingColumn( (wxDataViewColumn*) this );
@@ -391,9 +546,9 @@ wxString wxDataViewColumnBase::GetTitle()
     return m_title;
 }
 
-// --------------------------------------------------------- 
+// ---------------------------------------------------------
 // wxDataViewCtrlBase
-// --------------------------------------------------------- 
+// ---------------------------------------------------------
 
 IMPLEMENT_ABSTRACT_CLASS(wxDataViewCtrlBase, wxControl)
 
@@ -405,17 +560,12 @@ wxDataViewCtrlBase::wxDataViewCtrlBase()
 
 wxDataViewCtrlBase::~wxDataViewCtrlBase()
 {
-    if (m_model)
-        delete m_model;
 }
 
 bool wxDataViewCtrlBase::AssociateModel( wxDataViewListModel *model )
 {
-    if (m_model)
-        delete m_model;
-        
     m_model = model;
-    
+
     return true;
 }
 
@@ -457,7 +607,7 @@ size_t wxDataViewCtrlBase::GetNumberOfColumns()
     return m_cols.GetCount();
 }
 
-bool wxDataViewCtrlBase::DeleteColumn( size_t pos )
+bool wxDataViewCtrlBase::DeleteColumn( size_t WXUNUSED(pos) )
 {
     return false;
 }