Merge lp:~vaab/openobject-addons/project-has-wider-choices-for-set-priority-set-remaining-time into lp:openobject-addons

[Valentin Lab]

Adds generic methods ``.set_remaining_time_NN`` and ``.set_priority_NN`` in ``project.task`` model in the spirit of the existing ``.set_remaining_time_1``, ``.set_remaining_time_2``, ...

This allows kanban view a much wider range of action if needed. This code is used and tested in a modified kanban view I'm using.

Note that this avoids also hard written presets and messy copy-pasted code in all the "set_remaining_time_1/2/5/10".

Unmerged revisions

6663. By Valentin Lab on 2012-03-07

[NEW] wider choices for set_priority, and set_remaining_time available for view code (ie: kanban).

=== modified file 'project/project.py'
--- project/project.py	2012-02-28 14:08:16 +0000
+++ project/project.py	2012-03-08 13:10:26 +0000
@@ -681,10 +681,10 @@
 
     _order = "priority, sequence, date_start, name, id"
 
-    def set_priority(self, cr, uid, ids, priority):
+    def set_priority(self, cr, uid, ids, priority, context=None):
         """Set task priority
         """
-        return self.write(cr, uid, ids, {'priority' : priority})
+        return self.write(cr, uid, ids, {'priority' : str(priority)})
 
     def set_high_priority(self, cr, uid, ids, *args):
         """Set task priority to high
@@ -696,6 +696,27 @@
         """
         return self.set_priority(cr, uid, ids, '2')
 
+    ## Generic set_priority_* and set_remaining_time_*
+    def __getattr__(self, name):
+        for prefix in ["set_priority", "set_remaining_time"]:
+            if not name.startswith(prefix + "_"):
+                continue
+            arg = name[len(prefix) + 1:]
+            try:
+                arg = int(arg)
+            except ValueError:
+                continue ## defaults to standard get attr
+            method = getattr(self, prefix)
+
+            ## ``method`` and ``arg`` are stored in closure
+            def _custom_set(cr, uid, ids, context=None):
+                return method(cr, uid, ids, arg, context)
+
+            return _custom_set
+
+        ## default getattr
+        return super(osv.osv, self).__getattr__(name)
+
     def _check_recursion(self, cr, uid, ids, context=None):
         for id in ids:
             visited_branch = set()