# HG changeset patch
# User Sverre Rabbelier <srabbelier@gmail.com>
# Date 1235667059 0
# Node ID 75de65fcb017ebb494fb4f28b0f6c92e509e306a
# Parent  4a233f5a426459f2a9ed9fe8af6d145b3bd0bfd6
Move the hooks in base.Logic to the bottom of the class

This more closely follows 'important methods first'.

Patch by: Sverre Rabbelier

diff -r 4a233f5a4264 -r 75de65fcb017 app/soc/logic/models/base.py
--- a/app/soc/logic/models/base.py	Thu Feb 26 16:50:22 2009 +0000
+++ b/app/soc/logic/models/base.py	Thu Feb 26 16:50:59 2009 +0000
@@ -107,56 +107,6 @@
     depth = self._scope_logic.logic.getScopeDepth()
     return None if (depth is None) else (depth + 1)
 
-  def _updateField(self, entity, name, value):
-    """Hook called when a field is updated.
-
-    Base classes should override if any special actions need to be
-    taken when a field is updated. The field is not updated if the
-    method does not return a True value.
-
-    Args:
-      entity: the unaltered entity
-      name: the name of the field to be changed
-      value: the new value
-    """
-
-    if not entity:
-      raise NoEntityError
-
-    if not entity_properties or (name not in entity_properties):
-      raise InvalidArgumentError
-
-    return True
-
-  def _onCreate(self, entity):
-    """Called when an entity has been created.
-
-    Classes that override this can use it to do any post-creation operations.
-    """
-
-    if not entity:
-      raise NoEntityError
-
-    sidebar.flush()
-
-  def _onUpdate(self, entity):
-    """Called when an entity has been updated.
-
-    Classes that override this can use it to do any post-update operations.
-    """
-
-    if not entity:
-      raise NoEntityError
-
-  def _onDelete(self, entity):
-    """Called when an entity has been deleted.
-
-    Classes that override this can use it to do any post-deletion operations.
-    """
-
-    if not entity:
-      raise NoEntityError
-
   def getKeyNameFromFields(self, fields):
     """Returns the KeyName constructed from fields dict for this type of entity.
 
@@ -469,3 +419,53 @@
     entity.delete()
     # entity has been deleted call _onDelete
     self._onDelete(entity)
+
+  def _updateField(self, entity, name, value):
+    """Hook called when a field is updated.
+
+    Base classes should override if any special actions need to be
+    taken when a field is updated. The field is not updated if the
+    method does not return a True value.
+
+    Args:
+      entity: the unaltered entity
+      name: the name of the field to be changed
+      value: the new value
+    """
+
+    if not entity:
+      raise NoEntityError
+
+    if not entity_properties or (name not in entity_properties):
+      raise InvalidArgumentError
+
+    return True
+
+  def _onCreate(self, entity):
+    """Called when an entity has been created.
+
+    Classes that override this can use it to do any post-creation operations.
+    """
+
+    if not entity:
+      raise NoEntityError
+
+    sidebar.flush()
+
+  def _onUpdate(self, entity):
+    """Called when an entity has been updated.
+
+    Classes that override this can use it to do any post-update operations.
+    """
+
+    if not entity:
+      raise NoEntityError
+
+  def _onDelete(self, entity):
+    """Called when an entity has been deleted.
+
+    Classes that override this can use it to do any post-deletion operations.
+    """
+
+    if not entity:
+      raise NoEntityError