Improvement NPAL14675: fix a bug: take into account that activate() method can be...

diff --git a/src/SALOME_PYQT/SALOME_PYQT_GUI/SALOME_PYQT_Module.cxx b/src/SALOME_PYQT/SALOME_PYQT_GUI/SALOME_PYQT_Module.cxx
index 0a83e03d450b7ea52c4bbe67c558637f86587180..3ce9be1fe52d59b75c7f541015ce2314f38256c4 100644 (file)
--- a/src/SALOME_PYQT/SALOME_PYQT_GUI/SALOME_PYQT_Module.cxx
+++ b/src/SALOME_PYQT/SALOME_PYQT_GUI/SALOME_PYQT_Module.cxx
@@ -169,7 +169,7 @@ SALOME_PYQT_Module::SALOME_PYQT_Module()
   : SalomeApp_Module( __DEFAULT_NAME__ ),
     myModule( 0 ), 
     myXmlHandler ( 0 ),
-    myLastActivateStatus( false )
+    myLastActivateStatus( true )
 {
 }
 
@@ -252,6 +252,9 @@ bool SALOME_PYQT_Module::activateModule( SUIT_Study* theStudy )
   if ( !res )
     return res;
 
+  // reset the activation status to the default value
+  myLastActivateStatus = true;
+
   // ActivateReq: request class for internal activate() operation
   class ActivateReq : public PyInterp_Request
   {
@@ -830,17 +833,15 @@ void SALOME_PYQT_Module::activate( SUIT_Study* theStudy )
   // call Python module's activate() method (for the new modules)
   if(PyObject_HasAttrString(myModule , "activate")){
     PyObject* res1 = PyObject_CallMethod( myModule, "activate", "" );
-    if( !res1 || !PyBool_Check( res1 ) )
-      {
-       PyErr_Print();
-       //= true: for support of old modules
-       myLastActivateStatus = true;
-      }
-    else
-      {
-       //detect return status
-       myLastActivateStatus = PyObject_IsTrue( res1 );
-      }
+    if ( !res1 || !PyBool_Check( res1 ) ) {
+      PyErr_Print();
+      //= true: for support of old modules
+      myLastActivateStatus = true;
+    }
+    else {
+      //detect return status
+      myLastActivateStatus = PyObject_IsTrue( res1 );
+    }
   }
 }