From 4e66ad96cc73597f2e8e10c8e0b9aadeda2c7510 Mon Sep 17 00:00:00 2001
From: Yogesh Sakhreliya <ysa@tinyerp.com>
Date: Mon, 4 Oct 2010 19:38:41 +0530
Subject: [PATCH 01/10] [REM] remove role_id field in workflow.xml file.

bzr revid: ysa@tinyerp.com-20101004140841-abyzp16b5qaau505
---
 .../crossovered_budget_workflow.xml           |  4 --
 .../email_template_workflow.xml               |  4 --
 addons/hr_expense/hr_expense_workflow.xml     |  6 --
 addons/hr_holidays/hr_holidays_workflow.xml   |  5 --
 .../hr_timesheet_workflow.xml                 | 14 ++---
 .../process/hr_timesheet_sheet_process.xml    | 56 +++++++++----------
 addons/sale/process/sale_process.xml          |  8 ---
 7 files changed, 32 insertions(+), 65 deletions(-)

diff --git a/addons/account_budget/crossovered_budget_workflow.xml b/addons/account_budget/crossovered_budget_workflow.xml
index f304775bcca..16997d694f2 100644
--- a/addons/account_budget/crossovered_budget_workflow.xml
+++ b/addons/account_budget/crossovered_budget_workflow.xml
@@ -65,28 +65,24 @@
 		<field name="act_from" ref="act_confirm" />
 		<field name="act_to" ref="act_validate" />
 		<field name="signal">validate</field>
-		<field name="role_id" ref="crossovered_budget_role"/>
 	</record>
 
 	<record model="workflow.transition" id="t3">
 		<field name="act_from" ref="act_confirm" />
 		<field name="act_to" ref="act_cancel" />
 		<field name="signal">cancel</field>
-		<field name="role_id" ref="crossovered_budget_role"/>
 	</record>
 
 	<record model="workflow.transition" id="t4">
 		<field name="act_from" ref="act_validate" />
 		<field name="act_to" ref="act_cancel" />
 		<field name="signal">cancel</field>
-		<field name="role_id" ref="crossovered_budget_role"/>
 	</record>
 
 	<record model="workflow.transition" id="t5">
 		<field name="act_from" ref="act_validate" />
 		<field name="act_to" ref="act_done" />
 		<field name="signal">done</field>
-		<field name="role_id" ref="crossovered_budget_role"/>
 	</record>
 
 	<record model="workflow.transition" id="t6">
diff --git a/addons/email_template/email_template_workflow.xml b/addons/email_template/email_template_workflow.xml
index 911b8bf49ff..1a6a73cca77 100644
--- a/addons/email_template/email_template_workflow.xml
+++ b/addons/email_template/email_template_workflow.xml
@@ -47,26 +47,22 @@
             <field name="act_from" ref="act_draft"/>
             <field name="act_to" ref="act_approved"/>
             <field name="signal">button_approval</field>
-            <field name="role_id" ref="PE_ADMIN"/>
         </record>
 
         <record id="trans_approved_suspended" model="workflow.transition">
             <field name="act_from" ref="act_approved"/>
             <field name="act_to" ref="act_suspended"/>
             <field name="signal">button_suspended</field>
-            <field name="role_id" ref="PE_ADMIN"/>
         </record>
         <record id="trans_suspended_reapproved" model="workflow.transition">
             <field name="act_from" ref="act_suspended"/>
             <field name="act_to" ref="act_draft"/>
             <field name="signal">get_reapprove</field>
-            <field name="role_id" ref="PE_ADMIN"/>
         </record>
         <record id="trans_suspended_dummy" model="workflow.transition">
             <field name="act_from" ref="act_suspended"/>
             <field name="act_to" ref="act_dummy"/>
             <field name="signal">get_never</field>
-            <field name="role_id" ref="PE_ADMIN"/>
         </record>
     </data>
 </openerp>
diff --git a/addons/hr_expense/hr_expense_workflow.xml b/addons/hr_expense/hr_expense_workflow.xml
index cfeee8387ce..2bb0f51e26b 100644
--- a/addons/hr_expense/hr_expense_workflow.xml
+++ b/addons/hr_expense/hr_expense_workflow.xml
@@ -75,14 +75,12 @@
             <field name="act_from" ref="act_confirm"/>
             <field name="act_to" ref="act_accepted"/>
             <field name="signal">validate</field>
-            <field name="role_id" ref="HR"/>
         </record>
 
         <record id="t4" model="workflow.transition">
             <field name="act_from" ref="act_confirm"/>
             <field name="act_to" ref="act_refused"/>
             <field name="signal">refuse</field>
-            <field name="role_id" ref="HR"/>
         </record>
 
         <record id="t5" model="workflow.transition">
@@ -95,21 +93,18 @@
             <field name="act_from" ref="act_accepted"/>
             <field name="act_to" ref="act_refused"/>
             <field name="signal">refuse</field>
-            <field name="role_id" ref="HR"/>
         </record>
 
         <record id="t7" model="workflow.transition">
             <field name="act_from" ref="act_confirm"/>
             <field name="act_to" ref="act_draft"/>
             <field name="signal">draft</field>
-            <field name="role_id" ref="HR"/>
         </record>
 
         <record id="t8" model="workflow.transition">
             <field name="act_from" ref="act_accepted"/>
             <field name="act_to" ref="act_invoice"/>
             <field name="signal">invoice</field>
-            <field name="role_id" ref="HR_INV"/>
         </record>
 
         <record id="t9" model="workflow.transition">
@@ -128,7 +123,6 @@
             <field name="act_from" ref="act_refused"/>
             <field name="act_to" ref="act_draft"/>
             <field name="signal">draft</field>
-            <field name="role_id" ref="HR"/>
         </record>
 
     </data>
diff --git a/addons/hr_holidays/hr_holidays_workflow.xml b/addons/hr_holidays/hr_holidays_workflow.xml
index 6dcd55ff162..de2bd9f1131 100644
--- a/addons/hr_holidays/hr_holidays_workflow.xml
+++ b/addons/hr_holidays/hr_holidays_workflow.xml
@@ -77,28 +77,24 @@
         <field name="act_to" ref="act_validate" />
         <field name="signal">validate</field>
         <field name="condition">(holiday_status_id.double_validation == False)</field>
-        <field name="role_id" ref="HR_holidays"/>
     </record>
 
     <record model="workflow.transition" id="t3">
         <field name="act_from" ref="act_confirm" />
         <field name="act_to" ref="act_refuse" />
         <field name="signal">refuse</field>
-        <field name="role_id" ref="HR_holidays"/>
     </record>
 
     <record model="workflow.transition" id="t4">
         <field name="act_from" ref="act_validate" />
         <field name="act_to" ref="act_cancel" />
         <field name="signal">cancel</field>
-        <field name="role_id" ref="HR_holidays"/>
     </record>
 
     <record model="workflow.transition" id="t5">
         <field name="act_from" ref="act_refuse" />
         <field name="act_to" ref="act_cancel" />
         <field name="signal">cancel</field>
-        <field name="role_id" ref="HR_holidays"/>
     </record>
 
     <record model="workflow.transition" id="t7">
@@ -124,7 +120,6 @@
         <field name="act_to" ref="act_validate1" />
         <field name="condition">(holiday_status_id.double_validation == True)</field>
         <field name="signal">validate</field>
-        <field name="role_id" ref="HR_holidays"/>
     </record>
 
 </data>
diff --git a/addons/hr_timesheet_sheet/hr_timesheet_workflow.xml b/addons/hr_timesheet_sheet/hr_timesheet_workflow.xml
index 11af4574a4a..b664fd79c96 100644
--- a/addons/hr_timesheet_sheet/hr_timesheet_workflow.xml
+++ b/addons/hr_timesheet_sheet/hr_timesheet_workflow.xml
@@ -1,21 +1,21 @@
 <?xml version="1.0" encoding="utf-8"?>
 <openerp>
     <data>
-        
+
         <!-- Roles definition -->
-        
+
         <record id="HR" model="res.roles">
             <field name="name">Timesheets validation</field>
         </record>
-        
+
         <!-- Workflow definition -->
-        
+
         <record id="wkf_timesheet" model="workflow">
             <field name="name">hr_timesheet_sheet.sheet</field>
             <field name="osv">hr_timesheet_sheet.sheet</field>
             <field name="on_create">True</field>
         </record>
-        
+
         <record id="act_new" model="workflow.activity">
             <field name="wkf_id" ref="wkf_timesheet"/>
             <field name="name">new</field>
@@ -42,7 +42,7 @@
             <field name="action">write({'state':'done'})</field>
             <field name="flow_stop">True</field>
         </record>
-        
+
         <record id="t0" model="workflow.transition">
             <field name="act_from" ref="act_new"/>
             <field name="act_to" ref="act_draft"/>
@@ -56,13 +56,11 @@
             <field name="act_from" ref="act_confirm"/>
             <field name="act_to" ref="act_done"/>
             <field name="signal">done</field>
-            <field name="role_id" ref="HR"/>
         </record>
         <record id="t3" model="workflow.transition">
             <field name="act_from" ref="act_confirm"/>
             <field name="act_to" ref="act_draft"/>
             <field name="signal">cancel</field>
-            <field name="role_id" ref="HR"/>
         </record>
     </data>
 </openerp>
diff --git a/addons/hr_timesheet_sheet/process/hr_timesheet_sheet_process.xml b/addons/hr_timesheet_sheet/process/hr_timesheet_sheet_process.xml
index a2bf0aa24e5..22453dccfa4 100644
--- a/addons/hr_timesheet_sheet/process/hr_timesheet_sheet_process.xml
+++ b/addons/hr_timesheet_sheet/process/hr_timesheet_sheet_process.xml
@@ -1,21 +1,21 @@
 <?xml version="1.0" ?>
 <openerp>
     <data>
-        
-        <!-- 
+
+        <!--
             Process
         -->
-        
+
         <record id="process_process_hrtimesheetprocess0" model="process.process">
             <field eval="1" name="active"/>
             <field name="model_id" ref="hr_timesheet_sheet.model_hr_timesheet_sheet_sheet"/>
             <field eval="&quot;&quot;&quot;Hr Timesheet&quot;&quot;&quot;" name="name"/>
         </record>
-        
-        <!-- 
+
+        <!--
             Process Node
         -->
-    
+
         <record id="process_node_attendance0" model="process.node">
             <field name="model_id" ref="hr.model_hr_employee"/>
             <field eval="&quot;&quot;&quot;state&quot;&quot;&quot;" name="kind"/>
@@ -24,7 +24,7 @@
             <field name="process_id" ref="process_process_hrtimesheetprocess0"/>
             <field eval="1" name="flow_start"/>
         </record>
-    
+
         <record id="process_node_timesheet0" model="process.node">
             <field name="menu_id" ref="hr_timesheet_sheet.menu_act_hr_timesheet_sheet_form"/>
             <field name="model_id" ref="hr_timesheet_sheet.model_hr_timesheet_sheet_sheet"/>
@@ -35,7 +35,7 @@
             <field eval="&quot;&quot;&quot;object.state=='draft'&quot;&quot;&quot;" name="model_states"/>
             <field eval="0" name="flow_start"/>
         </record>
-    
+
         <record id="process_node_drafttimesheetsheet0" model="process.node">
             <field name="menu_id" ref="hr_timesheet_sheet.menu_act_hr_timesheet_sheet_form"/>
             <field name="model_id" ref="hr_timesheet_sheet.model_hr_timesheet_sheet_sheet"/>
@@ -46,7 +46,7 @@
             <field eval="&quot;&quot;&quot;object.state=='draft'&quot;&quot;&quot;" name="model_states"/>
             <field eval="0" name="flow_start"/>
         </record>
-    
+
         <record id="process_node_confirmedtimesheet0" model="process.node">
             <field name="menu_id" ref="hr_timesheet_sheet.menu_act_hr_timesheet_sheet_form"/>
             <field name="model_id" ref="hr_timesheet_sheet.model_hr_timesheet_sheet_sheet"/>
@@ -68,7 +68,7 @@
             <field eval="&quot;&quot;&quot;object.state=='done'&quot;&quot;&quot;" name="model_states"/>
             <field eval="0" name="flow_start"/>
         </record>
-    
+
         <record id="process_node_invoiceonwork0" model="process.node">
             <field name="menu_id" ref="account.menu_finance_receivables"/>
             <field name="model_id" ref="account.model_account_invoice"/>
@@ -91,19 +91,15 @@
             <field eval="&quot;&quot;&quot;object.state in ('open', pending', 'done', 'cancelled')&quot;&quot;&quot;" name="model_states"/>
             <field eval="1" name="flow_start"/>
         </record>
-        
-        <!-- 
+
+        <!--
             Process Transition
         -->
-        
+
         <record id="res_roles_hr0" model="res.roles">
             <field eval="&quot;&quot;&quot;HR&quot;&quot;&quot;" name="name"/>
         </record>
-        
-        <record id="hr_timesheet_sheet.t1" model="workflow.transition">
-            <field name="role_id" ref="res_roles_hr0"/>
-        </record>
-    
+
         <record id="process_transition_attendancetimesheet0" model="process.transition">
             <field eval="[(6,0,[])]" name="role_ids"/>
             <field eval="[(6,0,[])]" name="transition_ids"/>
@@ -112,7 +108,7 @@
             <field model="process.node" name="target_node_id" ref="process_node_drafttimesheetsheet0"/>
             <field model="process.node" name="source_node_id" ref="process_node_attendance0"/>
         </record>
-    
+
         <record id="process_transition_timesheetdraft0" model="process.transition">
             <field eval="[(6,0,[])]" name="role_ids"/>
             <field eval="[(6,0,[])]" name="transition_ids"/>
@@ -121,7 +117,7 @@
             <field model="process.node" name="target_node_id" ref="process_node_drafttimesheetsheet0"/>
             <field model="process.node" name="source_node_id" ref="process_node_timesheet0"/>
         </record>
-    
+
         <record id="process_transition_confirmtimesheet0" model="process.transition">
             <field eval="[(6,0,[])]" name="role_ids"/>
             <field eval="[(6,0,[])]" name="transition_ids"/>
@@ -131,7 +127,7 @@
             <field model="process.node" name="source_node_id" ref="process_node_drafttimesheetsheet0"/>
             <field eval="[(6,0,[ref('hr_timesheet_sheet.t1')])]" name="transition_ids"/>
         </record>
-    
+
         <record id="process_transition_validatetimesheet0" model="process.transition">
             <field eval="[(6,0,[])]" name="role_ids"/>
             <field eval="[(6,0,[])]" name="transition_ids"/>
@@ -140,7 +136,7 @@
             <field model="process.node" name="target_node_id" ref="process_node_validatedtimesheet0"/>
             <field model="process.node" name="source_node_id" ref="process_node_confirmedtimesheet0"/>
         </record>
-    
+
         <record id="process_transition_invoiceontimesheet0" model="process.transition">
             <field eval="[(6,0,[])]" name="role_ids"/>
             <field eval="[(6,0,[])]" name="transition_ids"/>
@@ -149,7 +145,7 @@
             <field model="process.node" name="target_node_id" ref="process_node_invoiceonwork0"/>
             <field model="process.node" name="source_node_id" ref="process_node_confirmedtimesheet0"/>
         </record>
-        
+
         <record id="process_transition_tasktimesheet0" model="process.transition">
             <field eval="[(6,0,[])]" name="role_ids"/>
             <field eval="[(6,0,[])]" name="transition_ids"/>
@@ -157,32 +153,32 @@
             <field eval="&quot;&quot;&quot;Moves task entry into the timesheet line&quot;&quot;&quot;" name="note"/>
             <field model="process.node" name="target_node_id" ref="process_node_timesheet0"/>
             <field model="process.node" name="source_node_id" ref="process_node_workontask0"/>
-         </record>        
-        
-        <!-- 
+         </record>
+
+        <!--
             Process Action
         -->
-        
+
         <record id="process_transition_action_draftconfirmtimesheet0" model="process.transition.action">
             <field eval="&quot;&quot;&quot;button_confirm&quot;&quot;&quot;" name="action"/>
             <field eval="&quot;&quot;&quot;object&quot;&quot;&quot;" name="state"/>
             <field eval="&quot;&quot;&quot;Confirm&quot;&quot;&quot;" name="name"/>
             <field name="transition_id" ref="process_transition_confirmtimesheet0"/>
         </record>
-        
+
         <record id="process_transition_action_validatetimesheet0" model="process.transition.action">
             <field eval="&quot;&quot;&quot;write({'state':'done'})&quot;&quot;&quot;" name="action"/>
             <field eval="&quot;&quot;&quot;object&quot;&quot;&quot;" name="state"/>
             <field eval="&quot;&quot;&quot;Validate&quot;&quot;&quot;" name="name"/>
             <field name="transition_id" ref="process_transition_validatetimesheet0"/>
         </record>
-        
+
         <record id="process_transition_action_refusetimesheet0" model="process.transition.action">
             <field eval="&quot;&quot;&quot;write({'state':'draft'})&quot;&quot;&quot;" name="action"/>
             <field eval="&quot;&quot;&quot;object&quot;&quot;&quot;" name="state"/>
             <field eval="&quot;&quot;&quot;Refuse&quot;&quot;&quot;" name="name"/>
             <field name="transition_id" ref="process_transition_validatetimesheet0"/>
         </record>
-        
+
     </data>
 </openerp>
diff --git a/addons/sale/process/sale_process.xml b/addons/sale/process/sale_process.xml
index b4a54d5afa5..4d6ca31d973 100644
--- a/addons/sale/process/sale_process.xml
+++ b/addons/sale/process/sale_process.xml
@@ -138,14 +138,6 @@
             <field eval="&quot;&quot;&quot;Salesman&quot;&quot;&quot;" name="name"/>
         </record>
 
-        <record id="sale.trans_draft_router" model="workflow.transition">
-            <field name="role_id" ref="res_roles_salesman0"/>
-        </record>
-
-        <record id="sale.trans_wait_invoice_invoice_manual" model="workflow.transition">
-            <field name="role_id" ref="res_roles_salesman0"/>
-        </record>
-
         <record id="process_transition_confirmquotation0" model="process.transition">
             <field eval="[(6,0,[])]" name="transition_ids"/>
             <field eval="&quot;&quot;&quot;Confirm Quotation&quot;&quot;&quot;" name="name"/>

From 4a1157c0572d2f2e2feb43fd7e7dd7fa73f15ef0 Mon Sep 17 00:00:00 2001
From: Fabien Pinckaers <fp@tinyerp.com>
Date: Tue, 5 Oct 2010 09:54:18 +0200
Subject: [PATCH 02/10] [IMP] useability improvements

bzr revid: fp@tinyerp.com-20101005075418-0t33zy0trdmiwbfj
---
 addons/base_calendar/base_calendar_view.xml   |  4 +++-
 addons/crm/crm_lead_data.xml                  |  3 ++-
 addons/crm/crm_meeting_view.xml               |  3 ++-
 addons/crm/crm_opportunity.py                 |  9 ++++----
 addons/crm/crm_opportunity_view.xml           | 22 ++++++-------------
 addons/crm/crm_view.xml                       |  1 +
 addons/hr/hr_installer.xml                    |  2 +-
 addons/hr/installer.py                        |  3 +--
 addons/mail_gateway/mail_gateway_view.xml     | 21 +-----------------
 addons/marketing/marketing_view.xml           |  3 +--
 addons/marketing/marketing_wizard.py          |  4 +---
 addons/product/product_view.xml               |  3 ++-
 addons/profile_tools/installer.py             | 10 ++++-----
 addons/profile_tools/misc_tools_installer.xml |  5 ++---
 addons/project/installer.py                   | 10 +++------
 addons/project/project_installer.xml          |  6 ++---
 addons/project_timesheet/__openerp__.py       |  2 +-
 addons/sale_crm/wizard/crm_make_sale.py       |  3 ++-
 addons/sale_crm/wizard/crm_make_sale_view.xml |  9 ++++----
 19 files changed, 46 insertions(+), 77 deletions(-)

diff --git a/addons/base_calendar/base_calendar_view.xml b/addons/base_calendar/base_calendar_view.xml
index 2a0ed14080f..ba91c4b75dd 100644
--- a/addons/base_calendar/base_calendar_view.xml
+++ b/addons/base_calendar/base_calendar_view.xml
@@ -163,7 +163,8 @@
         <!-- Invitation menu -->
 
            <menuitem id="menu_attendee_invitations"
-            name="Event Invitations" parent="base.menu_calendar_configuration"
+            parent="base.menu_calendar_configuration"
+            groups="base.group_extended"
             sequence="10" action="action_view_attendee_form" />
 
         <!--        ALARM FORM VIEW-->
@@ -496,6 +497,7 @@
 
         <menuitem id="menu_events"
                 name="Events" parent="base.menu_calendar_configuration"
+                groups="base.group_extended"
                 sequence="5" action="action_view_event" />
 
      </data>
diff --git a/addons/crm/crm_lead_data.xml b/addons/crm/crm_lead_data.xml
index cb80a7d28e9..2a49e43e683 100644
--- a/addons/crm/crm_lead_data.xml
+++ b/addons/crm/crm_lead_data.xml
@@ -33,9 +33,10 @@
         </record>
 
         <record model="crm.case.stage" id="stage_lead5">
-            <field name="name">Win</field>
+            <field name="name">Won</field>
             <field eval="'100'" name="probability"/>
             <field eval="'5'" name="sequence"/>
+            <field eval="1" name="on_change"/>
             <field name="object_id" search="[('model','=','crm.lead')]" model="ir.model"/>
         </record>
 
diff --git a/addons/crm/crm_meeting_view.xml b/addons/crm/crm_meeting_view.xml
index 2821024bfbf..98b311592b7 100644
--- a/addons/crm/crm_meeting_view.xml
+++ b/addons/crm/crm_meeting_view.xml
@@ -13,7 +13,8 @@
         <field name="context">{'object_id':'crm.meeting'}</field>
     </record>
 
-     <menuitem action="crm_meeting_categ_action" name="Categories of Meeting"
+     <menuitem action="crm_meeting_categ_action"
+        groups="base.group_extended"
         id="menu_crm_case_meeting-act" parent="base.menu_calendar_configuration" sequence="1"/>
 
     <!-- CRM Meetings Form View -->
diff --git a/addons/crm/crm_opportunity.py b/addons/crm/crm_opportunity.py
index 771212a171d..2b0062eb9d5 100644
--- a/addons/crm/crm_opportunity.py
+++ b/addons/crm/crm_opportunity.py
@@ -85,10 +85,11 @@ class crm_opportunity(osv.osv):
         """
         res = super(crm_opportunity, self).case_close(cr, uid, ids, args)
         stage_id = super(crm_opportunity, self).stage_next(cr, uid, ids, context={'force_domain': [('probability', '=', 0)]})
-        if not stage_id:
-            raise osv.except_osv(_('Warning !'), _('There is no stage for lost opportunities defined for this Sale Team.'))
-        value = self.onchange_stage_id(cr, uid, ids, stage_id, context={})['value']
-        value.update({'date_closed': time.strftime('%Y-%m-%d %H:%M:%S'), 'stage_id': stage_id})
+        value = {}
+        if stage_id:
+            value = self.onchange_stage_id(cr, uid, ids, stage_id, context={}).get('value', {})
+            value['stage_id'] = stage_id
+        value.update({'date_closed': time.strftime('%Y-%m-%d %H:%M:%S')})
 
         res = self.write(cr, uid, ids, value)
         for (id, name) in self.name_get(cr, uid, ids):
diff --git a/addons/crm/crm_opportunity_view.xml b/addons/crm/crm_opportunity_view.xml
index b52694d8b20..66b6f04553b 100644
--- a/addons/crm/crm_opportunity_view.xml
+++ b/addons/crm/crm_opportunity_view.xml
@@ -14,14 +14,14 @@
                             <label string="Stage:" align="1.0"/>
                             <group colspan="1" col="4">
                                 <field name="stage_id" nolabel="1"
+                                    widget="selection"
                                     on_change="onchange_stage_id(stage_id)"
-                                    domain="[('object_id.model', '=', 'crm.lead')]"
-                                    readonly="1" />
+                                    domain="[('object_id.model', '=', 'crm.lead')]"/>
                                 <button name="stage_previous"
-                                states="open,pending" type="object"
-                                icon="gtk-go-back" string="" />
-                                <button name="stage_next" states="open,pending"
-                                type="object" icon="gtk-go-forward" string="" />
+                                    states="draft,open,pending" type="object"
+                                    icon="gtk-go-back" string="" />
+                                <button name="stage_next" states="draft,open,pending"
+                                    type="object" icon="gtk-go-forward" string="" />
                             </group>
                             <field name="user_id"/>
 
@@ -88,7 +88,7 @@
                                     groups="base.group_extended"
                                     icon="gtk-go-up" />
                                 <button name="case_cancel" string="Cancel"
-                                    states="draft,open,pending" type="object"
+                                    states="draft" type="object"
                                     icon="gtk-close" />
                                 <button name="case_mark_lost" string="Mark Lost"
                                     states="open,pending" type="object"
@@ -283,14 +283,6 @@
                             string="Pending"  help="Pending Opportunities"
                             domain="[('state','=','pending')]"/>
                         <separator orientation="vertical"/>
-                        <filter icon="terp-go-today" string="Today"
-                            domain="[('create_date','&lt;', time.strftime('%%Y-%%m-%%d 23:59:59')), ('create_date','&gt;=', time.strftime('%%Y-%%m-%%d 00:00:00'))]"
-                            help="Todays' Opportunities" />
-                        <filter icon="terp-go-week" string="7 Days"
-                            help="Opportunities created during last 7 days"
-                            domain="[('create_date','&lt;', time.strftime('%%Y-%%m-%%d 23:59:59')),('create_date','&gt;=',(datetime.date.today()-datetime.timedelta(days=7)).strftime('%%Y-%%m-%%d 00:00:00'))]"
-                        />
-                        <separator orientation="vertical"/>
                         <field name="name" string="Opportunity"/>
                         <field name="partner_id"/>
                         <field name="user_id">
diff --git a/addons/crm/crm_view.xml b/addons/crm/crm_view.xml
index bb15b3631b8..daafc19a621 100644
--- a/addons/crm/crm_view.xml
+++ b/addons/crm/crm_view.xml
@@ -202,6 +202,7 @@
 
          <menuitem action="crm_case_resource_type_act"
             id="menu_crm_case_resource_type_act" sequence="4"
+            groups="base.group_extended"
             parent="base.menu_sale_config_sales" />
 
         <record id="crm_case_section_act_tree" model="ir.actions.act_window">
diff --git a/addons/hr/hr_installer.xml b/addons/hr/hr_installer.xml
index 490523b4e12..0f396e65a8b 100644
--- a/addons/hr/hr_installer.xml
+++ b/addons/hr/hr_installer.xml
@@ -27,8 +27,8 @@
             <field name="hr_recruitment"/>
             <field name="hr_timesheet_sheet"/>
             <field name="hr_contract"/>
-            <field name="hr_evaluation"/>
             <field name="hr_attendance"/>
+            <field name="hr_evaluation" groups="base.group_extended"/>
             <field name="hr_payroll" groups="base.group_extended"/>
             <field name="hr_payroll_account" groups="base.group_extended"/>
           </group>
diff --git a/addons/hr/installer.py b/addons/hr/installer.py
index 1d650d814e9..1bd615b8699 100644
--- a/addons/hr/installer.py
+++ b/addons/hr/installer.py
@@ -51,8 +51,7 @@ class hr_installer(osv.osv_memory):
     _defaults = {
         'hr_holidays': True,
         'hr_expense': True,
-        }
-
+    }
 hr_installer()
 
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
diff --git a/addons/mail_gateway/mail_gateway_view.xml b/addons/mail_gateway/mail_gateway_view.xml
index 706996ed55a..ab118295db5 100644
--- a/addons/mail_gateway/mail_gateway_view.xml
+++ b/addons/mail_gateway/mail_gateway_view.xml
@@ -135,16 +135,6 @@
         </record>
 
          <!--        Emails action-->
-        <record model="ir.actions.act_window" id="action_view_mailgate_message">
-            <field name="name">Emails</field>
-            <field name="res_model">mailgate.message</field>
-            <field name="view_mode">tree,form</field>
-            <field name="view_type">form</field>
-            <field name="domain">[('history', '=', True)]</field>
-            <field name="view_id" ref="view_mailgate_message_tree"/>
-            <field name="context">{"search_default_user_id":uid}</field>
-        </record>
-
         <record model="ir.actions.act_window" id="action_view_mailgate_thread">
             <field name="name">Mailgateway Threads</field>
             <field name="res_model">mailgate.thread</field>
@@ -196,18 +186,9 @@
             view_id="view_mailgate_message_tree"
             />
 
-
-      <menuitem name="Emails" id="menu_base_config_mail"
-        parent="base.menu_base_partner" sequence="5" />
-
-    <menuitem name="Emails" id="menu_mailgate_message"
-        parent="menu_base_config_mail"
-        action="mail_gateway.action_view_mailgate_message"
-       />
-
     <act_window
        id="act_res_partner_open_email" name="Attachments"
-        res_model="ir.attachment"
+       res_model="ir.attachment"
        src_model="mailgate.message"
        domain="[('res_id', '=', res_id),('res_model','=',model)]"/>
 
diff --git a/addons/marketing/marketing_view.xml b/addons/marketing/marketing_view.xml
index b96ba8fb7e3..6cd35f3a533 100644
--- a/addons/marketing/marketing_view.xml
+++ b/addons/marketing/marketing_view.xml
@@ -29,10 +29,9 @@
                             <attribute name="string">Configure</attribute>
                   </xpath>
                   <group colspan="8">
+                        <field name="marketing_campaign" />
                         <field name="email_template" />
                         <field name="crm_profiling" />
-                        <field name="marketing_campaign" />
-                        <field name="marketing_campaign_mailchimp" />
                   </group>
                 </data>
             </field>
diff --git a/addons/marketing/marketing_wizard.py b/addons/marketing/marketing_wizard.py
index 8d1a6fa50ed..4b95bbb86da 100644
--- a/addons/marketing/marketing_wizard.py
+++ b/addons/marketing/marketing_wizard.py
@@ -29,10 +29,8 @@ class marketing_installer(osv.osv_memory):
             help="Helps you to design templates of emails and integrate them in your different processes."),
         'marketing_campaign':fields.boolean('Marketing Campaigns',
             help="Helps you to manage marketing campaigns and automate actions and communication steps."),
-        'marketing_campaign_mailchimp':fields.boolean('Mailchimp Integration',
-            help="This modules integrate mailchimp.com's service with OpenERP to automate mass mailings."),
         'crm_profiling':fields.boolean('Profiling Tools',
-            help="Helps you to perform segmentation within partners and design segmentation questionnaires")
+            help="Helps you to perform segmentation of partners and design segmentation questionnaires")
     }
 marketing_installer()
 
diff --git a/addons/product/product_view.xml b/addons/product/product_view.xml
index 45ebaa8ee57..7e9ae7ce45e 100644
--- a/addons/product/product_view.xml
+++ b/addons/product/product_view.xml
@@ -257,11 +257,12 @@
             <field name="domain">[('parent_id','=',False)]</field>
             <field name="view_type">tree</field>
             <field name="view_id" ref="product_category_tree_view"/>
-            <field name="help">Products category opens a tree structure view of your products by category.</field>
+            <field name="help">This is a hyerarchical structure of your product's categories, you can enter a category to browse all products in this category and his children.</field>
         </record>
         <menuitem
             action="product_category_action"
             id="product.menu_products_category"
+            groups="base.group_extended"
             parent="base.menu_product"
             sequence="0"/>
         <record id="product_category_action_form" model="ir.actions.act_window">
diff --git a/addons/profile_tools/installer.py b/addons/profile_tools/installer.py
index 06be50a903f..54fd0602802 100644
--- a/addons/profile_tools/installer.py
+++ b/addons/profile_tools/installer.py
@@ -25,15 +25,13 @@ class misc_tools_installer(osv.osv_memory):
     _inherit = 'res.config.installer'
 
     _columns = {
-        'lunch':fields.boolean('Lunch',help='Helps to manage Lunch Orders.'),
-        'subscription':fields.boolean('Recurring Documents',help='Helps to add subscription on documents.'),
-        'survey':fields.boolean('Survey',help='Manages Custom Surveys.'),
-        'idea':fields.boolean('Idea',help='Manages ideas and votes'),
-        'audittrail':fields.boolean('Audit Trail',help="Lets you to track user's operations on specific Objects."),
+        'lunch':fields.boolean('Lunch',help='A simple module to help you to manage Lunch orders.'),
+        'subscription':fields.boolean('Recurring Documents',help='Helps to generate automatically recurring documents.'),
+        'survey':fields.boolean('Survey',help='Allows you to organize surveys.'),
+        'idea':fields.boolean('Ideas Box',help='Promote ideas of the employees, votes and discussion on best ideas.'),
     }
     _defaults = {
         'lunch': True,
     }
-
 misc_tools_installer()
 
diff --git a/addons/profile_tools/misc_tools_installer.xml b/addons/profile_tools/misc_tools_installer.xml
index c9b5f744b77..bc445e7818f 100644
--- a/addons/profile_tools/misc_tools_installer.xml
+++ b/addons/profile_tools/misc_tools_installer.xml
@@ -26,10 +26,9 @@
           </xpath>
           <group colspan="8">
             <field name="lunch"/>
-            <field name="subscription"/>
-            <field name="survey"/>
             <field name="idea"/>
-            <field name="audittrail"/>
+            <field name="survey"/>
+            <field name="subscription" groups="base.group_extended"/>
           </group>
         </data>
       </field>
diff --git a/addons/project/installer.py b/addons/project/installer.py
index 4662ebeda5d..7a147737768 100644
--- a/addons/project/installer.py
+++ b/addons/project/installer.py
@@ -34,14 +34,10 @@ class project_installer(osv.osv_memory):
         'hr_timesheet_sheet': fields.boolean('Timesheets',
             help="Tracks and helps employees encode and validate timesheets "
                  "and attendances."),
-        'hr_timesheet_invoice': fields.boolean('Invoice Based on Hours',
-            help="Helps generate invoices based on human resources "
-                 "costs and general expenses."),
+        'project_timesheet': fields.boolean('Bill Time on Tasks',
+            help="Helps generate invoices based on time spent on tasks, if activated on the project."),
         'account_budget': fields.boolean('Budgets',
             help="Helps accountants manage analytic and crossover budgets."),
-        'project_messages': fields.boolean('Project Messages',
-            help="Lets employees send messages to other members of the "
-                 "projects they're working on."),
         'project_issue': fields.boolean('Issues Tracker',
             help="Automatically synchronizes project tasks and crm cases."),
         # Methodologies
@@ -50,7 +46,7 @@ class project_installer(osv.osv_memory):
                  "in the SCRUM methodology."),
         'project_gtd': fields.boolean('Getting Things Done',
             help="GTD is a methodology to efficiently organise yourself and your tasks. This module fully integrates GTD principle with OpenERP's project management."),
-        }
+    }
 
     _defaults={
         'project_issue': True,
diff --git a/addons/project/project_installer.xml b/addons/project/project_installer.xml
index 2c22a4b6743..2b46cc0ab5f 100644
--- a/addons/project/project_installer.xml
+++ b/addons/project/project_installer.xml
@@ -27,11 +27,11 @@
             <field name="project_issue"/>
             <field name="project_long_term" groups="base.group_extended"/>
             <field name="hr_timesheet_sheet"/>
-            <field name="hr_timesheet_invoice"/>
+            <field name="project_timesheet"/>
             <field name="account_budget" groups="base.group_extended"/>
-            <field name="project_messages" groups="base.group_extended"/>
+            <newline/>
             <separator string="Methodologies" colspan="4"/>
-            <field name="project_scrum" groups="base.group_extended"/>
+            <field name="project_scrum"/>
             <field name="project_gtd"/>
           </group>
         </data>
diff --git a/addons/project_timesheet/__openerp__.py b/addons/project_timesheet/__openerp__.py
index 021d30da687..e1be86fe661 100644
--- a/addons/project_timesheet/__openerp__.py
+++ b/addons/project_timesheet/__openerp__.py
@@ -30,7 +30,7 @@
     """,
     'author': 'OpenERP SA',
     'website': 'http://www.openerp.com',
-    'depends': ['base', 'project', 'hr_timesheet_sheet', 'hr_timesheet_invoice'],
+    'depends': ['project', 'hr_timesheet_sheet', 'hr_timesheet_invoice'],
     'init_xml': [],
     'update_xml': ["security/ir.model.access.csv","process/project_timesheet_process.xml", "report/task_report_view.xml", "project_timesheet_view.xml"],
     'demo_xml': [],
diff --git a/addons/sale_crm/wizard/crm_make_sale.py b/addons/sale_crm/wizard/crm_make_sale.py
index ca6c8cb64c8..089a2389703 100644
--- a/addons/sale_crm/wizard/crm_make_sale.py
+++ b/addons/sale_crm/wizard/crm_make_sale.py
@@ -180,10 +180,11 @@ class crm_make_sale(osv.osv_memory):
         'partner_id': fields.many2one('res.partner', 'Customer', required=True), 
         'sale_order_line': fields.one2many('sale.order.make.line', 'opportunity_order_id', 'Product Line'), 
         'analytic_account': fields.many2one('account.analytic.account', 'Analytic Account'), 
-        'close': fields.boolean('Close Case', help='Check this to close the case after having created the sale order.'), 
+        'close': fields.boolean('Close Opportunity', help='Check this to close the opportunity after having created the sale order.'), 
     }
     _defaults = {
          'shop_id': _get_shop_id, 
+         'close': lambda *args: 1, 
          'partner_id': _selectPartner, 
     }
 
diff --git a/addons/sale_crm/wizard/crm_make_sale_view.xml b/addons/sale_crm/wizard/crm_make_sale_view.xml
index c6b589145c7..54550a08bf1 100644
--- a/addons/sale_crm/wizard/crm_make_sale_view.xml
+++ b/addons/sale_crm/wizard/crm_make_sale_view.xml
@@ -9,13 +9,13 @@
           <field name="type">form</field>
           <field name="arch" type="xml">
               <form string="Create a Sale Order" >
-              		<group width="660" height="300">
                     <group colspan="4">
                         <field name="partner_id" required="1"/>
                         <field name="shop_id" required="1" widget="selection"/>
-                        <field name="analytic_account" domain="[('parent_id','!=',False)]" />
-                        <field name="close" required="1"/>
-                    	</group>
+                        <field name="analytic_account" domain="[('type','=','normal')]"
+                            groups="base.group_extended"/>
+                        <field name="close"/>
+                    </group>
                             <field colspan="4" mode="tree,form,graph" name="sale_order_line" nolabel="1" widget="one2many_list">
                                 <form string="Sale Order Lines">
                                     <notebook>
@@ -65,7 +65,6 @@
                     <button special="cancel" string="_Close" icon="gtk-cancel"/>
                     <button name="makeOrder" string="_Create" type="object" icon='gtk-ok'/>
                 </group>
-                </group>
             </form>
           </field>
        </record>

From 4988819428f5a692a216264561c8a2b5fa975e27 Mon Sep 17 00:00:00 2001
From: Fabien Pinckaers <fp@tinyerp.com>
Date: Tue, 5 Oct 2010 09:59:22 +0200
Subject: [PATCH 03/10] [IMP] removed board on users

bzr revid: fp@tinyerp.com-20101005075922-d0n6h4g0kz25p0bn
---
 addons/account/__openerp__.py           |  1 -
 addons/account/board_account_demo.xml   | 13 -------------
 addons/crm/__openerp__.py               |  1 -
 addons/crm/board_crm_demo.xml           | 14 --------------
 addons/event/__openerp__.py             |  2 +-
 addons/event/board_association_demo.xml | 10 ----------
 addons/mrp/__openerp__.py               |  1 -
 addons/mrp/board_manufacturing_demo.xml |  9 ---------
 addons/project/board_project_demo.xml   |  9 +++------
 9 files changed, 4 insertions(+), 56 deletions(-)
 delete mode 100644 addons/account/board_account_demo.xml
 delete mode 100644 addons/crm/board_crm_demo.xml
 delete mode 100644 addons/event/board_association_demo.xml
 delete mode 100644 addons/mrp/board_manufacturing_demo.xml

diff --git a/addons/account/__openerp__.py b/addons/account/__openerp__.py
index 823bb56a7a9..271cc92e41c 100644
--- a/addons/account/__openerp__.py
+++ b/addons/account/__openerp__.py
@@ -120,7 +120,6 @@ module named account_voucher.
         'project/analytic_account_demo.xml',
         'demo/account_minimal.xml',
         'account_unit_test.xml',
-        'board_account_demo.xml',
     ],
     'test': [
         'test/account_customer_invoice.yml',
diff --git a/addons/account/board_account_demo.xml b/addons/account/board_account_demo.xml
deleted file mode 100644
index 93bfed6d56d..00000000000
--- a/addons/account/board_account_demo.xml
+++ /dev/null
@@ -1,13 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<openerp>
-    <data noupdate="1">
-        <!--
-		Administrator shortcut
-		Demo user startup menu
-		-->
-        <record id="base.user_root" model="res.users">
-            <field name="action_id" ref="open_board_account"/>
-        </record>
-
-    </data>
-</openerp>
diff --git a/addons/crm/__openerp__.py b/addons/crm/__openerp__.py
index 5de8de6de68..89dc61bef59 100644
--- a/addons/crm/__openerp__.py
+++ b/addons/crm/__openerp__.py
@@ -119,7 +119,6 @@ Create dashboard for CRM that includes:
         'crm_meeting_demo.xml',
         'crm_opportunity_demo.xml',
         'crm_phonecall_demo.xml',
-        'board_crm_demo.xml',
     ],
     'test': [
             'test/test_crm_lead.yml',
diff --git a/addons/crm/board_crm_demo.xml b/addons/crm/board_crm_demo.xml
deleted file mode 100644
index fd8bb480d68..00000000000
--- a/addons/crm/board_crm_demo.xml
+++ /dev/null
@@ -1,14 +0,0 @@
-<?xml version="1.0"?>
-<openerp>
-    <data noupdate="1">
-
-        <!--
-         Administrator shortcut
-         Demo user startup menu
-         -->
-        <record model="res.users" id="base.user_root">
-            <field name="action_id" ref="open_board_crm"/>
-        </record>
-
-    </data>
-</openerp>
diff --git a/addons/event/__openerp__.py b/addons/event/__openerp__.py
index fdf64388a19..4e65009f422 100644
--- a/addons/event/__openerp__.py
+++ b/addons/event/__openerp__.py
@@ -53,7 +53,7 @@
         'board_association_view.xml',
         'res_partner_view.xml',
     ],
-    'demo_xml': ['event_demo.xml', 'board_association_demo.xml'],
+    'demo_xml': ['event_demo.xml'],
     'test': ['test/test_event.yml'],
     'installable': True,
     'active': False,
diff --git a/addons/event/board_association_demo.xml b/addons/event/board_association_demo.xml
deleted file mode 100644
index 1e66696aa8b..00000000000
--- a/addons/event/board_association_demo.xml
+++ /dev/null
@@ -1,10 +0,0 @@
-<?xml version="1.0"?>
-<openerp>
-	<data noupdate="1">
-
-		<record model="res.users" id="base.user_root">
-			<field name="action_id" ref="open_board_associations_manager"/>
-		</record>
-	</data>
-</openerp>
-
diff --git a/addons/mrp/__openerp__.py b/addons/mrp/__openerp__.py
index c56466501e0..f267764970a 100644
--- a/addons/mrp/__openerp__.py
+++ b/addons/mrp/__openerp__.py
@@ -85,7 +85,6 @@
     ],
     'demo_xml': [
          'mrp_demo.xml',
-         'board_manufacturing_demo.xml'
     ],
     'test': [
          'test/mrp_procurement.yml',
diff --git a/addons/mrp/board_manufacturing_demo.xml b/addons/mrp/board_manufacturing_demo.xml
deleted file mode 100644
index af8f07d2871..00000000000
--- a/addons/mrp/board_manufacturing_demo.xml
+++ /dev/null
@@ -1,9 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<openerp>
-    <data noupdate="1">
-        <record id="base.user_root" model="res.users">
-            <field name="action_id" ref="open_board_manufacturing"/>
-        </record>
-
-    </data>
-</openerp>
diff --git a/addons/project/board_project_demo.xml b/addons/project/board_project_demo.xml
index 6a063405ff3..85bf3140d26 100644
--- a/addons/project/board_project_demo.xml
+++ b/addons/project/board_project_demo.xml
@@ -3,12 +3,9 @@
     <data noupdate="1">
 
         <!--
-		 Administrator shortcut
-		 Demo user startup menu
-		 -->
-        <record id="base.user_root" model="res.users">
-            <field name="action_id" ref="open_board_project"/>
-        </record>
+         Administrator shortcut
+         Demo user startup menu
+         -->
 
         <record id="note_project" model="board.note">
             <field name="name">Don't forget the new development policies.</field>

From f291106692e566de1e318e0807ae3e7128265d3c Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Tue, 5 Oct 2010 09:59:48 +0200
Subject: [PATCH 04/10] [IMP] mrp_repair: no need to rountrip via string to get
 today's datetime

bzr revid: xmo@openerp.com-20101005075948-1rq455369nr0rz6b
---
 addons/mrp_repair/mrp_repair.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addons/mrp_repair/mrp_repair.py b/addons/mrp_repair/mrp_repair.py
index 6cec29650f3..010cb725c5c 100644
--- a/addons/mrp_repair/mrp_repair.py
+++ b/addons/mrp_repair/mrp_repair.py
@@ -687,7 +687,7 @@ class mrp_repair_line(osv.osv, ProductChangeMixin):
         if type == 'add':
             stock_id = self.pool.get('stock.location').search(cr, uid, [('name','=','Stock')])[0]
             to_invoice = False
-            if guarantee_limit and datetime.strptime(date.today().strftime('%Y-%m-%d'), '%Y-%m-%d') > datetime.strptime(guarantee_limit, '%Y-%m-%d'):
+            if guarantee_limit and datetime.now() > datetime.strptime(guarantee_limit, '%Y-%m-%d'):
                 to_invoice=True
 
             return {'value': {

From a392e830d7a4621b7d4eec6941bcba6c41de70d9 Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Tue, 5 Oct 2010 10:00:13 +0200
Subject: [PATCH 05/10] [IMP] mrp_repair: reverse boolean test to reduce
 nesting

bzr revid: xmo@openerp.com-20101005080013-7ssjz1tnlqe0cqdz
---
 addons/mrp_repair/mrp_repair.py | 24 +++++++++++++-----------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/addons/mrp_repair/mrp_repair.py b/addons/mrp_repair/mrp_repair.py
index 010cb725c5c..6fe86fd3f2b 100644
--- a/addons/mrp_repair/mrp_repair.py
+++ b/addons/mrp_repair/mrp_repair.py
@@ -684,22 +684,24 @@ class mrp_repair_line(osv.osv, ProductChangeMixin):
                     }
             }
         produc_id = self.pool.get('stock.location').search(cr, uid, [('name','=','Production')])[0]
-        if type == 'add':
-            stock_id = self.pool.get('stock.location').search(cr, uid, [('name','=','Stock')])[0]
-            to_invoice = False
-            if guarantee_limit and datetime.now() > datetime.strptime(guarantee_limit, '%Y-%m-%d'):
-                to_invoice=True
 
+        if type != 'add':
             return {'value': {
-                        'to_invoice': to_invoice,
-                        'location_id': stock_id,
-                        'location_dest_id': produc_id
-                    }
-            }
-        return {'value': {
                 'to_invoice': False,
                 'location_id': produc_id,
                 'location_dest_id': False
+                }
+            }
+
+        stock_id = self.pool.get('stock.location').search(cr, uid, [('name','=','Stock')])[0]
+        to_invoice = False
+        if guarantee_limit and datetime.now() > datetime.strptime(guarantee_limit, '%Y-%m-%d'):
+            to_invoice=True
+
+        return {'value': {
+            'to_invoice': to_invoice,
+            'location_id': stock_id,
+            'location_dest_id': produc_id
             }
         }
 

From be1a8c1e604749abf7ee3480839b37552209b85e Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Tue, 5 Oct 2010 10:01:07 +0200
Subject: [PATCH 06/10] [FIX] mrp_repair: typo in variable name

bzr revid: xmo@openerp.com-20101005080107-reomihila8y8g9uj
---
 addons/mrp_repair/mrp_repair.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/addons/mrp_repair/mrp_repair.py b/addons/mrp_repair/mrp_repair.py
index 6fe86fd3f2b..a4bfe9457e7 100644
--- a/addons/mrp_repair/mrp_repair.py
+++ b/addons/mrp_repair/mrp_repair.py
@@ -683,12 +683,12 @@ class mrp_repair_line(osv.osv, ProductChangeMixin):
                         'location_dest_id': False
                     }
             }
-        produc_id = self.pool.get('stock.location').search(cr, uid, [('name','=','Production')])[0]
+        product_id = self.pool.get('stock.location').search(cr, uid, [('name','=','Production')])[0]
 
         if type != 'add':
             return {'value': {
                 'to_invoice': False,
-                'location_id': produc_id,
+                'location_id': product_id,
                 'location_dest_id': False
                 }
             }
@@ -701,7 +701,7 @@ class mrp_repair_line(osv.osv, ProductChangeMixin):
         return {'value': {
             'to_invoice': to_invoice,
             'location_id': stock_id,
-            'location_dest_id': produc_id
+            'location_dest_id': product_id
             }
         }
 

From 14559bb42e5d4a48ee76dc88f3c539fec5ac620f Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Tue, 5 Oct 2010 10:03:12 +0200
Subject: [PATCH 07/10] [IMP] mrp_repair: simplify boolean temp variable
 setting

bzr revid: xmo@openerp.com-20101005080312-h3nzrhvjogoc7o6q
---
 addons/mrp_repair/mrp_repair.py | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/addons/mrp_repair/mrp_repair.py b/addons/mrp_repair/mrp_repair.py
index a4bfe9457e7..1a9705ce122 100644
--- a/addons/mrp_repair/mrp_repair.py
+++ b/addons/mrp_repair/mrp_repair.py
@@ -683,8 +683,8 @@ class mrp_repair_line(osv.osv, ProductChangeMixin):
                         'location_dest_id': False
                     }
             }
-        product_id = self.pool.get('stock.location').search(cr, uid, [('name','=','Production')])[0]
 
+        product_id = self.pool.get('stock.location').search(cr, uid, [('name','=','Production')])[0]
         if type != 'add':
             return {'value': {
                 'to_invoice': False,
@@ -694,10 +694,8 @@ class mrp_repair_line(osv.osv, ProductChangeMixin):
             }
 
         stock_id = self.pool.get('stock.location').search(cr, uid, [('name','=','Stock')])[0]
-        to_invoice = False
-        if guarantee_limit and datetime.now() > datetime.strptime(guarantee_limit, '%Y-%m-%d'):
-            to_invoice=True
-
+        to_invoice = (guarantee_limit and
+                      datetime.strptime(guarantee_limit, '%Y-%m-%d') < datetime.now()):
         return {'value': {
             'to_invoice': to_invoice,
             'location_id': stock_id,

From 2300f9b49d9fe3a432bb90201ba4d4f7099949ec Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Tue, 5 Oct 2010 10:05:02 +0200
Subject: [PATCH 08/10] [IMP] mrp_repair: make indentation of return dicts
 coherent in onchange_operation_type

bzr revid: xmo@openerp.com-20101005080502-3lehfh2fk7dn5vw5
---
 addons/mrp_repair/mrp_repair.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/addons/mrp_repair/mrp_repair.py b/addons/mrp_repair/mrp_repair.py
index 1a9705ce122..af44f19052a 100644
--- a/addons/mrp_repair/mrp_repair.py
+++ b/addons/mrp_repair/mrp_repair.py
@@ -679,9 +679,9 @@ class mrp_repair_line(osv.osv, ProductChangeMixin):
         """
         if not type:
             return {'value': {
-                        'location_id': False,
-                        'location_dest_id': False
-                    }
+                'location_id': False,
+                'location_dest_id': False
+                }
             }
 
         product_id = self.pool.get('stock.location').search(cr, uid, [('name','=','Production')])[0]

From 491585cfc27a31741c0fb89535f5b3537733209e Mon Sep 17 00:00:00 2001
From: "RME(OpenERP),JMA(OpenERP)" <>
Date: Tue, 5 Oct 2010 14:12:35 +0530
Subject: [PATCH 09/10] [FIX] account_payment : Missing reference of statement
 line variable fixed

bzr revid: ach@tinyerp.com-20101005084235-5x5wluw3jun63hn1
---
 .../wizard/account_payment_populate_statement.py                | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addons/account_payment/wizard/account_payment_populate_statement.py b/addons/account_payment/wizard/account_payment_populate_statement.py
index d223c58316f..7ebb9d933de 100644
--- a/addons/account_payment/wizard/account_payment_populate_statement.py
+++ b/addons/account_payment/wizard/account_payment_populate_statement.py
@@ -114,7 +114,7 @@ class account_payment_populate_statement(osv.osv_memory):
                     'voucher_id': voucher_id,
                     }, context=context)
 
-            line_obj.write(cr, uid, [line.id], {'bank_statement_line_id': st_line_id})
+                line_obj.write(cr, uid, [line.id], {'bank_statement_line_id': st_line_id})
         return {'type': 'ir.actions.act_window_close'}
 
 account_payment_populate_statement()

From c36a3d33908bf17dd4ade84985a9c7d7ee213456 Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Tue, 5 Oct 2010 11:10:22 +0200
Subject: [PATCH 10/10] [FIX] mrp_repair: syntax error typo

bzr revid: xmo@openerp.com-20101005091022-dmwju7zimmja2fzg
---
 addons/mrp_repair/mrp_repair.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addons/mrp_repair/mrp_repair.py b/addons/mrp_repair/mrp_repair.py
index af44f19052a..f7a6837aace 100644
--- a/addons/mrp_repair/mrp_repair.py
+++ b/addons/mrp_repair/mrp_repair.py
@@ -695,7 +695,7 @@ class mrp_repair_line(osv.osv, ProductChangeMixin):
 
         stock_id = self.pool.get('stock.location').search(cr, uid, [('name','=','Stock')])[0]
         to_invoice = (guarantee_limit and
-                      datetime.strptime(guarantee_limit, '%Y-%m-%d') < datetime.now()):
+                      datetime.strptime(guarantee_limit, '%Y-%m-%d') < datetime.now())
         return {'value': {
             'to_invoice': to_invoice,
             'location_id': stock_id,