From 77c01d1ff560d8ed8988d0b62cb3e825f589e7bb Mon Sep 17 00:00:00 2001 From: Ajmal JK Date: Thu, 6 Feb 2020 09:11:23 +0530 Subject: [PATCH] [FIX] Bug Fixed 'invoice_stock_move' --- invoice_stock_move/__manifest__.py | 2 +- invoice_stock_move/doc/RELEASE_NOTES.md | 6 ++ invoice_stock_move/models/invoice_stock.py | 94 +++++++++++++++++++ .../views/invoice_stock_move_view.xml | 4 +- 4 files changed, 103 insertions(+), 3 deletions(-) create mode 100644 invoice_stock_move/doc/RELEASE_NOTES.md diff --git a/invoice_stock_move/__manifest__.py b/invoice_stock_move/__manifest__.py index 6862507b8..aa6d52c9e 100644 --- a/invoice_stock_move/__manifest__.py +++ b/invoice_stock_move/__manifest__.py @@ -19,7 +19,7 @@ ############################################################################## { 'name': "Stock Picking From Invoice", - 'version': '10.0.1.0.0', + 'version': '10.0.1.1.1', 'summary': """Stock Picking From Customer/Supplier Invoice""", 'description': """This Module Enables To Create Stocks Picking From Customer/Supplier Invoice""", 'author': "Cybrosys Techno Solutions", diff --git a/invoice_stock_move/doc/RELEASE_NOTES.md b/invoice_stock_move/doc/RELEASE_NOTES.md new file mode 100644 index 000000000..32905757f --- /dev/null +++ b/invoice_stock_move/doc/RELEASE_NOTES.md @@ -0,0 +1,6 @@ +## Module + +#### 20.01.2020 +#### Version 10.0.1.1.1 +#### FIX +- Bug Fixed. diff --git a/invoice_stock_move/models/invoice_stock.py b/invoice_stock_move/models/invoice_stock.py index a0af462ad..22793b94a 100644 --- a/invoice_stock_move/models/invoice_stock.py +++ b/invoice_stock_move/models/invoice_stock.py @@ -19,6 +19,7 @@ ############################################################################## from odoo.exceptions import UserError from odoo import models, fields, api, _ +from odoo.tools.safe_eval import safe_eval class InvoiceStockMove(models.Model): @@ -186,5 +187,98 @@ class SupplierInvoiceLine(models.Model): done += moves.create(template) return done +class AccountInvoiceRefund(models.TransientModel): + _inherit = 'account.invoice.refund' + + @api.model + def compute_refund(self, mode='refund'): + company_id = self.env.context.get('company_id') or self.env.user.company_id.id + inv_obj = self.env['account.invoice'] + inv_tax_obj = self.env['account.invoice.tax'] + inv_line_obj = self.env['account.invoice.line'] + context = dict(self._context or {}) + xml_id = False + + for form in self: + created_inv = [] + date = False + description = False + for inv in inv_obj.browse(context.get('active_ids')): + if inv.state in ['draft', 'proforma2', 'cancel']: + raise UserError(_('Cannot refund draft/proforma/cancelled invoice.')) + if inv.reconciled and mode in ('cancel', 'modify'): + raise UserError(_( + 'Cannot refund invoice which is already reconciled, invoice should be unreconciled first. You can only refund this invoice.')) + + date = form.date or False + description = form.description or inv.name + refund = inv.refund(form.date_invoice, date, description, inv.journal_id.id) + + created_inv.append(refund.id) + if inv.picking_transfer_id.code == 'outgoing': + data = self.env['stock.picking.type'].search( + [('warehouse_id.company_id', '=', company_id), ('code', '=', 'incoming')], limit=1) + refund.picking_transfer_id = data.id + if inv.picking_type_id.code == 'incoming': + data = self.env['stock.picking.type'].search( + [('warehouse_id.company_id', '=', company_id), ('code', '=', 'outgoing')], limit=1) + refund.picking_type_id = data.id + if mode in ('cancel', 'modify'): + movelines = inv.move_id.line_ids + to_reconcile_ids = {} + to_reconcile_lines = self.env['account.move.line'] + for line in movelines: + if line.account_id.id == inv.account_id.id: + to_reconcile_lines += line + to_reconcile_ids.setdefault(line.account_id.id, []).append(line.id) + if line.reconciled: + line.remove_move_reconcile() + refund.action_invoice_open() + for tmpline in refund.move_id.line_ids: + if tmpline.account_id.id == inv.account_id.id: + to_reconcile_lines += tmpline + to_reconcile_lines.filtered(lambda l: l.reconciled == False).reconcile() + if mode == 'modify': + invoice = inv.read(inv_obj._get_refund_modify_read_fields()) + invoice = invoice[0] + del invoice['id'] + invoice_lines = inv_line_obj.browse(invoice['invoice_line_ids']) + invoice_lines = inv_obj.with_context(mode='modify')._refund_cleanup_lines(invoice_lines) + tax_lines = inv_tax_obj.browse(invoice['tax_line_ids']) + tax_lines = inv_obj._refund_cleanup_lines(tax_lines) + invoice.update({ + 'type': inv.type, + 'date_invoice': form.date_invoice, + 'state': 'draft', + 'number': False, + 'invoice_line_ids': invoice_lines, + 'tax_line_ids': tax_lines, + 'date': date, + 'origin': inv.origin, + 'fiscal_position_id': inv.fiscal_position_id.id, + }) + for field in inv_obj._get_refund_common_fields(): + if inv_obj._fields[field].type == 'many2one': + invoice[field] = invoice[field] and invoice[field][0] + else: + invoice[field] = invoice[field] or False + inv_refund = inv_obj.create(invoice) + if inv_refund.payment_term_id.id: + inv_refund._onchange_payment_term_date_invoice() + created_inv.append(inv_refund.id) + xml_id = (inv.type in ['out_refund', 'out_invoice']) and 'action_invoice_tree1' or \ + (inv.type in ['in_refund', 'in_invoice']) and 'action_invoice_tree2' + # Put the reason in the chatter + subject = _("Invoice refund") + body = description + refund.message_post(body=body, subject=subject) + if xml_id: + result = self.env.ref('account.%s' % (xml_id)).read()[0] + invoice_domain = safe_eval(result['domain']) + invoice_domain.append(('id', 'in', created_inv)) + result['domain'] = invoice_domain + return result + return True + diff --git a/invoice_stock_move/views/invoice_stock_move_view.xml b/invoice_stock_move/views/invoice_stock_move_view.xml index 1cbe89d6e..921b4c340 100644 --- a/invoice_stock_move/views/invoice_stock_move_view.xml +++ b/invoice_stock_move/views/invoice_stock_move_view.xml @@ -8,7 +8,7 @@