[FIX] website_hr_recruitment: missing required 'name', which leaded to crash, and the phone field is partner_phone, not just phone. Do you even test what you do ?

bzr revid: dle@openerp.com-20140303174445-j733tpmccc9x855u

1 files changed, 4 insertions, 1 deletions

diff --git a/addons/website_hr_recruitment/controllers/main.py b/addons/website_hr_recruitment/controllers/main.py
index a3cbbe3cbc4..8140a39dc9b 100644
--- a/addons/website_hr_recruitment/controllers/main.py
+++ b/addons/website_hr_recruitment/controllers/main.py
@@ -86,11 +86,14 @@ class website_hr_recruitment(http.Controller):
 
         value = {
             'source_id' : imd.xmlid_to_res_id(cr, SUPERUSER_ID, 'hr_recruitment.source_website_company'),
+            'name': '%s\'s Application' % post.get('partner_name'), 
         }
-        for f in ['phone', 'email_from', 'partner_name', 'description']:
+        for f in ['email_from', 'partner_name', 'description']:
             value[f] = post.get(f)
         for f in ['department_id', 'job_id']:
             value[f] = int(post.get(f) or 0)
+        # Retro-compatibility for saas-3. "phone" field should be replace by "partner_phone" in the template in trunk.
+        value['partner_phone'] = post.pop('phone', False)
 
         applicant_id = request.registry['hr.applicant'].create(cr, SUPERUSER_ID, value, context=context)
         if post['ufile']: