From bb8b60c1e9ee36f65bd7859b0c410824cae95bc7 Mon Sep 17 00:00:00 2001
From: bth-openerp <bth@tinyerp.com>
Date: Fri, 20 Sep 2013 16:24:21 +0530
Subject: [PATCH] [IMP]IMproved code for subscribe.

bzr revid: bth@tinyerp.com-20130920105421-20w2i2jvdwe2qkwy
---
 addons/website_hr_recruitment/controllers/main.py  | 12 ++++++------
 .../views/website_hr_recruitment.xml               | 14 +++++++++++++-
 2 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/addons/website_hr_recruitment/controllers/main.py b/addons/website_hr_recruitment/controllers/main.py
index df9432924ab..234a4d0d781 100644
--- a/addons/website_hr_recruitment/controllers/main.py
+++ b/addons/website_hr_recruitment/controllers/main.py
@@ -86,7 +86,7 @@ class website_hr_recruitment(http.Controller):
         return request.website.render("website_hr_recruitment.thankyou", values)
 
     @website.route(['/job/detail/<int:job_id>/subscribe'], type='http', auth="public")
-    def subscribe(self, event_id=None, **post):
+    def subscribe(self, job_id=None, **post):
         partner_obj = request.registry['res.partner']
         job_obj = request.registry['hr.job']
         user_obj = request.registry['res.users']
@@ -107,12 +107,12 @@ class website_hr_recruitment(http.Controller):
                 partner_ids = [user_obj.browse(
                     request.cr, request.uid, request.uid,
                     context=request.context).partner_id.id]
-            job_obj.check_access_rule(request.cr, request.uid, [event_id],
+            job_obj.check_access_rule(request.cr, request.uid, [job_id],
                                         'read', request.context)
-            job_obj.message_subscribe(request.cr, SUPERUSER_ID, [event_id],
+            job_obj.message_subscribe(request.cr, SUPERUSER_ID, [job_id],
                                         partner_ids, request.context)
 
-        return self.detail(job_id=job_id)
+        return self.detail(id=job_id)
 
     @website.route(['/job/detail/<int:job_id>/unsubscribe'], type='http', auth="public")
     def unsubscribe(self, job_id=None, **post):
@@ -127,10 +127,10 @@ class website_hr_recruitment(http.Controller):
                     context=request.context)
             else:
                 partner_ids = [user_obj.browse(request.cr, request.uid, request.uid, request.context).partner_id.id]
-            job_obj.check_access_rule(request.cr, request.uid, [event_id], 'read', request.context)
+            job_obj.check_access_rule(request.cr, request.uid, [job_id], 'read', request.context)
             job_obj.message_unsubscribe(request.cr, SUPERUSER_ID, [job_id], partner_ids, request.context)
 
-        return self.detail(job_id=job_id)
+        return self.detail(id=job_id)
 
     @website.route('/recruitment/published', type='json', auth="admin")
     def published (self, **post):
diff --git a/addons/website_hr_recruitment/views/website_hr_recruitment.xml b/addons/website_hr_recruitment/views/website_hr_recruitment.xml
index 137b5704034..b7b7ac7b8e4 100644
--- a/addons/website_hr_recruitment/views/website_hr_recruitment.xml
+++ b/addons/website_hr_recruitment/views/website_hr_recruitment.xml
@@ -73,7 +73,19 @@
 		</t>
                 <t t-set="title">Job Detail</t>
 		<div id="wrap">
-		    <div class="oe_structure">
+			<div class="col-md-4">
+                        <form action="./subscribe" method="POST" class="form-inline" t-if="not subscribe">
+                            <div class="col-lg-7">
+                                <input placeholder="Email Address" type="email" name="email" class="form-control" t-if="is_public_user"/>
+                            </div>
+                            <button type="submit" class="btn btn-primary" name="subscribe">Subscribe</button>
+                        </form>
+                        <form action="./unsubscribe" method="POST" class="form-inline" t-if="subscribe">
+                            <input type="hidden" name="email" t-att-value="subscribe"/>
+                            <button type="submit" class="btn btn-default" name="unsubscribe">Unsubscribe</button>
+                        </form>
+                    </div>
+		    <div class="oe_structure" style="clear:both;">
 	                <h1 class="text-center" t-field="job.name"></h1>
 			<h4 class="text-center">
                             <i class="icon-time"></i> <span><t t-esc="vals_date"/></span>