[16.0][ADD] stock_available_location_get_domain #11
Conversation
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
|
Is this feature available for v.15 too? |
github-actions
bot
removed
the
stale
@Sridharksolves Not yet. But if we merge this, we can backport. Could you review this ? |
| field = leaf.split(".")[0] | ||
| if field in location_fields and field != "location_id": | ||
| location_domain.append(element) | ||
| elif field == "location_id": |
There was a problem hiding this comment.
You miss the use case where the leaf is just "location_id"
| elif field == "location_id": | |
| elif leaf == "location_id": | |
| location_domain.append(("id", element[1], element[2])) | |
| elif field == "location_id": |
| location_fields = self.env["stock.location"].fields_get() | ||
| for element in quant_domain: | ||
| leaf = expression.is_leaf(element) | ||
| if leaf: |
There was a problem hiding this comment.
leaf can also be a boolean
| if leaf: | |
| if leaf and isinstance(leaf, str): |
jbaudoux
changed the title
|
@jbaudoux Thanks for review. I will add those cases. |
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
bot
added
the
stale
No description provided.