Complete the LeFactory generated code tests, add a __hash__ method for fieldtypes

EditorialModel/fieldtypes/generic.py

@@ -82,6 +82,15 @@ class GenericFieldType(object):
     @staticmethod
     def module_name(fieldtype_name):
         return 'EditorialModel.fieldtypes.%s'%(fieldtype_name)
+
+    ## @brief __hash__ implementation for fieldtypes
+    def __hash__(self):
+        hash_dats = [ self.__class__.__module__ ]
+        for kdic in sorted([k for k in self.__dict__.keys() if not k.startswith('_')]):
+            hash_dats.append((kdic, getattr(self, kdic)))
+        return hash(tuple(hash_dats))
+
+
     
     ## @brief Transform a value into a valid python representation according to the fieldtype
     # @param value ? : The value to cast

leobject/lefactory.py

@@ -31,6 +31,8 @@ class LeFactory(object):
     # @return name.title()
     @staticmethod
     def name2classname(name):
+        if not isinstance(name, str):
+            raise AttributeError("Argument name should be a str and not a %s"%type(name))
         return name.title()
 
     ## @brief Return a call to a FieldType constructor given an EmField
@@ -64,12 +66,12 @@ class LeFactory(object):
                 cls_fieldgroup[fieldgroup.name].append(field.name)
         
         return """
-{name}._fildtypes = {ftypes}
+{name}._fieldtypes = {ftypes}
 {name}._linked_types = {ltypes}
 {name}._fieldgroups = {fgroups}
 """.format(
     name = LeFactory.name2classname(emclass.name),
-    ftypes = repr(cls_fields),
+    ftypes = "{"+(','.join([ '\n%s:%s'%(repr(f),v) for f,v in cls_fields.items()]))+"}",
     ltypes = "{"+(','.join(cls_linked_types))+'}',
     fgroups = repr(cls_fieldgroup)
 )