feat: fix PR comments.

https://gravitee.atlassian.net/browse/APIM-11696

Author: kovaren

gravitee-apim-rest-api/gravitee-apim-rest-api-management-v2/gravitee-apim-rest-api-management-v2-rest/src/main/java/io/gravitee/rest/api/management/v2/rest/resource/environment/PortalNavigationItemsResource.java

@@ -78,7 +78,7 @@ public PortalNavigationItemsResponse getPortalNavigationItems(
     }
 
     @POST
-    @Permissions({ @Permission(value = RolePermission.ENVIRONMENT_SETTINGS, acls = { RolePermissionAction.UPDATE }) })
+    @Permissions({ @Permission(value = RolePermission.ENVIRONMENT_PAGE, acls = { RolePermissionAction.UPDATE }) })
     @Consumes(MediaType.APPLICATION_JSON)
     @Produces(MediaType.APPLICATION_JSON)
     public Response createPortalNavigationItem(@Valid @NotNull final BaseCreatePortalNavigationItem createPortalNavigationItem) {

gravitee-apim-rest-api/gravitee-apim-rest-api-model/src/main/java/io/gravitee/rest/api/model/permissions/EnvironmentPermission.java

@@ -49,7 +49,8 @@ public enum EnvironmentPermission implements Permission {
     IDENTITY_PROVIDER_ACTIVATION("IDENTITY_PROVIDER_ACTIVATION", 3700),
     INTEGRATION("INTEGRATION", 3800),
     SHARED_POLICY_GROUP("SHARED_POLICY_GROUP", 3900),
-    CLUSTER("CLUSTER", 4000);
+    CLUSTER("CLUSTER", 4000),
+    PAGE("PAGE", 4100);
 
     String name;
     int mask;

gravitee-apim-rest-api/gravitee-apim-rest-api-model/src/main/java/io/gravitee/rest/api/model/permissions/RolePermission.java

@@ -88,6 +88,7 @@ public enum RolePermission {
     ENVIRONMENT_INTEGRATION(RoleScope.ENVIRONMENT, EnvironmentPermission.INTEGRATION),
     ENVIRONMENT_SHARED_POLICY_GROUP(RoleScope.ENVIRONMENT, EnvironmentPermission.SHARED_POLICY_GROUP),
     ENVIRONMENT_CLUSTER(RoleScope.ENVIRONMENT, EnvironmentPermission.CLUSTER),
+    ENVIRONMENT_PAGE(RoleScope.ENVIRONMENT, EnvironmentPermission.PAGE),
 
     ORGANIZATION_USERS(RoleScope.ORGANIZATION, OrganizationPermission.USER),
     ORGANIZATION_USERS_TOKEN(RoleScope.ORGANIZATION, OrganizationPermission.USER_TOKEN),

gravitee-apim-rest-api/gravitee-apim-rest-api-service/src/main/java/io/gravitee/apim/core/portal_page/exception/HomepageAlreadyExistsException.java

@@ -19,7 +19,7 @@
 
 public class HomepageAlreadyExistsException extends ValidationDomainException {
 
-    public HomepageAlreadyExistsException(String message) {
-        super(message);
+    public HomepageAlreadyExistsException() {
+        super("Homepage already exists");
     }
 }

gravitee-apim-rest-api/gravitee-apim-rest-api-service/src/main/java/io/gravitee/apim/core/portal_page/exception/ParentAreaMismatchException.java

@@ -19,7 +19,7 @@
 
 public class ParentAreaMismatchException extends ValidationDomainException {
 
-    public ParentAreaMismatchException(String message) {
-        super(message);
+    public ParentAreaMismatchException(String parentId) {
+        super(String.format("Parent item with id %s belongs to a different area than the child item", parentId));
     }
 }

gravitee-apim-rest-api/gravitee-apim-rest-api-service/src/main/java/io/gravitee/apim/core/portal_page/exception/ParentNotFoundException.java

@@ -19,7 +19,7 @@
 
 public class ParentNotFoundException extends ValidationDomainException {
 
-    public ParentNotFoundException(String message) {
-        super(message);
+    public ParentNotFoundException(String parentId) {
+        super(String.format("Parent item with id %s does not exist", parentId));
     }
 }

gravitee-apim-rest-api/gravitee-apim-rest-api-service/src/main/java/io/gravitee/apim/core/portal_page/exception/ParentTypeMismatchException.java

@@ -19,7 +19,7 @@
 
 public class ParentTypeMismatchException extends ValidationDomainException {
 
-    public ParentTypeMismatchException(String message) {
-        super(message);
+    public ParentTypeMismatchException(String parentId) {
+        super(String.format("Parent item with id %s is not a folder", parentId));
     }
 }

gravitee-apim-rest-api/gravitee-apim-rest-api-service/src/main/java/io/gravitee/apim/core/portal_page/exception/PortalPageSpecificationException.java

@@ -16,7 +16,6 @@
 package io.gravitee.apim.core.portal_page.model;
 
 import jakarta.annotation.Nonnull;
-import jakarta.annotation.Nullable;
 import lombok.Getter;
 import lombok.Setter;
 

gravitee-apim-rest-api/gravitee-apim-rest-api-service/src/main/java/io/gravitee/apim/core/portal_page/model/PortalNavigationPage.java

@@ -72,15 +72,13 @@ private void validate(PortalNavigationItem item) {
         if (parentId != null) {
             final var parentItem = portalNavigationItemsQueryService.findByIdAndEnvironmentId(item.getEnvironmentId(), parentId);
             if (parentItem == null) {
-                throw new ParentNotFoundException(String.format("Parent item with id %s does not exist", parentId));
+                throw new ParentNotFoundException(parentId.toString());
             }
             if (!parentItem.getClass().equals(PortalNavigationFolder.class)) {
-                throw new ParentTypeMismatchException(String.format("Parent item with id %s is not a folder", parentId));
+                throw new ParentTypeMismatchException(parentId.toString());
             }
             if (!parentItem.getArea().equals(item.getArea())) {
-                throw new ParentAreaMismatchException(
-                    String.format("Parent item with id %s belongs to a different area than the child item", parentId)
-                );
+                throw new ParentAreaMismatchException(parentId.toString());
             }
         }
 
@@ -90,7 +88,7 @@ private void validate(PortalNavigationItem item) {
                 item.getArea()
             );
             if (!existingHomepage.isEmpty()) {
-                throw new HomepageAlreadyExistsException("Homepage already exists");
+                throw new HomepageAlreadyExistsException();
             }
         }