[qet] [1998] Prevented multiple messages from being displayed when opening a recent file .

[subversion@xxxxxxxxxxxxx]

Revision: 1998
Author:   xavier
Date:     2013-01-03 22:53:26 +0100 (Thu, 03 Jan 2013)
Log Message:
-----------
Prevented multiple messages from being displayed when opening a recent file.

Modified Paths:
--------------
    trunk/sources/editor/qetelementeditor.cpp
    trunk/sources/qetdiagrameditor.cpp

Modified: trunk/sources/editor/qetelementeditor.cpp
===================================================================
--- trunk/sources/editor/qetelementeditor.cpp	2013-01-03 17:26:08 UTC (rev 1997)
+++ trunk/sources/editor/qetelementeditor.cpp	2013-01-03 21:53:26 UTC (rev 1998)
@@ -915,10 +915,8 @@
 	@see openElement
 */
 void QETElementEditor::openRecentFile(const QString &filepath) {
-	if (!QFile::exists ( filepath )) 
-		QMessageBox::warning(this, tr("Attention"),
-							tr("Le fichier semble ne plus exister...")+"\n"+
-							tr("Fichier: ")+filepath+"\n");
+	// small hack to prevent all element editors from trying to topen the required
+	// recent file at the same time
 	if (qApp -> activeWindow() != this) return;
 	openElement(filepath);
 }
@@ -932,6 +930,18 @@
 */
 void QETElementEditor::openElement(const QString &filepath) {
 	if (filepath.isEmpty()) return;
+	// we have to test the file existence here because QETApp::openElementFiles()
+	// will discard non-existent files through QFileInfo::canonicalFilePath()
+	if (!QFile::exists(filepath)) {
+		QET::MessageBox::critical(
+			this,
+			tr("Impossible d'ouvrir le fichier", "message box title"),
+			QString(
+				tr("Il semblerait que le fichier %1 que vous essayez d'ouvrir"
+				" n'existe pas ou plus.")
+			).arg(filepath)
+		);
+	}
 	QETApp::instance() -> openElementFiles(QStringList() << filepath);
 }
 

Modified: trunk/sources/qetdiagrameditor.cpp
===================================================================
--- trunk/sources/qetdiagrameditor.cpp	2013-01-03 17:26:08 UTC (rev 1997)
+++ trunk/sources/qetdiagrameditor.cpp	2013-01-03 21:53:26 UTC (rev 1998)
@@ -637,10 +637,8 @@
 	@see openAndAddDiagram
 */
 bool QETDiagramEditor::openRecentFile(const QString &filepath) {
-	if (!QFile::exists ( filepath )) 
-		QMessageBox::warning(this, tr("Attention"),
-							tr("Le fichier semble ne plus exister...")+"\n"+
-							tr("Fichier: ")+filepath+"\n");
+	// small hack to prevent all diagram editors from trying to topen the required
+	// recent file at the same time
 	if (qApp -> activeWindow() != this) return(false);
 	return(openAndAddProject(filepath));
 }
@@ -735,12 +733,26 @@
 		}
 	}
 	
+	// check the file exists
+	if (!filepath_info.exists()) {
+		if (interactive) {
+			QET::MessageBox::critical(
+				this,
+				tr("Impossible d'ouvrir le fichier", "message box title"),
+				QString(
+					tr("Il semblerait que le fichier %1 que vous essayez d'ouvrir"
+					" n'existe pas ou plus.")
+				).arg(filepath)
+			);
+		}
+	}
+	
 	// verifie que le fichier est accessible en lecture
 	if (!filepath_info.isReadable()) {
 		if (interactive) {
 			QET::MessageBox::critical(
 				this,
-				tr("Impossible d'ouvrir le fichier"),
+				tr("Impossible d'ouvrir le fichier", "message box title"),
 				tr("Il semblerait que le fichier que vous essayez d'ouvrir ne "
 				"soit pas accessible en lecture. Il est donc impossible de "
 				"l'ouvrir. Veuillez v\351rifier les permissions du fichier.")
@@ -754,7 +766,7 @@
 		if (interactive) {
 			QET::MessageBox::warning(
 				this,
-				tr("Ouverture du projet en lecture seule"),
+				tr("Ouverture du projet en lecture seule", "message box title"),
 				tr("Il semblerait que le projet que vous essayez d'ouvrir ne "
 				"soit pas accessible en \351criture. Il sera donc ouvert en "
 				"lecture seule.")