Bugfix on EM groups handling (now we have to set actives groups in settings)

install/conf.d/lodel2.ini

@@ -10,7 +10,7 @@ filename = -
 context = True
 
 [lodel2.editorialmodel]
-groups = 
+groups = base_group, editorial_abstract, editorial_person
 emfile = editorial_model.pickle
 dyncode = leapi_dyncode.py
 

lodel/editorial_model/components.py

@@ -96,6 +96,8 @@ class EmClass(EmComponent):
  fetching arbitrary datas from DB"},
                 data_handler = 'LeobjectSubclassIdentifier',
                 internal = True)
+        if group is not None:
+            group.add_components([self])
     
     ##@brief Property that represent a dict of all fields (the EmField defined in this class and all its parents)
     # @todo use Settings.editorialmodel.groups to determine wich fields should be returned

lodel/editorial_model/model.py

@@ -5,7 +5,7 @@ import importlib
 import copy
 
 from lodel.utils.mlstring import MlString
-from lodel.logger import logger
+from lodel import logger
 from lodel.settings import Settings
 from lodel.settings.utils import SettingsError
 
@@ -128,19 +128,26 @@ class EditorialModel(object):
     #EditorialModel.__active_classes attibutes
     def __set_actives(self):
         if Settings.editorialmodel.editormode:
+            logger.warning("All EM groups active because editormode in ON")
             # all groups & classes actives because we are in editor mode
             self.__active_groups = self.__groups
             self.__active_classes = self.__classes
         else:
             #determine groups first
             self.__active_groups = dict()
+            self.__active_classes = dict()
             for agrp in Settings.editorialmodel.groups:
                 if agrp not in self.__groups:
                     raise SettingsError('Invalid group found in settings : %s' % agrp)
+                logger.debug("Set group '%s' as active" % agrp)
                 grp = self.__groups[agrp]
                 self.__active_groups[grp.uid] = grp
                 for acls in grp.components():
                     self.__active_classes[acls.uid] = acls
+            if len(self.__active_groups) == 0:
+                raise RuntimeError("No groups activated, abording...")
+            if len(self.__active_classes) == 0:
+                raise RuntimeError("No active class found. Abording")
     
     ##@brief EmField getter
     # @param uid str : An EmField uid represented by "CLASSUID.FIELDUID"

lodel/leapi/lefactory.py

@@ -4,6 +4,7 @@ import functools
 from lodel.editorial_model.components import *
 from lodel.leapi.leobject import LeObject
 from lodel.leapi.datahandlers.base_classes import DataHandler
+from lodel import logger
 
 ##@brief Generate python module code from a given model
 # @param model lodel.editorial_model.model.EditorialModel
@@ -111,6 +112,7 @@ def generate_classes(model):
     bootstrap = ""
     # Generating field list for LeObjects generated from EmClass
     for em_class in get_classes(model):
+        logger.info("Generating a dynamic class for %s" % em_class.uid)
         uid = list()        # List of fieldnames that are part of the EmClass primary key
         parents = list()    # List of parents EmClass
         # Determine pk