Hotfix package

database/mysql/oscarinit.sql

@@ -13206,4 +13206,4 @@ CREATE TABLE IF NOT EXISTS specialty (
   region varchar(5) default '',
   specialty char(2) default '',
   specialtydesc varchar(100) default ''
-);
+);

database/mysql/updates/update-2026-01-26-tickler-indexes.sql

@@ -0,0 +1,14 @@
+-- Performance indexes for tickler queries
+-- These indexes support common filter operations and batch fetching
+
+-- Index for batch fetching tickler links
+CREATE INDEX IF NOT EXISTS `idx_tickler_link_tickler_no` ON `tickler_link` (`tickler_no`);
+
+-- Index for composite key
+CREATE INDEX IF NOT EXISTS `idx_tickler_status_service_date` ON `tickler` (`status`, `service_date`);
+
+-- Index for "assigned to" filter queries
+CREATE INDEX IF NOT EXISTS `idx_tickler_task_assigned_to` ON `tickler` (`task_assigned_to`);
+
+-- Index for "created by" filter queries
+CREATE INDEX IF NOT EXISTS `idx_tickler_creator` ON `tickler` (`creator`);

src/main/java/ca/openosp/openo/commn/dao/CtlDocTypeDaoImpl.java

@@ -29,10 +29,12 @@
 
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Locale;
 
 import javax.persistence.Query;
 
 import ca.openosp.openo.commn.model.CtlDocType;
+import ca.openosp.openo.documentManager.EDocUtil;
 import org.springframework.stereotype.Repository;
 
 @Repository
@@ -43,10 +45,11 @@ public CtlDocTypeDaoImpl() {
     }
 
     public void changeDocType(String docType, String module, String status) {
-        String sql = "UPDATE CtlDocType SET status =?1 WHERE module =?2 AND doctype =?3";
+        List<String> modules = EDocUtil.getModulesForQuery(module);
+        String sql = "UPDATE CtlDocType SET status =?1 WHERE module in (?2) AND doctype =?3";
         Query query = entityManager.createQuery(sql);
         query.setParameter(1, status);
-        query.setParameter(2, module);
+        query.setParameter(2, modules);
         query.setParameter(3, docType);
 
         query.executeUpdate();
@@ -61,27 +64,32 @@ public List<CtlDocType> findByStatusAndModule(String[] status, String module) {
     }
 
     public List<CtlDocType> findByStatusAndModule(List<String> status, String module) {
-        Query query = entityManager.createQuery("select c from CtlDocType c where c.status in (?1) and c.module=?2");
+        List<String> modules = EDocUtil.getModulesForQuery(module);
+        Query query = entityManager.createQuery("select c from CtlDocType c where c.status in (?1) and c.module in (?2)");
         query.setParameter(1, status);
-        query.setParameter(2, module);
+        query.setParameter(2, modules);
         @SuppressWarnings("unchecked")
         List<CtlDocType> results = query.getResultList();
         return results;
     }
 
     public List<CtlDocType> findByDocTypeAndModule(String docType, String module) {
-        Query query = entityManager.createQuery("select c from CtlDocType c where c.docType=?1 and c.module=?2");
+        List<String> modules = EDocUtil.getModulesForQuery(module);
+        Query query = entityManager.createQuery("select c from CtlDocType c where c.docType=?1 and c.module in (?2)");
         query.setParameter(1, docType);
-        query.setParameter(2, module);
+        query.setParameter(2, modules);
         @SuppressWarnings("unchecked")
         List<CtlDocType> results = query.getResultList();
         return results;
     }
 
     public void addDocType(String docType, String module) {
+        if (module == null) {
+            throw new IllegalArgumentException("module cannot be null");
+        }
         CtlDocType d = new CtlDocType();
         d.setDocType(docType);
-        d.setModule(module);
+        d.setModule(module.toLowerCase(Locale.ROOT));
         d.setStatus("A");
         entityManager.persist(d);
     }

src/main/java/ca/openosp/openo/commn/dao/DocumentDaoImpl.java

@@ -45,6 +45,7 @@
 import ca.openosp.openo.commn.model.EFormDocs;
 import org.springframework.stereotype.Repository;
 
+import ca.openosp.openo.documentManager.EDocUtil;
 import ca.openosp.openo.documentManager.EDocUtil.EDocSort;
 
 @Repository
@@ -325,10 +326,11 @@ public List<Object[]> findDocuments(String module, String moduleid, String docTy
             throw new IllegalArgumentException("moduleid must be a valid integer, but got:" + moduleid, e);
         }
 
+        List<String> modules = EDocUtil.getModulesForQuery(module);
         StringBuilder buf = new StringBuilder("SELECT DISTINCT c, d " +
                 "FROM Document d, CtlDocument c " +
-                "WHERE c.id.documentNo = d.id AND c.id.module = :module");
-        params.put("module", module);
+                "WHERE c.id.documentNo = d.id AND c.id.module IN :modules");
+        params.put("modules", modules);
 
         boolean isShowingAllDocuments = docType == null || docType.equals("all") || docType.length() == 0;
 

src/main/java/ca/openosp/openo/commn/dao/PreventionDaoImpl.java

@@ -184,7 +184,7 @@ public List<Prevention> findByTypeAndDate(String preventionType, Date startDate,
 
     @Override
     public List<Prevention> findByTypeAndDemoNo(String preventionType, Integer demoNo) {
-        String sqlCommand = "select x from " + modelClass.getSimpleName() + " x where preventionType=?1 and demographicId=?2 and deleted='0' order by preventionDate desc";
+        String sqlCommand = "select x from " + modelClass.getSimpleName() + " x where preventionType=?1 and demographicId=?2 and deleted='0' order by preventionDate asc";
 
         Query query = entityManager.createQuery(sqlCommand);
         query.setParameter(1, preventionType);

src/main/java/ca/openosp/openo/commn/dao/TicklerDao.java

@@ -35,6 +35,7 @@
 
 import ca.openosp.openo.commn.model.CustomFilter;
 import ca.openosp.openo.commn.model.Tickler;
+import ca.openosp.openo.tickler.dto.TicklerListDTO;
 
 public interface TicklerDao extends AbstractDao<Tickler> {
 
@@ -68,4 +69,37 @@ public List<Tickler> findByDemographicIdTaskAssignedToAndMessage(Integer demogra
     public List<Tickler> getTicklers(CustomFilter filter);
 
     public int getNumTicklers(CustomFilter filter);
+
+    /**
+     * Returns a list of TicklerListDTOs matching the filter criteria with pagination.
+     * <p>
+     * This method uses JPQL constructor expressions to fetch only the fields needed
+     * for display, reducing database queries from ~25 per page load to a small fixed set
+     * of queries. Comments and links are each batch-loaded in separate queries.
+     * </p>
+     *
+     * @param filter CustomFilter the filter criteria including status, provider, date range, etc.
+     * @param offset int the starting position for pagination (0-based)
+     * @param limit int the maximum number of results to return
+     * @return List&lt;TicklerListDTO&gt; matching the filter with comments and links populated,
+     *         or empty list if no matches found
+     * @since 2026-01-30
+     */
+    List<TicklerListDTO> getTicklerDTOs(CustomFilter filter, int offset, int limit);
+
+    /**
+     * Returns a list of TicklerListDTOs matching the filter criteria.
+     * <p>
+     * This method uses JPQL constructor expressions to fetch only the fields needed
+     * for display, reducing database queries from ~25 per page load to a small fixed set
+     * of queries. Comments and links are each batch-loaded in separate queries.
+     * Uses default pagination with MAX_LIST_RETURN_SIZE limit.
+     * </p>
+     *
+     * @param filter CustomFilter the filter criteria including status, provider, date range, etc.
+     * @return List&lt;TicklerListDTO&gt; matching the filter with comments and links populated,
+     *         or empty list if no matches found
+     * @since 2026-01-30
+     */
+    List<TicklerListDTO> getTicklerDTOs(CustomFilter filter);
 }