Merge pull request #913 from catalyst/fix-sql-format-405

[#911] fix sql cleaning
catalyst · Dec 9, 2024 · c61b76b · c61b76b
2 parents 9034f9c + fc05b1f
commit c61b76b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/classes/local/step/reader_sql.php b/classes/local/step/reader_sql.php
@@ -48,7 +48,8 @@ class reader_sql extends reader_step {
      */
     public static function form_define_fields(): array {
         return [
-            'sql' => ['type' => PARAM_TEXT, 'required' => true],
+            // Must be PARAM_RAW to avoid clean_param mangling the sql.
+            'sql' => ['type' => PARAM_RAW, 'required' => true],
             'counterfield' => ['type' => PARAM_TEXT],
             'countervalue' => ['type' => PARAM_TEXT],
         ];

diff --git a/classes/local/step/sql_trait.php b/classes/local/step/sql_trait.php
@@ -175,7 +175,8 @@ public function validate_config($config) {
      */
     public static function form_define_fields(): array {
         return [
-            'sql' => ['type' => PARAM_TEXT, 'required' => true],
+            // Must be PARAM_RAW to avoid clean_param mangling the sql.
+            'sql' => ['type' => PARAM_RAW, 'required' => true],
         ];
     }