X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/0dd48109b4d2983fe600b3faf260b31cb82353c4..8b93348e82c0906364ef2e020fd1adaaa49f519f:/src/common/dobjcmn.cpp

diff --git a/src/common/dobjcmn.cpp b/src/common/dobjcmn.cpp
index 4929160986..20d6b2999c 100644
--- a/src/common/dobjcmn.cpp
+++ b/src/common/dobjcmn.cpp
@@ -94,29 +94,48 @@ wxDataObjectComposite::~wxDataObjectComposite()
 }
 
 wxDataObjectSimple *
-wxDataObjectComposite::GetObject(const wxDataFormat& format) const
+wxDataObjectComposite::GetObject(const wxDataFormat& format, wxDataObjectBase::Direction dir) const
 {
     wxSimpleDataObjectList::compatibility_iterator node = m_dataObjects.GetFirst();
+
     while ( node )
     {
         wxDataObjectSimple *dataObj = node->GetData();
 
-        if ( dataObj->GetFormat() == format )
-        {
-            return dataObj;
-        }
-
+        if (dataObj->IsSupported(format,dir))
+          return dataObj;
         node = node->GetNext();
     }
-
     return NULL;
 }
 
 void wxDataObjectComposite::Add(wxDataObjectSimple *dataObject, bool preferred)
 {
+   // check if the data format of the passed object already exists in the composite data object, if this is the case
+   // do not add the data object and display a message in debug mode (otherwise this method fails silently):
+   // start checking if the data format exists for the 'GET' direction:
+    size_t indexFormats;
+    size_t noOfFormats;
+    wxDataFormat* formats;
+    
+    noOfFormats = dataObject->GetFormatCount(wxDataObjectBase::Get);
+    formats = new wxDataFormat[noOfFormats];
+    for (indexFormats=0; indexFormats<noOfFormats; ++indexFormats)
+      wxCHECK_RET(this->GetObject(formats[indexFormats],wxDataObjectBase::Get) == NULL,
+                  _("The data format for the GET-direction of the to be added data object already exists"));
+    delete[] formats;
+   // do the same with the 'SET' direction:
+    noOfFormats = dataObject->GetFormatCount(wxDataObjectBase::Set);
+    
+    formats = new wxDataFormat[noOfFormats];
+    for (indexFormats=0; indexFormats<noOfFormats; ++indexFormats)
+      wxCHECK_RET(this->GetObject(formats[indexFormats],wxDataObjectBase::Set) == NULL,
+                  _("The data format for the SET-direction of the to be added data object already exists"));
+    delete[] formats;
+
+   // if we reach this location the data object can simply be appended:
     if ( preferred )
         m_preferred = m_dataObjects.GetCount();
-
     m_dataObjects.Append( dataObject );
 }