IDEMPIERE-4693 2Pack Hangs in 7.1+ / IDEMPIERE-2767 (#570)

* IDEMPIERE-4693 2Pack Hangs in 7.1+ / IDEMPIERE-2767

* Decrease log level to INFO

* Add javadoc note
This commit is contained in:
Carlos Ruiz 2021-02-10 21:33:19 +01:00 committed by GitHub
parent ee77620186
commit 92195c08a4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 3 deletions

View File

@ -273,6 +273,8 @@ public final class MRole extends X_AD_Role implements ImmutablePOSupport
/************************************************************************** /**************************************************************************
* Standard Constructor * Standard Constructor
* NOTE - This method must not be used when the role is being requested to manage permissions,
* in such case is necessary to use one of the get methods setting the userID
* @param ctx context * @param ctx context
* @param AD_Role_ID id * @param AD_Role_ID id
* @param trxName transaction * @param trxName transaction
@ -303,6 +305,8 @@ public final class MRole extends X_AD_Role implements ImmutablePOSupport
/** /**
* Load Constructor * Load Constructor
* NOTE - This method must not be used when the role is being requested to manage permissions,
* in such case is necessary to use one of the get methods setting the userID
* @param ctx context * @param ctx context
* @param rs result set * @param rs result set
* @param trxName transaction * @param trxName transaction
@ -809,7 +813,7 @@ public final class MRole extends X_AD_Role implements ImmutablePOSupport
private void loadOrgAccessUser(ArrayList<OrgAccess> list) private void loadOrgAccessUser(ArrayList<OrgAccess> list)
{ {
if (getAD_User_ID() == -1) { if (getAD_User_ID() == -1) {
log.severe("Trying to load Org Access from User but user has not been set"); log.info("Trying to load Org Access from User but user has not been set");
} }
PreparedStatement pstmt = null; PreparedStatement pstmt = null;
ResultSet rs = null; ResultSet rs = null;
@ -2868,7 +2872,7 @@ public final class MRole extends X_AD_Role implements ImmutablePOSupport
final int AD_User_ID = getAD_User_ID(); final int AD_User_ID = getAD_User_ID();
if (AD_User_ID < 0) if (AD_User_ID < 0)
{ {
log.severe("Trying to load Child Roles but user has not been set"); log.info("Trying to load Child Roles but user has not been set");
//throw new IllegalStateException("AD_User_ID is not set"); //throw new IllegalStateException("AD_User_ID is not set");
return ; return ;
} }
@ -2904,7 +2908,7 @@ public final class MRole extends X_AD_Role implements ImmutablePOSupport
final int AD_User_ID = getAD_User_ID(); final int AD_User_ID = getAD_User_ID();
if (AD_User_ID < 0) if (AD_User_ID < 0)
{ {
log.severe("Trying to load Substituted Roles but user has not been set"); log.info("Trying to load Substituted Roles but user has not been set");
//throw new IllegalStateException("AD_User_ID is not set"); //throw new IllegalStateException("AD_User_ID is not set");
return; return;
} }