done #20801
This commit is contained in:
parent
2c69842452
commit
bfd9e01841
|
@ -615,10 +615,7 @@ public class CKANPublisherServicesImpl extends RemoteServiceServlet implements C
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* The method tries to retrieve the scope related to the organization using the map first,
|
* The method tries to retrieve the scope related to the organization using the map first,
|
||||||
* if no match is found, it retrieves such information by using liferay.
|
* if no match is found, it retrieves such information by using liferay
|
||||||
*
|
|
||||||
* @param orgName the org name
|
|
||||||
* @return the scope from org name
|
|
||||||
*/
|
*/
|
||||||
private String getScopeFromOrgName(String orgName){
|
private String getScopeFromOrgName(String orgName){
|
||||||
|
|
||||||
|
@ -629,11 +626,16 @@ public class CKANPublisherServicesImpl extends RemoteServiceServlet implements C
|
||||||
String toReturn = null;
|
String toReturn = null;
|
||||||
|
|
||||||
if(isWithinPortal()){
|
if(isWithinPortal()){
|
||||||
if(mapOrganizationScope.containsKey(orgName))
|
if(mapOrganizationScope.containsKey(orgName)) {
|
||||||
toReturn = mapOrganizationScope.get(orgName);
|
toReturn = mapOrganizationScope.get(orgName);
|
||||||
else{
|
}else{
|
||||||
try{
|
try{
|
||||||
String evaluatedScope = GenericUtils.retrieveScopeFromOrganizationName(orgName);
|
String evaluatedScope = GenericUtils.retrieveScopeFromOrganizationName(orgName);
|
||||||
|
//see #20801
|
||||||
|
if(evaluatedScope==null || evaluatedScope.isEmpty()) {
|
||||||
|
logger.warn("Scope detected for OrganizationName: "+orgName+" is null or empty, skipping filling 'mapOrganizationScope' and returning null");
|
||||||
|
return toReturn;
|
||||||
|
}
|
||||||
mapOrganizationScope.put(orgName, evaluatedScope);
|
mapOrganizationScope.put(orgName, evaluatedScope);
|
||||||
toReturn = evaluatedScope;
|
toReturn = evaluatedScope;
|
||||||
}catch(Exception e){
|
}catch(Exception e){
|
||||||
|
|
|
@ -52,13 +52,18 @@ public class GenericUtils {
|
||||||
GroupManager gm = new LiferayGroupManager();
|
GroupManager gm = new LiferayGroupManager();
|
||||||
List<GCubeGroup> groups = gm.listGroups();
|
List<GCubeGroup> groups = gm.listGroups();
|
||||||
for (GCubeGroup gCubeGroup : groups) {
|
for (GCubeGroup gCubeGroup : groups) {
|
||||||
if(gCubeGroup.getGroupName().equalsIgnoreCase(organizationName))
|
if(gCubeGroup.getGroupName().equalsIgnoreCase(organizationName)) {
|
||||||
return gm.getInfrastructureScope(gCubeGroup.getGroupId());
|
String theScope = gm.getInfrastructureScope(gCubeGroup.getGroupId());
|
||||||
|
logger.info("For organizationName: " + organizationName+" by using "+LiferayGroupManager.class.getSimpleName()+ " got the scope: "+theScope +", returning it");
|
||||||
|
return theScope;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
logger.info("No scope detected for organizationName: " + organizationName +" by using "+LiferayGroupManager.class.getSimpleName());
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* First check to retrieve the token, else create it.
|
* First check to retrieve the token, else create it.
|
||||||
*
|
*
|
||||||
|
|
Loading…
Reference in New Issue