better handling of python errors raised during initialization.

wonder · wonder · commit c47bcc86ba17 · 2008-05-21T21:09:37.000Z
git-svn-id: http://svn.osgeo.org/qgis/trunk/qgis@8491 c8812cc2-4d05-0410-92ff-de0c093fc19c
diff --git a/src/app/qgspythondialog.cpp b/src/app/qgspythondialog.cpp
@@ -43,7 +43,7 @@ void QgsPythonDialog::on_edtCmdLine_returnPressed()
   
   // when using Py_single_input the return value will be always null
   // we're using custom hooks for output and exceptions to show output in console
-  if (QgsPythonUtils::runString(command))
+  if (QgsPythonUtils::runStringUnsafe(command))
   {
     QgsPythonUtils::evalString("sys.stdout.data", output);
     QgsPythonUtils::runString("sys.stdout.data = ''");
diff --git a/src/app/qgspythonutils.cpp b/src/app/qgspythonutils.cpp
@@ -22,6 +22,7 @@
 
 #include "qgsapplication.h"
 #include "qgslogger.h"
+#include "qgsmessageoutput.h"
 
 #include <QMessageBox>
 #include <QStringList>
@@ -51,32 +52,25 @@ void QgsPythonUtils::initPython(QgisInterface* interface)
   runString("sys.path = [\"" + homePluginsPath()  + "\", \"" + pythonPath() + "\", \"" + pluginsPath() + "\"] + sys.path");
 
   // import SIP
-  if (!runString("from sip import wrapinstance, unwrapinstance"))
+  if (!runString("from sip import wrapinstance, unwrapinstance",
+       QObject::tr("Couldn't load SIP module.") + "\n" + QObject::tr("Python support will be disabled.")))
   {
-    QMessageBox::warning(0, QObject::tr("Python error"),
-                         QObject::tr("Couldn't load SIP module.\nPython support will be disabled."));
-    PyErr_Clear();
     exitPython();
     return;
   }
   
   // import Qt bindings
-  if (!runString("from PyQt4 import QtCore, QtGui"))
+  if (!runString("from PyQt4 import QtCore, QtGui",
+       QObject::tr("Couldn't load PyQt4.") + "\n" + QObject::tr("Python support will be disabled.")))
   {
-    QMessageBox::warning(0, QObject::tr("Python error"),
-                         QObject::tr("Couldn't load PyQt bindings.\nPython support will be disabled."));
-    PyErr_Clear();
     exitPython();
     return;
   }
   
   // import QGIS bindings
-  if (!runString("from qgis.core import *") ||
-      !runString("from qgis.gui import *"))
+  QString error_msg = QObject::tr("Couldn't load PyQGIS.") + "\n" + QObject::tr("Python support will be disabled.");
+  if (!runString("from qgis.core import *", error_msg) || !runString("from qgis.gui import *", error_msg))
   {
-    QMessageBox::warning(0, QObject::tr("Python error"),
-                         QObject::tr("Couldn't load QGIS bindings.\nPython support will be disabled."));
-    PyErr_Clear();
     exitPython();
     return;
   }
@@ -162,13 +156,43 @@ void QgsPythonUtils::uninstallConsoleHooks()
 }
 
 
-bool QgsPythonUtils::runString(const QString& command)
+bool QgsPythonUtils::runStringUnsafe(const QString& command)
 {
   PyRun_String(command.toLocal8Bit().data(), Py_single_input, mMainDict, mMainDict);
-  
   return (PyErr_Occurred() == 0);
 }
 
+bool QgsPythonUtils::runString(const QString& command, QString msgOnError)
+{
+  bool res = runStringUnsafe(command);
+  if (res)
+    return true;
+    
+  // an error occured
+  // fetch error details and show it
+  QString err_type, err_value;
+  getError(err_type, err_value);
+  
+  if (msgOnError.isEmpty())
+  {
+    // use some default message if custom hasn't been specified
+    msgOnError = QObject::tr("An error occured during execution of following code:") + "\n<tt>" + command + "</tt>";
+  }
+  msgOnError.replace("\n", "<br>");
+  QString str = msgOnError + "<br><br>" + QObject::tr("Error details:") + "<br>"
+      + QObject::tr("Type:")  + " <b>" + err_type + "</b><br>"
+      + QObject::tr("Value:") + " <b>" + err_value + "</b>";
+  
+  // TODO: show sys.path, local variables, traceback
+  
+  QgsMessageOutput* msg = QgsMessageOutput::createMessageOutput();
+  msg->setTitle(QObject::tr("Python error"));
+  msg->setMessage(str, QgsMessageOutput::MessageHtml);
+  msg->showMessage();
+  
+  return res;
+}
+
 
 QString QgsPythonUtils::getTypeAsString(PyObject* obj)
 {
@@ -207,7 +231,7 @@ bool QgsPythonUtils::getError(QString& errorClassName, QString& errorText)
   PyObject* err_value;
   PyObject* err_tb;
   
-  // get the exception information
+  // get the exception information and clear error
   PyErr_Fetch(&err_type, &err_value, &err_tb);
   
   // get exception's class name
@@ -223,8 +247,10 @@ bool QgsPythonUtils::getError(QString& errorClassName, QString& errorText)
   else
     errorText.clear();
   
-  // clear exception
-  PyErr_Clear();
+  // cleanup
+  Py_XDECREF(err_type);
+  Py_XDECREF(err_value);
+  Py_XDECREF(err_tb);
   
   return true;
 }
@@ -373,29 +399,16 @@ bool QgsPythonUtils::startPlugin(QString packageName)
 {
   QString pluginPythonVar = "plugins['" + packageName + "']";
   
-  // create an instance of the plugin
-  if (!runString(pluginPythonVar + " = " + packageName + ".classFactory(iface)"))
-  {
-    PyErr_Print(); // just print to console
-    PyErr_Clear();
+  QString errMsg = QObject::tr("Couldn't load plugin ") + packageName;
     
-    QMessageBox::warning(0, QObject::tr("Python error"),
-                         QObject::tr("Couldn't load plugin ") + packageName +
-                         QObject::tr(" due an error when calling its classFactory() method"));
+  // create an instance of the plugin
+  if (!runString(pluginPythonVar + " = " + packageName + ".classFactory(iface)",
+                 errMsg + QObject::tr(" due an error when calling its classFactory() method")))
     return false;
-  }
   
   // initGui
-  if (!runString(pluginPythonVar + ".initGui()"))
-  {
-    PyErr_Print(); // just print to console
-    PyErr_Clear();
-    
-    QMessageBox::warning(0, QObject::tr("Python error"),
-                         QObject::tr("Couldn't load plugin ") + packageName +
-                         QObject::tr(" due an error when calling its initGui() method"));
+  if (!runString(pluginPythonVar + ".initGui()", errMsg + QObject::tr(" due an error when calling its initGui() method")))
     return false;
-  }
   
   return true;
 }
@@ -406,16 +419,12 @@ bool QgsPythonUtils::unloadPlugin(QString packageName)
   // unload and delete plugin!
   QString varName = "plugins['" + packageName + "']";
   
-  if (!runString(varName + ".unload()") ||
-      !runString("del " + varName))
-  {
-    PyErr_Print(); // just print to console
-    PyErr_Clear();
-    
-    QMessageBox::warning(0, QObject::tr("Python error"),
-                         QObject::tr("Error while unloading plugin ") + packageName);
+  QString errMsg = QObject::tr("Error while unloading plugin ") + packageName;
+  
+  if (!runString(varName + ".unload()", errMsg))
+    return false;
+  if (!runString("del " + varName, errMsg))
     return false;
-  }
   
   return true;
 }
diff --git a/src/app/qgspythonutils.h b/src/app/qgspythonutils.h
@@ -53,8 +53,13 @@ class QgsPythonUtils
     
     //! run a statement (wrapper for PyRun_String)
     //! this command is more advanced as enables error checking etc.
+    //! when an exception is raised, it shows dialog with exception details
     //! @return true if no error occured
-    static bool runString(const QString& command);
+    static bool runString(const QString& command, QString msgOnError = QString());
+    
+    //! run a statement, error reporting is not done
+    //! @return true if no error occured
+    static bool runStringUnsafe(const QString& command);
     
     static bool evalString(const QString& command, QString& result);
     

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ void QgsPythonDialog::on_edtCmdLine_returnPressed()`
`43`	`43`
`44`	`44`	`// when using Py_single_input the return value will be always null`
`45`	`45`	`// we're using custom hooks for output and exceptions to show output in console`
`46`		`- if (QgsPythonUtils::runString(command))`
	`46`	`+ if (QgsPythonUtils::runStringUnsafe(command))`
`47`	`47`	`{`
`48`	`48`	`QgsPythonUtils::evalString("sys.stdout.data", output);`
`49`	`49`	`QgsPythonUtils::runString("sys.stdout.data = ''");`