Apr 8 [UPDT] : Updated 'stock_move_invoice'

stock_move_invoice/doc/RELEASE_NOTES.md

@@ -9,3 +9,8 @@
 #### Version 16.0.1.0.0
 ##### UPDATE
 - Update for Invoice From Stock Picking
+
+#### 13.02.2024
+#### Version 16.0.1.0.1
+##### BUG FIX
+- Updated the module by removing the stock move line issue

stock_move_invoice/models/stock_picking.py

@@ -50,48 +50,40 @@ class StockPicking(models.Model):
             current_user = self.env.uid
             if picking_id.picking_type_id.code == 'outgoing':
                 customer_journal_id = \
-                    picking_id.env['ir.config_parameter'].sudo().\
+                    picking_id.env['ir.config_parameter'].sudo(). \
                         get_param('stock_move_invoice.customer_journal_id') or \
                     False
                 if not customer_journal_id:
                     raise UserError(
                         _("Please configure the journal from settings"))
                 invoice_line_list = []
-                for move_ids_without_package in picking_id.\
+                for move_ids_without_package in picking_id.move_ids_without_package:
-                        move_ids_without_package:
                     vals = (0, 0, {
                         'name': move_ids_without_package.description_picking,
                         'product_id': move_ids_without_package.product_id.id,
-                        'price_unit':
+                        'price_unit': move_ids_without_package.product_id.lst_price,
-                            move_ids_without_package.product_id.lst_price,
+                        'account_id': move_ids_without_package.product_id.property_account_income_id.id if
-                        'account_id':
+                        move_ids_without_package.product_id.property_account_income_id
-                            move_ids_without_package.product_id.
+                        else move_ids_without_package.product_id.categ_id.property_account_income_categ_id.id,
-                            property_account_income_id.id if
-                            move_ids_without_package.
-                            product_id.property_account_income_id
-                        else move_ids_without_package.
-                            product_id.categ_id.
-                            property_account_income_categ_id.id,
                         'tax_ids': [(6, 0, [
                             picking_id.company_id.account_sale_tax_id.id])],
                         'quantity': move_ids_without_package.quantity_done,
                     })
                     invoice_line_list.append(vals)
-                    invoice = picking_id.env['account.move'].create({
+                invoice = picking_id.env['account.move'].create({
-                        'move_type': 'out_invoice',
+                    'move_type': 'out_invoice',
-                        'invoice_origin': picking_id.name,
+                    'invoice_origin': picking_id.name,
-                        'invoice_user_id': current_user,
+                    'invoice_user_id': current_user,
-                        'narration': picking_id.name,
+                    'narration': picking_id.name,
-                        'partner_id': picking_id.partner_id.id,
+                    'partner_id': picking_id.partner_id.id,
-                        'currency_id':
+                    'currency_id': picking_id.env.user.company_id.currency_id.id,
-                            picking_id.env.user.company_id.currency_id.id,
+                    'journal_id': int(customer_journal_id),
-                        'journal_id': int(customer_journal_id),
+                    'payment_reference': picking_id.name,
-                        'payment_reference': picking_id.name,
+                    'picking_id': picking_id.id,
-                        'picking_id': picking_id.id,
+                    'invoice_line_ids': invoice_line_list,
-                        'invoice_line_ids': invoice_line_list,
+                    'transfer_ids': self
-                        'transfer_ids': self
+                })
-                    })
+                return invoice
-                    return invoice
 
     def create_bill(self):
         """This is the function for creating vendor bill
@@ -106,7 +98,7 @@ class StockPicking(models.Model):
                     raise UserError(
                         _("Please configure the journal from the settings."))
                 invoice_line_list = []
-                for move_ids_without_package in picking_id.\
+                for move_ids_without_package in picking_id. \
                         move_ids_without_package:
                     vals = (0, 0, {
                         'name': move_ids_without_package.description_picking,
@@ -118,7 +110,7 @@ class StockPicking(models.Model):
                             property_account_income_id.id if
                             move_ids_without_package.product_id.
                             property_account_income_id
-                        else move_ids_without_package.product_id.categ_id.
+                            else move_ids_without_package.product_id.categ_id.
                             property_account_income_categ_id.id,
                         'tax_ids': [(6, 0, [
                             picking_id.company_id.account_purchase_tax_id.id])],
@@ -148,14 +140,14 @@ class StockPicking(models.Model):
             current_user = picking_id.env.uid
             if picking_id.picking_type_id.code == 'incoming':
                 customer_journal_id = \
-                    picking_id.env['ir.config_parameter'].sudo().\
+                    picking_id.env['ir.config_parameter'].sudo(). \
                         get_param('stock_move_invoice.customer_journal_id') or \
                     False
                 if not customer_journal_id:
                     raise UserError(
                         _("Please configure the journal from settings"))
                 invoice_line_list = []
-                for move_ids_without_package in picking_id.\
+                for move_ids_without_package in picking_id. \
                         move_ids_without_package:
                     vals = (0, 0, {
                         'name': move_ids_without_package.description_picking,
@@ -202,7 +194,7 @@ class StockPicking(models.Model):
                     raise UserError(
                         _("Please configure the journal from the settings."))
                 invoice_line_list = []
-                for move_ids_without_package in picking_id.\
+                for move_ids_without_package in picking_id. \
                         move_ids_without_package:
                     vals = (0, 0, {
                         'name': move_ids_without_package.description_picking,
@@ -260,14 +252,14 @@ class StockPicking(models.Model):
                     partner_id = self.partner_id
                     invoice_line_list = []
                     customer_journal_id = \
-                        self.env['ir.config_parameter'].sudo().\
+                        self.env['ir.config_parameter'].sudo(). \
                             get_param('stock_move_invoice.customer_journal_id') \
                         or False
                     if not customer_journal_id:
                         raise UserError(
                             _("Please configure the journal from settings"))
                     for picking_id in self:
-                        for move_ids_without_package in picking_id.\
+                        for move_ids_without_package in picking_id. \
                                 move_ids_without_package:
                             vals = (0, 0, {
                                 'name':
@@ -312,14 +304,14 @@ class StockPicking(models.Model):
                     partner_id = self.partner_id
                     bill_line_list = []
                     vendor_journal_id = \
-                        self.env['ir.config_parameter'].sudo().\
+                        self.env['ir.config_parameter'].sudo(). \
                             get_param('stock_move_invoice.vendor_journal_id') \
                         or False
                     if not vendor_journal_id:
                         raise UserError(_("Please configure the journal from "
                                           "the settings."))
                     for picking_id in self:
-                        for move_ids_without_package in picking_id.\
+                        for move_ids_without_package in picking_id. \
                                 move_ids_without_package:
                             vals = (0, 0, {
                                 'name':