[13.0][ADD] product_quantity_update_force_inventory

[HviorForgeFlow]

Product Quantity Update Force Inventory

This module forces users to create inventory adjustments instead of updating quantities on product card.

It could be considered as a work around to manage Stock Accounting properly combining:

• https://github.com/OCA/stock-logistics-warehouse/tree/13.0/stock_change_qty_reason
• https://github.com/OCA/stock-logistics-warehouse/tree/13.0/stock_account_change_qty_reason

CC @ForgeFlow

[rousseldenis]

I'm not really in favor of such things as you lose all the inheritance mechanism.

Isn't there any other mean doing the same thing ?

[HviorForgeFlow]

Create a new action and overwrite on views. What is preferable?

[rousseldenis]

And in this case, why not overriding original method directly with normal inheritance ???

[HviorForgeFlow]

At this point, it is a matter of design, or am I missing anything?

This way it will also work! 😺

# Copyright 2018-20 ForgeFlow <http://www.forgeflow.com>

from odoo import api, models


class StockQuant(models.Model):
    _inherit = 'stock.quant'

    @api.model
    def _get_quants_action(self, domain=None, extend=False):
        action = super()._get_quants_action(domain=domain, extend=extend)
        action['view_id'] = self.env.ref('stock.view_stock_quant_tree').id
        # Enables form view in readonly list
        action.update({
            'view_mode': 'tree,form',
            'views': [
                (action['view_id'], 'list'),
                (self.env.ref('stock.view_stock_quant_form').id, 'form'),
            ],
        })
        return action

[HviorForgeFlow]

I changed the override for that code again, it should perform the same, could we move it forward?

[AdriaGForgeFlow]

LGTM 👍

Could you clean the commits, I think you have some unwanted commits there

[rousseldenis]

This is great! Indeed I would change module name in order to let people understand what this do.

[HviorForgeFlow]

Any suggestion for module name change?

[rousseldenis]

Any suggestion for module name change?

product_quantity_update_force_inventory or something like that. More functional oriented.

[LoisRForgeFlow]

LGTM 👍

[AaronHForgeFlow]

👍 It works as it says

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[HaraldPanten]

Nice one. Functional review 👍

[LoisRForgeFlow]

/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 13.0-ocabot-merge-pr-846-by-LoisRForgeFlow-bump-nobump, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at e494f7d. Thanks a lot for contributing to OCA. ❤️