fix: dramatically increase performance for get_my_slots

Author: RiedleroD

lbplanner/classes/helpers/slot_helper.php

@@ -107,6 +107,29 @@ public static function get_all_slots(): array {
         return $slotsobj;
     }
 
+    /**
+     * Returns a list of all slots relevant for a vintage.
+     *
+     * @param string $vintage the vintage to filter for
+     * @return slot[] An array of the slots.
+     */
+    public static function get_vintage_slots(string $vintage): array {
+        global $DB;
+        $slots = $DB->get_records_sql(
+            'SELECT slot.* FROM {' . self::TABLE_SLOTS . '} as slot ' .
+            'INNER JOIN {'. self::TABLE_SLOT_FILTERS . '} as filter ON slot.id=filter.slotid ' .
+            'WHERE filter.vintage=? OR filter.vintage=NULL',
+            [$vintage]
+        );
+
+        $slotsobj = [];
+        foreach ($slots as $slot) {
+            array_push($slotsobj, slot::from_db($slot));
+        }
+
+        return $slotsobj;
+    }
+
     /**
      * Returns a list of all slots belonging to a supervisor.
      * @param int $supervisorid userid of the supervisor in question

lbplanner/services/slots/get_my_slots.php

@@ -43,7 +43,7 @@ public static function get_my_slots_parameters(): external_function_parameters {
     public static function get_my_slots(): array {
         global $USER;
 
-        $allslots = slot_helper::get_all_slots();
+        $allslots = slot_helper::get_vintage_slots($USER->address);
 
         $myslots = slot_helper::filter_slots_for_user($allslots, $USER);