engineIOR() made pure virtual, to avoid serious logical errors

diff --git a/src/SalomeApp/SalomeApp_Module.cxx b/src/SalomeApp/SalomeApp_Module.cxx
index d1cc44ba1b8c881030c2e1ce3becf4191c6c8a1a..4eb303e59d25573ed5c5604bba9dab31409d3e0b 100644 (file)
--- a/src/SalomeApp/SalomeApp_Module.cxx
+++ b/src/SalomeApp/SalomeApp_Module.cxx
@@ -65,15 +65,6 @@ void SalomeApp_Module::viewManagers( QStringList& ) const
 {
 }
 
-QString SalomeApp_Module::engineIOR() const
-{
-  // Return an empty string by default, to avoid any "default behavior".
-  // It means that SALOMEDS persistence is not used by a module by default.
-  // Each module can override this method to return either its actual engine's IOR,
-  // or simple the result returned by SalomeApp_Application::defaultEngineIOR()
-  return QString( "" );
-}
-
 SalomeApp_Application* SalomeApp_Module::getApp() const
 {
   return (SalomeApp_Application*)application();

diff --git a/src/SalomeApp/SalomeApp_Module.h b/src/SalomeApp/SalomeApp_Module.h
index 8be396c33f9d2a65dfd05916774fbbbbae380119..0668c193a78f191886cdbd60f3555b50b703246b 100644 (file)
--- a/src/SalomeApp/SalomeApp_Module.h
+++ b/src/SalomeApp/SalomeApp_Module.h
@@ -44,7 +44,15 @@ public:
   virtual void                        initialize( CAM_Application* );
   virtual void                        windows( QMap<int, int>& ) const;
   virtual void                        viewManagers( QStringList& ) const;
-  virtual QString                     engineIOR() const;
+
+  // engineIOR() should be a pure virtual method, to avoid logical errors!
+  //
+  // Implementation in derived classes can return the following values:
+  // "" (empty string)   - means that this is a light module, default engine should be used for interaction with SALOMEDS persistence
+  // "-1"                - means that this is a light module, SALOMEDS persistence is not used at all
+  // module`s engine IOR - means that this is a standard SALOME module with a CORBA engine
+  //
+  virtual QString                     engineIOR() const = 0;
 
   virtual void                        contextMenuPopup( const QString&, QPopupMenu*, QString& );