Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0025709Openbravo ERPA. Platformpublic2014-02-12 10:332014-02-18 22:19
Reportereduardo_Argal 
Assigned ToAugustoMauch 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR14Q2 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0025709: SecuredProcess preference not working as expected in some conditions
DescriptionSecuredProcess is a preference you can set so that permissions for processes/buttons within a window are not directly inherited by roles which have access to that window.
This is not working properly in certain conditions
Steps To ReproduceCreate a preference for Financial account window using SecuredProcess property.
Create a role which has access just to financial account window
create a user and assign just previously created role
Log in using this new user

Realize that buttons in the header of the window are not grayed out, but that when selecting a record on any child tab those are not grayed out when they should. Some of those even when they are not grayed out throws an error when trying to execute them while others are just enabled for the user to be executed.

All buttons should be grayed out, and all should raise an error when trying to execute them
Proposed SolutionAll buttons should be grayed out, and all should raise an error when trying to execute them
Additional Information
TagsNo tags attached.
Relationships
related to feature request 00257203.0PR14Q2 closed alostale Adds configuration option for processes to explicitly require permissions to be able to execute it 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2014-02-12 10:33eduardo_ArgalNew Issue
2014-02-12 10:33eduardo_ArgalAssigned To => AugustoMauch
2014-02-12 10:33eduardo_ArgalModules => Core
2014-02-12 10:33eduardo_ArgalOBNetwork customer => Yes
2014-02-12 10:33eduardo_ArgalTriggers an Emergency Pack => No
2014-02-13 11:45eduardo_ArgalOBNetwork customerYes => No
2014-02-13 11:45eduardo_ArgalTarget Version3.0MP32 =>
2014-02-17 13:44alostaleRelationship addedrelated to 0025720
2014-02-18 07:57alostaleNote Added: 0064306
2014-02-18 07:57alostaleReview Assigned To => shankarb
2014-02-18 08:00hgbotCheckin
2014-02-18 08:00hgbotNote Added: 0064307
2014-02-18 08:00hgbotStatusnew => resolved
2014-02-18 08:00hgbotResolutionopen => fixed
2014-02-18 08:00hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/2d05b68d4f4f14367c2d0c316a0d4c90f4d92fba [^]
2014-02-18 19:46hudsonbotCheckin
2014-02-18 19:46hudsonbotNote Added: 0064331
2014-02-18 22:19AugustoMauchReview Assigned Toshankarb => AugustoMauch
2014-02-18 22:19AugustoMauchNote Added: 0064335
2014-02-18 22:19AugustoMauchStatusresolved => closed
2014-02-18 22:19AugustoMauchFixed in Version => 3.0MP32

Notes
(0064306)
alostale   
2014-02-18 07:57   
The problem is in buttons defined in a parent tab when the focus is in one of its children. In this case only the first button in the parent tab is grayed out.
(0064307)
hgbot   
2014-02-18 08:00   
Repository: erp/devel/pi
Changeset: 2d05b68d4f4f14367c2d0c316a0d4c90f4d92fba
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Feb 18 07:58:51 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/2d05b68d4f4f14367c2d0c316a0d4c90f4d92fba [^]

fixed bug 25709: header buttons not correctly disabled focusing in child tab

---
M modules/org.openbravo.client.application/src/org/openbravo/client/application/WindowSettingsActionHandler.java
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-window.js
---
(0064331)
hudsonbot   
2014-02-18 19:46   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/bf9e3b7ea60c [^]
Maturity status: Test
(0064335)
AugustoMauch   
2014-02-18 22:19   
Code reviewed and verified in pi@e7619b809092