Fixed one wrong "return false" plus minor code reorder...

diff --git a/lib/xmldb/classes/generators/XMLDBGenerator.class.php b/lib/xmldb/classes/generators/XMLDBGenerator.class.php
index acacaefdd64881424d61311b9a78f7df5fd99441..113dd3261f801ac9c79e644691827bb76ba21478 100644 (file)
--- a/lib/xmldb/classes/generators/XMLDBGenerator.class.php
+++ b/lib/xmldb/classes/generators/XMLDBGenerator.class.php
@@ -179,7 +179,7 @@ class XMLDBgenerator {
         $table = 'CREATE TABLE ' . $this->getTableName($xmldb_table) . ' (';
 
         if (!$xmldb_fields = $xmldb_table->getFields()) {
-            return false;
+            return $results;
         }
 
     /// Prevent tables without prefix to be duplicated (part of MDL-6614)
@@ -576,14 +576,14 @@ class XMLDBgenerator {
                                                                              $this->alter_column_skip_default,
                                                                              $this->alter_column_skip_notnull), $alter);
 
-    /// Build the standard alter table modify
-        $results[] = $alter;
-
     /// Add the after clause if necesary
         if ($this->add_after_clause && $xmldb_field->getPrevious()) {
             $alter .= ' after ' . $this->getEncQuoted($xmldb_field->getPrevious());
         }
 
+    /// Build the standard alter table modify
+        $results[] = $alter;
+
         return $results;
     }