MDL-14679 table/column meta cache is reset on each request and DDL operation.

No need to call it with false parameter but in very exceptional ocasions.

diff --git a/lib/ddl/database_manager.php b/lib/ddl/database_manager.php
index 2b0d0101965dd8df333498124c4007fac1aa9c72..60a469a4d3174c1cdb2659523f0d594a40019292 100644 (file)
--- a/lib/ddl/database_manager.php
+++ b/lib/ddl/database_manager.php
@@ -152,7 +152,7 @@ class database_manager {
         }
 
     /// Get list of fields in table
-        $columns = $this->mdb->get_columns($tablename, false);
+        $columns = $this->mdb->get_columns($tablename);
 
         $exists = array_key_exists($fieldname,  $columns);
 
@@ -1010,7 +1010,7 @@ class database_manager {
             }
 
             // a) check for required fields
-            $dbfields = $this->mdb->get_columns($tablename, false);
+            $dbfields = $this->mdb->get_columns($tablename);
             $fields   = $table->getFields();
             foreach ($fields as $field) {
                 $fieldname = $field->getName();

diff --git a/lib/ddl/mssql_sql_generator.php b/lib/ddl/mssql_sql_generator.php
index eb56aba40bad80426c92701f593788de547cce74..9392aed3abf64f496549d4f444917af253643e7c 100644 (file)
--- a/lib/ddl/mssql_sql_generator.php
+++ b/lib/ddl/mssql_sql_generator.php
@@ -299,7 +299,7 @@ class mssql_sql_generator extends sql_generator {
         $fieldname = $xmldb_field->getName();
 
     /// Take a look to field metadata
-        $meta = $this->mdb->get_columns($tablename, false);
+        $meta = $this->mdb->get_columns($tablename);
         $metac = $meta[$fieldname];
         $oldmetatype = $metac->meta_type;
 

diff --git a/lib/ddl/mysql_sql_generator.php b/lib/ddl/mysql_sql_generator.php
index b3f5d06c182347fabf36aa0051ffaed20f539f86..74b092f3215bc85f49b9c7400e3670b7d2972938 100644 (file)
--- a/lib/ddl/mysql_sql_generator.php
+++ b/lib/ddl/mysql_sql_generator.php
@@ -308,7 +308,7 @@ class mysql_sql_generator extends sql_generator {
         $tablename = $xmldb_table->getName($xmldb_table);
 
     /// Fetch all the columns in the table
-        if (!$columns = $this->mdb->get_columns($tablename, false)) {
+        if (!$columns = $this->mdb->get_columns($tablename)) {
             return array();
         }
 

diff --git a/lib/ddl/postgres_sql_generator.php b/lib/ddl/postgres_sql_generator.php
index 19c83d1d86c9d173bf56c441669896f1f12efbc7..9d78fbcc61e9093cfb161db5353ded80003755ca 100644 (file)
--- a/lib/ddl/postgres_sql_generator.php
+++ b/lib/ddl/postgres_sql_generator.php
@@ -198,7 +198,7 @@ class postgres_sql_generator extends sql_generator {
         $fieldname = $xmldb_field->getName();
 
     /// Take a look to field metadata
-        $meta = $this->mdb->get_columns($tablename, false);
+        $meta = $this->mdb->get_columns($tablename);
         $metac = $meta[$xmldb_field->getName()];
         $oldmetatype = $metac->meta_type;
         $oldlength = $metac->max_length;