fix(core): Update modDate when unpublishing and archive (dotCMS#32227)

Add the `updateModDate` to the archive and unpublish functions, this way
now when unpublishing or archiving a content the Last Edition Date will
we updated

***Note**: As the modDate now will be updated each time we unpublish or
archive, `Assertions.assertEquals(site2.modDate(),
serverSite.entity().modDate());` was removed from `Test_Archive_Site()`

---------

Co-authored-by: erickgonzalez <[email protected]>

Author: gortiz-dotcms

dotCMS/src/main/java/com/dotcms/content/elasticsearch/business/ESContentletAPIImpl.java

@@ -3719,6 +3719,8 @@ private void internalArchive(final Contentlet contentlet, final User user,
 
             CacheLocator.getHTMLPageCache().remove(contentlet.getInode());
         }
+        final Set<String> inodes = Stream.of(contentlet).map(Contentlet::getInode).collect(Collectors.toSet());
+        updateModDate(inodes, user);
         invalidateLanguageVariableCache(contentlet);
         HibernateUtil.addCommitListener(
                 () -> this.contentletSystemEventUtil.pushArchiveEvent(workingContentlet), 1000);
@@ -4154,6 +4156,9 @@ private void internalUnpublish(final Contentlet contentlet, final User user, fin
             this.cleanFileAssetCache(contentlet, user, false);
         }
 
+        final Set<String> inodes = Stream.of(contentlet).map(Contentlet::getInode).collect(Collectors.toSet());
+        updateModDate(inodes, user);
+
         invalidateLanguageVariableCache(contentlet);
 
         new ContentletLoader().invalidate(contentlet, PageMode.LIVE);

dotcms-integration/src/test/java/com/dotcms/content/elasticsearch/business/ESContentletAPIImplTest.java

@@ -4814,4 +4814,79 @@ public void testValidationExceptionForMissingRequiredRelationship() throws Excep
 
     }
 
+    /**
+     * Method to test: {@link ESContentletAPIImpl#unpublish(Contentlet, User, boolean)}
+     * Given Scenario: Unpublish a {@link Contentlet}
+     * Expected Result: The modDate field should be updated
+     *
+     * @throws DotDataException
+     */
+    @Test
+    public void checkModDateAfterUnpublishContent() throws DotDataException, DotSecurityException {
+        final Host host = new SiteDataGen().nextPersisted();
+        final ContentType contentType = new ContentTypeDataGen()
+                .host(host)
+                .nextPersisted();
+        final Contentlet contentlet = new ContentletDataGen(contentType)
+                .host(host)
+                .nextPersistedAndPublish();
+
+        try {
+            assertNotNull(contentlet.getModDate());
+            Thread.sleep(500);
+            APILocator.getContentletAPI().unpublish(contentlet, APILocator.systemUser(), false);
+
+            final Contentlet contentletByIdentifier = APILocator.getContentletAPI()
+                    .findContentletByIdentifier(contentlet.getIdentifier(),
+                            false, contentlet.getLanguageId(), APILocator.systemUser(), false);
+
+            assertNotEquals(contentlet.getModDate(), contentletByIdentifier.getModDate());
+        } catch (InterruptedException e) {
+            throw new RuntimeException(e);
+        } finally {
+            ContentTypeDataGen.remove(contentType);
+            ContentletDataGen.remove(contentlet);
+        }
+
+    }
+    /**
+     * Method to test: {@link ESContentletAPIImpl#archive(Contentlet, User, boolean)}
+     * Given Scenario: Archive a {@link Contentlet}
+     * Expected Result: The modDate field should be updated
+     *
+     * @throws DotDataException
+     */
+    @Test
+    public void checkModDateAfterArchiveContent() throws DotDataException, DotSecurityException {
+        final Host host = new SiteDataGen().nextPersisted();
+        final ContentType contentType = new ContentTypeDataGen()
+                .host(host)
+                .nextPersisted();
+        final Contentlet contentlet = new ContentletDataGen(contentType)
+                .host(host)
+                .nextPersistedAndPublish();
+
+        final Date initialModDate = contentlet.getModDate();
+
+        try {
+            assertNotNull(contentlet.getModDate());
+            Thread.sleep(500);
+            APILocator.getContentletAPI().archive(contentlet, APILocator.systemUser(), false);
+
+            final Contentlet contentletByIdentifier = APILocator.getContentletAPI()
+                    .findContentletByIdentifier(contentlet.getIdentifier(),
+                            false, contentlet.getLanguageId(), APILocator.systemUser(), false);
+
+            final Date finalModDate = contentletByIdentifier.getModDate();
+
+            assertNotEquals(initialModDate, finalModDate);
+        } catch (InterruptedException e) {
+            throw new RuntimeException(e);
+        } finally {
+            ContentTypeDataGen.remove(contentType);
+            ContentletDataGen.remove(contentlet);
+        }
+
+    }
+
 }

tools/dotcms-cli/cli/src/test/java/com/dotcms/cli/command/site/SiteCommandIT.java

@@ -1142,7 +1142,6 @@ void Test_Archive_Site() throws IOException {
                 Assertions.assertNotNull(serverSite.entity().identifier());
                 Assertions.assertEquals(site2.siteName(), serverSite.entity().siteName());
                 Assertions.assertEquals(site2.identifier(), serverSite.entity().identifier());
-                Assertions.assertEquals(site2.modDate(), serverSite.entity().modDate());
                 Assertions.assertEquals(site2.isArchived(), serverSite.entity().isArchived());
             }
         } finally {