Merge lp:~openerp-dev/openobject-addons/6.1-opw-573199-ksa into lp:openobject-addons/6.1

[Kirti Savalia(OpenERP)]

Hello,

Fix the problem when stage id is not in the list.

Thanks
KSA

[Naresh(OpenERP) (nch-openerp)]

Hello,

This bug was qualified as Not Relevant on Trunk (means that architecture on trunk has changed and so this bug has no meaning anymore). If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh

Unmerged revisions

6717. By Kirti Savalia(OpenERP) on 2012-03-30

[FIX]: when stage id is not in the list

=== modified file 'hr_recruitment/hr_recruitment.py'
--- hr_recruitment/hr_recruitment.py	2011-12-31 07:57:20 +0000
+++ hr_recruitment/hr_recruitment.py	2012-03-30 11:01:22 +0000
@@ -243,7 +243,7 @@
             department = (case.department_id.id or False)
             st = case.stage_id.id  or False
             stage_ids = stage_obj.search(cr, uid, ['|',('department_id','=',department),('department_id','=',False)], context=context)
-            if st and stage_ids.index(st):
+            if st in stage_ids and stage_ids.index(st):
                 self.write(cr, uid, [case.id], {'stage_id': stage_ids[stage_ids.index(st)-1]}, context=context)
             else:
                 self.write(cr, uid, [case.id], {'stage_id': False}, context=context)
@@ -263,7 +263,7 @@
             st = case.stage_id.id  or False
             stage_ids = stage_obj.search(cr, uid, ['|',('department_id','=',department),('department_id','=',False)], context=context)
             val = False
-            if st and len(stage_ids) != stage_ids.index(st)+1:
+            if st in stage_ids and len(stage_ids) != stage_ids.index(st)+1:
                 val = stage_ids[stage_ids.index(st)+1]
             elif (not st) and stage_ids:
                 val = stage_ids[0]