Skip to content

[14.0][MIG] stock_warehouse_security - #2602

Open
sebalix wants to merge 15 commits into
OCA:14.0from
akretion:14.0-mig-stock_warehouse_security
Open

sebalix wants to merge 15 commits into
OCA:14.0from
akretion:14.0-mig-stock_warehouse_security

Conversation

@sebalix

@sebalix sebalix commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Backported from 16.0.
To satisfy the security rules on stock.location, stock.quant and stock.quant.package models, the dependency on stock_location_warehouse has been added. These rules for quants and packages have been adapted to use the warehouse_id field of the location too.

@OCA-git-bot OCA-git-bot added mod:stock_warehouse_security Module stock_warehouse_security series:14.0 labels Jun 23, 2026
@florian-dacosta

Copy link
Copy Markdown
Contributor

Hello @sebalix
Did you check #1927 ?
It was using #1924 to add warehouse_id where it is missing.

I am not following these module anymore though, I don't know how it is now on recent version and everything, I am just letting you know in case you were not aware of these old PRs

@sebalix

sebalix commented Jun 23, 2026

Copy link
Copy Markdown
Contributor Author

@florian-dacosta huge thanks, I didn't spot that one 🤦‍♂️ I'll test it and close mine afterwards

@sebalix
sebalix force-pushed the 14.0-mig-stock_warehouse_security branch 2 times, most recently from f53b3ad to 64bd5a1 Compare June 23, 2026 10:36
Backported from 16.0.
To satisfy the security rules on `stock.location`, `stock.quant` and
`stock.quant.package` models, the dependency 'stock_location_warehouse'
has been added. These rules for quants and packages have been adapted to
use the `warehouse_id` field of the location too.
@sebalix
sebalix force-pushed the 14.0-mig-stock_warehouse_security branch from 64bd5a1 to 951e5f2 Compare June 23, 2026 10:41
@sebalix

sebalix commented Jun 23, 2026

Copy link
Copy Markdown
Contributor Author

@florian-dacosta I reviewed #1927 and I will finally keep my PR for now (with the addition of stock_location_warehouse dependency which makes current security rules working as expected, thanks).

Also, #1924 contains a breaking change that is not welcome with the workflow I'm working on sadly.

@sebalix
sebalix marked this pull request as ready for review June 29, 2026 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:stock_warehouse_security Module stock_warehouse_security series:14.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants