From 35fbb93b1ed44c42a814a31ba533b6e65dc54447 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= <tde@openerp.com>
Date: Fri, 1 Jun 2012 11:23:52 +0200
Subject: [PATCH] [CLEAN] crm_claim: cleaned code before merging (mostly about
 buttons, and also a forgotten access rights thing)

bzr revid: tde@openerp.com-20120601092352-60zr9zhjs4zwhokb
---
 addons/crm_claim/crm_claim.py                 | 29 +++++++++++++------
 addons/crm_claim/crm_claim_view.xml           | 19 ++++++------
 addons/crm_claim/security/ir.model.access.csv |  1 +
 3 files changed, 31 insertions(+), 18 deletions(-)

diff --git a/addons/crm_claim/crm_claim.py b/addons/crm_claim/crm_claim.py
index 6e2138720da..c4e19a19dfd 100644
--- a/addons/crm_claim/crm_claim.py
+++ b/addons/crm_claim/crm_claim.py
@@ -159,15 +159,6 @@ class crm_claim(base_stage, osv.osv):
             return stage_ids[0]
         return False
 
-    def case_get_note_msg_prefix(self, cr, uid, id, context=None):
-        return 'Claim'
-
-    def case_refuse_send_note(self, cr, uid, ids, context=None):
-        for id in ids:
-            msg = _('%s has been <b>refused</b>.') % (self.case_get_note_msg_prefix(cr, uid, id, context=context))
-            self.message_append_note(cr, uid, [id], body=msg, context=context)
-        return True
-
     def case_refuse(self, cr, uid, ids, context=None):
         """ Mark the case as refused: state=done and case_refused=True """
         for lead in self.browse(cr, uid, ids):
@@ -242,6 +233,26 @@ class crm_claim(base_stage, osv.osv):
             res = self.write(cr, uid, [case.id], values, context=context)
         return res
 
+    # ---------------------------------------------------
+    # OpenChatter methods and notifications
+    # ---------------------------------------------------
+
+    def case_get_note_msg_prefix(self, cr, uid, id, context=None):
+        """ Override of default prefix for notifications. """
+        return 'Claim'
+
+    def case_refuse_send_note(self, cr, uid, ids, context=None):
+        for id in ids:
+            msg = _('%s has been <b>refused</b>.') % (self.case_get_note_msg_prefix(cr, uid, id, context=context))
+            self.message_append_note(cr, uid, [id], body=msg, context=context)
+        return True
+
+    def stage_set_send_note(self, cr, uid, ids, stage_id, context=None):
+        """ Override of the (void) default notification method. """
+        stage_name = self.pool.get('crm.claim.stage').name_get(cr, uid, [stage_id], context=context)[0][1]
+        return self.message_append_note(cr, uid, ids, body= _("Stage changed to <b>%s</b>.") % (stage_name), context=context)
+
+
 class res_partner(osv.osv):
     _inherit = 'res.partner'
     _columns = {
diff --git a/addons/crm_claim/crm_claim_view.xml b/addons/crm_claim/crm_claim_view.xml
index e7762866c4e..c6e7db167a8 100644
--- a/addons/crm_claim/crm_claim_view.xml
+++ b/addons/crm_claim/crm_claim_view.xml
@@ -79,15 +79,12 @@
                     <field name="date_deadline" invisible="1"/>
                     <field name="date_closed" invisible="1"/>
                     <field name="state" groups="base.group_no_one"/>
-                    <button name="case_open" string="Open"
-                        states="draft,pending" type="object"
-                        icon="gtk-go-forward" groups="base.group_no_one"/>
-                    <button name="case_close" string="Done"
-                        states="draft,open,pending" type="object"
-                        icon="gtk-close" groups="base.group_no_one"/>
-                    <button name="case_refuse" string="Refuse"
-                        states="draft,open,pending" type="object"
-                        icon="gtk-cancel" groups="base.group_no_one"/>
+                    <button name="case_open" string="Open" type="object"
+                        states="draft,pending" icon="gtk-go-forward"/>
+                    <button name="case_close" string="Done" type="object"
+                        states="draft,open,pending" icon="gtk-close"/>
+                    <button name="case_refuse" string="Refuse" type="object"
+                        states="draft,open,pending" icon="gtk-cancel"/>
                 </tree>
             </field>
         </record>
@@ -105,6 +102,10 @@
                             states="open,pending"/>
                     <button name="case_refuse" string="Refuse" type="object"
                             states="draft,open,pending"/>
+                    <button name="stage_previous" string="Previous Stage" type="object"
+                            states="open,pending" icon="gtk-go-back" attrs="{'invisible': [('stage_id','=', False)]}"/>
+                    <button name="stage_next" string="Next Stage" type="object"
+                            states="open,pending" icon="gtk-go-forward" attrs="{'invisible': [('stage_id','=', False)]}"/>
                     <button name="case_reset" string="Reset to Draft" type="object"
                             states="cancel,done"/>
                     <button name="case_cancel" string="Cancel" type="object"
diff --git a/addons/crm_claim/security/ir.model.access.csv b/addons/crm_claim/security/ir.model.access.csv
index 5a05fbd4d87..303624288ce 100644
--- a/addons/crm_claim/security/ir.model.access.csv
+++ b/addons/crm_claim/security/ir.model.access.csv
@@ -1,5 +1,6 @@
 id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
 access_crm_claim_manager,crm.claim.manager,model_crm_claim,base.group_sale_manager,1,1,1,1
 access_crm_claim_user,crm.claim.user,model_crm_claim,base.group_sale_salesman,1,1,1,0
+access_crm_claim_stage_user,crm.claim.stage.user,model_crm_claim_stage,base.group_sale_salesman,1,1,1,1
 access_crm_claim_report_manager,crm.claim.report.manager,model_crm_claim_report,base.group_sale_manager,1,1,1,1
 access_crm_claim_partner_manager,crm.claim.partner.manager,model_crm_claim,base.group_partner_manager,1,0,0,0