From e00efa5eed00b793f373edf971203b9278b7ae09 Mon Sep 17 00:00:00 2001
From: Rajvi Patel <rajvi2232000@gmail.com>
Date: Thu, 11 Dec 2025 16:05:50 -0600
Subject: [PATCH 1/3] [CITE-242] Update template for job prune button

---
 .../webapp/WEB-INF/views/auth/jobs/list.html  | 72 ++++++++++++++++---
 1 file changed, 63 insertions(+), 9 deletions(-)

diff --git a/citesphere/src/main/webapp/WEB-INF/views/auth/jobs/list.html b/citesphere/src/main/webapp/WEB-INF/views/auth/jobs/list.html
index bad98d32e..f6768d29e 100644
--- a/citesphere/src/main/webapp/WEB-INF/views/auth/jobs/list.html
+++ b/citesphere/src/main/webapp/WEB-INF/views/auth/jobs/list.html
@@ -4,15 +4,36 @@
 <script th:inline="javascript">
 //# sourceURL=page.js
 $(function() {
-    $('#pagination-top').twbsPagination({
-        totalPages: [(${total})],
-        startPage: [(${page})],
-        prev: "«",
-        next: "»",
-        visiblePages: 10,
-        initiateStartPageClick: false,
-        onPageClick: function (event, page) {
-            window.location.href = "[(@{|/auth/jobs/sync/list|})]?page=" + (page - 1);
+    var totalPages = [(${total})];
+    var startPage = [(${page})];
+    if (totalPages > 0 && startPage > 0 && startPage <= totalPages) {
+        $('#pagination-top').twbsPagination({
+            totalPages: totalPages,
+            startPage: startPage,
+            prev: "«",
+            next: "»",
+            visiblePages: 10,
+            initiateStartPageClick: false,
+            onPageClick: function (event, page) {
+                window.location.href = "[(@{|/auth/jobs/sync/list|})]?page=" + (page - 1);
+            }
+        });
+    }
+    
+    $('#openPruneModal').on('click', function() {
+        $('#pruneModal').modal('show');
+    });
+    
+    $('#pruneForm').on('submit', function(e) {
+        var date = $('#pruneDate').val();
+        if (!date) {
+            e.preventDefault();
+            alert('Please select a date to prune jobs.');
+            return;
+        }
+        var message = 'This will delete all jobs up to and including ' + date + '. This cannot be undone. Continue?';
+        if (!confirm(message)) {
+            e.preventDefault();
         }
     });
 });
@@ -23,6 +44,39 @@
 <div layout:fragment="content">
 <h1>Jobs</h1>
 
+<div class="row" style="margin-bottom: 15px;">
+    <div class="col-md-12">
+        <button id="openPruneModal" class="btn btn-danger btn-sm">Prune jobs</button>
+    </div>
+</div>
+
+<div class="modal fade" id="pruneModal" tabindex="-1" role="dialog" aria-labelledby="pruneModalLabel">
+  <div class="modal-dialog" role="document">
+    <div class="modal-content">
+      <form id="pruneForm" th:action="@{/auth/jobs/sync/prune}" method="post">
+        <input type="hidden" th:name="${_csrf.parameterName}" th:value="${_csrf.token}" />
+        <div class="modal-header">
+          <button type="button" class="close" data-dismiss="modal" aria-label="Close"><span aria-hidden="true">&times;</span></button>
+          <h4 class="modal-title" id="pruneModalLabel">Prune jobs</h4>
+        </div>
+        <div class="modal-body">
+          <div class="form-group">
+            <label for="pruneDate">Delete jobs created on or before</label>
+            <input type="date" id="pruneDate" name="pruneDate" class="form-control" required />
+          </div>
+          <div class="alert alert-warning" role="alert">
+            This will permanently delete all jobs up to and including the selected date. This cannot be undone.
+          </div>
+        </div>
+        <div class="modal-footer">
+          <button type="button" class="btn btn-default" data-dismiss="modal">Cancel</button>
+          <button type="submit" class="btn btn-danger">Delete jobs</button>
+        </div>
+      </form>
+    </div>
+  </div>
+</div>
+
 <ul id="pagination-top" class="pagination-sm"></ul>
 
 <table class="table table-striped">

From 5d850b189ad03b69b8b0853ff20e06e96f67518a Mon Sep 17 00:00:00 2001
From: Rajvi Patel <rajvi2232000@gmail.com>
Date: Fri, 12 Dec 2025 13:44:47 -0600
Subject: [PATCH 2/3] [CITE-242] Add prune function in controller

---
 .../jobs/GroupSyncJobRepository.java          |  2 ++
 .../core/service/jobs/ISyncJobManager.java    |  3 +++
 .../service/jobs/impl/SyncJobManager.java     | 13 ++++++++-
 .../web/user/jobs/SyncJobsController.java     | 27 +++++++++++++++++--
 4 files changed, 42 insertions(+), 3 deletions(-)

diff --git a/citesphere/src/main/java/edu/asu/diging/citesphere/core/repository/jobs/GroupSyncJobRepository.java b/citesphere/src/main/java/edu/asu/diging/citesphere/core/repository/jobs/GroupSyncJobRepository.java
index 77f266f21..6e2b60ebe 100644
--- a/citesphere/src/main/java/edu/asu/diging/citesphere/core/repository/jobs/GroupSyncJobRepository.java
+++ b/citesphere/src/main/java/edu/asu/diging/citesphere/core/repository/jobs/GroupSyncJobRepository.java
@@ -15,4 +15,6 @@ public interface GroupSyncJobRepository extends PagingAndSortingRepository<Group
     public List<GroupSyncJob> findByGroupIdIn(List<String> groupIds, Pageable page);
     
     public long countByGroupIdIn(List<String> groupIds);
+    
+    public long deleteByGroupIdInAndCreatedOnBefore(List<String> groupIds, java.time.OffsetDateTime threshold);
 }
diff --git a/citesphere/src/main/java/edu/asu/diging/citesphere/core/service/jobs/ISyncJobManager.java b/citesphere/src/main/java/edu/asu/diging/citesphere/core/service/jobs/ISyncJobManager.java
index bb245da6c..8fdf15790 100644
--- a/citesphere/src/main/java/edu/asu/diging/citesphere/core/service/jobs/ISyncJobManager.java
+++ b/citesphere/src/main/java/edu/asu/diging/citesphere/core/service/jobs/ISyncJobManager.java
@@ -1,6 +1,7 @@
 package edu.asu.diging.citesphere.core.service.jobs;
 
 import java.util.List;
+import java.time.OffsetDateTime;
 
 import org.springframework.data.domain.Pageable;
 
@@ -18,5 +19,7 @@ public interface ISyncJobManager {
     long getJobsCount(IUser user);
 
     void cancelJob(String jobId);
+    
+    long pruneJobs(IUser user, OffsetDateTime before);
 
 }
\ No newline at end of file
diff --git a/citesphere/src/main/java/edu/asu/diging/citesphere/core/service/jobs/impl/SyncJobManager.java b/citesphere/src/main/java/edu/asu/diging/citesphere/core/service/jobs/impl/SyncJobManager.java
index fa4fb77df..4d4c7681a 100644
--- a/citesphere/src/main/java/edu/asu/diging/citesphere/core/service/jobs/impl/SyncJobManager.java
+++ b/citesphere/src/main/java/edu/asu/diging/citesphere/core/service/jobs/impl/SyncJobManager.java
@@ -8,12 +8,12 @@
 import java.util.stream.Collectors;
 
 import javax.annotation.PostConstruct;
+import javax.transaction.Transactional;
 
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.data.domain.Pageable;
 import org.springframework.stereotype.Service;
 
-import edu.asu.diging.citesphere.core.model.jobs.IJob;
 import edu.asu.diging.citesphere.core.model.jobs.JobStatus;
 import edu.asu.diging.citesphere.core.model.jobs.impl.GroupSyncJob;
 import edu.asu.diging.citesphere.core.repository.jobs.GroupSyncJobRepository;
@@ -89,4 +89,15 @@ public void cancelJob(String jobId) {
             jobRepo.save(job);
         }
     }
+    
+    @Override
+    @Transactional
+    public long pruneJobs(IUser user, OffsetDateTime before) {
+        List<ICitationGroup> groups = citationManager.getGroups(user);
+        if (groups == null || groups.isEmpty()) {
+            return 0;
+        }
+        List<String> groupIds = groups.stream().map(g -> g.getGroupId() + "").collect(Collectors.toList());
+        return jobRepo.deleteByGroupIdInAndCreatedOnBefore(groupIds, before);
+    }
 }
diff --git a/citesphere/src/main/java/edu/asu/diging/citesphere/web/user/jobs/SyncJobsController.java b/citesphere/src/main/java/edu/asu/diging/citesphere/web/user/jobs/SyncJobsController.java
index cf13e4dfd..c1dbc70d2 100644
--- a/citesphere/src/main/java/edu/asu/diging/citesphere/web/user/jobs/SyncJobsController.java
+++ b/citesphere/src/main/java/edu/asu/diging/citesphere/web/user/jobs/SyncJobsController.java
@@ -1,5 +1,9 @@
 package edu.asu.diging.citesphere.web.user.jobs;
 
+import java.time.LocalDate;
+import java.time.OffsetDateTime;
+import java.time.ZoneOffset;
+
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.data.domain.Pageable;
 import org.springframework.data.domain.Sort.Direction;
@@ -8,6 +12,10 @@
 import org.springframework.stereotype.Controller;
 import org.springframework.ui.Model;
 import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RequestMethod;
+import org.springframework.web.bind.annotation.RequestParam;
+import org.springframework.web.servlet.mvc.support.RedirectAttributes;
+import org.springframework.format.annotation.DateTimeFormat;
 
 import edu.asu.diging.citesphere.core.service.jobs.ISyncJobManager;
 import edu.asu.diging.citesphere.user.IUser;
@@ -26,8 +34,23 @@ public String list(Model model, @PageableDefault(sort = { "createdOn" }, directi
             return "redirect:/";
         }
         model.addAttribute("jobs", jobManager.getJobs((IUser) authentication.getPrincipal(), page));
-        model.addAttribute("total", Math.ceil(total/page.getPageSize()));
-        model.addAttribute("page", page.getPageNumber() + 1);
+        long totalPages = total > 0 ? (long) Math.ceil((double) total / page.getPageSize()) : 1;
+        model.addAttribute("total", totalPages);
+        model.addAttribute("page", Math.max(1, Math.min(page.getPageNumber() + 1, totalPages)));
         return "auth/jobs/list";
     }
+    
+    @RequestMapping(value = "/auth/jobs/sync/prune", method = RequestMethod.POST)
+    public String prune(@RequestParam("pruneDate") @DateTimeFormat(iso = DateTimeFormat.ISO.DATE) LocalDate pruneDate, Authentication authentication,
+            RedirectAttributes redirectAttributes) {
+        OffsetDateTime pruneBefore = pruneDate.plusDays(1).atStartOfDay().atOffset(ZoneOffset.UTC);
+        long deleted = jobManager.pruneJobs((IUser) authentication.getPrincipal(), pruneBefore);
+        
+        redirectAttributes.addFlashAttribute("show_alert", true);
+        redirectAttributes.addFlashAttribute("alert_type", "success");
+        redirectAttributes.addFlashAttribute("alert_msg",
+                String.format("%d job(s) deleted up to and including %s.", deleted, pruneDate));
+        
+        return "redirect:/auth/jobs/sync/list";
+    }
 }

From acefd396b5401b76892a39625660fb9d1ba83987 Mon Sep 17 00:00:00 2001
From: Girik1105 <girik1105@gmail.com>
Date: Mon, 15 Dec 2025 12:36:55 -0700
Subject: [PATCH 3/3] [CITE-242] Better UI and alers

---
 .../webapp/WEB-INF/views/auth/jobs/list.html  | 50 +++++++++++++++----
 1 file changed, 41 insertions(+), 9 deletions(-)

diff --git a/citesphere/src/main/webapp/WEB-INF/views/auth/jobs/list.html b/citesphere/src/main/webapp/WEB-INF/views/auth/jobs/list.html
index f6768d29e..43a9f8329 100644
--- a/citesphere/src/main/webapp/WEB-INF/views/auth/jobs/list.html
+++ b/citesphere/src/main/webapp/WEB-INF/views/auth/jobs/list.html
@@ -28,13 +28,22 @@
         var date = $('#pruneDate').val();
         if (!date) {
             e.preventDefault();
-            alert('Please select a date to prune jobs.');
+            alert('Please select a date.');
             return;
         }
-        var message = 'This will delete all jobs up to and including ' + date + '. This cannot be undone. Continue?';
-        if (!confirm(message)) {
-            e.preventDefault();
-        }
+        e.preventDefault();
+        $('#confirmDate').text(date);
+        $('#pruneModal').modal('hide');
+        $('#confirmDeleteModal').modal('show');
+    });
+
+    $('#confirmDelete').on('click', function() {
+        $('#pruneForm')[0].submit();
+    });
+
+    $('#cancelConfirm').on('click', function() {
+        $('#confirmDeleteModal').modal('hide');
+        $('#pruneModal').modal('show');
     });
 });
 </script>
@@ -42,11 +51,12 @@
 
 <body>
 <div layout:fragment="content">
-<h1>Jobs</h1>
-
 <div class="row" style="margin-bottom: 15px;">
-    <div class="col-md-12">
-        <button id="openPruneModal" class="btn btn-danger btn-sm">Prune jobs</button>
+    <div class="col-md-6">
+        <h1 style="margin: 0;">Jobs</h1>
+    </div>
+    <div class="col-md-6 text-right" style="padding-top: 10px;">
+        <button id="openPruneModal" class="btn btn-danger btn-sm">Prune Jobs</button>
     </div>
 </div>
 
@@ -77,6 +87,28 @@ <h4 class="modal-title" id="pruneModalLabel">Prune jobs</h4>
   </div>
 </div>
 
+<div class="modal fade" id="confirmDeleteModal" tabindex="-1" role="dialog">
+  <div class="modal-dialog" role="document">
+    <div class="modal-content">
+      <div class="modal-header">
+        <button type="button" class="close" data-dismiss="modal"><span>&times;</span></button>
+        <h4 class="modal-title">Confirm Deletion</h4>
+      </div>
+      <div class="modal-body">
+        <div class="alert alert-danger">
+          <strong>Warning!</strong> This is an irreversible action. All jobs up to and including
+          <span id="confirmDate"></span> will be permanently deleted.
+        </div>
+        <p>Are you sure you want to continue?</p>
+      </div>
+      <div class="modal-footer">
+        <button type="button" class="btn btn-default" id="cancelConfirm">Cancel</button>
+        <button type="button" class="btn btn-danger" id="confirmDelete">Delete Jobs</button>
+      </div>
+    </div>
+  </div>
+</div>
+
 <ul id="pagination-top" class="pagination-sm"></ul>
 
 <table class="table table-striped">