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

Syncing from upstream Vauxoo/addons-vauxoo (16.0) #325

Merged
merged 7 commits into from
May 17, 2024

Conversation

bt-admin
Copy link

bt_gitbot

Alejandro Garza and others added 7 commits May 15, 2024 17:17
- New module added in order to allow the user to create stock transfers
  manually.
Due to upstream updates adding support for @api.ondelete and
pre-commit-vauxoo detecting the override of .unlink() method as
a code smell, this transforms the old logic into more
idiomatic odoo code.
…inal commit

While the original fix (cbd70ff)
did fix the immediate lint messages, we didn't actually stop it from
calling super().unlink(), thus it didn't actually fix what it intended
to fix. Here we remove that call to super().unlink() so the issue is
properly fixed.
Add domain to route_id field to show the only the routes that are
supplied_wh_id = warehouse_id. This is to avoid
inconsistencies between the rule and the warehouse.
To prevent any potential duplication, this fix introduces a
return-early pattern. Now, transfers are not created if the manual
transfer record has already been published.

Issue reported on helpdesk ticket #25879
- Changed name for deprecated models like stock.location.route to
  stock.route and stock move to Operations(stock.move.line) to Stock
  Moves.
- Added user to group uom to fix unit tests and avoid error for not
  null violation in product uom.
…ithout group

A validation error is raised when a user without the "Manage Multiple
Units of Measure" access tries to save or validate a manual transfer.

This fix allows any user to create manual transfers, but only users with
"Units of Measure" access can modify this attribute on the lines.
@bt-admin bt-admin added the 16.0 label May 17, 2024
@bt-admin bt-admin merged commit 60dbc12 into brain-tec:16.0 May 17, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants