search

frappe / erpnext

Free and Open Source Enterprise Resource Planning (ERP)
https://erpnext.com
GNU General Public License v3.0
21.91k stars 7.34k forks source link

Warehouse limit restriction in inventory reports #875

Closed lxnow closed 7 years ago

lxnow commented 11 years ago

Currently anyone who has access to inventory reports (warehouse-wise and stock level) can view inventory in all locations. We would like to be able to limit what the user can view based on his/her warehouse access. In other words, the warehouse restrictions (warehouse_user child table) is ignored when it comes to reporting.

The ideal solution should allow user to have a separate "read" access and separate "write" access so that a user could make transactions to warehouses he/she has authorization over, while being able to view inventory in other (but not all) warehouses.

Bounty: $100

--- Want to back this issue? **[Place a bounty on it!](https://www.bountysource.com/issues/988717-warehouse-limit-restriction-in-inventory-reports?utm_campaign=plugin&utm_content=tracker%2F266088&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F266088&utm_medium=issues&utm_source=github).
lxnow commented 11 years ago

Increased bounty.

lxnow commented 11 years ago

Or maybe ability to create and save custom reports that are then distributable to specific users?

rmehta commented 11 years ago

The issue here is a custom permission logic for warehouse - this needs to be re-thought. Maybe if User Property is used to restrict access, then the problem would not exist.

The problem in using user property is that the "warehouse" fieldname is separate for each transaction.

Another good solution (Really Good Idea) is that User Properties should be on Masters not fields (as it is currently designed). If this is the case, then for each master (like Warehouse, or Company) there could be a user list. And this standard condition should be automatically added to all queries like the current match condition. Big project, but this will solve a lot of permission related problems.

rmehta commented 10 years ago

The new user permissions should fix this?

anandpdoshi commented 10 years ago

Need to check in script reports

Sent from phone

On 08-May-2014, at 10:10 am, Rushabh Mehta notifications@github.com wrote:

The new user permissions should fix this?

— Reply to this email directly or view it on GitHub.