diff --git a/addons/hr/hr.py b/addons/hr/hr.py
index 9133bc38593..7f2b358ea2e 100644
--- a/addons/hr/hr.py
+++ b/addons/hr/hr.py
@@ -135,6 +135,10 @@ class hr_job(osv.osv):
         return True
 
     def job_open(self, cr, uid, ids, *args):
+        self.write(cr, uid, ids, {'state': 'open', 'no_of_recruitment': 0})
+        return True
+
+    def job_done(self, cr, uid, ids, *args):
         self.write(cr, uid, ids, {'state': 'done', 'no_of_recruitment': 0})
         return True
 
diff --git a/addons/hr/hr_view.xml b/addons/hr/hr_view.xml
index 70a903bb58b..97b4370b646 100644
--- a/addons/hr/hr_view.xml
+++ b/addons/hr/hr_view.xml
@@ -347,7 +347,7 @@
                 <form layout="manual">
                 <header>
                     <button name="job_recruitement" string="In Recruitement" states="open" type="object"/>
-                    <button name="job_open" string="Recruitment Done" states="recruit" type="object"/>
+                    <button name="job_done" string="Recruitment Done" states="recruit" type="object"/>
                     <field name="state" widget="statusbar" statusbar_visible="recruit,open"/>
                 </header>
                 <sheet string="Job" layout="auto">
diff --git a/addons/hr/test/open2recruit2close_job.yml b/addons/hr/test/open2recruit2close_job.yml
index 0f14828694c..9ed5b2b61aa 100644
--- a/addons/hr/test/open2recruit2close_job.yml
+++ b/addons/hr/test/open2recruit2close_job.yml
@@ -26,13 +26,13 @@
   !python {model: hr.employee}: |
     self.write(cr, uid, [ref('employee_niv')], {'job_id':ref('job_developer')})
 -
-  Now I dont want to recruit new employee for this position. so I mark it as Old.
+  Now I dont want to recruit new employee for this position. so I mark it as Done.
 -
   !python {model: hr.job}: |
-    self.job_old(cr, uid, [ref('job_developer')])
+    self.job_done(cr, uid, [ref('job_developer')])
 -
   I check 'state' and number of 'Expected in Recruitment' after closing the Job position.
 -
-  !assert {model: hr.job, id: job_developer, string: Job position is in 'old' state and no_of_recruitment = 0}:
-    - state == 'old'
+  !assert {model: hr.job, id: job_developer, string: Job position is in 'done' state and no_of_recruitment = 0}:
+    - state == 'done'
     - no_of_recruitment == 0