Closes #8808 - Convert MapRoleEntity to interface

model/map/src/main/java/org/keycloak/models/map/role/MapRoleAdapter.java

@@ -21,6 +21,7 @@ import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
+import java.util.Set;
 import java.util.stream.Stream;
 import org.jboss.logging.Logger;
 import static org.keycloak.common.util.StackUtil.getShortStackTrace;
@@ -65,21 +66,23 @@ public class MapRoleAdapter extends AbstractRoleModel<MapRoleEntity> implements
 
     @Override
     public boolean isComposite() {
-        return ! entity.getCompositeRoles().isEmpty();
+        return ! (entity.getCompositeRoles() == null || entity.getCompositeRoles().isEmpty());
     }
 
     @Override
     public Stream<RoleModel> getCompositesStream() {
-        LOG.tracef("%% %s(%s).getCompositesStream():%d - %s", entity.getName(), entity.getId(), entity.getCompositeRoles().size(), getShortStackTrace());
+        Set<String> compositeRoles = entity.getCompositeRoles() == null ? Collections.emptySet() : entity.getCompositeRoles();
-        return entity.getCompositeRoles().stream()
+        LOG.tracef("%% %s(%s).getCompositesStream():%d - %s", entity.getName(), entity.getId(), compositeRoles.size(), getShortStackTrace());
+        return compositeRoles.stream()
                 .map(uuid -> session.roles().getRoleById(realm, uuid))
                 .filter(Objects::nonNull);
     }
 
     @Override
     public Stream<RoleModel> getCompositesStream(String search, Integer first, Integer max) {
-        LOG.tracef("%% (%s).getCompositesStream(%s, %d, %d):%d - %s", this, search, first, max, entity.getCompositeRoles().size(), getShortStackTrace());
+        Set<String> compositeRoles = entity.getCompositeRoles() == null ? Collections.emptySet() : entity.getCompositeRoles();
-        return session.roles().getRolesStream(realm, entity.getCompositeRoles().stream(), search, first, max);
+        LOG.tracef("%% (%s).getCompositesStream(%s, %d, %d):%d - %s", this, search, first, max, compositeRoles.size(), getShortStackTrace());
+        return session.roles().getRolesStream(realm, compositeRoles.stream(), search, first, max);
     }
 
     @Override
@@ -96,7 +99,8 @@ public class MapRoleAdapter extends AbstractRoleModel<MapRoleEntity> implements
 
     @Override
     public boolean isClientRole() {
-        return entity.isClientRole();
+        final Boolean clientRole = entity.isClientRole();
+        return clientRole == null ? false : clientRole;
     }
 
     @Override
@@ -126,7 +130,8 @@ public class MapRoleAdapter extends AbstractRoleModel<MapRoleEntity> implements
 
     @Override
     public Map<String, List<String>> getAttributes() {
-        return entity.getAttributes();
+        Map<String, List<String>> attributes = entity.getAttributes();
+        return attributes == null ? Collections.emptyMap() : attributes;
     }
 
     @Override

model/map/src/main/java/org/keycloak/models/map/role/MapRoleEntity.java

@@ -16,131 +16,70 @@
  */
 package org.keycloak.models.map.role;
 
-import java.util.HashMap;
-import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
-import java.util.Objects;
 import java.util.Set;
+import org.keycloak.models.map.annotations.GenerateEntityImplementations;
 import org.keycloak.models.map.common.AbstractEntity;
+import org.keycloak.models.map.common.DeepCloner;
 import org.keycloak.models.map.common.UpdatableEntity;
 
-public class MapRoleEntity extends UpdatableEntity.Impl implements AbstractEntity {
+@GenerateEntityImplementations(
+  inherits = "org.keycloak.models.map.role.MapRoleEntity.AbstractRoleEntity"
+)
+@DeepCloner.Root
+public interface MapRoleEntity extends AbstractEntity, UpdatableEntity {
 
-    private String id;
+    public abstract class AbstractRoleEntity extends UpdatableEntity.Impl implements MapRoleEntity {
-    private String realmId;
 
-    private String name;
+        private String id;
-    private String description;
-    private boolean clientRole;
-    private String clientId;
-    private Set<String> compositeRoles = new HashSet<>();
-    private Map<String, List<String>> attributes = new HashMap<>();
 
-    /**
+        @Override
-     * Flag signalizing that any of the setters has been meaningfully used.
+        public String getId() {
-     */
+            return this.id;
+        }
 
-    public MapRoleEntity() {}
+        @Override
+        public void setId(String id) {
+            if (this.id != null) throw new IllegalStateException("Id cannot be changed");
+            this.id = id;
+            this.updated |= id != null;
+        }
 
-    public MapRoleEntity(String id, String realmId) {
-        this.id = id;
-        this.realmId = realmId;
     }
 
-    @Override
+    default Boolean isComposite() {
-    public String getId() {
+        return ! (getCompositeRoles() == null || getCompositeRoles().isEmpty());
-        return this.id;
     }
 
-    @Override
+    Boolean isClientRole();
-    public void setId(String id) {
-        if (this.id != null) throw new IllegalStateException("Id cannot be changed");
-        this.id = id;
-        this.updated |= id != null;
-    }
 
-    public String getName() {
+    String getRealmId();
-        return name;
-    }
 
-    public void setName(String name) {
+    String getClientId();
-        this.updated |= ! Objects.equals(this.name, name);
-        this.name = name;
-    }
 
-    public String getDescription() {
+    String getName();
-        return description;
-    }
 
-    public void setDescription(String description) {
+    String getDescription();
-        this.updated |= ! Objects.equals(this.description, description);
-        this.description = description;
-    }
 
-    public Map<String, List<String>> getAttributes() {
+    void setClientRole(Boolean clientRole);
-        return attributes;
-    }
 
-    public void setAttributes(Map<String, List<String>> attributes) {
+    void setRealmId(String realmId);
-        this.updated |= ! Objects.equals(this.attributes, attributes);
-        this.attributes = attributes;
-    }
 
-    public void setAttribute(String name, List<String> values) {
+    void setClientId(String clientId);
-        this.updated |= ! Objects.equals(this.attributes.put(name, values), values);
-    }
 
-    public void removeAttribute(String name) {
+    void setName(String name);
-        this.updated |= this.attributes.remove(name) != null;
-    }
 
-    public String getRealmId() {
+    void setDescription(String description);
-        return realmId;
-    }
 
-    public void setRealmId(String realmId) {
+    Set<String> getCompositeRoles();
-        this.updated |= ! Objects.equals(this.realmId, realmId);
+    void setCompositeRoles(Set<String> compositeRoles);
-        this.realmId = realmId;
+    void addCompositeRole(String roleId);
-    }
+    void removeCompositeRole(String roleId);
 
-    public boolean isClientRole() {
+    Map<String, List<String>> getAttributes();
-        return clientRole;
+    void setAttributes(Map<String, List<String>> attributes);
-    }
+    void setAttribute(String name, List<String> values);
+    void removeAttribute(String name);
 
-    public void setClientRole(boolean clientRole) {
-        this.updated |= ! Objects.equals(this.clientRole, clientRole);
-        this.clientRole = clientRole;
-    }
-
-    public boolean isComposite() {
-        return ! (compositeRoles == null || compositeRoles.isEmpty());
-    }
-
-    public Set<String> getCompositeRoles() {
-        return compositeRoles;
-    }
-
-    public String getClientId() {
-        return clientId;
-    }
-
-    public void setClientId(String clientId) {
-        this.updated |= ! Objects.equals(this.clientId, clientId);
-        this.clientId = clientId;
-    }
-
-    public void setCompositeRoles(Set<String> compositeRoles) {
-        this.updated |= ! Objects.equals(this.compositeRoles, compositeRoles);
-        this.compositeRoles.clear();
-        this.compositeRoles.addAll(compositeRoles);
-    }
-
-    public void addCompositeRole(String roleId) {
-        this.updated |= this.compositeRoles.add(roleId);
-    }
-
-    public void removeCompositeRole(String roleId) {
-        this.updated |= this.compositeRoles.remove(roleId);
-    }
 }

model/map/src/main/java/org/keycloak/models/map/role/MapRoleProvider.java

@@ -66,9 +66,10 @@ public class MapRoleProvider implements RoleProvider {
 
         LOG.tracef("addRealmRole(%s, %s, %s)%s", realm, id, name, getShortStackTrace());
 
-        MapRoleEntity entity = new MapRoleEntity(id, realm.getId());
+        MapRoleEntity entity = new MapRoleEntityImpl();
-        entity.setName(name);
+        entity.setId(id);
         entity.setRealmId(realm.getId());
+        entity.setName(name);
         if (tx.read(entity.getId()) != null) {
             throw new ModelDuplicateException("Role exists: " + id);
         }
@@ -121,7 +122,9 @@ public class MapRoleProvider implements RoleProvider {
 
         LOG.tracef("addClientRole(%s, %s, %s)%s", client, id, name, getShortStackTrace());
 
-        MapRoleEntity entity = new MapRoleEntity(id, client.getRealm().getId());
+        MapRoleEntity entity = new MapRoleEntityImpl();
+        entity.setId(id);
+        entity.setRealmId(client.getRealm().getId());
         entity.setName(name);
         entity.setClientRole(true);
         entity.setClientId(client.getId());

model/map/src/main/java/org/keycloak/models/map/storage/chm/ConcurrentHashMapStorageProviderFactory.java

@@ -56,6 +56,7 @@ import org.keycloak.models.map.group.MapGroupEntityImpl;
 import org.keycloak.models.map.loginFailure.MapUserLoginFailureEntity;
 import org.keycloak.models.map.realm.MapRealmEntity;
 import org.keycloak.models.map.role.MapRoleEntity;
+import org.keycloak.models.map.role.MapRoleEntityImpl;
 import com.fasterxml.jackson.databind.JavaType;
 import java.io.File;
 import java.io.IOException;
@@ -106,6 +107,7 @@ public class ConcurrentHashMapStorageProviderFactory implements AmphibianProvide
       .constructorDC(MapClientEntityImpl.class,         MapClientEntityImpl::new)
       .constructor(MapProtocolMapperEntity.class,       MapProtocolMapperEntityImpl::new)
       .constructor(MapGroupEntityImpl.class,            MapGroupEntityImpl::new)
+      .constructor(MapRoleEntityImpl.class,             MapRoleEntityImpl::new)
       .build();
 
     public static final Map<Class<?>, String> MODEL_TO_NAME = new HashMap<>();
@@ -154,10 +156,9 @@ public class ConcurrentHashMapStorageProviderFactory implements AmphibianProvide
     static {
         INTERFACE_TO_IMPL.put(MapClientEntity.class, MapClientEntityImpl.class);
 //        INTERFACE_TO_IMPL.put(MapClientScopeEntity.class, MapClientScopeEntityImpl.class);
-//        INTERFACE_TO_IMPL.put(MapClientEntity.class, MapClientEntityImpl.class);
         INTERFACE_TO_IMPL.put(MapGroupEntity.class, MapGroupEntityImpl.class);
 //        INTERFACE_TO_IMPL.put(MapRealmEntity.class, MapRealmEntityImpl.class);
-//        INTERFACE_TO_IMPL.put(MapRoleEntity.class, MapRoleEntityImpl.class);
+        INTERFACE_TO_IMPL.put(MapRoleEntity.class, MapRoleEntityImpl.class);
 //        INTERFACE_TO_IMPL.put(MapRootAuthenticationSessionEntity.class, MapRootAuthenticationSessionEntityImpl.class);
 //        INTERFACE_TO_IMPL.put(MapUserLoginFailureEntity.class, MapUserLoginFailureEntityImpl.class);
 //        INTERFACE_TO_IMPL.put(MapUserEntity.class, MapUserEntityImpl.class);