Merge branch 'mig/14.0/hr_expense_change_analytic' into '14.0'

mig/14.0/hr_expense_change_analytic into 14.0

See merge request hibou-io/hibou-odoo/suite!645

hr_expense_change/tests/test_expense_change.py

@@ -141,12 +141,12 @@ class TestAccountEntry(test_expenses.TestExpenses):
         self.assertEqual(expense_sheet.accounting_date, expense_sheet.account_move_id.date)
 
         expense = expense_sheet.expense_line_ids[0]
-
+        self.expense = expense
         ctx = {'active_model': 'hr.expense', 'active_ids': expense.ids}
         change = self.env['hr.expense.change'].sudo(flag=True).with_context(ctx).create({})
         self.assertEqual(change.date, expense.date)
 
-        change_date = '2018-01-01'
+        change_date = '2017-01-31'
         change.write({'date': change_date})
 
         change.affect_change()

hr_expense_change_analytic/__init__.py

@@ -0,0 +1 @@
+from . import wizard

hr_expense_change_analytic/__manifest__.py

@@ -0,0 +1,23 @@
+{
+    'name': 'HR Expense Change - Analytic',
+    'author': 'Hibou Corp. <hello@hibou.io>',
+    'version': '14.0.1.0.0',
+    'category': 'Employees',
+    'sequence': 96,
+    'summary': 'Change Analytic Account on Expense.',
+    'description': """
+Adds fields and functionality to change the analytic account on expense 
+and subsequent documents.
+
+    """,
+    'website': 'https://hibou.io/',
+    'depends': [
+        'hr_expense_change',
+        'analytic',
+    ],
+    'data': [
+        'wizard/expense_change_views.xml',
+    ],
+    'installable': True,
+    'application': False,
+}

hr_expense_change_analytic/tests/__init__.py

@@ -0,0 +1 @@
+from . import test_expense_change

hr_expense_change_analytic/tests/test_expense_change.py

@@ -0,0 +1,36 @@
+from odoo.addons.hr_expense_change.tests import test_expense_change
+from odoo.tests import tagged
+from odoo import fields
+
+
+@tagged('-at_install', 'post_install')
+class TestWizard(test_expense_change.TestAccountEntry):
+
+    def test_expense_change_basic(self):
+        super(TestWizard, self).test_expense_values()
+
+        # Tests Adding an Analytic Account
+
+        self.analytic_account = self.env['account.analytic.account'].create({
+            'name': 'test account',
+        })
+        self.analytic_account2 = self.env['account.analytic.account'].create({
+            'name': 'test account2',
+        })
+
+        self.assertNotEqual(self.expense.analytic_account_id, self.analytic_account)
+        ctx = {'active_model': 'hr.expense', 'active_ids': self.expense.ids}
+        change = self.env['hr.expense.change'].sudo().with_context(ctx).create({})
+        change.analytic_account_id = self.analytic_account
+        change.affect_change()
+        self.assertEqual(self.expense.analytic_account_id, self.analytic_account)
+
+        # Tests Changing
+        change.analytic_account_id = self.analytic_account2
+        change.affect_change()
+        self.assertEqual(self.expense.analytic_account_id, self.analytic_account2)
+
+        # Tests Removing
+        change.analytic_account_id = False
+        change.affect_change()
+        self.assertFalse(self.expense.analytic_account_id)

hr_expense_change_analytic/wizard/__init__.py

@@ -0,0 +1 @@
+from . import expense_change

hr_expense_change_analytic/wizard/expense_change.py

@@ -0,0 +1,58 @@
+from odoo import api, fields, models, _
+
+
+class ExpenseChangeWizard(models.TransientModel):
+    _inherit = 'hr.expense.change'
+
+    analytic_account_id = fields.Many2one('account.analytic.account', string='Analytic Account')
+    analytic_account_warning = fields.Char(string='Analytic Account Warning', compute='_compute_analytic_warning')
+
+    @api.model
+    def default_get(self, fields):
+        rec = super(ExpenseChangeWizard, self).default_get(fields)
+        expense = self.env['hr.expense'].browse(rec['expense_id'])
+        rec.update({
+            'analytic_account_id': expense.analytic_account_id.id,
+        })
+        return rec
+
+    @api.onchange('expense_id', 'analytic_account_id')
+    def _compute_analytic_warning(self):
+        self.ensure_one()
+        expenses = self._find_expenses_to_write_analytic(self.expense_id.analytic_account_id.id)
+        if len(expenses) <= 1:
+            self.analytic_account_warning = ''
+        else:
+            other_expenses = expenses - self.expense_id
+            self.analytic_account_warning = '%d other expenses will be changed. (%s)' % \
+                                            (len(other_expenses), ', '.join(other_expenses.mapped('name')))
+
+
+
+    def affect_change(self):
+        old_analytic_id = self.expense_id.analytic_account_id.id
+        res = super(ExpenseChangeWizard, self).affect_change()
+        self._affect_analytic_change(old_analytic_id)
+        return res
+
+    def _find_expenses_to_write_analytic(self, old_analytic_id):
+        if self.analytic_account_id.id == old_analytic_id:
+            return []
+        # Essentially, if you have a move, you must write all related expenses and lines.
+        if not self.expense_id.sheet_id.account_move_id:
+            return self.expense_id
+        return self.expense_id.sheet_id.expense_line_ids\
+                   .filtered(lambda l: l.analytic_account_id.id == old_analytic_id)
+
+    def _affect_analytic_change(self, old_analytic_id):
+        expenses_to_affect = self._find_expenses_to_write_analytic(old_analytic_id)
+        if expenses_to_affect:
+            prev_state = expenses_to_affect.state
+            expenses_to_affect.state = "draft"
+            expenses_to_affect.write({'analytic_account_id': self.analytic_account_id.id})
+
+            lines_to_affect = self.expense_id.sheet_id.account_move_id \
+                .line_ids.filtered(lambda l: l.analytic_account_id.id == old_analytic_id and l.debit)
+            lines_to_affect.write({'analytic_account_id': self.analytic_account_id.id})
+            lines_to_affect.create_analytic_lines()
+            expenses_to_affect.state = prev_state

hr_expense_change_analytic/wizard/expense_change_views.xml

@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<odoo>
+    <record id="hr_expense_change_form_inherit" model="ir.ui.view">
+        <field name="name">hr.expense.change.form.inherit</field>
+        <field name="model">hr.expense.change</field>
+        <field name="inherit_id" ref="hr_expense_change.hr_expense_change_form"/>
+        <field name="arch" type="xml">
+            <xpath expr="//group[@name='group_right']" position="inside">
+                <field name="analytic_account_id" domain="[('company_id', '=', expense_company_id)]"/>
+                <field name="analytic_account_warning" attrs="{'invisible': [('analytic_account_warning', '=', '')]}"/>
+            </xpath>
+        </field>
+    </record>
+</odoo>