Merge lp:~camptocamp/contract-management/hours_block_vre into lp:~contract-management-core-editors/contract-management/7.0

[Matthieu Dietrich @ camptocamp]

- a new field for the department, which is filterable/searchable,
- a new button in the form view in order to display the hours block related to a project.

[lp:~camptocamp/contract-management/hours_block_vre]

9. By Lorenzo Battistini on 2014-02-28

[FIX] Can't install analytic_hours_block

10. By Daniel Reis on 2014-02-28

New project_sla module

[Matthieu Dietrich @ camptocamp (mdietrich-c2c)]

To expand a bit: this MP is to add 2 things related to hours block:
- a new field for the department, which is filterable/searchable,
- a new button in the project form view, in order to get the hours block related to a project.

[Yannick Vaucher @ Camptocamp (yvaucher-c2c)]

If department is changed in invoice the related field won't be updated.

Please set the proper store triggers.

Cheers,

review: Needs Fixing

[lp:~camptocamp/contract-management/hours_block_vre]

11. By Matthieu Dietrich @ camptocamp on 2014-04-01

[FIX] fixed conflict with latest version + modified store conditions for related fields

[Matthieu Dietrich @ camptocamp (mdietrich-c2c)]

I fixed the store fields (not only department_id; other related() as well) to be refreshed if the distant field is modified.

Regards,
Matthieu

[Yannick Vaucher @ Camptocamp (yvaucher-c2c)]

Well done Thanks

LGTM

review: Approve (code review, no test)

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

42 + 'account.hours.block': (lambda self, cr, uid, ids, c=None:
43 + ids, ['invoice_id'], 10),

this is a very bad line split, please keep the 'ids' on the same line as the lambda

missing i18n update

review: Needs Fixing (code review, no test)

[lp:~camptocamp/contract-management/hours_block_vre]

12. By Matthieu Dietrich @ camptocamp on 2014-04-02

[FIX] reformatting

[Matthieu Dietrich @ camptocamp (mdietrich-c2c)]

@afayolle: thanks, I re-formatted it now.

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

review: Approve (code review, no test)

=== modified file 'analytic_hours_block/__init__.py'
--- analytic_hours_block/__init__.py	2012-12-17 12:31:50 +0000
+++ analytic_hours_block/__init__.py	2014-04-02 14:53:22 +0000
@@ -20,3 +20,4 @@
 ##############################################################################
 import hours_block
 import report
+import project
 \ No newline at end of file
 
=== modified file 'analytic_hours_block/__openerp__.py'
--- analytic_hours_block/__openerp__.py	2014-02-17 08:33:04 +0000
+++ analytic_hours_block/__openerp__.py	2014-04-02 14:53:22 +0000
@@ -40,13 +40,16 @@
     "depends": [
         "account",
         "hr_timesheet_invoice",
-        "analytic"
+        "analytic",
+        "project",
         ],
     "data": [
         "report.xml",
         "hours_block_view.xml",
         "hours_block_data.xml",
         "hours_block_menu.xml",
+        "project_view.xml",
+        "report.xml",
         "security/hours_block_security.xml",
         "security/ir.model.access.csv",
         ],
 
=== modified file 'analytic_hours_block/hours_block.py'
--- analytic_hours_block/hours_block.py	2014-01-31 14:45:54 +0000
+++ analytic_hours_block/hours_block.py	2014-04-02 14:53:22 +0000
@@ -267,74 +267,130 @@
             'invoice_id', 'date_invoice',
             type="date",
             string="Invoice Date",
-            store=True,
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['date_invoice'], 10),
+            },
             readonly=True),
         'user_id': fields.related(
             'invoice_id', 'user_id',
             type="many2one",
             relation="res.users",
             string="Salesman",
-            store=True,
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['user_id'], 10),
+            },
             readonly=True),
         'partner_id': fields.related(
             'invoice_id', 'partner_id',
             type="many2one",
             relation="res.partner",
             string="Partner",
-            store=True,
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['partner_id'], 10),
+            },
             readonly=True),
         'name': fields.related(
             'invoice_id', 'name',
             type="char",
             string="Description",
-            store=True,
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['name'], 10),
+            },
             readonly=True),
         'number': fields.related(
             'invoice_id', 'number',
             type="char",
             string="Number",
-            store=True,
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['number'], 10),
+            },
             readonly=True),
         'journal_id': fields.related(
             'invoice_id', 'journal_id',
             type="many2one",
             relation="account.journal",
             string="Journal",
-            store=True,
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['journal_id'], 10),
+            },
             readonly=True),
         'period_id': fields.related(
             'invoice_id', 'period_id',
             type="many2one",
             relation="account.period",
             string="Period",
-            store=True,
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['period_id'], 10),
+            },
             readonly=True),
         'company_id': fields.related(
             'invoice_id', 'company_id',
             type="many2one",
             relation="res.company",
             string="Company",
-            store=True,
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['company_id'], 10),
+            },
             readonly=True),
         'currency_id': fields.related(
             'invoice_id', 'currency_id',
             type="many2one",
             relation="res.currency",
             string="Currency",
-            store=True,
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['currency_id'], 10),
+            },
             readonly=True),
         'residual': fields.related(
             'invoice_id', 'residual',
             type="float",
             string="Residual",
-            store=True,
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['residual'], 10),
+            },
             readonly=True),
         'amount_total': fields.related(
             'invoice_id', 'amount_total',
             type="float",
             string="Total",
-            store=True,
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['amount_total'], 10),
+            },
             readonly=True),
+        'department_id': fields.related(
+            'invoice_id', 'department_id',
+             type='many2one',
+             relation='hr.department',
+             string='Department',
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['department_id'], 10),
+            },
+             readonly=True),
+
         'state': fields.related(
             'invoice_id', 'state',
             type='selection',
@@ -348,7 +404,11 @@
             ],
             string='State',
             readonly=True,
-            store=True),
+            store={
+                'account.hours.block': (lambda self, cr, uid, ids, c=None: ids,
+                                        ['invoice_id'], 10),
+                'account.invoice': (_get_invoice, ['state'], 10),
+            }),
     }
 
 
 
=== modified file 'analytic_hours_block/hours_block_menu.xml'
--- analytic_hours_block/hours_block_menu.xml	2012-12-17 15:37:44 +0000
+++ analytic_hours_block/hours_block_menu.xml	2014-04-02 14:53:22 +0000
@@ -6,6 +6,7 @@
         Hours block menu
         -->
         <record model="ir.actions.act_window" id="action_all_block_hour">
+			<field name="context">{'search_default_running': 1, 'search_default_group_department_id': 1}</field>
             <field name="name">Hours Blocks</field>
             <field name="res_model">account.hours.block</field>
             <field name="view_type">form</field>
 
=== modified file 'analytic_hours_block/hours_block_view.xml'
--- analytic_hours_block/hours_block_view.xml	2014-01-08 11:55:30 +0000
+++ analytic_hours_block/hours_block_view.xml	2014-04-02 14:53:22 +0000
@@ -18,15 +18,18 @@
                         <separator orientation="vertical"/>
                         <field name="number"/>
                         <field name="partner_id"/>
+                        <field name="department_id" string="Department"/>
                         <field name="user_id" select="1" widget="selection" string="Salesman">
                         <filter domain="[('user_id','=',uid)]" help="My invoices" icon="terp-personal" separator="1"/>
                         </field>
+                        
                         <field name="company_id" widget="selection"/>
                     </group>
                     <newline/>
                     <group expand="0" string="Group By...">
                         <filter string="Partner" icon="terp-partner" domain="[]" context="{'group_by':'partner_id'}"/>
                         <filter string="Responsible" icon="terp-personal" domain="[]"  context="{'group_by':'user_id'}"/>
+                        <filter string="Department" icon="terp-personal" domain="[]"  context="{'group_by':'department_id'}"/>
                         <filter string="Invoice State" icon="terp-stock_effects-object-colorize" domain="[]"  context="{'group_by':'state'}"/>
                     </group>
                </search>
@@ -71,6 +74,7 @@
                             <field name="partner_id" groups="base.group_user"/>
                             <field name="user_id"/>
                             <field name="company_id" groups="base.group_multi_company" widget="selection"/>
+                            <field name="department_id" widget="selection"/>
 
                             <field name="journal_id" invisible="1"/>
                             <field name="period_id" invisible="1" groups="account.group_account_user"/>
@@ -108,6 +112,7 @@
                     <field name="journal_id" invisible="1"/>
                     <field name="period_id" invisible="1" groups="account.group_account_user"/>
                     <field name="company_id" groups="base.group_multi_company" widget="selection"/>
+                    <field name="department_id" widget="selection"/>
                     <field name="user_id"/>
                     <field name="currency_id"/>
                     <field name="residual" sum="Residual Amount"/>
 
=== added file 'analytic_hours_block/project.py'
--- analytic_hours_block/project.py	1970-01-01 00:00:00 +0000
+++ analytic_hours_block/project.py	2014-04-02 14:53:22 +0000
@@ -0,0 +1,33 @@
+# -*- coding: utf-8 -*-
+from osv import orm
+from openerp.tools.translate import _
+
+
+class project_project(orm.Model):
+    _inherit = 'project.project'
+
+    def hours_block_tree_view(self, cr, uid, ids, context):
+        invoice_line_obj = self.pool.get('account.invoice.line')
+        hours_block_obj = self.pool.get('account.hours.block')
+        project = self.browse(cr, uid , ids)[0]
+        invoice_line_ids = invoice_line_obj.search(cr, uid, [('account_analytic_id', '=', project.analytic_account_id.id)])
+        invoice_lines =  invoice_line_obj.browse(cr, uid, invoice_line_ids)
+        invoice_ids = [x.invoice_id.id for x in invoice_lines]
+        res_ids = hours_block_obj.search(cr, uid, [('invoice_id','in',invoice_ids)])
+        domain=False
+        if res_ids:
+            domain = [('id', 'in', res_ids)]
+        else:
+            raise orm.except_orm(_('Warning'), _("No Hours Block for this project"))
+
+        return  {
+            'name': _('Hours Blocks'),
+            'domain': domain,
+            'res_model': 'account.hours.block',
+            'type': 'ir.actions.act_window',
+            'view_id': False,
+            'view_mode': 'tree,form',
+            'view_type': 'form',
+            'limit': 80,
+            'res_id' : res_ids or False,
+        }
 
=== added file 'analytic_hours_block/project_view.xml'
--- analytic_hours_block/project_view.xml	1970-01-01 00:00:00 +0000
+++ analytic_hours_block/project_view.xml	2014-04-02 14:53:22 +0000
@@ -0,0 +1,19 @@
+<?xml version="1.0" ?>
+<openerp>
+    <data>
+
+        <record model="ir.ui.view" id="edit_project_hours_block_link">
+            <field name="name">project.project.form.hours.block.link</field>
+            <field name="model">project.project</field>
+            <field name="inherit_id" ref="project.edit_project"/>
+            <field name="type">form</field>
+            <field name="arch" type="xml">
+                <xpath expr="//button[@name='attachment_tree_view']" position="after">
+                        <button name="hours_block_tree_view" string="Hours Block" type="object"/>
+                </xpath>
+            </field>
+        </record>
+
+
+    </data>
+</openerp>