MDL-68793 core_table: fix WS name to comply with guidelines
authorSimey Lameze <simey@moodle.com>
Fri, 22 May 2020 02:08:16 +0000 (10:08 +0800)
committerSimey Lameze <simey@moodle.com>
Tue, 2 Jun 2020 03:49:04 +0000 (11:49 +0800)
lib/db/services.php
lib/table/amd/build/local/dynamic/repository.min.js
lib/table/amd/build/local/dynamic/repository.min.js.map
lib/table/amd/src/local/dynamic/repository.js
lib/table/classes/external/dynamic/get.php [moved from lib/table/classes/external/dynamic/fetch.php with 99% similarity]
lib/table/tests/external/dynamic/get_test.php [moved from lib/table/tests/external/dynamic/fetch_test.php with 94% similarity]
version.php

index e8c1350..1a9ae7f 100644 (file)
@@ -2764,10 +2764,10 @@ $functions = array(
         'capabilities'  => '',
         'services'      => [MOODLE_OFFICIAL_MOBILE_SERVICE],
     ],
-    'core_table_dynamic_fetch' => [
-        'classname' => 'core_table\external\dynamic\fetch',
+    'core_table_get_dynamic_table_content' => [
+        'classname' => 'core_table\external\dynamic\get',
         'methodname' => 'execute',
-        'description' => 'Fetch a dynamic table view raw html',
+        'description' => 'Get the dynamic table content raw html',
         'type' => 'read',
         'ajax' => true,
         'services' => [MOODLE_OFFICIAL_MOBILE_SERVICE],
index ff5ca26..e07cb96 100644 (file)
Binary files a/lib/table/amd/build/local/dynamic/repository.min.js and b/lib/table/amd/build/local/dynamic/repository.min.js differ
index a8cb000..4cc7125 100644 (file)
Binary files a/lib/table/amd/build/local/dynamic/repository.min.js.map and b/lib/table/amd/build/local/dynamic/repository.min.js.map differ
index 3e43ac9..e548e97 100644 (file)
@@ -54,7 +54,7 @@ export const fetch = (component, handler, uniqueid, {
         hiddenColumns = {}
     } = {}, resetPreferences = false) => {
     return fetchMany([{
-        methodname: `core_table_dynamic_fetch`,
+        methodname: `core_table_get_dynamic_table_content`,
         args: {
             component,
             handler,
similarity index 99%
rename from lib/table/classes/external/dynamic/fetch.php
rename to lib/table/classes/external/dynamic/get.php
index ad2fcda..f21e463 100644 (file)
@@ -31,7 +31,6 @@ use external_multiple_structure;
 use external_single_structure;
 use external_value;
 use external_warnings;
-use moodle_url;
 
 /**
  * Core table external functions.
@@ -41,7 +40,7 @@ use moodle_url;
  * @copyright  2020 Simey Lameze <simey@moodle.com>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class fetch extends external_api {
+class get extends external_api {
 
     /**
      * Describes the parameters for fetching the table html.
@@ -140,7 +139,7 @@ class fetch extends external_api {
     }
 
     /**
-     * External function to fetch a table view.
+     * External function to get the table view content.
      *
      * @param string $component The component.
      * @param string $handler Dynamic table class name.
@@ -15,7 +15,7 @@
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
 /**
- * Unit tests for core_table\external\fetch;
+ * Unit tests for core_table\external\dynamic\get;
  *
  * @package   core_table
  * @category  test
@@ -31,14 +31,14 @@ use core_table\local\filter\filter;
 use advanced_testcase;
 
 /**
- * Unit tests for core_table\external\fetch;
+ * Unit tests for core_table\external\dynamic\get;
  *
  * @package   core_table
  * @category  test
  * @copyright  2020 Simey Lameze <simey@moodle.com>
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class fetch_test extends advanced_testcase {
+class get_test extends advanced_testcase {
 
     /**
      * Setup before class.
@@ -55,7 +55,7 @@ class fetch_test extends advanced_testcase {
         $this->resetAfterTest();
 
         $this->expectException(\invalid_parameter_exception::class);
-        fetch::execute(
+        get::execute(
             "core-user",
             "participants",
             "",
@@ -79,7 +79,7 @@ class fetch_test extends advanced_testcase {
         $this->resetAfterTest();
 
         $this->expectException(\UnexpectedValueException::class);
-        fetch::execute(
+        get::execute(
             "core_users",
             "participants",
             "",
@@ -106,7 +106,7 @@ class fetch_test extends advanced_testcase {
         $this->expectExceptionMessage("Table handler class {$handler} not found. Please make sure that your table handler class is under the \\core_user\\table namespace.");
 
         // Tests that invalid users_participants_table class gets an exception.
-        fetch::execute(
+        get::execute(
             "core_user",
             "users_participants_table",
             "",
@@ -143,7 +143,7 @@ class fetch_test extends advanced_testcase {
         $this->expectExceptionMessage("Invalid parameter value detected (filters => Invalid parameter value detected " .
         "(Missing required key in single structure: name): Missing required key in single structure: name");
 
-        fetch::execute(
+        get::execute(
             "core_user",
             "participants", "user-index-participants-{$course->id}",
             $this->get_sort_array(['firstname' => SORT_ASC]),
@@ -153,9 +153,9 @@ class fetch_test extends advanced_testcase {
     }
 
     /**
-     * Test execute fetch table.
+     * Test execute method.
      */
-    public function test_execute_fetch_table(): void {
+    public function test_table_get_execute(): void {
         $this->resetAfterTest();
 
         $course = $this->getDataGenerator()->create_course();
@@ -176,7 +176,7 @@ class fetch_test extends advanced_testcase {
             ]
         ];
 
-        $participantstable = fetch::execute(
+        $participantstable = get::execute(
             "core_user",
             "participants",
             "user-index-participants-{$course->id}",
index 32f956c..17fac9b 100644 (file)
@@ -29,7 +29,7 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-$version  = 2020052900.00;              // YYYYMMDD      = weekly release date of this DEV branch.
+$version  = 2020052900.01;              // YYYYMMDD      = weekly release date of this DEV branch.
                                         //         RR    = release increments - 00 in DEV branches.
                                         //           .XX = incremental changes.
 $release  = '3.9dev+ (Build: 20200529)'; // Human-friendly version name