comparison framework/Document.cpp @ 676:0d4236961c97

Ensure that AggregateModels are actually deleted on release despite not being in model list (ugh, this is ugly)

     if (model == m_mainModel) {
         return;
     }
 
     bool toDelete = false;
+    bool isInModelList = false; // should become true for any "normal" model
 
     ModelList::iterator mitr = findModelInList(model);
-    
+
     if (mitr != m_models.end()) {
+
         if (mitr->refcount == 0) {
             SVCERR << "WARNING: Document::releaseModel: model " << model
                    << " reference count is zero already!" << endl;
         } else {
 #ifdef DEBUG_DOCUMENT

 #endif
             if (--mitr->refcount == 0) {
                 toDelete = true;
             }
         }
+        isInModelList = true;
+
     } else if (m_aggregateModels.find(model) != m_aggregateModels.end()) {
 #ifdef DEBUG_DOCUMENT
         SVDEBUG << "Document::releaseModel: is an aggregate model" << endl;
 #endif
         toDelete = true;

                     << model << " even though it is source for "
                     << sourceCount << " other derived model(s) -- resetting "
                     << "their source fields appropriately" << endl;
         }
 
-        deleteModelFromList(model);
-
-#ifdef DEBUG_DOCUMENT
-        SVDEBUG << "Document::releaseModel: Deleted model " << model << endl;
-        SVDEBUG << "Models now: ";
-        for (const auto &r: m_models) {
-            SVDEBUG << r.model << " ";
-        } 
-        SVDEBUG << endl;
-#endif
+        if (isInModelList) {
+            deleteModelFromList(model);
+
+#ifdef DEBUG_DOCUMENT
+            SVDEBUG << "Document::releaseModel: Deleted model " << model << endl;
+            SVDEBUG << "Models now: ";
+            for (const auto &r: m_models) {
+                SVDEBUG << r.model << " ";
+            } 
+            SVDEBUG << endl;
+#endif
+        } else {
+            model->aboutToDelete();
+            emit modelAboutToBeDeleted(model);
+            delete model;
+
+#ifdef DEBUG_DOCUMENT
+            SVDEBUG << "Document::releaseModel: Deleted awkward model " << model << endl;
+#endif
+        }
     }
 }
 
 void
 Document::deleteLayer(Layer *layer, bool force)