Fix automated tests

moodleou · Jul 7, 2021 · 29660f6 · 29660f6
1 parent 517bc76
commit 29660f6
Show file tree

Hide file tree

Showing 6 changed files with 29 additions and 14 deletions.
diff --git a/classes/external/get_users.php b/classes/external/get_users.php
@@ -59,24 +59,29 @@ public static function execute(string $query, string $capability): array {
         self::validate_context($context);
         require_capability('report/customsql:definequeries', $context);
 
-        $extrafields = get_extra_user_fields($context);
+        if (class_exists('\core_user\fields')) {
+            $extrafields = \core_user\fields::for_identity($context, false)->get_required_fields();
+            $fields = \core_user\fields::for_identity($context,
+                    false)->with_userpic()->get_sql('u', false, '', '', false)->selects;
+        } else {
+            $extrafields = get_extra_user_fields($context);
+            $fields = \user_picture::fields('u', $extrafields);
+        }
 
-        $fields = array_merge(['u.id'], preg_split('~\s*,\s*~',
-                \user_picture::fields('u')), $extrafields);
         $withcapabilityjoin = get_with_capability_join($context, $capability, 'u.id');
         [$wherecondition, $whereparams] = users_search_sql($query, 'u', true, $extrafields);
         [$sort, $sortparams] = users_order_by_sql('u', $query, $context);
 
         $users = $DB->get_records_sql("
-                SELECT " . implode(', ', $fields) . "
+                SELECT $fields
                   FROM {user} u
 
                   JOIN (
 
                      SELECT id
                       FROM {user} u
-                      {$withcapabilityjoin->joins}
-                     WHERE {$withcapabilityjoin->wheres}
+                      $withcapabilityjoin->joins
+                     WHERE $withcapabilityjoin->wheres
 
                      UNION
 

diff --git a/classes/privacy/provider.php b/classes/privacy/provider.php
@@ -229,9 +229,9 @@ public static function delete_data_for_users(request\approved_userlist $userlist
      */
     protected static function you_or_somebody_else($userid, $user) {
         if ($userid == $user->id) {
-            return get_string('privacy_you', 'mod_bookingsystem');
+            return get_string('privacy_you', 'report_customsql');
         } else {
-            return get_string('privacy_somebodyelse', 'mod_bookingsystem');
+            return get_string('privacy_somebodyelse', 'report_customsql');
         }
     }
 }
diff --git a/edit_form.php b/edit_form.php
@@ -120,10 +120,18 @@ public function definition() {
                 if (!$user) {
                     return false;
                 }
+
+                if (class_exists('\core_user\fields')) {
+                    $extrafields = \core_user\fields::for_identity(\context_system::instance(),
+                            false)->get_required_fields();
+                } else {
+                    $extrafields = get_extra_user_fields(context_system::instance());
+                }
+
                 return $OUTPUT->render_from_template(
                         'report_customsql/form-user-selector-suggestion',
                         \report_customsql\external\get_users::prepare_result_object(
-                                $user, get_extra_user_fields(context_system::instance()))
+                                $user, $extrafields)
                         );
             }
         ];

diff --git a/templates/form-user-selector-suggestion.mustache b/templates/form-user-selector-suggestion.mustache
@@ -32,7 +32,8 @@
     * identity concatenated list of identity fields.
 
     Example context (json):
-    { "id": "1",
+    {
+      "id": "1",
       "fullname": "Admin User",
       "hasidentity": true,
       "identity": "[email protected], 0144114141",

diff --git a/templates/form_report_information.mustache b/templates/form_report_information.mustache
@@ -31,9 +31,10 @@
     * usermodified Last modified by.
 
     Example context (json):
-    { "timecreated": "Wednesday, 5 May 2021",
+    {
+      "timecreated": "Wednesday, 5 May 2021",
       "timemodified": "Wednesday, 5 May 2021",
-      "usermodified": "Admin User",
+      "usermodified": "Admin User"
     }
 }}
 <div class="report-customsql-infomation">

diff --git a/tests/behat/report_customsql.feature b/tests/behat/report_customsql.feature
@@ -218,8 +218,8 @@ Feature: Ad-hoc database queries report
 
   Scenario: View an Ad-hoc database query that returns data that confuses PHP CSV parsing
     Given the following custom sql report exists:
-      | name     | Test query                            |
-      | querysql | SELECT '\' AS Backslash, ',' AS Comma |
+      | name     | Test query                                    |
+      | querysql | SELECT CHR(92) AS Backslash, CHR(44) AS Comma |
     When I log in as "admin"
     And I view the "Test query" custom sql report
     Then "\" row "Comma" column of "report_customsql_results" table should contain ","