Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[18.0][MIG] stock_available: Migration to 18.0 #31

Open
wants to merge 53 commits into
base: 18.0
Choose a base branch
from

Conversation

LuisBrose
Copy link

No description provided.

Lionel Sausin (Numérigraphe) and others added 30 commits December 5, 2024 11:52
Generic module to compute the stock quantity available to promise using several implementations.
stock_available_immediatly is changed to become the first optional implementation.
Cherry pick of commit 0b060f619fa5d60f9fb343afe1154acd5c730148 from the v7 branch

[IMP] stock_available* uses new API

[ADD] stock_available_mrp

Module to take immediate manufaturing capability into account in the stock quantity available to promise.

Conflicts:
	stock_available/res_config.py

[DEL] move stock_available_mrp to __unported__

[ADD] stock_available_sale

Take sale quotations into account in the stock quantity available to promise
Cherry-picked from 497068f5f5f1d65fdd4a6e90a787eba37b4e3232

Conflicts:
	stock_available/res_config.py

[DEL] move stock_available_sale to __unported__

[IMP] READMEs and TODOs

Cherry-picked from v7 at 8add4bea7e91924983fb29966e14a1f0d4d5332e

Conflicts:
	__unported__/stock_available_mrp/__openerp__.py
	stock_available/__openerp__.py
	stock_available_immediately/__openerp__.py

[IMP] respect product decimal precision

odoo/odoo#5512 and odoo-dev/odoo@b3e5a94 makes it clear the standard intends to support decimal precision on the product form.
…uct and

product.template, now takes in account variants and correctly displays value.
[FLAKE8]

Removing duplicate modules and moving README.rst into __unported__

[ADD} location calculations

[FIX] typo
not using internal qty_available that seems not to
take in consideration reserved quants.

[ADD] Tests
Commit 6c16913 changed the way we compute the immediately_usable_qty: instead of using the virtual stock, we used the sum of quants without reservations. But a quant may actually be reserved and still be available (for example it may be reserved for an internal move).
Fixes OCA/stock-logistics-warehouse#79

Remove loop and use correct decorator

Restore the features of stock_available_immediately

The previous fix restored stock_available but then there was no way to exclude the incomming moves from the count. This belongs in stock_available_immediately, restoring it cleanly.
This commit also takes care to respect the distinction between templates and variants, so it should fix OCA/stock-logistics-warehouse#73 too.

Restore the qty avail. to promise on variant treeview

PEP8
* fix the dependencies for the computed field

* use api.multi instead of api.one to avoid calling
  super()._immediately_usable_qty in a loop (this improves perfs on a tree view
  display)
There are cases where we dot NOT want to simply sum the quantities of all the
variants. For example when dealing with manufacturing capacities, we may have
to chose between variants because we can't make ALL of them with the same
components.

So instead of a simple non-modular implementation, we'll let each module define
his own implementation of how to compute the product template's quantity
available for sale.

Conflicts:
	stock_available/__openerp__.py
	stock_available_immediately/__openerp__.py
[CHG] improve code regarding code review

[ADD] add test

[CHG] optimize stock computation by avoiding to call useless compute
… related unit test

Increase version number of the module
Currently translated at 61.9% (13 of 21 strings)

Translation: stock-logistics-warehouse-11.0/stock-logistics-warehouse-11.0-stock_available
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-11-0/stock-logistics-warehouse-11-0-stock_available/ar/
Currently translated at 100.0% (21 of 21 strings)

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_available
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_available/pt_BR/
Currently translated at 23.8% (5 of 21 strings)

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_available
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_available/zh_CN/
marielejeune and others added 12 commits December 5, 2024 12:24
Currently translated at 63.1% (12 of 19 strings)

Translation: stock-logistics-availability-16.0/stock-logistics-availability-16.0-stock_available
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-availability-16-0/stock-logistics-availability-16-0-stock_available/es/
Currently translated at 63.1% (12 of 19 strings)

Translation: stock-logistics-availability-16.0/stock-logistics-availability-16.0-stock_available
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-availability-16-0/stock-logistics-availability-16-0-stock_available/es/
Currently translated at 100.0% (19 of 19 strings)

Translation: stock-logistics-availability-16.0/stock-logistics-availability-16.0-stock_available
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-availability-16-0/stock-logistics-availability-16-0-stock_available/fr/
Currently translated at 100.0% (19 of 19 strings)

Translation: stock-logistics-availability-16.0/stock-logistics-availability-16.0-stock_available
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-availability-16-0/stock-logistics-availability-16-0-stock_available/es/
Currently translated at 100.0% (19 of 19 strings)

Translation: stock-logistics-availability-16.0/stock-logistics-availability-16.0-stock_available
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-availability-16-0/stock-logistics-availability-16-0-stock_available/it/
Currently translated at 100.0% (11 of 11 strings)

Translation: stock-logistics-availability-17.0/stock-logistics-availability-17.0-stock_available
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-availability-17-0/stock-logistics-availability-17-0-stock_available/pt_BR/
@LuisBrose LuisBrose force-pushed the 18.0-mig-stock_available branch 2 times, most recently from 2215710 to 2775f20 Compare December 5, 2024 14:02
@OCA-git-bot
Copy link
Contributor

Sorry @LuisBrose you are not allowed to mark the addon tobe migrated.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@LuisBrose LuisBrose mentioned this pull request Dec 5, 2024
1 task
@LuisBrose LuisBrose force-pushed the 18.0-mig-stock_available branch from 2775f20 to a7bd1c1 Compare December 5, 2024 14:20
@rousseldenis
Copy link
Contributor

/ocabot migration stock_available

@LuisBrose LuisBrose force-pushed the 18.0-mig-stock_available branch from a7bd1c1 to 1733291 Compare December 10, 2024 09:42
stock_available/models/product_product.py Outdated Show resolved Hide resolved
stock_available/models/product_product.py Outdated Show resolved Hide resolved
stock_available/models/product_template.py Outdated Show resolved Hide resolved
@LuisBrose LuisBrose force-pushed the 18.0-mig-stock_available branch from 1733291 to 114fce1 Compare December 10, 2024 10:31
Copy link
Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.