Merge lp:~openerp-dev/openobject-addons/trunk-bug-735348-rme into lp:openobject-addons

[Ravindra Mekhiya(OpenERP)]

Hello,

Code optimization has been corrected.

Thanks.

[Bhumika Shrimali (sbh-openerp)]

Hello,

It is copy paste the code of crm module, you need to change on stage method.

Thanks

[OpenERP buildbot (openerp-buildbot)]

hr_evaluation/test/test_hr_evaluation.yml: ('warning', 'Warning !')
project_timesheet/test/worktask_entry_to_timesheetline_entry.yml[warning]: Assertion "After hr manager's work task" FAILED test: len(timesheet_ids) > 0 values: ! 0 > 0
~/addons/anonymization/anonymization.py:298[pywarn]: The behavior of this method will change in future versions. Use specific 'len(elem)' or 'elem is not None' test instead.
~/addons/anonymization/anonymization.py:300[pywarn]: The behavior of this method will change in future versions. Use specific 'len(elem)' or 'elem is not None' test instead.

review: Disapprove

[OpenERP buildbot (openerp-buildbot)]

hr_evaluation/test/test_hr_evaluation.yml: ('warning', 'Warning !')
project_timesheet/test/worktask_entry_to_timesheetline_entry.yml[warning]: Assertion "After hr manager's work task" FAILED test: len(timesheet_ids) > 0 values: ! 0 > 0
~/addons/anonymization/anonymization.py:298[pywarn]: The behavior of this method will change in future versions. Use specific 'len(elem)' or 'elem is not None' test instead.
~/addons/anonymization/anonymization.py:300[pywarn]: The behavior of this method will change in future versions. Use specific 'len(elem)' or 'elem is not None' test instead.

review: Disapprove

=== modified file 'crm_claim/crm_claim.py'
--- crm_claim/crm_claim.py	2011-02-15 09:42:10 +0000
+++ crm_claim/crm_claim.py	2011-03-16 12:47:58 +0000
@@ -24,6 +24,7 @@
 import time
 import binascii
 import tools
+from tools.translate import _
 
 CRM_CLAIM_PENDING_STATES = (
     crm.AVAILABLE_STATES[2][0], # Cancelled
@@ -81,6 +82,20 @@
         'message_ids': fields.one2many('mailgate.message', 'res_id', 'Messages', domain=[('model','=',_name)]),
     }
     
+    def stage_next(self, cr, uid, ids, context=None):
+        stage = super(crm_claim, self).stage_next(cr, uid, ids, context=context)
+        if stage:
+            stage_obj = self.pool.get('crm.case.stage').browse(cr, uid, stage, context=context)
+            self.history(cr, uid, ids, _("Changed Stage to: ") + stage_obj.name)
+        return stage
+
+    def stage_previous(self, cr, uid, ids, context=None):
+        stage = super(crm_claim, self).stage_previous(cr, uid, ids, context=context)
+        if stage:
+            stage_obj = self.pool.get('crm.case.stage').browse(cr, uid, stage, context=context)
+            self.history(cr, uid, ids, _("Changed Stage to: ") + stage_obj.name)
+        return stage
+    
     def _get_stage_id(self, cr, uid, context=None):
         """Finds type of stage according to object.
         @param self: The object pointer