removed -SNAPSHOT from lower ranges

Reviewed-on: #4

CHANGELOG.md

@@ -4,7 +4,17 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [v2.2.1-SNAPSHOT] - 2024-29-02
+## [v2.2.2] - 2024-05-14
+
+**Bug fixes**
+
+- Incident Catalogue edit is leading to duplicate fields [#27455]
+
+**Enhancement**
+
+- The Catalogue should read the (Liferay) Highest Role in the VRE (i.e. the Organization) [#27467]
+
+## [v2.2.1] - 2024-03-19
 
 **Enhancement**
 

pom.xml

@@ -13,7 +13,7 @@
 
 	<groupId>org.gcube.portlets.widgets</groupId>
 	<artifactId>ckan-metadata-publisher-widget</artifactId>
-	<version>2.2.1-SNAPSHOT</version>
+	<version>2.2.2</version>
 	<name>gCube Ckan metadata publisher widget</name>
 
 	<description>
@@ -97,7 +97,7 @@
 		<dependency>
 			<groupId>org.gcube.portlets.widgets</groupId>
 			<artifactId>metadata-profile-form-builder-widget</artifactId>
-			<version>[2.0.0-SNAPSHOT, 3.0.0-SNAPSHOT)</version>
+			<version>[2.0.0, 3.0.0-SNAPSHOT)</version>
 		</dependency>
 		<dependency>
 			<groupId>org.gcube.data-catalogue</groupId>
@@ -107,7 +107,7 @@
 		<dependency>
 			<groupId>org.gcube.portlets.user</groupId>
 			<artifactId>uri-resolver-manager</artifactId>
-			<version>[1.0.0, 2.0.0-SNAPSHOT)</version>
+			<version>[1.7.0, 2.0.0-SNAPSHOT)</version>
 		</dependency>
 		<dependency>
 			<groupId>junit</groupId>
@@ -154,7 +154,7 @@
 		<dependency>
 			<groupId>org.gcube.datacatalogue</groupId>
 			<artifactId>catalogue-util-library</artifactId>
-			<version>[1.0.0-SNAPSHOT, 2.0.0)</version>
+			<version>[1.0.0, 2.0.0)</version>
 			<scope>compile</scope>
 		</dependency>
 

src/main/java/org/gcube/portlets/widgets/ckandatapublisherwidget/client/ui/form/UpdateDatasetForm.java

@@ -9,6 +9,7 @@ import java.util.List;
 import java.util.Map;
 import java.util.Map.Entry;
 import java.util.Set;
+import java.util.stream.Collectors;
 
 import org.gcube.portlets.widgets.ckandatapublisherwidget.client.CKanMetadataPublisher;
 import org.gcube.portlets.widgets.ckandatapublisherwidget.client.CKanPublisherServiceAsync;
@@ -566,6 +567,8 @@ public class UpdateDatasetForm extends Composite {
 											public void onSuccess(
 													final MetadataProfileBeanForUpdate metadataProfileForUpdate) {
 												setAlertBlock("", null, AlertType.ERROR, false);
+												
+												GWT.log("MetadataProfileBeanForUpdate: "+metadataProfileForUpdate);
 
 												List<MetaDataProfileBean> profiles = metadataProfileForUpdate
 														.getListProfileBean();
@@ -817,6 +820,7 @@ public class UpdateDatasetForm extends Composite {
 
 				GWT.log("There are " + categories.size() + " categories for profile " + bean.getTitle());
 
+				//Adding fields without categories
 				if (categories == null || categories.isEmpty()) {
 					for (MetadataFieldWrapper field : fields) {
 
@@ -830,12 +834,41 @@ public class UpdateDatasetForm extends Composite {
 						}
 					}
 				} else {
+					
+					// create the categories, then parse the fields. Fields do not belonging to a
+					// category are put at the end
+					
+					//Collecting and showing the fields for categories
+					for (CategoryWrapper categoryWrapper : categories) {
+						
+						List<MetadataFieldWrapper> fieldsForThisCategory = fields.stream().filter(f -> f.getOwnerCategory().getId()==categoryWrapper.getId()).collect(Collectors.toList());
+						
+						CategoryPanel cp = new CategoryPanel(categoryWrapper.getTitle(),
+								categoryWrapper.getDescription());
+						fields.removeAll(fieldsForThisCategory);
+
+						for (MetadataFieldWrapper metadataFieldWrapper : fieldsForThisCategory) {
+
+							MetaDataField fieldWidget;
+							try {
+								fieldWidget = new MetaDataField(metadataFieldWrapper, eventBusPublisherWidget,
+										operation);
+								cp.addField(fieldWidget);
+								listOfMetadataFields.add(fieldWidget);
+							} catch (Exception e) {
+								GWT.log("Unable to build such widget", e);
+							}
+						}
+						metadataFieldsPanel.add(cp);
+					}
+					
 
 					// create the categories, then parse the fields. Fields do not belonging to a
 					// category are put at the end
-					for (CategoryWrapper categoryWrapper : categories) {
+					/*for (CategoryWrapper categoryWrapper : categories) {
 						if (categoryWrapper.getFieldsForThisCategory() != null
 								&& categoryWrapper.getFieldsForThisCategory().size() > 0) {
+							
 							CategoryPanel cp = new CategoryPanel(categoryWrapper.getTitle(),
 									categoryWrapper.getDescription());
 							List<MetadataFieldWrapper> fieldsForThisCategory = categoryWrapper
@@ -856,7 +889,7 @@ public class UpdateDatasetForm extends Composite {
 							}
 							metadataFieldsPanel.add(cp);
 						}
-					}
+					}*/
 
 					// add the remaining one at the end of the categories
 					CategoryPanel extrasCategory = new CategoryPanel("Other", null);

src/main/java/org/gcube/portlets/widgets/ckandatapublisherwidget/server/CKANPublisherServicesImpl.java

@@ -29,6 +29,7 @@ import org.gcube.datacatalogue.utillibrary.shared.jackan.model.CkanDataset;
 import org.gcube.datacatalogue.utillibrary.shared.jackan.model.CkanGroup;
 import org.gcube.datacatalogue.utillibrary.shared.jackan.model.CkanLicense;
 import org.gcube.datacatalogue.utillibrary.shared.jackan.model.CkanOrganization;
+import org.gcube.datacatalogue.utillibrary.shared.jackan.model.CkanPair;
 import org.gcube.datacatalogue.utillibrary.shared.jackan.model.CkanResource;
 import org.gcube.datacatalogue.utillibrary.shared.jackan.model.CkanTag;
 import org.gcube.portlets.user.uriresolvermanager.UriResolverManager;
@@ -485,9 +486,12 @@ public class CKANPublisherServicesImpl extends RemoteServiceServlet implements C
 			}
 			bean.setResources(list);
 		}
-
+		
+		logger.debug("extras are #: "+dataset.getExtras().size());
 		// Settings the dataset type
 		Map<String, List<String>> extras = dataset.getListExtrasAsHashMap();
+		printExtras(dataset);
+	
 		if (extras != null) {
 			List<String> theDatasetType = extras.get(SYS_TYPE);
 			if (theDatasetType != null && theDatasetType.size() > 0) {
@@ -499,6 +503,18 @@ public class CKANPublisherServicesImpl extends RemoteServiceServlet implements C
 		logger.info("Returning the bean for dataset title {} and type {}" + bean.getTitle(), bean.getChosenType());
 		return bean;
 	}
+	
+	private void printExtras(CkanDataset dataset) {
+		if(logger.isDebugEnabled()) {
+			logger.trace("extras are: "+dataset.getExtras());
+			if (dataset.getExtras() != null) {
+				int i = 0;
+				for (CkanPair cp : dataset.getExtras()) {
+					logger.debug(++i+") key:" + cp.getKey() + ", value: "+cp.getValue());
+				}
+			}
+		}
+	}
 
 	/**
 	 * Discover from the IS the vocabulary of tags for this scope, if present.
@@ -1277,7 +1293,7 @@ public class CKANPublisherServicesImpl extends RemoteServiceServlet implements C
 			String scopePerCurrentUrl = GenericUtils.getScopeFromClientUrl(getThreadLocalRequest());
 
 			if (!isWithinPortal()) {
-				role = RolesCkanGroupOrOrg.EDITOR;
+				role = RolesCkanGroupOrOrg.ADMIN;
 				logger.warn("OUT FROM PORTAL SETTING HARD-CODED ROLE " + role);
 			} else {
 				String keyPerScopeRole = CatalogueUtilMethods

src/main/java/org/gcube/portlets/widgets/ckandatapublisherwidget/server/utils/CatalogueRoleManager.java

@@ -2,6 +2,7 @@ package org.gcube.portlets.widgets.ckandatapublisherwidget.server.utils;
 
 import java.util.HashSet;
 import java.util.List;
+import java.util.Optional;
 import java.util.Set;
 
 import org.gcube.datacatalogue.utillibrary.server.DataCatalogue;
@@ -23,17 +24,18 @@ import org.gcube.vomanagement.usermanagement.model.GatewayRolesNames;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-
-
 /**
  * Facilities to check roles into the catalogue.
  */
 public class CatalogueRoleManager {
-	
+
 	private static final Logger logger = LoggerFactory.getLogger(CatalogueRoleManager.class);
 
 	/**
-	 * Retrieve the highest ckan role the user has and also retrieve the list of organizations (scopes) in which the user has the ckan-admin or ckan-editor role
+	 * Retrieve the highest ckan role the user has and also retrieve the list of
+	 * organizations (scopes) in which the user has the ckan-admin or ckan-editor
+	 * role
+	 * 
 	 * @param currentScope
 	 * @param username
 	 * @param groupName
@@ -41,12 +43,14 @@ public class CatalogueRoleManager {
 	 * @param orgsInWhichAtLeastEditorRole
 	 * @return the highest among the roles
 	 */
-	public static RolesCkanGroupOrOrg getHighestRole(String currentScope, String username, String groupName, CKANPublisherServicesImpl gcubeCkanDataCatalogServiceImpl, List<OrganizationBean> orgsInWhichAtLeastEditorRole, String gatewayHostname){
+	public static RolesCkanGroupOrOrg getHighestRole(String currentScope, String username, String groupName,
+			CKANPublisherServicesImpl gcubeCkanDataCatalogServiceImpl,
+			List<OrganizationBean> orgsInWhichAtLeastEditorRole, String gatewayHostname) {
 
 		// base role as default value
 		RolesCkanGroupOrOrg toReturn = RolesCkanGroupOrOrg.MEMBER;
 
-		try{
+		try {
 
 			UserManager userManager = new LiferayUserManager();
 			RoleManager roleManager = new LiferayRoleManager();
@@ -58,80 +62,89 @@ public class CatalogueRoleManager {
 			// retrieve current group id
 			long currentGroupId = groupManager.getGroupIdFromInfrastructureScope(currentScope);
 
-			logger.debug("Group id is " + currentGroupId + " and scope is " + currentScope + "and gateway is "+gatewayHostname);
-			
+			logger.debug("Group id is " + currentGroupId + " and scope is " + currentScope + "and gateway is "
+					+ gatewayHostname);
+
 			Set<GCubeGroup> groups;
-			//Updated by Francesco see Task #12480
-			if(gatewayHostname!=null && !gatewayHostname.isEmpty()) {
+			// Updated by Francesco see Task #12480
+			if (gatewayHostname != null && !gatewayHostname.isEmpty()) {
 				// retrieve the list of organizations for the current user filtered for gateway
 				groups = groupManager.listGroupsByUserAndSite(userid, gatewayHostname);
 				for (GCubeGroup gCubeGroup : groups) {
-					logger.info("Found group (alias VRE) belonging to "+gatewayHostname+": "+gCubeGroup.getGroupName());
+					logger.info("Found group (alias VRE) belonging to " + gatewayHostname + ": "
+							+ gCubeGroup.getGroupName());
 				}
-			}else {
+			} else {
 				// retrieve the flat list of organizations for the current user
 				List<GCubeGroup> listGroups = groupManager.listGroupsByUser(userid);
 				groups = new HashSet<GCubeGroup>(listGroups);
 			}
-			
-			//Used by Workspace?
-			// root (so check into the root, the VOs and the VRES)
-			if(groupManager.isRootVO(currentGroupId)){
 
-				logger.info("The current scope is the Root Vo, so the list of organizations of the user " + username + " has " + groups.size() + " group/s");
+			// Used by Workspace?
+			// root (so check into the root, the VOs and the VRES)
+			if (groupManager.isRootVO(currentGroupId)) {
+
+				logger.info("The current scope is the Root Vo, so the list of organizations of the user " + username
+						+ " has " + groups.size() + " group/s");
 
 				for (GCubeGroup gCubeGroup : groups) {
-					
-					if(!groupManager.isVRE(gCubeGroup.getGroupId()))
+
+					if (!groupManager.isVRE(gCubeGroup.getGroupId()))
 						continue;
 
 					// get the name of this group
 					String gCubeGroupName = gCubeGroup.getGroupName();
-					
+
 					logger.info("Cheking role of the user " + username + " in the VRE " + gCubeGroupName);
-					
+
 					// get the role of the users in this group
-					List<GCubeRole> roles = roleManager.listRolesByUserAndGroup(userid, groupManager.getGroupId(gCubeGroupName));
+					List<GCubeRole> roles = roleManager.listRolesByUserAndGroup(userid,
+							groupManager.getGroupId(gCubeGroupName));
 
 					// get highest role
 					RolesCkanGroupOrOrg correspondentRoleToCheck = getLiferayHighestRoleInOrg(roles);
 
 					// be sure it is so
-					checkIfRoleIsSetInCkanInstance(username, gCubeGroupName, gCubeGroup.getGroupId(), 
-							correspondentRoleToCheck, groupManager, gcubeCkanDataCatalogServiceImpl, orgsInWhichAtLeastEditorRole);
+					checkIfRoleIsSetInCkanInstance(username, gCubeGroupName, gCubeGroup.getGroupId(),
+							correspondentRoleToCheck, groupManager, gcubeCkanDataCatalogServiceImpl,
+							orgsInWhichAtLeastEditorRole);
 
 					toReturn = RolesCkanGroupOrOrg.getHigher(toReturn, correspondentRoleToCheck);
-					
-					logger.info("Found the role "+toReturn+" for " + username + " in the VRE " + gCubeGroupName);
+
+					logger.info("Found the role " + toReturn + " for " + username + " in the VRE " + gCubeGroupName);
 
 				}
 
-			}else if(groupManager.isVO(currentGroupId)){
+			} else if (groupManager.isVO(currentGroupId)) {
 
-				logger.debug("The list of organizations of the user " + username + " to scan is the one under the VO " + groupName);
+				logger.debug("The list of organizations of the user " + username + " to scan is the one under the VO "
+						+ groupName);
 
 				for (GCubeGroup gCubeGroup : groups) {
 
 					// if the gCubeGroup is not under the VO or it is not the VO continue
-					if(currentGroupId != gCubeGroup.getParentGroupId() || currentGroupId != gCubeGroup.getGroupId())
+					if (currentGroupId != gCubeGroup.getParentGroupId() || currentGroupId != gCubeGroup.getGroupId())
 						continue;
 
 					String gCubeGroupName = gCubeGroup.getGroupName();
 
-					List<GCubeRole> roles = roleManager.listRolesByUserAndGroup(userid, groupManager.getGroupId(gCubeGroupName));
+					List<GCubeRole> roles = roleManager.listRolesByUserAndGroup(userid,
+							groupManager.getGroupId(gCubeGroupName));
 
 					// get highest role
 					RolesCkanGroupOrOrg correspondentRoleToCheck = getLiferayHighestRoleInOrg(roles);
 
 					// be sure it is so
-					checkIfRoleIsSetInCkanInstance(username, gCubeGroupName, gCubeGroup.getGroupId(), 
-							correspondentRoleToCheck, groupManager, gcubeCkanDataCatalogServiceImpl, orgsInWhichAtLeastEditorRole);
+					checkIfRoleIsSetInCkanInstance(username, gCubeGroupName, gCubeGroup.getGroupId(),
+							correspondentRoleToCheck, groupManager, gcubeCkanDataCatalogServiceImpl,
+							orgsInWhichAtLeastEditorRole);
 
 					toReturn = RolesCkanGroupOrOrg.getHigher(toReturn, correspondentRoleToCheck);
 				}
 
-			}else if(groupManager.isVRE(currentGroupId)){
-				List<GCubeRole> roles = roleManager.listRolesByUserAndGroup(userManager.getUserId(username), groupManager.getGroupId(groupName));
+			} else if (groupManager.isVRE(currentGroupId)) {
+				List<GCubeRole> roles = roleManager.listRolesByUserAndGroup(userManager.getUserId(username),
+						groupManager.getGroupId(groupName));
 
 				logger.debug("The current scope is the vre " + groupName);
 
@@ -139,24 +152,26 @@ public class CatalogueRoleManager {
 				RolesCkanGroupOrOrg correspondentRoleToCheck = getLiferayHighestRoleInOrg(roles);
 
 				// be sure it is so
-				checkIfRoleIsSetInCkanInstance(username, groupName, currentGroupId, 
-						correspondentRoleToCheck, groupManager, gcubeCkanDataCatalogServiceImpl, orgsInWhichAtLeastEditorRole);
+				checkIfRoleIsSetInCkanInstance(username, groupName, currentGroupId, correspondentRoleToCheck,
+						groupManager, gcubeCkanDataCatalogServiceImpl, orgsInWhichAtLeastEditorRole);
 
 				toReturn = correspondentRoleToCheck;
 
 			}
-		}catch(Exception e){
+		} catch (Exception e) {
 			logger.error("Unable to retrieve the role information for this user. Returning member role", e);
 			return RolesCkanGroupOrOrg.MEMBER;
 		}
 
 		// return the role
-		logger.debug("Returning role "  + toReturn + " for user " + username);
+		logger.debug("Returning role " + toReturn + " for user " + username);
 		return toReturn;
 	}
 
 	/**
-	 * Check if the role admin is set or must be set into the ckan instance at this scope
+	 * Check if the role admin is set or must be set into the ckan instance at this
+	 * scope
+	 * 
 	 * @param username
 	 * @param gCubeGroupName
 	 * @param groupId
@@ -167,43 +182,53 @@ public class CatalogueRoleManager {
 	 * @throws UserManagementSystemException
 	 * @throws GroupRetrievalFault
 	 */
-	private static void checkIfRoleIsSetInCkanInstance(String username,
-			String gCubeGroupName, long groupId,
-			RolesCkanGroupOrOrg correspondentRoleToCheck,
-			GroupManager groupManager, CKANPublisherServicesImpl ckanPublisherServicesImpl, List<OrganizationBean> orgs) throws UserManagementSystemException, GroupRetrievalFault {
+	private static void checkIfRoleIsSetInCkanInstance(String username, String gCubeGroupName, long groupId,
+			RolesCkanGroupOrOrg correspondentRoleToCheck, GroupManager groupManager,
+			CKANPublisherServicesImpl ckanPublisherServicesImpl, List<OrganizationBean> orgs)
+			throws UserManagementSystemException, GroupRetrievalFault {
 
-		// with this invocation, we check if the role is present in ckan and if it is not it will be added
+		// with this invocation, we check if the role is present in ckan and if it is
+		// not it will be added
 		DataCatalogue catalogue = ckanPublisherServicesImpl.getCatalogue(groupManager.getInfrastructureScope(groupId));
 
 		// if there is an instance of ckan in this scope..
-		if(catalogue != null){
+		if (catalogue != null) {
 			boolean res = catalogue.checkRoleIntoOrganization(username, gCubeGroupName, correspondentRoleToCheck);
-			if(res && !correspondentRoleToCheck.equals(RolesCkanGroupOrOrg.MEMBER)){
+			if (res && !correspondentRoleToCheck.equals(RolesCkanGroupOrOrg.MEMBER)) {
 				// get the orgs of the user and retrieve its title and name
 				CkanOrganization organization = catalogue.getOrganizationByName(gCubeGroupName.toLowerCase());
 				orgs.add(new OrganizationBean(organization.getTitle(), organization.getName(), true));
 			}
-		}else
-			logger.warn("It seems there is no ckan instance into scope " + groupManager.getInfrastructureScope(groupId));
+		} else
+			logger.warn(
+					"It seems there is no ckan instance into scope " + groupManager.getInfrastructureScope(groupId));
 	}
 
 	/**
-	 * Retrieve the ckan roles among a list of liferay roles
+	 * Retrieve the highest ckan role among a list of liferay roles
+	 * 
 	 * @param roles
-	 * @return
+	 * @return Updated by Francesco M. see #27467
 	 */
-	private static RolesCkanGroupOrOrg getLiferayHighestRoleInOrg(
-			List<GCubeRole> roles) {
-		// NOTE: it is supposed that there is just one role for this person correspondent to the one in the catalog
-		for (GCubeRole gCubeRole : roles) {
-			if(gCubeRole.getRoleName().equalsIgnoreCase(GatewayRolesNames.CATALOGUE_ADMIN.getRoleName())){
-				return RolesCkanGroupOrOrg.ADMIN;
-			}
-			if(gCubeRole.getRoleName().equalsIgnoreCase(GatewayRolesNames.CATALOGUE_EDITOR.getRoleName())){
-				return RolesCkanGroupOrOrg.EDITOR;
-			}
-		}
+	private static RolesCkanGroupOrOrg getLiferayHighestRoleInOrg(List<GCubeRole> roles) {
+
+		if (roles == null || roles.isEmpty())
+			return RolesCkanGroupOrOrg.MEMBER;
+
+		Optional<GCubeRole> admin = roles.stream()
+				.filter(r -> r.getRoleName().equalsIgnoreCase(GatewayRolesNames.CATALOGUE_ADMIN.getRoleName()))
+				.findFirst();
+
+		if (admin.isPresent())
+			return RolesCkanGroupOrOrg.ADMIN;
+
+		Optional<GCubeRole> editor = roles.stream()
+				.filter(r -> r.getRoleName().equalsIgnoreCase(GatewayRolesNames.CATALOGUE_EDITOR.getRoleName()))
+				.findFirst();
+
+		if (editor.isPresent())
+			return RolesCkanGroupOrOrg.EDITOR;
+
 		return RolesCkanGroupOrOrg.MEMBER;
 	}
-	
 }