MDL-45171 log reports: fix sql for cross support
authorEloy Lafuente (stronk7) <stronk7@moodle.org>
Fri, 18 Apr 2014 15:34:06 +0000 (17:34 +0200)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Fri, 18 Apr 2014 15:34:06 +0000 (17:34 +0200)
Rules for cross-db support are:

- Always use the 'AS' keyword for columns.
- Never use the 'AS' keyword for tables/views.

report/log/classes/table_log.php
report/loglive/classes/table_log.php

index a41c0da..76ed093 100644 (file)
@@ -481,7 +481,7 @@ class report_log_table_log extends table_sql {
             $ccselect = ', ' . context_helper::get_preload_record_columns_sql('ctx');
             $ccjoin = "LEFT JOIN {context} ctx ON (ctx.instanceid = c.id AND ctx.contextlevel = :contextlevel)";
             $courseparams['contextlevel'] = CONTEXT_COURSE;
-            $sql = "SELECT c.id,c.shortname $ccselect FROM {course} as c
+            $sql = "SELECT c.id,c.shortname $ccselect FROM {course} c
                    $ccjoin
                      WHERE c.id " . $coursesql;
 
@@ -496,4 +496,4 @@ class report_log_table_log extends table_sql {
             }
         }
     }
-}
\ No newline at end of file
+}
index 04614b6..c124de6 100644 (file)
@@ -387,7 +387,7 @@ class report_loglive_table_log extends table_sql {
             $ccselect = ', ' . context_helper::get_preload_record_columns_sql('ctx');
             $ccjoin = "LEFT JOIN {context} ctx ON (ctx.instanceid = c.id AND ctx.contextlevel = :contextlevel)";
             $courseparams['contextlevel'] = CONTEXT_COURSE;
-            $sql = "SELECT c.id,c.shortname $ccselect FROM {course} as c
+            $sql = "SELECT c.id,c.shortname $ccselect FROM {course} c
                    $ccjoin
                      WHERE c.id " . $coursesql;