Feature #14738

Take into account the change in the signature of the callbacks in
CDIResourceEventListener.

In the ResourcesManager component, remove the custom listeners for
change in the 'responsable' role users list and use instead the new
notification mechanism about modifications in user lists of the roles
in the different component instances.

Author: mmoqui

kmelia/kmelia-library/src/main/java/org/silverpeas/components/kmelia/service/KmeliaUserEventListener.java

@@ -41,7 +41,7 @@ public class KmeliaUserEventListener extends CDIResourceEventListener<UserEvent>
   private KmeliaService kmeliaService;
 
   @Override
-  public void onDeletion(final UserEvent event) throws Exception {
+  public void onDeletion(final UserEvent event) {
     UserDetail user = event.getTransition().getBefore();
     kmeliaService.userHaveBeenDeleted(user.getId());
   }

kmelia/kmelia-library/src/main/java/org/silverpeas/components/kmelia/service/KmeliaWysiwygEventListener.java

@@ -41,19 +41,16 @@ public class KmeliaWysiwygEventListener extends CDIResourceEventListener<Wysiwyg
   private KmeliaService kmeliaService;
 
   @Override
-  public void onUpdate(final WysiwygEvent event) throws Exception {
+  public void onUpdate(final WysiwygEvent event) {
     WysiwygContent content = event.getTransition().getAfter();
     if (content != null) {
       anExternalPublicationElementHaveChanged(content);
     }
   }
 
   @Override
-  public void onCreation(final WysiwygEvent event) throws Exception {
-    WysiwygContent content = event.getTransition().getAfter();
-    if (content != null) {
-      anExternalPublicationElementHaveChanged(content);
-    }
+  public void onCreation(final WysiwygEvent event) {
+    onUpdate(event);
   }
 
   private void anExternalPublicationElementHaveChanged(WysiwygContent content) {

kmelia/kmelia-war/src/main/java/org/silverpeas/components/kmelia/notification/VersionSupportUpdater.java

@@ -47,7 +47,7 @@ public class VersionSupportUpdater extends CDIResourceEventListener<ComponentIns
   private Administration administration;
 
   @Override
-  public void onUpdate(final ComponentInstanceEvent event) throws Exception {
+  public void onUpdate(final ComponentInstanceEvent event) {
     StateTransition<ComponentInst> transition = event.getTransition();
     String instanceType = event.getTransition().getBefore().getName();
     if ("kmelia".equalsIgnoreCase(instanceType)) {

resourcesManager/resourcesManager-library/src/integration-test/java/org/silverpeas/components/resourcesmanager/ResourcesManagerInstancePreDestructionIT.java

@@ -92,7 +92,7 @@ public void preDestroyAResourcesManagerInstance() throws Exception {
         try (PreparedStatement stmt = connection.prepareStatement(sql)) {
           ResultSet rs = stmt.executeQuery();
           assertThat(rs.next(), is(true));
-          assertThat(rs.getLong(1), is(0l));
+          assertThat(rs.getLong(1), is(0L));
         }
       }
     }

resourcesManager/resourcesManager-library/src/main/java/org/silverpeas/components/resourcesmanager/GroupUserLinkEventListener.java

@@ -0,0 +1,57 @@
+/*
+ * Copyright (C) 2000 - 2025 Silverpeas
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * As a special exception to the terms and conditions of version 3.0 of
+ * the GPL, you may redistribute this Program in connection with Free/Libre
+ * Open Source Software ("FLOSS") applications as described in Silverpeas's
+ * FLOSS exception.  You should have received a copy of the text describing
+ * the FLOSS exception, and it is also available here:
+ * "https://www.silverpeas.org/legal/floss_exception.html"
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+package org.silverpeas.components.resourcesmanager;
+
+import org.silverpeas.core.admin.user.notification.role.UserRoleEvent;
+import org.silverpeas.core.annotation.Service;
+import org.silverpeas.core.notification.system.CDIResourceEventListener;
+
+import javax.inject.Inject;
+import java.util.Set;
+
+/**
+ * Listeners of events about changes in the users list of the role named 'responsable'. For any
+ * users who don't play anymore this role, they are removed from the managers in charge of the
+ * validation for all the bookable resources handled by the Resources Manager applications.
+ *
+ * @author mmoquillon
+ */
+@Service
+public class UserRoleEventListener extends CDIResourceEventListener<UserRoleEvent> {
+
+  private static final String MANAGER_ROLE = "responsable";
+
+  @Inject
+  private ResourcesManagersSynchronizer synchronizer;
+
+  @Override
+  public void onDeletion(UserRoleEvent event) {
+    if (event.getRole().equals(MANAGER_ROLE)) {
+      Set<String> userIds = event.getUserIds();
+      event.getInstanceIds().forEach(a -> synchronizer.synchronize(a, userIds));
+    }
+  }
+}
+  

resourcesManager/resourcesManager-library/src/main/java/org/silverpeas/components/resourcesmanager/ProfileInstEventListener.java

@@ -45,12 +45,12 @@ public class WebPagesWysiwygEventListener extends CDIResourceEventListener<Wysiw
   private OrganizationController organisationController;
 
   @Override
-  public void onUpdate(final WysiwygEvent event) throws Exception {
+  public void onUpdate(final WysiwygEvent event) {
     notifyUsersAboutChange(event.getTransition().getAfter());
   }
 
   @Override
-  public void onCreation(final WysiwygEvent event) throws Exception {
+  public void onCreation(final WysiwygEvent event) {
     notifyUsersAboutChange(event.getTransition().getAfter());
   }
 

resourcesManager/resourcesManager-library/src/main/java/org/silverpeas/components/resourcesmanager/UserRoleEventListener.java

@@ -46,10 +46,9 @@ public class GroupEventListener extends CDIResourceEventListener<GroupEvent> {
    * In that case, as a group can be referred by a yellowpages instance (as a group
    * of inner contacts), take caution to remove also this group from the yellowpages instance.
    * @param event the event on the deletion of a group.
-   * @throws Exception if an error occurs while treating the event.
    */
   @Override
-  public void onDeletion(final GroupEvent event) throws Exception {
+  public void onDeletion(final GroupEvent event) {
     yellowpagesService.removeGroup(event.getTransition().getBefore().getId());
   }
 }