[Fixes #9550] Show creator/editor in 'Edit properties'

geosolutions-it · Jan 17, 2024 · 8d2e9dd · 8d2e9dd
1 parent 01758ec
commit 8d2e9dd
Show file tree

Hide file tree

Showing 24 changed files with 376 additions and 80 deletions.
diff --git a/doc/sql/002_create_schema_oracle.sql b/doc/sql/002_create_schema_oracle.sql
@@ -26,6 +26,8 @@
         metadata varchar2(4000 char),
         name varchar2(255 char) not null,
         category_id number(19,0) not null,
+        creator varchar2(255 char),
+        editor varchar2(255 char),
         primary key (id),
         unique (name)
     );

diff --git a/doc/sql/002_create_schema_postgres.sql b/doc/sql/002_create_schema_postgres.sql
@@ -41,6 +41,8 @@ psql -U geostore_test -d geostore -f 002_create_schema_postgres.sql
         metadata varchar(30000),
         name varchar(255) not null,
         category_id int8 not null,
+        creator varchar(255),
+        editor varchar(255),
         primary key (id),
         unique (name)
     );

diff --git a/...on/h2/h2-migration-from-v.1.5.0-to-v2.0.0 → ...2/h2-migration-from-v.1.5.0-to-v2.0.0.sql b/...on/h2/h2-migration-from-v.1.5.0-to-v2.0.0 → ...2/h2-migration-from-v.1.5.0-to-v2.0.0.sql
@@ -14,3 +14,6 @@ create index idx_user_group_attr_text on gs_user_group_attribute (string);
 create index idx_attr_user_group on gs_user_group_attribute (userGroup_id);
 
 alter table gs_user_group_attribute add constraint fk_ugattrib_user_group foreign key (userGroup_id) references gs_usergroup;
+
+alter table gs_resource add column creator varchar(255);
+alter table gs_resource add column editor varchar(255);
diff --git a/...e/oracle-migration-from-v.1.5.0-to-v2.0.0 → ...acle-migration-from-v.1.5.0-to-v2.0.0.sql b/...e/oracle-migration-from-v.1.5.0-to-v2.0.0 → ...acle-migration-from-v.1.5.0-to-v2.0.0.sql
@@ -15,3 +15,6 @@ create index idx_user_group_attr_text on gs_user_group_attribute (string);
 create index idx_attr_user_group on gs_user_group_attribute (userGroup_id);
 
 alter table gs_user_group_attribute add constraint fk_ugattrib_user_group foreign key (userGroup_id) references gs_usergroup;
+
+alter table gs_resource add column creator varchar2(255 char);
+alter table gs_resource add column editor varchar2(255 char);
diff --git a/...stgresql-migration-from-v.1.5.0-to-v2.0.0 → ...esql-migration-from-v.1.5.0-to-v2.0.0.sql b/...stgresql-migration-from-v.1.5.0-to-v2.0.0 → ...esql-migration-from-v.1.5.0-to-v2.0.0.sql
@@ -15,3 +15,6 @@ create index idx_user_group_attr_text on gs_user_group_attribute (string);
 create index idx_attr_user_group on gs_user_group_attribute (userGroup_id);
 
 alter table gs_user_group_attribute add constraint fk_ugattrib_user_group foreign key (userGroup_id) references gs_usergroup;
+
+alter table gs_resource add column creator varchar(255);
+alter table gs_resource add column editor varchar(255);
diff --git a/src/core/model/src/main/java/it/geosolutions/geostore/core/model/Resource.java b/src/core/model/src/main/java/it/geosolutions/geostore/core/model/Resource.java
@@ -94,6 +94,12 @@ public class Resource implements Serializable, CycleRecoverable {
     @Temporal(TemporalType.TIMESTAMP)
     private Date lastUpdate;
 
+    @Column(nullable = true, updatable = true)
+    private String creator;
+
+    @Column(nullable = true, updatable = true)
+    private String editor;
+
     @Column(nullable = true, updatable = true, length = 30000)
     private String metadata;
 
@@ -256,6 +262,34 @@ public void setSecurity(List<SecurityRule> security) {
         this.security = security;
     }
 
+    /**
+     * @return the creator username
+     */
+    public String getCreator() {
+        return creator;
+    }
+
+    /**
+     * @param creator the creator username
+     */
+    public void setCreator(String creator) {
+        this.creator = creator;
+    }
+
+    /**
+     * @return the editor username
+     */
+    public String getEditor() {
+        return editor;
+    }
+
+    /**
+     * @param editor the creator username
+     */
+    public void setEditor(String editor) {
+        this.editor = editor;
+    }
+
     /*
      * (non-Javadoc) @see java.lang.Object#toString()
      */
@@ -290,17 +324,27 @@ public String toString() {
 
         if (attribute != null) {
             builder.append(", ");
-            builder.append("attribute=").append(attribute.toString());
+            builder.append("attribute=").append(attribute);
         }
 
         if (data != null) {
             builder.append(", ");
-            builder.append("data=").append(data.toString());
+            builder.append("data=").append(data);
         }
 
         if (category != null) {
             builder.append(", ");
-            builder.append("category=").append(category.toString());
+            builder.append("category=").append(category);
+        }
+
+        if (creator != null) {
+            builder.append(", ");
+            builder.append("creator=").append(creator);
+        }
+
+        if (editor != null) {
+            builder.append(", ");
+            builder.append("editor=").append(editor);
         }
 
         builder.append(']');
@@ -325,6 +369,8 @@ public int hashCode() {
         result = (prime * result) + ((metadata == null) ? 0 : metadata.hashCode());
         result = (prime * result) + ((name == null) ? 0 : name.hashCode());
         result = (prime * result) + ((security == null) ? 0 : security.hashCode());
+        result = (prime * result) + ((creator == null) ? 0 : creator.hashCode());
+        result = (prime * result) + ((editor == null) ? 0 : editor.hashCode());
 
         return result;
     }
@@ -409,12 +455,20 @@ public boolean equals(Object obj) {
             return false;
         }
         if (security == null) {
-            if (other.security != null) {
-                return false;
-            }
+            return other.security == null;
         } else if (!security.equals(other.security)) {
             return false;
         }
+        if (creator == null) {
+            return other.creator == null;
+        } else if (!creator.equals(other.creator)) {
+            return false;
+        }
+        if (editor == null) {
+            return other.editor == null;
+        } else if (!editor.equals(other.editor)) {
+            return false;
+        }
 
         return true;
     }
@@ -432,7 +486,10 @@ public Object onCycleDetected(Context arg0) {
         r.setName(this.name);
         r.setAttribute(null);
         r.setData(null);
+        r.setCreator(this.creator);
+        r.setEditor(this.editor);
 
         return r;
     }
+
 }
diff --git a/src/core/persistence/src/main/java/it/geosolutions/geostore/core/dao/SecurityDAO.java b/src/core/persistence/src/main/java/it/geosolutions/geostore/core/dao/SecurityDAO.java
@@ -45,7 +45,7 @@ public interface SecurityDAO extends RestrictedGenericDAO<SecurityRule> {
 
     /**
      *
-     * @param userName
+     * @param groupNames
      * @param resourceId
      * @return
      */

diff --git a/...ore/persistence/src/main/java/it/geosolutions/geostore/core/dao/impl/SecurityDAOImpl.java b/...ore/persistence/src/main/java/it/geosolutions/geostore/core/dao/impl/SecurityDAOImpl.java
@@ -23,6 +23,8 @@
 import java.util.List;
 
 
+import it.geosolutions.geostore.core.dao.ResourceDAO;
+import it.geosolutions.geostore.core.model.Resource;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.springframework.transaction.annotation.Transactional;
@@ -48,6 +50,9 @@ public class SecurityDAOImpl extends BaseDAO<SecurityRule, Long> implements Secu
     private static final Logger LOGGER = LogManager.getLogger(SecurityDAOImpl.class);
 
     private UserGroupDAO userGroupDAO;
+
+    private ResourceDAO resourceDAO;
+
     /*
      * (non-Javadoc)
      * 
@@ -60,10 +65,11 @@ public void persist(SecurityRule... entities) {
         }
         for (SecurityRule rule : entities) {
             validateGroup(rule);
+            validateCreatorAndEditor(rule);
         }
         super.persist(entities);
     }
-    
+
     protected void validateGroup(SecurityRule rule) throws InternalError {
         if (rule.getGroup() != null) {
             UserGroup ug = userGroupDAO.find(rule.getGroup().getId());
@@ -74,6 +80,26 @@ protected void validateGroup(SecurityRule rule) throws InternalError {
         }
     }
 
+    private void validateCreatorAndEditor(SecurityRule rule) {
+        if (rule.getResource() != null && (rule.getUser() != null || rule.getUsername() != null)) {
+            Resource resource = rule.getResource();
+            boolean updated = false;
+            if (resource.getCreator() == null) {
+                resource.setCreator(
+                        rule.getUser() != null ? rule.getUser().getName() : rule.getUsername());
+                updated = true;
+            }
+            if (resource.getEditor() == null) {
+                resource.setEditor(
+                        rule.getUser() != null ? rule.getUser().getName() : rule.getUsername());
+                updated = true;
+            }
+            if (updated) {
+                resourceDAO.merge(resource);
+            }
+        }
+    }
+
     /*
      * (non-Javadoc)
      * 
@@ -214,6 +240,12 @@ public UserGroupDAO getUserGroupDAO() {
     public void setUserGroupDAO(UserGroupDAO userGroupDAO) {
         this.userGroupDAO = userGroupDAO;
     }
-
-
+
+    public ResourceDAO getResourceDAO() {
+        return resourceDAO;
+    }
+
+    public void setResourceDAO(ResourceDAO resourceDAO) {
+        this.resourceDAO = resourceDAO;
+    }
 }
diff --git a/src/core/persistence/src/test/java/it/geosolutions/geostore/core/dao/ResourceDAOTest.java b/src/core/persistence/src/test/java/it/geosolutions/geostore/core/dao/ResourceDAOTest.java
@@ -63,6 +63,8 @@ public void testPersistResource() throws Exception {
             resource.setName(NAME1);
             resource.setCreation(new Date());
             resource.setCategory(category);
+            resource.setCreator("USER1");
+            resource.setEditor("USER2");
 
             resourceDAO.persist(resource);
             resourceId = resource.getId();
@@ -103,14 +105,20 @@ public void testPersistResource() throws Exception {
             assertNotNull("Can't retrieve resource", loaded);
 
             assertEquals(NAME1, loaded.getName());
+            assertEquals("USER1", loaded.getCreator());
+            assertEquals("USER2", loaded.getEditor());
             loaded.setName(NAME2);
+            loaded.setCreator("USER1Updated");
+            loaded.setEditor("USER2Updated");
             resourceDAO.merge(loaded);
         }
 
         {
             Resource loaded = resourceDAO.find(resourceId);
             assertNotNull("Can't retrieve resource", loaded);
             assertEquals(NAME2, loaded.getName());
+            assertEquals("USER1Updated", loaded.getCreator());
+            assertEquals("USER2Updated", loaded.getEditor());
         }
 
         //

diff --git a/src/core/persistence/src/test/java/it/geosolutions/geostore/core/dao/SecurityDAOTest.java b/src/core/persistence/src/test/java/it/geosolutions/geostore/core/dao/SecurityDAOTest.java
@@ -46,7 +46,6 @@ public class SecurityDAOTest extends BaseDAOTest {
      */
     @Test
     public void testPersistSecurity() throws Exception {
-
         final String NAME = "NAME";
 
         if (LOGGER.isDebugEnabled()) {
@@ -81,6 +80,11 @@ public void testPersistSecurity() throws Exception {
             assertEquals(1, resourceDAO.count(null));
             assertEquals(1, resourceDAO.findAll().size());
 
+            Resource loaded = resourceDAO.find(resourceId);
+            assertNotNull(loaded);
+            assertNull(loaded.getCreator());
+            assertNull(loaded.getEditor());
+
             // SecurityRule security = new SecurityRule();
             // security.setCanRead(true);
             // security.setCanWrite(true);
@@ -105,16 +109,21 @@ public void testPersistSecurity() throws Exception {
         // PERSIST
         //
         {
+            Resource loaded = resourceDAO.find(resourceId);
             SecurityRule security = new SecurityRule();
             security.setCanRead(true);
             security.setCanWrite(true);
-            security.setResource(resourceDAO.find(resourceId));
+            security.setResource(loaded);
 
             securityDAO.persist(security);
             securityId = security.getId();
 
             assertEquals(1, securityDAO.count(null));
             assertEquals(1, securityDAO.findAll().size());
+
+            loaded = resourceDAO.find(resourceId);
+            assertNull(loaded.getCreator());
+            assertNull(loaded.getEditor());
         }
 
         //
@@ -146,7 +155,6 @@ public void testPersistSecurity() throws Exception {
 
     @Test
     public void testPersistSecurityUsingNames() throws Exception {
-
         final String NAME = "NAME";
         final String USERNAME= "USER";
         final String GROUPNAME= "GROUP";
@@ -200,6 +208,9 @@ public void testPersistSecurityUsingNames() throws Exception {
             assertEquals(1, resourceDAO.count(null));
             assertEquals(1, resourceDAO.findAll().size());
 
+            Resource loaded = resourceDAO.find(resourceId);
+            assertNull(loaded.getCreator());
+            assertNull(loaded.getEditor());
         }
 
         //
@@ -219,18 +230,30 @@ public void testPersistSecurityUsingNames() throws Exception {
             SecurityRule rule = securityDAO.find(securityId);
             assertNotNull(rule);
             assertNotNull(rule.getUsername());
+
+            Resource loaded = resourceDAO.find(resourceId);
+            assertNotNull(loaded.getCreator());
+            assertNotNull(loaded.getEditor());
+            assertEquals("testuser", loaded.getCreator());
+            assertEquals("testuser", loaded.getEditor());
+
             securityDAO.removeById(securityId);
         }
 
         //
         // PERSIST WITH USER
         //
         {
+            Resource loaded = resourceDAO.find(resourceId);
+            loaded.setEditor(null);
+            resourceDAO.merge(loaded);
+
             SecurityRule security = new SecurityRule();
             security.setCanRead(true);
             security.setCanWrite(true);
             security.setResource(resourceDAO.find(resourceId));
             User testUser = new User();
+            testUser.setName("TheTestUser");
             testUser.setId(userId);
             security.setUser(testUser);
             securityDAO.persist(security);
@@ -241,6 +264,13 @@ public void testPersistSecurityUsingNames() throws Exception {
             SecurityRule rule = securityDAO.find(securityId);
             assertNotNull(rule);
             assertNotNull(rule.getUser());
+
+            loaded = resourceDAO.find(resourceId);
+            assertNotNull(loaded.getCreator());
+            assertNotNull(loaded.getEditor());
+            assertEquals("testuser", loaded.getCreator());
+            assertEquals(testUser.getName(), loaded.getEditor());
+
             securityDAO.removeById(securityId);
         }