midsuit
/
core-jgi
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

minor fix on MRole - misleading comment - typos on comments - add useful log information

This commit is contained in:
Carlos Ruiz 2015-07-28 09:03:12 -05:00
parent f05a59eeab
commit 8726f5eb77
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
org.adempiere.base/src/org/compiere/model/MRole.java
View File

@ -2154,10 +2154,10 @@ public final class MRole extends X_AD_Role
/** /**
* UPADATE - Can I Update the record. * UPDATE - Can I Update the record.
* Access error info (AccessTableNoUpdate) is saved in the log * Access error info (AccessTableNoUpdate) is saved in the log
* *
* @param AD_Client_ID comntext to derive client/org/user level * @param AD_Client_ID context to derive client/org/user level
* @param AD_Org_ID number of the current window to retrieve context * @param AD_Org_ID number of the current window to retrieve context
* @param AD_Table_ID table * @param AD_Table_ID table
* @param Record_ID record id * @param Record_ID record id
@ -2170,7 +2170,7 @@ public final class MRole extends X_AD_Role
{ {
String userLevel = getUserLevel(); // Format 'SCO' String userLevel = getUserLevel(); // Format 'SCO'
if (userLevel.indexOf('S') != -1) // System cannot change anything if (userLevel.indexOf('S') != -1) // System can change anything
return true; return true;
boolean retValue = true; boolean retValue = true;
@ -2208,6 +2208,7 @@ public final class MRole extends X_AD_Role
// Client Access: Verify if the role has access to the given client - teo_sarca, BF [ 1982398 ] // Client Access: Verify if the role has access to the given client - teo_sarca, BF [ 1982398 ]
if (retValue) { if (retValue) {
retValue = isClientAccess(AD_Client_ID, true); // r/w access retValue = isClientAccess(AD_Client_ID, true); // r/w access
whatMissing += "C";
} }
// Org Access: Verify if the role has access to the given organization - teo_sarca, patch [ 1628050 ] // Org Access: Verify if the role has access to the given organization - teo_sarca, patch [ 1628050 ]