Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0038659 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
feature request | [Openbravo ERP] C. Security | minor | have not tried | 2018-05-30 15:54 | 2018-06-04 09:57 | |||
Reporter | alostale | View Status | public | |||||
Assigned To | alostale | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | 3.0PR18Q3 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 5ae34663c033 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | caristu | |||||||
OBNetwork customer | No | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0038659: new roles should be manual by default | |||||||
Description | When creating new Roles, it is possible to define whether access for all objects will be granted automatically or not (through the Manual flag) also in case of automatic access if access to advanced features will be granted, see doc [1]. Currently new roles are defaulted to automatically be granted to all possible objects by being not manual and advanced features. It is more common to have restricted roles with access to only some specific objects than roles with access to all of them. Current defaults are error prone: if the user creating a new role forgets to flag default "Manual" field and saves the new role, all objects will be automatically granted. Setting it correctly afterwards, forces to remove access to all those objects. --- [1] http://wiki.openbravo.com/wiki/Role [^] | |||||||
Steps To Reproduce | 1. Create a new role keeping all default values -> check there is access to all windows, reports, etc | |||||||
Proposed Solution | -Change defaults to Manual: Yes Advanced: No - If a role is updated afterwards from manual to automatic, grant access to all objects automatically | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
||||||||
|
![]() |
|
(0104842) hgbot (developer) 2018-05-31 14:25 |
Repository: erp/devel/pi Changeset: 5ae34663c0339d067e787c2e4f9784df253bbbfe Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Wed May 30 13:03:45 2018 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/5ae34663c0339d067e787c2e4f9784df253bbbfe [^] fixes 38659: new roles should be manual by default * By default, new roles are now maked as manual and not advanced * When a role is moved from manual to automatic, permissions are now recalculated * Roles created in Initial Client and Org setup are kept as privileged --- M src-db/database/model/tables/AD_ROLE.xml M src-db/database/model/triggers/AD_ROLE_TRG.xml M src-db/database/sourcedata/AD_COLUMN.xml M src/org/openbravo/erpCommon/businessUtility/InitialSetupUtility.java --- |
(0104904) caristu (viewer) 2018-06-04 09:57 |
Code reviewed + tested OK. |
![]() |
|||
Date Modified | Username | Field | Change |
2018-05-30 15:54 | alostale | New Issue | |
2018-05-30 15:54 | alostale | Assigned To | => alostale |
2018-05-30 15:54 | alostale | OBNetwork customer | => No |
2018-05-30 15:54 | alostale | Modules | => Core |
2018-05-30 15:54 | alostale | Triggers an Emergency Pack | => No |
2018-05-31 14:25 | hgbot | Checkin | |
2018-05-31 14:25 | hgbot | Note Added: 0104842 | |
2018-05-31 14:25 | hgbot | Status | new => resolved |
2018-05-31 14:25 | hgbot | Resolution | open => fixed |
2018-05-31 14:25 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/5ae34663c0339d067e787c2e4f9784df253bbbfe [^] |
2018-05-31 14:28 | alostale | Review Assigned To | => caristu |
2018-06-04 09:54 | caristu | Issue cloned | 0038689 |
2018-06-04 09:54 | caristu | Relationship added | related to 0038689 |
2018-06-04 09:57 | caristu | Note Added: 0104904 | |
2018-06-04 09:57 | caristu | Status | resolved => closed |
2018-06-04 09:57 | caristu | Fixed in Version | => 3.0PR18Q3 |
Copyright © 2000 - 2009 MantisBT Group |