[FIX] stock_by_warehouse: add some sudos lost during migration to 13.0 i#17886

Vauxoo / addons-vauxoo

All our modules related to developments that solves generic issues on Odoo, or that solve internal problems on Odoo Core, if something is here, maybe it is solving an issue in your company, try it and report what you see.

http://www.vauxoo.com

193 stars 288 forks source link

[FIX] stock_by_warehouse: add some sudos lost during migration to 13.0 i#17886 #1523

Closed luisg123v closed 2 years ago

luisg123v commented 2 years ago

When migration of this module was performed from 12.0 to 13.0 (#1460), some sudo's were removed because we considered them no longer required. However, they actually are, in case there are record rules defined for the warehouse model. Even if the current user doesn't have enough access to see some warehouses, they need to know if there's available stock elsewhere.

Dummy MR:

vauxoo/typ!936

luisg123v commented 2 years ago

@desdelinux, @keylor2906, @hugho-ad,

Could you review, please?

CC @deivislaya

desdelinux commented 2 years ago

LGTM :)

There is a visual bug that we must attack in another mr.

When there are many warehouses: there is no scroll that allows you to scroll down to see them all, so you only see a certain amount.

cc @keylor2906 @deivislaya @hugho-ad @luisg123v