Fix #2371 by left join maintenance_timeslot

server/model/maintenance.js

@@ -188,13 +188,13 @@ class Maintenance extends BeanModel {
      */
     static getActiveMaintenanceSQLCondition() {
         return `
-
+            (
-            (maintenance_timeslot.start_date <= DATETIME('now')
+                (maintenance_timeslot.start_date <= DATETIME('now')
-            AND maintenance_timeslot.end_date >= DATETIME('now')
+                AND maintenance_timeslot.end_date >= DATETIME('now')
-            AND maintenance.active = 1)
+                AND maintenance.active = 1)
-            OR
+                OR
-            (maintenance.strategy = 'manual' AND active = 1)
+                (maintenance.strategy = 'manual' AND active = 1)
-
+            )
         `;
     }
 
@@ -204,10 +204,12 @@ class Maintenance extends BeanModel {
      */
     static getActiveAndFutureMaintenanceSQLCondition() {
         return `
-            ((maintenance_timeslot.end_date >= DATETIME('now')
+            (
-            AND maintenance.active = 1)
+                ((maintenance_timeslot.end_date >= DATETIME('now')
-            OR
+                AND maintenance.active = 1)
-            (maintenance.strategy = 'manual' AND active = 1))
+                OR
+                (maintenance.strategy = 'manual' AND active = 1))
+            )
         `;
     }
 }

server/model/status_page.js

@@ -282,11 +282,13 @@ class StatusPage extends BeanModel {
             let activeCondition = Maintenance.getActiveMaintenanceSQLCondition();
             let maintenanceBeanList = R.convertToBeans("maintenance", await R.getAll(`
                 SELECT maintenance.*
-                FROM maintenance, maintenance_status_page msp, maintenance_timeslot
+                FROM maintenance
-                WHERE msp.maintenance_id = maintenance.id
+                JOIN maintenance_status_page
-                    AND maintenance_timeslot.maintenance_id = maintenance.id
+                    ON maintenance_status_page.maintenance_id = maintenance.id
-                    AND msp.status_page_id = ?
+                    AND maintenance_status_page.status_page_id = ?
-                    AND ${activeCondition}
+                LEFT JOIN maintenance_timeslot
+                    ON maintenance_timeslot.maintenance_id = maintenance.id
+                WHERE ${activeCondition}
                 ORDER BY maintenance.end_date
             `, [ statusPageId ]));