User cache

Author: watson8

classes/models/user.php

@@ -51,6 +51,12 @@
 class user extends table_base implements allocatable, moderatable {
     use allocatable_functions;
 
+    /**
+     * Cache area where objects by ID are stored.
+     * @var string
+     */
+    const CACHE_AREA_IDS = 'userids';
+
     /**
      * @var string
      */
@@ -67,6 +73,23 @@ public function __construct($data = false) {
         parent::__construct($data);
     }
 
+
+    /**
+     * Overrride from parent.
+     * Get child object from its ID, from the database.
+     * @param int $id
+     * @return \stdClass|null
+     */
+    protected static function get_db_record_from_id(int $id, int $strictness): ?object {
+        // In the case of user we override this so we don't get all DB fields, to avoid caching unnecessary data incl user hashed password.
+        $userfields = 'id,email,suspended' . implode(',', \core_user\fields::get_name_fields());
+        $user = \core_user::get_user($id, $userfields);
+        if (!$user || $user->deleted) {
+            return null;
+        }
+        return $user;
+    }
+
     /**
      * Get the user's full name.
      * @return string
@@ -88,7 +111,8 @@ public function name(): string {
             return core_user::get_fullname($data);
         }
 
-        return core_user::get_fullname($this->get_raw_record());
+        $record = self::get_db_record_from_id($this->id, IGNORE_MISSING);
+        return $record ? core_user::get_fullname($record) : '';
     }
 
     /**

classes/observer.php

@@ -26,6 +26,7 @@
 use core\event\group_member_removed;
 use core\event\role_assigned;
 use core\event\role_unassigned;
+use mod_coursework\models\user;
 
 defined('MOODLE_INTERNAL') || die();
 require_once($CFG->dirroot . '/mod/coursework/lib.php');
@@ -81,4 +82,15 @@ public static function process_allocations_when_group_member_removed(group_membe
     public static function remove_teacher_from_dropdown_when_unenrolled(core\event\role_unassigned $event) {
         teacher_removed_allocated_not_graded($event);
     }
+
+    /**
+     * @param core\event\user_updated $event
+     * @throws dml_exception
+     */
+    public static function user_updated(core\event\user_updated $event) {
+        $user = user::get_from_id($event->objectid);
+        if ($user) {
+            $user->clear_cache();
+        }
+    }
 }

db/caches.php

@@ -79,6 +79,11 @@
         'simplekeys' => true,
         'simpledata' => false,
     ],
+    user::CACHE_AREA_IDS => [
+        'mode' => cache_store::MODE_APPLICATION,
+        'simplekeys' => true,
+        'simpledata' => false,
+    ],
     feedback::CACHE_AREA_BY_SUBMISSION => [
         'mode' => cache_store::MODE_APPLICATION,
         'simplekeys' => true,

db/events.php

@@ -59,4 +59,8 @@
         'eventname' => 'core\event\role_unassigned',
         'callback' => 'mod_coursework_observer::remove_teacher_from_dropdown_when_unenrolled',
     ],
+    [
+        'eventname' => 'core\event\user_updated',
+        'callback' => 'mod_coursework_observer::user_updated',
+    ],
 ];

db/upgrade.php

@@ -365,6 +365,31 @@ function xmldb_coursework_upgrade($oldversion) {
         upgrade_mod_savepoint(true, 2026010801, 'coursework');
     }
 
+    //todo debugging only - delete this
+    if ($oldversion < 2026022301) {
+
+        // Define table coursework_temp_logs to be created.
+        $table = new xmldb_table('coursework_temp_logs');
+
+        // Adding fields to table coursework_temp_logs.
+        $table->add_field('id', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, XMLDB_SEQUENCE, null);
+        $table->add_field('classname', XMLDB_TYPE_CHAR, '255', null, null, null, null);
+        $table->add_field('notes', XMLDB_TYPE_CHAR, '255', null, null, null, null);
+        $table->add_field('stacktrace', XMLDB_TYPE_CHAR, '255', null, null, null, null);
+        $table->add_field('timecreated', XMLDB_TYPE_INTEGER, '10', null, null, null, null);
+
+        // Adding keys to table coursework_temp_logs.
+        $table->add_key('primary', XMLDB_KEY_PRIMARY, ['id']);
+
+        // Conditionally launch create table for coursework_temp_logs.
+        if (!$dbman->table_exists($table)) {
+            $dbman->create_table($table);
+        }
+
+        // Coursework savepoint reached.
+        upgrade_mod_savepoint(true, 2026022301, 'coursework');
+    }
+
     // Always needs to return true.
     return true;
 }

version.php

@@ -24,7 +24,7 @@
 
 $plugin->component = 'mod_coursework';
 
-$plugin->version = 2026022000;
+$plugin->version = 2026022302;
 $plugin->requires = 2024100700;
 
 $plugin->cron = 300; // Period for cron to check this module (secs).