comparison main/MainWindow.cpp @ 194:c851c49c79fe

* Make drop behave more sensibly (replace current pane)

         return;
     }
 
     Model *prevPlaybackModel = m_viewManager->getPlaybackModel();
 
-    std::set<Model *> soloModels;
-
-    for (int i = 0; i < p->getLayerCount(); ++i) {
-        Layer *layer = p->getLayer(i);
-        if (dynamic_cast<TimeRulerLayer *>(layer)) {
-            continue;
-        }
-        if (layer && layer->getModel()) {
-            Model *model = layer->getModel();
-            if (dynamic_cast<RangeSummarisableTimeValueModel *>(model)) {
-                m_viewManager->setPlaybackModel(model);
-            }
-            soloModels.insert(model);
+    View::ModelSet soloModels = p->getModels();
+    
+    for (View::ModelSet::iterator mi = soloModels.begin();
+         mi != soloModels.end(); ++mi) {
+        if (dynamic_cast<RangeSummarisableTimeValueModel *>(*mi)) {
+            m_viewManager->setPlaybackModel(*mi);
         }
     }
     
     RangeSummarisableTimeValueModel *a = 
         dynamic_cast<RangeSummarisableTimeValueModel *>(prevPlaybackModel);

 	delete newModel;
         m_openingAudioFile = false;
 	return FileOpenFailed;
     }
 
-    bool setAsMain = true;
-    static bool prevSetAsMain = true;
-
     bool realFile = (location == path);
 
-    if (mode == CreateAdditionalModel) setAsMain = false;
-    else if (mode == AskUser) {
-        if (m_document->getMainModel()) {
-
+    std::cerr << "mode = " << mode << std::endl;
+
+    if (mode == AskUser) {
+        if (getMainModel()) {
+
+            std::cerr << "ask user, have main model" << std::endl;
+
+            static bool prevSetAsMain = true;
+            bool setAsMain = true;
+            
             QStringList items;
             items << tr("Replace the existing main waveform")
                   << tr("Load this file into a new waveform pane");
 
             bool ok = false;

                 return FileOpenCancelled;
             }
             
             setAsMain = (item == items[0]);
             prevSetAsMain = setAsMain;
-        }
-    }
-
-    if (setAsMain) {
+
+            if (setAsMain) mode = ReplaceMainModel;
+            else mode = CreateAdditionalModel;
+
+        } else {
+            mode = ReplaceMainModel;
+        }
+    }
+
+    if (mode == ReplaceCurrentPane) {
+
+        std::cerr << "replace current pane" << std::endl;
+
+        Pane *pane = m_paneStack->getCurrentPane();
+        if (pane) {
+            std::cerr << "have pane" << std::endl;
+
+            if (getMainModel()) {
+                std::cerr << "have main model" << std::endl;
+
+                View::ModelSet models(pane->getModels());
+                if (models.find(getMainModel()) != models.end()) {
+                    std::cerr << "main model is in pane, setting to ReplaceMainModel" << std::endl;
+                    mode = ReplaceMainModel;
+                }
+            } else {
+                std::cerr << "no main model, setting to ReplaceMainModel" << std::endl;
+                mode = ReplaceMainModel;
+            }
+        } else {
+            std::cerr << "no pane, setting to CreateAdditionalModel" << std::endl;
+            mode = CreateAdditionalModel;
+        }
+    }
+
+    if (mode == CreateAdditionalModel && !getMainModel()) {
+        std::cerr << "mode is CreateAdditionalModel and no main model, setting to ReplaceMainModel" << std::endl;
+        mode = ReplaceMainModel;
+    }
+
+    std::cerr << "mode now " << mode << std::endl;
+
+    if (mode == ReplaceMainModel) {
 
         Model *prevMain = getMainModel();
         if (prevMain) {
             m_playSource->removeModel(prevMain);
             PlayParameterRepository::getInstance()->removeModel(prevMain);
         }
-
         PlayParameterRepository::getInstance()->addModel(newModel);
 
 	m_document->setMainModel(newModel);
 	setupMenus();
 

 	    }
 	}
 
         if (realFile) m_audioFile = path;
 
-    } else { // !setAsMain
+    } else if (mode == CreateAdditionalModel) {
 
 	CommandHistory::getInstance()->startCompoundOperation
 	    (tr("Import \"%1\"").arg(QFileInfo(location).fileName()), true);
 
 	m_document->addImportedModel(newModel);

 	    m_timeRulerLayer = m_document->createMainModelLayer
 		(LayerFactory::TimeRuler);
 	}
 
 	m_document->addLayerToView(pane, m_timeRulerLayer);
+
+	Layer *newLayer = m_document->createImportedLayer(newModel);
+
+	if (newLayer) {
+	    m_document->addLayerToView(pane, newLayer);
+	}
+	
+	CommandHistory::getInstance()->endCompoundOperation();
+
+    } else if (mode == ReplaceCurrentPane) {
+
+        // We know there is a current pane, otherwise we would have
+        // reset the mode to CreateAdditionalModel above; and we know
+        // the current pane does not contain the main model, otherwise
+        // we would have reset it to ReplaceMainModel.  But we don't
+        // know whether the pane contains a waveform model at all.
+        
+        Pane *pane = m_paneStack->getCurrentPane();
+        Layer *replace = 0;
+
+        for (int i = 0; i < pane->getLayerCount(); ++i) {
+            Layer *layer = pane->getLayer(i);
+            if (dynamic_cast<WaveformLayer *>(layer)) {
+                replace = layer;
+                break;
+            }
+        }
+
+	CommandHistory::getInstance()->startCompoundOperation
+	    (tr("Import \"%1\"").arg(QFileInfo(location).fileName()), true);
+
+	m_document->addImportedModel(newModel);
+
+        if (replace) {
+            m_document->removeLayerFromView(pane, replace);
+        }
 
 	Layer *newLayer = m_document->createImportedLayer(newModel);
 
 	if (newLayer) {
 	    m_document->addLayerToView(pane, newLayer);

 {
     if (pane) m_paneStack->setCurrentPane(pane);
 
     for (QStringList::iterator i = uriList.begin(); i != uriList.end(); ++i) {
 
-        FileOpenStatus status =
-            openURL(*i, (m_document->getMainModel() ?
-                         CreateAdditionalModel : ReplaceMainModel));
+        FileOpenStatus status = openURL(*i, ReplaceCurrentPane);
 
         if (status == FileOpenFailed) {
             QMessageBox::critical(this, tr("Failed to open dropped URL"),
                                   tr("<b>Open failed</b><p>Dropped URL \"%1\" could not be opened").arg(*i));
         } else if (status == FileOpenWrongMode) {