Release 2025112400

Author: rschrenk

README.md

@@ -1,7 +1,46 @@
-# moodle-local_table_sql
+# local_table_sql Moodle Plugin
+**Enhanced table_sql functionality with advanced features and a modern, responsive interface**
 
-TODO
+Moodle provides a `table_sql` class for building tables through a unified API. However, it lacks modern features such as AJAX-based pagination and filtering.
 
-## Licenses for JavaScripts
+`local_table_sql` is a drop-in replacement for Moodle’s core `table_sql` class and adds a wide range of powerful enhancements, including AJAX navigation, full-text search and advanced filtering for each column.
 
-see: LICENSE.js.txt
+Plugins can declare this plugin as a dependency in their `version.php`.
+Tables built using `local_table_sql` provide the following features:
+
+## Key Features
+
+- **AJAX-driven data loading** – Load only the data that is needed, improving performance.
+- **Built-in filtering and sorting** – Global search or column-specific filters included.
+- **Full-text search support**
+- **Dynamic column visibility** – Show or hide columns instantly.
+- **Customizable UI actions** – Add action buttons or context menus for each row.
+- **Row selection and bulk actions**
+- **SQL-based data source** – Provide your own SQL query as input.
+- **Custom form integration** – Display forms for adding or editing rows.
+
+
+## Usage
+
+See the demo folder for example usage of the `local_table_sql` plugin.
+
+## Information for developers regarding the react-app directory
+
+The `react-app` directory contains the React frontend responsible for rendering table data, built using **Material React Table**.
+
+To build the React app (only needed if you made changes to the react code, a precompiled version is included in the plugin):
+
+1. Run the script:
+   ```bash
+   cd react-table
+   ./build.sh
+   ```
+
+2. The compiled output will be copied into the js/main.js directory.
+
+The react-app directory is not required for normal plugin usage**—only for development**.
+It is included in the repository for transparency and open-source collaboration.
+
+## JavaScript Licenses
+
+See: LICENSE.js.txt

classes/privacy/provider.php

@@ -23,7 +23,7 @@
 
 namespace local_table_sql\privacy;
 
-defined('MOODLE_INTERNAL') || die;
+defined('MOODLE_INTERNAL') || die();
 
 class provider implements \core_privacy\local\metadata\null_provider {
     /**

classes/table_sql.php

@@ -86,11 +86,12 @@ public function __construct($uniqueid = null) {
 
         if ($uniqueid === null) {
             // use the whole classname and namespace, then create an md5 to hide implementation details to the client
-            $uniqueid = str_replace('\\', '.', get_class($this));
+            $uniqueid = str_replace('\\', '.', $this->get_class_name());
+            $uniqueid = md5($uniqueid);
         } elseif (is_array($uniqueid)) {
             // create a uniqueid of the parameters and the current class
-            // don't use __CLASS__, because it will give you local_table_sql/sql_table
-            $uniqueid = join('-', array_merge([str_replace('\\', '.', get_class($this))], $uniqueid));
+            $uniqueid = join('-', array_merge([str_replace('\\', '.', $this->get_class_name())], $uniqueid));
+            $uniqueid = md5($uniqueid);
         } elseif (is_string($uniqueid)) {
             // ok
         } else {
@@ -175,6 +176,32 @@ public function __construct($uniqueid = null) {
         }
     }
 
+    /**
+     * Returns a class name. For anonymous classes, it returns the file path and line number, which IS NOT stable against code changes above the class definition.
+     * But good enough for us
+     * PHP's get_class() on anonymous functions also returns file + line number, but also adds some random string
+     * And that is not exactly defined in the PHP manual
+     *
+     * @return string The class name
+     */
+    function get_class_name(): string {
+        $reflection = new \ReflectionClass($this);
+
+        // 1. Handle Named Classes
+        if (!$reflection->isAnonymous()) {
+            return $reflection->getName();
+        }
+
+        // 2. Handle Anonymous Classes (MD5 of Location)
+        $fileName = $reflection->getFileName();
+        $startLine = $reflection->getStartLine();
+
+        // Create the volatile location string
+        $locationString = $fileName . ':' . $startLine;
+
+        return $locationString;
+    }
+
     /**
      * Add a single header.
      */
@@ -473,14 +500,6 @@ protected function sql_format_timestamp(string $column, ?string $format = null)
         }
     }
 
-    private function _uniqueid(string $prefix = 'unique') {
-        static $uniqueid = 0;
-
-        $uniqueid++;
-
-        return $prefix . $uniqueid;
-    }
-
     protected function no_filter($column) {
         $this->column_nofilter[] = $column;
     }

classes/table_sql_form.php

@@ -25,7 +25,7 @@
 
 use local_table_sql\local\js_call_amd;
 
-defined('MOODLE_INTERNAL') || die;
+defined('MOODLE_INTERNAL') || die();
 
 require_once("$CFG->libdir/tablelib.php");
 

classes/table_sql_subform.php

@@ -25,7 +25,7 @@
 
 use moodleform;
 
-defined('MOODLE_INTERNAL') || die;
+defined('MOODLE_INTERNAL') || die();
 
 global $CFG;
 require_once($CFG->libdir . "/formslib.php");

db/upgrade.php

@@ -23,7 +23,7 @@
  */
 
 
-defined('MOODLE_INTERNAL') || die;
+defined('MOODLE_INTERNAL') || die();
 
 function xmldb_local_table_sql_upgrade($oldversion) {
     global $DB;

version.php

@@ -21,9 +21,9 @@
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
-defined('MOODLE_INTERNAL') || die;
+defined('MOODLE_INTERNAL') || die();
 
-$plugin->version = 2025111701;
+$plugin->version = 2025112400;
 $plugin->requires = 2022041900;
 $plugin->component = 'local_table_sql';
 $plugin->release = '1.0';