EmComponent: get_max_rank() should return an int

EditorialModel/components.py

@@ -116,7 +116,7 @@ class EmComponent(object):
     # @return A positive integer or -1 if no components
     def get_max_rank(self):
         components = self.same_rank_group()
-        return 1 if len(components) == 0 else components[-1].rank
+        return len(components) + 1
 
     ## Return an array of instances that are concerned by the same rank
     # @return An array of instances that are concerned by the same rank

EditorialModel/model.py

@@ -141,10 +141,11 @@ class Model(object):
         datas['uid'] = self.new_uid()
         em_component = em_obj(self, **datas)
 
+        em_component.rank = em_component.get_max_rank()
+
         self._components['uids'][em_component.uid] = em_component
         self._components[self.name_from_emclass(em_component.__class__)].append(em_component)
 
-        em_component.rank = em_component.get_max_rank()
         if rank != 'last':
             em_component.set_rank(1 if rank == 'first' else rank)