keycloak-uncached

KEYCLOAK-302 Ensure that all tables and columns have maximum

2/18/2014 4:53:45 PM

Details

diff --git a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/ApplicationEntity.java b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/ApplicationEntity.java
index 5da99fc..e44da8b 100755
--- a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/ApplicationEntity.java
+++ b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/ApplicationEntity.java
@@ -37,7 +37,7 @@ public class ApplicationEntity {
     Collection<ApplicationRoleEntity> roles = new ArrayList<ApplicationRoleEntity>();
 
     @OneToMany(fetch = FetchType.LAZY, cascade ={CascadeType.REMOVE}, orphanRemoval = true)
-    @JoinTable(name="APPLICATION_DEFAULT_ROLES")
+    @JoinTable(name="ApplicationDefaultRoles")
     Collection<RoleEntity> defaultRoles = new ArrayList<RoleEntity>();
 
     public String getId() {
diff --git a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/ApplicationRoleEntity.java b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/ApplicationRoleEntity.java
index 61a850c..71343f7 100755
--- a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/ApplicationRoleEntity.java
+++ b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/ApplicationRoleEntity.java
@@ -16,7 +16,7 @@ import javax.persistence.NamedQuery;
 @Entity
 public class ApplicationRoleEntity extends RoleEntity {
     @ManyToOne
-    @JoinTable(name = "APPLICATION_ROLE")
+    @JoinTable(name = "ApplicationRole")
     private ApplicationEntity application;
 
     public ApplicationEntity getApplication() {
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
index 4bfbcff..e6967e3 100755
--- 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
@@ -38,6 +38,8 @@ public class RealmEntity {
     protected boolean verifyEmail;
     protected boolean resetPasswordAllowed;
     protected boolean social;
+
+    @Column(name="updateProfileOnInitSocLogin")
     protected boolean updateProfileOnInitialSocialLogin;
     protected String passwordPolicy;
 
@@ -54,16 +56,16 @@ public class RealmEntity {
     protected String accountTheme;
 
     @OneToMany(cascade ={CascadeType.REMOVE}, orphanRemoval = true)
-    @JoinTable(name="USER_REQUIRED_CREDENTIALS")
+    @JoinTable(name="User_RequiredCreds")
     Collection<RequiredCredentialEntity> requiredCredentials = new ArrayList<RequiredCredentialEntity>();
 
     @OneToMany(cascade ={CascadeType.REMOVE}, orphanRemoval = true)
-    @JoinTable(name="APPLICATION_REQUIRED_CREDENTIALS")
-    Collection<RequiredCredentialEntity> requiredApplicationCredentials = new ArrayList<RequiredCredentialEntity>();
+    @JoinTable(name="App_RequiredCreds")
+    Collection<RequiredCredentialEntity> requiredAppCredentials = new ArrayList<RequiredCredentialEntity>();
 
     @OneToMany(cascade ={CascadeType.REMOVE}, orphanRemoval = true)
-    @JoinTable(name="OAUTH_CLIENT_REQUIRED_CREDENTIALS")
-    Collection<RequiredCredentialEntity> requiredOAuthClientCredentials = new ArrayList<RequiredCredentialEntity>();
+    @JoinTable(name="OAuthClient_RequiredCreds")
+    Collection<RequiredCredentialEntity> requiredOAuthClCredentials = new ArrayList<RequiredCredentialEntity>();
 
     @OneToMany(cascade ={CascadeType.REMOVE}, orphanRemoval = true)
     Collection<ApplicationEntity> applications = new ArrayList<ApplicationEntity>();
@@ -84,7 +86,7 @@ public class RealmEntity {
     protected Map<String, String> socialConfig = new HashMap<String, String>();
 
     @OneToMany(fetch = FetchType.LAZY, cascade ={CascadeType.REMOVE}, orphanRemoval = true)
-    @JoinTable(name="REALM_DEFAULT_ROLES")
+    @JoinTable(name="RealmDefaultRoles")
     Collection<RoleEntity> defaultRoles = new ArrayList<RoleEntity>();
 
     public String getId() {
@@ -207,20 +209,20 @@ public class RealmEntity {
         this.requiredCredentials = requiredCredentials;
     }
 
-    public Collection<RequiredCredentialEntity> getRequiredApplicationCredentials() {
-        return requiredApplicationCredentials;
+    public Collection<RequiredCredentialEntity> getRequiredAppCredentials() {
+        return requiredAppCredentials;
     }
 
-    public void setRequiredApplicationCredentials(Collection<RequiredCredentialEntity> requiredApplicationCredentials) {
-        this.requiredApplicationCredentials = requiredApplicationCredentials;
+    public void setRequiredAppCredentials(Collection<RequiredCredentialEntity> requiredAppCredentials) {
+        this.requiredAppCredentials = requiredAppCredentials;
     }
 
-    public Collection<RequiredCredentialEntity> getRequiredOAuthClientCredentials() {
-        return requiredOAuthClientCredentials;
+    public Collection<RequiredCredentialEntity> getRequiredOAuthClCredentials() {
+        return requiredOAuthClCredentials;
     }
 
-    public void setRequiredOAuthClientCredentials(Collection<RequiredCredentialEntity> requiredOAuthClientCredentials) {
-        this.requiredOAuthClientCredentials = requiredOAuthClientCredentials;
+    public void setRequiredOAuthClCredentials(Collection<RequiredCredentialEntity> requiredOAuthClCredentials) {
+        this.requiredOAuthClCredentials = requiredOAuthClCredentials;
     }
 
     public Collection<ApplicationEntity> getApplications() {
diff --git a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/RoleEntity.java b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/RoleEntity.java
index acb40c9..7e43cea 100755
--- a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/RoleEntity.java
+++ b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/RoleEntity.java
@@ -6,9 +6,7 @@ import javax.persistence.GeneratedValue;
 import javax.persistence.Id;
 import javax.persistence.Inheritance;
 import javax.persistence.InheritanceType;
-import javax.persistence.JoinTable;
 import javax.persistence.ManyToMany;
-import javax.persistence.OneToMany;
 import java.util.ArrayList;
 import java.util.Collection;
 
diff --git a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/SocialLinkEntity.java b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/SocialLinkEntity.java
index 9063948..ab85191 100755
--- a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/SocialLinkEntity.java
+++ b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/SocialLinkEntity.java
@@ -2,14 +2,12 @@ package org.keycloak.models.jpa.entities;
 
 import javax.persistence.Entity;
 import javax.persistence.GeneratedValue;
-import javax.persistence.GenerationType;
 import javax.persistence.Id;
 import javax.persistence.ManyToOne;
 import javax.persistence.NamedQueries;
 import javax.persistence.NamedQuery;
 
 import org.hibernate.annotations.GenericGenerator;
-import org.keycloak.models.jpa.utils.JpaIdGenerator;
 
 /**
  * @author <a href="mailto:bill@burkecentral.com">Bill Burke</a>
@@ -27,8 +25,6 @@ public class SocialLinkEntity {
     @GeneratedValue(generator = "keycloak_generator")
     private String id;
 
-    private String str = JpaIdGenerator.class.getName();
-
     @ManyToOne
     private UserEntity user;
 
diff --git a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/UserEntity.java b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/UserEntity.java
index 04af485..58a9292 100755
--- a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/UserEntity.java
+++ b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/UserEntity.java
@@ -9,7 +9,6 @@ import javax.persistence.Column;
 import javax.persistence.ElementCollection;
 import javax.persistence.Entity;
 import javax.persistence.GeneratedValue;
-import javax.persistence.GenerationType;
 import javax.persistence.Id;
 import javax.persistence.ManyToOne;
 import javax.persistence.MapKeyColumn;
diff --git a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/UserRoleMappingEntity.java b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/UserRoleMappingEntity.java
index 6a9aec7..b5a7a2e 100755
--- a/model/jpa/src/main/java/org/keycloak/models/jpa/entities/UserRoleMappingEntity.java
+++ b/model/jpa/src/main/java/org/keycloak/models/jpa/entities/UserRoleMappingEntity.java
@@ -1,11 +1,6 @@
 package org.keycloak.models.jpa.entities;
 
 import javax.persistence.Entity;
-import javax.persistence.GeneratedValue;
-import javax.persistence.GenerationType;
-import javax.persistence.Id;
-import javax.persistence.ManyToOne;
-import javax.persistence.MappedSuperclass;
 import javax.persistence.NamedQueries;
 import javax.persistence.NamedQuery;
 
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
index 8b7712d..7fbc340 100755
--- a/model/jpa/src/main/java/org/keycloak/models/jpa/RealmAdapter.java
+++ b/model/jpa/src/main/java/org/keycloak/models/jpa/RealmAdapter.java
@@ -1,6 +1,5 @@
 package org.keycloak.models.jpa;
 
-import org.bouncycastle.openssl.PEMWriter;
 import org.keycloak.models.RoleContainerModel;
 import org.keycloak.models.jpa.entities.ApplicationEntity;
 import org.keycloak.models.jpa.entities.CredentialEntity;
@@ -15,7 +14,6 @@ import org.keycloak.models.jpa.entities.UserRoleMappingEntity;
 import org.keycloak.models.jpa.entities.UserScopeMappingEntity;
 import org.keycloak.models.utils.KeycloakModelUtils;
 import org.keycloak.models.utils.Pbkdf2PasswordEncoder;
-import org.keycloak.util.PemUtils;
 import org.keycloak.models.ApplicationModel;
 import org.keycloak.models.OAuthClientModel;
 import org.keycloak.models.PasswordPolicy;
@@ -29,8 +27,7 @@ import org.keycloak.models.utils.TimeBasedOTP;
 
 import javax.persistence.EntityManager;
 import javax.persistence.TypedQuery;
-import java.io.IOException;
-import java.io.StringWriter;
+
 import java.security.PrivateKey;
 import java.security.PublicKey;
 import java.util.ArrayList;
@@ -285,7 +282,7 @@ public class RealmAdapter implements RealmModel {
     @Override
     public List<RequiredCredentialModel> getRequiredApplicationCredentials() {
         List<RequiredCredentialModel> requiredCredentialModels = new ArrayList<RequiredCredentialModel>();
-        Collection<RequiredCredentialEntity> entities = realm.getRequiredApplicationCredentials();
+        Collection<RequiredCredentialEntity> entities = realm.getRequiredAppCredentials();
         if (entities == null) return requiredCredentialModels;
         for (RequiredCredentialEntity entity : entities) {
             RequiredCredentialModel model = new RequiredCredentialModel();
@@ -301,7 +298,7 @@ public class RealmAdapter implements RealmModel {
     @Override
     public List<RequiredCredentialModel> getRequiredOAuthClientCredentials() {
         List<RequiredCredentialModel> requiredCredentialModels = new ArrayList<RequiredCredentialModel>();
-        Collection<RequiredCredentialEntity> entities = realm.getRequiredOAuthClientCredentials();
+        Collection<RequiredCredentialEntity> entities = realm.getRequiredOAuthClCredentials();
         if (entities == null) return requiredCredentialModels;
         for (RequiredCredentialEntity entity : entities) {
             RequiredCredentialModel model = new RequiredCredentialModel();
@@ -321,7 +318,7 @@ public class RealmAdapter implements RealmModel {
         entity.setType(model.getType());
         entity.setFormLabel(model.getFormLabel());
         em.persist(entity);
-        realm.getRequiredOAuthClientCredentials().add(entity);
+        realm.getRequiredOAuthClCredentials().add(entity);
         em.flush();
     }
 
@@ -339,7 +336,7 @@ public class RealmAdapter implements RealmModel {
         entity.setType(model.getType());
         entity.setFormLabel(model.getFormLabel());
         em.persist(entity);
-        realm.getRequiredApplicationCredentials().add(entity);
+        realm.getRequiredAppCredentials().add(entity);
         em.flush();
     }
 
@@ -352,7 +349,7 @@ public class RealmAdapter implements RealmModel {
 
     @Override
     public void updateRequiredOAuthClientCredentials(Set<String> creds) {
-        Collection<RequiredCredentialEntity> relationships = realm.getRequiredOAuthClientCredentials();
+        Collection<RequiredCredentialEntity> relationships = realm.getRequiredOAuthClCredentials();
         if (relationships == null) relationships = new ArrayList<RequiredCredentialEntity>();
 
         Set<String> already = new HashSet<String>();
@@ -378,7 +375,7 @@ public class RealmAdapter implements RealmModel {
 
     @Override
     public void updateRequiredApplicationCredentials(Set<String> creds) {
-        Collection<RequiredCredentialEntity> relationships = realm.getRequiredApplicationCredentials();
+        Collection<RequiredCredentialEntity> relationships = realm.getRequiredAppCredentials();
         if (relationships == null) relationships = new ArrayList<RequiredCredentialEntity>();
 
         Set<String> already = new HashSet<String>();