diff --git a/account_invoice_change/__init__.py b/account_invoice_change/__init__.py new file mode 100644 index 00000000..40272379 --- /dev/null +++ b/account_invoice_change/__init__.py @@ -0,0 +1 @@ +from . import wizard diff --git a/account_invoice_change/__manifest__.py b/account_invoice_change/__manifest__.py new file mode 100644 index 00000000..d6938c02 --- /dev/null +++ b/account_invoice_change/__manifest__.py @@ -0,0 +1,31 @@ +{ + 'name': 'Account Invoice Change', + 'author': 'Hibou Corp. ', + 'version': '14.0.1.0.0', + 'category': 'Accounting', + 'sequence': 95, + 'summary': 'Technical foundation for changing invoices.', + 'description': """ +Technical foundation for changing invoices. + +Creates wizard and permissions for making invoice changes that can be +handled by other individual modules. + +This module implements, as examples, how to change the Salesperson and Date fields. + +Abstractly, individual 'changes' should come from specific 'fields' or capability +modules that handle the consequences of changing that field in whatever state the +the invoice is currently in. + + """, + 'website': 'https://hibou.io/', + 'depends': [ + 'account', + ], + 'data': [ + 'wizard/invoice_change_views.xml', + 'security/ir.model.access.csv', + ], + 'installable': True, + 'application': False, +} diff --git a/account_invoice_change/security/ir.model.access.csv b/account_invoice_change/security/ir.model.access.csv new file mode 100644 index 00000000..a9aaa525 --- /dev/null +++ b/account_invoice_change/security/ir.model.access.csv @@ -0,0 +1,3 @@ +id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink +access_invoice_change,access_invoice_change,model_account_invoice_change,base.group_user,1,0,0,0 +manage_invoice_change,manage_invoice_change,model_account_invoice_change,base.group_system,1,1,1,1 diff --git a/account_invoice_change/tests/__init__.py b/account_invoice_change/tests/__init__.py new file mode 100644 index 00000000..defd2696 --- /dev/null +++ b/account_invoice_change/tests/__init__.py @@ -0,0 +1 @@ +from . import test_invoice_change diff --git a/account_invoice_change/tests/test_invoice_change.py b/account_invoice_change/tests/test_invoice_change.py new file mode 100644 index 00000000..e2c362bd --- /dev/null +++ b/account_invoice_change/tests/test_invoice_change.py @@ -0,0 +1,51 @@ +from odoo.addons.account.tests.account_test_users import AccountTestUsers +from odoo import fields + + +class TestInvoiceChange(AccountTestUsers): + + def test_invoice_change_basic(self): + self.account_invoice_obj = self.env['account.move'] + self.payment_term = self.env.ref('account.account_payment_term_advance') + self.journalrec = self.env['account.journal'].search([('type', '=', 'sale')])[0] + self.partner3 = self.env.ref('base.res_partner_3') + self.invoice_basic = self.account_invoice_obj.with_user(self.account_user).create({ + 'type': 'out_invoice', + 'name': "Test Customer Invoice", + 'invoice_payment_term_id': self.payment_term.id, + 'journal_id': self.journalrec.id, + 'partner_id': self.partner3.id, + # account_id=self.account_rec1_id.id, + 'line_ids': [(0, 0, { + 'product_id': self.env.ref('product.product_product_5').id, + 'quantity': 10.0, + 'account_id': self.env['account.account'].search( + [('user_type_id', '=', self.env.ref('account.data_account_type_revenue').id)], limit=1).id, + 'name': 'product test 5', + 'price_unit': 100.00, + })], + }) + self.assertEqual(self.invoice_basic.state, 'draft') + self.invoice_basic.action_post() + self.assertEqual(self.invoice_basic.state, 'posted') + self.assertEqual(self.invoice_basic.date, fields.Date.today()) + self.assertEqual(self.invoice_basic.invoice_date, fields.Date.today()) + self.assertEqual(self.invoice_basic.invoice_user_id, self.account_user) + self.assertEqual(self.invoice_basic.line_ids[0].date, fields.Date.today()) + + ctx = {'active_model': 'account.move', 'active_ids': [self.invoice_basic.id]} + change = self.env['account.invoice.change'].with_context(ctx).create({}) + self.assertEqual(change.date, self.invoice_basic.date) + self.assertEqual(change.invoice_date, self.invoice_basic.invoice_date) + self.assertEqual(change.invoice_user_id, self.invoice_basic.invoice_user_id) + + change_date = '2018-01-01' + change_invoice_date = '2019-01-01' + change_user = self.env.user + change.write({'invoice_user_id': change_user.id, 'date': change_date, 'invoice_date': change_invoice_date}) + + change.affect_change() + self.assertEqual(str(self.invoice_basic.date), change_date) + self.assertEqual(str(self.invoice_basic.invoice_date), change_invoice_date) + self.assertEqual(self.invoice_basic.invoice_user_id, change_user) + self.assertEqual(str(self.invoice_basic.line_ids[0].date), change_date) diff --git a/account_invoice_change/wizard/__init__.py b/account_invoice_change/wizard/__init__.py new file mode 100644 index 00000000..8ec5d6da --- /dev/null +++ b/account_invoice_change/wizard/__init__.py @@ -0,0 +1 @@ +from . import invoice_change \ No newline at end of file diff --git a/account_invoice_change/wizard/invoice_change.py b/account_invoice_change/wizard/invoice_change.py new file mode 100644 index 00000000..63baae08 --- /dev/null +++ b/account_invoice_change/wizard/invoice_change.py @@ -0,0 +1,57 @@ +from odoo import api, fields, models, _ +from odoo.exceptions import UserError + + +class InvoiceChangeWizard(models.TransientModel): + _name = 'account.invoice.change' + _description = 'Invoice Change' + + move_id = fields.Many2one('account.move', string='Invoice', readonly=True, required=True) + move_company_id = fields.Many2one('res.company', readonly=True, related='move_id.company_id') + invoice_user_id = fields.Many2one('res.users', string='Salesperson') + date = fields.Date(string='Accounting Date') + invoice_date = fields.Date(string='Invoice Date') + + @api.model + def default_get(self, fields): + rec = super(InvoiceChangeWizard, self).default_get(fields) + context = dict(self._context or {}) + active_model = context.get('active_model') + active_ids = context.get('active_ids') + + # Checks on context parameters + if not active_model or not active_ids: + raise UserError( + _("Programmation error: wizard action executed without active_model or active_ids in context.")) + if active_model != 'account.move': + raise UserError(_( + "Programmation error: the expected model for this action is 'account.move'. The provided one is '%d'.") % active_model) + + # Checks on received invoice records + invoice = self.env[active_model].browse(active_ids) + if len(invoice) != 1: + raise UserError(_("Invoice Change expects only one invoice.")) + rec.update({ + 'move_id': invoice.id, + 'invoice_user_id': invoice.invoice_user_id.id, + 'date': invoice.date, + 'invoice_date': invoice.invoice_date, + }) + return rec + + def _new_move_vals(self): + vals = {} + if self.move_id.invoice_user_id != self.invoice_user_id: + vals['invoice_user_id'] = self.invoice_user_id.id + if self.move_id.date != self.date: + vals['date'] = self.date + if self.move_id.invoice_date != self.invoice_date: + vals['invoice_date'] = self.invoice_date + return vals + + def affect_change(self): + self.ensure_one() + vals = self._new_move_vals() + if vals: + self.move_id.write(vals) + return True diff --git a/account_invoice_change/wizard/invoice_change_views.xml b/account_invoice_change/wizard/invoice_change_views.xml new file mode 100644 index 00000000..5e233688 --- /dev/null +++ b/account_invoice_change/wizard/invoice_change_views.xml @@ -0,0 +1,48 @@ + + + + Invoice Change + account.invoice.change + +
+ + + + + + + + + + +
+
+ +
+
+ + + Invoice Change Wizard + ir.actions.act_window + account.invoice.change + form + new + + + + account.move.form.inherit.invoice.change + account.move + + + +