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

[16.0][ADD] stock_available_location_get_domain #11

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions setup/stock_available_location_get_domain/setup.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
import setuptools

setuptools.setup(
setup_requires=['setuptools-odoo'],
odoo_addon=True,
)
84 changes: 84 additions & 0 deletions stock_available_location_get_domain/README.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,84 @@
===================================
Stock Available Location Get Domain
===================================

..
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!! This file is generated by oca-gen-addon-readme !!
!! changes will be overwritten. !!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!! source digest: sha256:943a988f24484f4a3db1de40304f1bf42d40ce80274f9ae48c5b41fc408848f6
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
:target: https://odoo-community.org/page/development-status
:alt: Beta
.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png
:target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
:alt: License: AGPL-3
.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fstock--logistics--availability-lightgray.png?logo=github
:target: https://github.com/OCA/stock-logistics-availability/tree/16.0/stock_available_location_get_domain
:alt: OCA/stock-logistics-availability
.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
:target: https://translation.odoo-community.org/projects/stock-logistics-availability-16-0/stock-logistics-availability-16-0-stock_available_location_get_domain
:alt: Translate me on Weblate
.. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
:target: https://runboat.odoo-community.org/builds?repo=OCA/stock-logistics-availability&target_branch=16.0
:alt: Try me on Runboat

|badge1| |badge2| |badge3| |badge4| |badge5|

This is a technical helper module in order to reuse the standard
_get_domain_locations function for locations and not quants.

**Table of contents**

.. contents::
:local:

Usage
=====

Call the new _get_domain_location_for_locations() function on products in order
to retrieve a domain for stock locations with the same context behavior as
for product availability.

Bug Tracker
===========

Bugs are tracked on `GitHub Issues <https://github.com/OCA/stock-logistics-availability/issues>`_.
In case of trouble, please check there if your issue has already been reported.
If you spotted it first, help us to smash it by providing a detailed and welcomed
`feedback <https://github.com/OCA/stock-logistics-availability/issues/new?body=module:%20stock_available_location_get_domain%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.

Do not contact contributors directly about support or help with technical issues.

Credits
=======

Authors
~~~~~~~

* ACSONE SA/NV

Contributors
~~~~~~~~~~~~

* Denis Roussel <[email protected]>

Maintainers
~~~~~~~~~~~

This module is maintained by the OCA.

.. image:: https://odoo-community.org/logo.png
:alt: Odoo Community Association
:target: https://odoo-community.org

OCA, or the Odoo Community Association, is a nonprofit organization whose
mission is to support the collaborative development of Odoo features and
promote its widespread use.

This module is part of the `OCA/stock-logistics-availability <https://github.com/OCA/stock-logistics-availability/tree/16.0/stock_available_location_get_domain>`_ project on GitHub.

You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
1 change: 1 addition & 0 deletions stock_available_location_get_domain/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
from . import models
14 changes: 14 additions & 0 deletions stock_available_location_get_domain/__manifest__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
# Copyright 2023 ACSONE SA/NV
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).

{
"name": "Stock Available Location Get Domain",
"summary": """
This is a technical helper module in order to reuse the standard
_get_domain_locations() function for locations and not quants""",
"version": "16.0.1.0.0",
"license": "AGPL-3",
"author": "ACSONE SA/NV,Odoo Community Association (OCA)",
"website": "https://github.com/OCA/stock-logistics-availability",
"depends": ["stock"],
}
1 change: 1 addition & 0 deletions stock_available_location_get_domain/models/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
from . import product_product
33 changes: 33 additions & 0 deletions stock_available_location_get_domain/models/product_product.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
# Copyright 2023 ACSONE SA/NV
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
from odoo import models
from odoo.osv import expression


class ProductProduct(models.Model):

_inherit = "product.product"

def _get_domain_location_for_locations(self):
"""
Adapt the domain computed for stock.quant for stock.location
"""
quant_domain = self._get_domain_locations()[0]
# Adapt the domain on quants by replacing location_id by ""
# Be sure to exclude potential fields that couldn't belong to stock.location
# and replace by nothing
location_domain = []
location_fields = self.env["stock.location"].fields_get()
for element in quant_domain:
leaf = expression.is_leaf(element)
if leaf:

Choose a reason for hiding this comment

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

leaf can also be a boolean

Suggested change
if leaf:
if leaf and isinstance(leaf, str):

field = leaf.split(".")[0]
if field in location_fields and field != "location_id":
location_domain.append(element)

Check warning on line 26 in stock_available_location_get_domain/models/product_product.py

View check run for this annotation

Codecov / codecov/patch

stock_available_location_get_domain/models/product_product.py#L26

Added line #L26 was not covered by tests
elif field == "location_id":

Choose a reason for hiding this comment

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

You miss the use case where the leaf is just "location_id"

Suggested change
elif field == "location_id":
elif leaf == "location_id":
location_domain.append(("id", element[1], element[2]))
elif field == "location_id":

location_domain.append(
(element[0].replace("location_id.", ""), element[1], element[2])
)
else:
location_domain.append(element)
return location_domain
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
* Denis Roussel <[email protected]>
2 changes: 2 additions & 0 deletions stock_available_location_get_domain/readme/DESCRIPTION.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
This is a technical helper module in order to reuse the standard
_get_domain_locations function for locations and not quants.
3 changes: 3 additions & 0 deletions stock_available_location_get_domain/readme/USAGE.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
Call the new _get_domain_location_for_locations() function on products in order
to retrieve a domain for stock locations with the same context behavior as
for product availability.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading