From 6a81815e3f0c64f4003a014b71b96c1d141830cb Mon Sep 17 00:00:00 2001 From: AjmalCybro Date: Mon, 30 Jan 2023 15:03:56 +0530 Subject: [PATCH] Jan 30 [UPDT] : Bug Fixed 'stock_move_invoice' --- stock_move_invoice/__manifest__.py | 2 +- stock_move_invoice/models/stock_picking.py | 104 ++++++++++----------- 2 files changed, 53 insertions(+), 53 deletions(-) diff --git a/stock_move_invoice/__manifest__.py b/stock_move_invoice/__manifest__.py index 3b20563a1..f7682ec43 100644 --- a/stock_move_invoice/__manifest__.py +++ b/stock_move_invoice/__manifest__.py @@ -22,7 +22,7 @@ { 'name': "Invoice From Stock Picking", - 'version': '14.0.1.0.0', + 'version': '14.0.1.0.1', 'summary': """In this module creating customer invoice,vendor bill, customer credit note and refund from stock picking""", 'description': """In this module creating customer invoice,vendor bill, customer diff --git a/stock_move_invoice/models/stock_picking.py b/stock_move_invoice/models/stock_picking.py index d8142be78..e8e2b736d 100644 --- a/stock_move_invoice/models/stock_picking.py +++ b/stock_move_invoice/models/stock_picking.py @@ -60,19 +60,19 @@ class StockPicking(models.Model): 'quantity': move_ids_without_package.quantity_done, }) invoice_line_list.append(vals) - invoice = picking_id.env['account.move'].create({ - 'move_type': 'out_invoice', - 'invoice_origin': picking_id.name, - 'invoice_user_id': current_user, - 'narration': picking_id.name, - 'partner_id': picking_id.partner_id.id, - 'currency_id': picking_id.env.user.company_id.currency_id.id, - 'journal_id': int(customer_journal_id), - 'payment_reference': picking_id.name, - 'picking_id': picking_id.id, - 'invoice_line_ids': invoice_line_list - }) - return invoice + invoice = picking_id.env['account.move'].create({ + 'move_type': 'out_invoice', + 'invoice_origin': picking_id.name, + 'invoice_user_id': current_user, + 'narration': picking_id.name, + 'partner_id': picking_id.partner_id.id, + 'currency_id': picking_id.env.user.company_id.currency_id.id, + 'journal_id': int(customer_journal_id), + 'payment_reference': picking_id.name, + 'picking_id': picking_id.id, + 'invoice_line_ids': invoice_line_list + }) + return invoice def create_bill(self): """This is the function for creating vendor bill @@ -96,19 +96,19 @@ class StockPicking(models.Model): 'quantity': move_ids_without_package.quantity_done, }) invoice_line_list.append(vals) - invoice = picking_id.env['account.move'].create({ - 'move_type': 'in_invoice', - 'invoice_origin': picking_id.name, - 'invoice_user_id': current_user, - 'narration': picking_id.name, - 'partner_id': picking_id.partner_id.id, - 'currency_id': picking_id.env.user.company_id.currency_id.id, - 'journal_id': int(vendor_journal_id), - 'payment_reference': picking_id.name, - 'picking_id': picking_id.id, - 'invoice_line_ids': invoice_line_list - }) - return invoice + invoice = picking_id.env['account.move'].create({ + 'move_type': 'in_invoice', + 'invoice_origin': picking_id.name, + 'invoice_user_id': current_user, + 'narration': picking_id.name, + 'partner_id': picking_id.partner_id.id, + 'currency_id': picking_id.env.user.company_id.currency_id.id, + 'journal_id': int(vendor_journal_id), + 'payment_reference': picking_id.name, + 'picking_id': picking_id.id, + 'invoice_line_ids': invoice_line_list + }) + return invoice def create_customer_credit(self): """This is the function for creating customer credit note @@ -132,19 +132,19 @@ class StockPicking(models.Model): 'quantity': move_ids_without_package.quantity_done, }) invoice_line_list.append(vals) - invoice = picking_id.env['account.move'].create({ - 'move_type': 'out_refund', - 'invoice_origin': picking_id.name, - 'invoice_user_id': current_user, - 'narration': picking_id.name, - 'partner_id': picking_id.partner_id.id, - 'currency_id': picking_id.env.user.company_id.currency_id.id, - 'journal_id': int(customer_journal_id), - 'payment_reference': picking_id.name, - 'picking_id': picking_id.id, - 'invoice_line_ids': invoice_line_list - }) - return invoice + invoice = picking_id.env['account.move'].create({ + 'move_type': 'out_refund', + 'invoice_origin': picking_id.name, + 'invoice_user_id': current_user, + 'narration': picking_id.name, + 'partner_id': picking_id.partner_id.id, + 'currency_id': picking_id.env.user.company_id.currency_id.id, + 'journal_id': int(customer_journal_id), + 'payment_reference': picking_id.name, + 'picking_id': picking_id.id, + 'invoice_line_ids': invoice_line_list + }) + return invoice def create_vendor_credit(self): """This is the function for creating refund @@ -168,19 +168,19 @@ class StockPicking(models.Model): 'quantity': move_ids_without_package.quantity_done, }) invoice_line_list.append(vals) - invoice = picking_id.env['account.move'].create({ - 'move_type': 'in_refund', - 'invoice_origin': picking_id.name, - 'invoice_user_id': current_user, - 'narration': picking_id.name, - 'partner_id': picking_id.partner_id.id, - 'currency_id': picking_id.env.user.company_id.currency_id.id, - 'journal_id': int(vendor_journal_id), - 'payment_reference': picking_id.name, - 'picking_id': picking_id.id, - 'invoice_line_ids': invoice_line_list - }) - return invoice + invoice = picking_id.env['account.move'].create({ + 'move_type': 'in_refund', + 'invoice_origin': picking_id.name, + 'invoice_user_id': current_user, + 'narration': picking_id.name, + 'partner_id': picking_id.partner_id.id, + 'currency_id': picking_id.env.user.company_id.currency_id.id, + 'journal_id': int(vendor_journal_id), + 'payment_reference': picking_id.name, + 'picking_id': picking_id.id, + 'invoice_line_ids': invoice_line_list + }) + return invoice def action_open_picking_invoice(self): """This is the function of the smart button which redirect to the