comparison document/SVFileReader.cpp @ 173:102317ae3970

* Better abbreviation modes for layer names in pane (and input model combo
  of plugin parameter dialog)
* Avoid crash when loading SV file containing model derived from nonexistent
  model (shouldn't happen of course, but see bug #1771769)
* Remember last-used input model in plugin parameter dialog
* Don't override a layer colour loaded from a session file with the generated
  default colour when attaching it to a view

 	    
 	    bool foundInAwaiting = false;
 
 	    for (std::map<int, int>::iterator i = m_awaitingDatasets.begin();
 		 i != m_awaitingDatasets.end(); ++i) {
-		if (m_models[i->second] == m_currentDataset) {
+		if (haveModel(i->second) &&
+                    m_models[i->second] == m_currentDataset) {
 		    m_awaitingDatasets.erase(i);
 		    foundInAwaiting = true;
 		    break;
 		}
 	    }

 
         if (!m_currentDerivedModel) {
             if (m_currentDerivedModel < 0) {
                 std::cerr << "WARNING: SV-XML: Bad derivation output model id "
                           << m_currentDerivedModelId << std::endl;
-            } else if (m_models[m_currentDerivedModelId]) {
+            } else if (haveModel(m_currentDerivedModelId)) {
                 std::cerr << "WARNING: SV-XML: Derivation has existing model "
                           << m_currentDerivedModelId
                           << " as target, not regenerating" << std::endl;
             } else {
                 m_currentDerivedModel = m_models[m_currentDerivedModelId] =

 {
     bool ok = false;
 
     READ_MANDATORY(int, id, toInt);
 
-    if (m_models.find(id) != m_models.end()) {
+    if (haveModel(id)) {
 	std::cerr << "WARNING: SV-XML: Ignoring duplicate model id " << id
 		  << std::endl;
 	return false;
     }
 

 	int modelId;
 	bool modelOk = false;
 	modelId = attributes.value("model").trimmed().toInt(&modelOk);
 
 	if (modelOk) {
-	    if (m_models.find(modelId) != m_models.end()) {
+	    if (haveModel(modelId)) {
 		Model *model = m_models[modelId];
 		m_document->setModel(layer, model);
 	    } else {
 		std::cerr << "WARNING: SV-XML: Unknown model id " << modelId
 			  << " in layer definition" << std::endl;

     }
     
     int modelId = m_awaitingDatasets[id];
     
     Model *model = 0;
-    if (m_models.find(modelId) != m_models.end()) {
+    if (haveModel(modelId)) {
 	model = m_models[modelId];
     } else {
 	std::cerr << "WARNING: SV-XML: Internal error: Unknown model " << modelId
 		  << " expecting dataset " << id << std::endl;
 	return false;

 	return false;
     }
 
     QString transform = attributes.value("transform");
 
-    if (m_models.find(modelId) != m_models.end()) {
+    if (haveModel(modelId)) {
         m_currentDerivedModel = m_models[modelId];
     } else {
         // we'll regenerate the model when the derivation element ends
         m_currentDerivedModel = 0;
     }

     
     int sourceId = 0;
     bool sourceOk = false;
     sourceId = attributes.value("source").trimmed().toInt(&sourceOk);
 
-    if (sourceOk && m_models[sourceId]) {
+    if (sourceOk && haveModel(sourceId)) {
         m_currentTransformSource = m_models[sourceId];
     } else {
         m_currentTransformSource = m_document->getMainModel();
     }
 

     if (!modelOk) {
 	std::cerr << "WARNING: SV-XML: No model id specified for play parameters" << std::endl;
 	return false;
     }
 
-    if (m_models.find(modelId) != m_models.end()) {
+    if (haveModel(modelId)) {
 
         bool ok = false;
 
         PlayParameters *parameters = PlayParameterRepository::getInstance()->
             getPlayParameters(m_models[modelId]);