Renamed entity master admin app to master admin client

keycloak · Apr 14, 2015 · c70d12a · c70d12a
1 parent d4f138c
commit c70d12a
Show file tree

Hide file tree

Showing 7 changed files with 21 additions and 17 deletions.
diff --git a/connections/jpa-liquibase/src/main/resources/META-INF/jpa-changelog-1.2.0.RC1.xml b/connections/jpa-liquibase/src/main/resources/META-INF/jpa-changelog-1.2.0.RC1.xml
@@ -48,5 +48,6 @@
         </update>
         <dropColumn tableName="CLIENT" columnName="DTYPE"/>
         <renameColumn tableName="CLIENT" newColumnName="CLIENT_ID" oldColumnName="NAME"/>
+        <renameColumn tableName="REALM" newColumnName="MASTER_ADMIN_CLIENT" oldColumnName="MASTER_ADMIN_APP"/>
     </changeSet>
 </databaseChangeLog>
diff --git a/...te/src/main/java/org/keycloak/connections/mongo/updater/impl/updates/Update1_2_0_RC1.java b/...te/src/main/java/org/keycloak/connections/mongo/updater/impl/updates/Update1_2_0_RC1.java
@@ -19,6 +19,9 @@ public String getId() {
     public void update(KeycloakSession session) {
         convertApplicationsToClients();
         convertOAuthClientsToClients();
+
+        db.getCollection("realms").update(new BasicDBObject(), new BasicDBObject("$rename", new BasicDBObject("adminAppId", "clientId")), false, true);
+
     }
 
     private void convertApplicationsToClients() {

diff --git a/model/api/src/main/java/org/keycloak/models/entities/RealmEntity.java b/model/api/src/main/java/org/keycloak/models/entities/RealmEntity.java
@@ -64,7 +64,7 @@ public class RealmEntity extends AbstractIdentifiableEntity {
     private List<String> eventsListeners = new ArrayList<String>();
     private List<String> enabledEventTypes = new ArrayList<String>();
 
-    private String adminAppId;
+    private String masterAdminClient;
 
     private boolean internationalizationEnabled;
     private List<String> supportedLocales = new ArrayList<String>();
@@ -391,12 +391,12 @@ public void setEnabledEventTypes(List<String> enabledEventTypes) {
         this.enabledEventTypes = enabledEventTypes;
     }
 
-    public String getAdminAppId() {
+    public String getMasterAdminClient() {
-        return adminAppId;
+        return masterAdminClient;
     }
 
-    public void setAdminAppId(String adminAppId) {
+    public void setMasterAdminClient(String masterAdminClient) {
-        this.adminAppId = adminAppId;
+        this.masterAdminClient = masterAdminClient;
     }
 
     public List<UserFederationProviderEntity> getUserFederationProviders() {

diff --git a/model/file/src/main/java/org/keycloak/models/file/adapter/RealmAdapter.java b/model/file/src/main/java/org/keycloak/models/file/adapter/RealmAdapter.java
@@ -966,14 +966,14 @@ public ClientModel getMasterAdminClient() {
     @Override
     public void setMasterAdminClient(ClientModel client) {
         if (client == null) {
-            realm.setAdminAppId(null);
+            realm.setMasterAdminClient(null);
             this.masterAdminApp = null;
         } else {
             String appId = client.getId();
             if (appId == null) {
                 throw new IllegalStateException("Master Admin app not initialized.");
             }
-            realm.setAdminAppId(appId);
+            realm.setMasterAdminClient(appId);
             this.masterAdminApp = client;
         }
     }

diff --git a/model/jpa/src/main/java/org/keycloak/models/jpa/RealmAdapter.java b/model/jpa/src/main/java/org/keycloak/models/jpa/RealmAdapter.java
@@ -1066,13 +1066,13 @@ public void setEnabledEventTypes(Set<String> enabledEventTypes) {
 
     @Override
     public ClientModel getMasterAdminClient() {
-        return new ClientAdapter(this, em, session, realm.getMasterAdminApp());
+        return new ClientAdapter(this, em, session, realm.getMasterAdminClient());
     }
 
     @Override
     public void setMasterAdminClient(ClientModel client) {
         ClientEntity appEntity = client !=null ? em.getReference(ClientEntity.class, client.getId()) : null;
-        realm.setMasterAdminApp(appEntity);
+        realm.setMasterAdminClient(appEntity);
         em.flush();
     }
 

diff --git a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/RealmEntity.java b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/RealmEntity.java
@@ -136,8 +136,8 @@ public class RealmEntity {
     protected Set<String> enabledEventTypes = new HashSet<String>();
 
     @OneToOne
-    @JoinColumn(name="MASTER_ADMIN_APP")
+    @JoinColumn(name="MASTER_ADMIN_CLIENT")
-    protected ClientEntity masterAdminApp;
+    protected ClientEntity masterAdminClient;
 
     @OneToMany(cascade ={CascadeType.REMOVE}, orphanRemoval = true, mappedBy = "realm")
     protected List<IdentityProviderEntity> identityProviders = new ArrayList<IdentityProviderEntity>();
@@ -437,12 +437,12 @@ public void setEnabledEventTypes(Set<String> enabledEventTypes) {
         this.enabledEventTypes = enabledEventTypes;
     }
 
-    public ClientEntity getMasterAdminApp() {
+    public ClientEntity getMasterAdminClient() {
-        return masterAdminApp;
+        return masterAdminClient;
     }
 
-    public void setMasterAdminApp(ClientEntity masterAdminApp) {
+    public void setMasterAdminClient(ClientEntity masterAdminClient) {
-        this.masterAdminApp = masterAdminApp;
+        this.masterAdminClient = masterAdminClient;
     }
 
     public List<UserFederationProviderEntity> getUserFederationProviders() {

diff --git a/model/mongo/src/main/java/org/keycloak/models/mongo/keycloak/adapters/RealmAdapter.java b/model/mongo/src/main/java/org/keycloak/models/mongo/keycloak/adapters/RealmAdapter.java
@@ -979,14 +979,14 @@ public void setEnabledEventTypes(Set<String> enabledEventTypes) {
 
     @Override
     public ClientModel getMasterAdminClient() {
-        MongoClientEntity appData = getMongoStore().loadEntity(MongoClientEntity.class, realm.getAdminAppId(), invocationContext);
+        MongoClientEntity appData = getMongoStore().loadEntity(MongoClientEntity.class, realm.getMasterAdminClient(), invocationContext);
         return appData != null ? new ClientAdapter(session, this, appData, invocationContext) : null;
     }
 
     @Override
     public void setMasterAdminClient(ClientModel client) {
         String adminAppId = client != null ? client.getId() : null;
-        realm.setAdminAppId(adminAppId);
+        realm.setMasterAdminClient(adminAppId);
         updateRealm();
     }