Actually brute-force switches make more sense just after their "normal" version. Also, getSchema() method is preferably to be called before getColumns(), see next commit for reason

bdamele · bdamele · commit a23ca952e443 · 2011-04-29T21:09:07.000Z
diff --git a/lib/controller/action.py b/lib/controller/action.py
@@ -90,12 +90,18 @@ def action():
     if conf.getTables:
         conf.dumper.dbTables(conf.dbmsHandler.getTables())
 
-    if conf.getColumns:
-        conf.dumper.dbTableColumns(conf.dbmsHandler.getColumns())
+    if conf.commonTables:
+        conf.dumper.dbTables(tableExists(paths.COMMON_TABLES))
 
     if conf.getSchema:
         conf.dumper.dbTableColumns(conf.dbmsHandler.getSchema())
 
+    if conf.getColumns:
+        conf.dumper.dbTableColumns(conf.dbmsHandler.getColumns())
+
+    if conf.commonColumns:
+        conf.dumper.dbTableColumns(columnExists(paths.COMMON_COLUMNS))
+
     if conf.dumpTable:
         conf.dumper.dbTableValues(conf.dbmsHandler.dumpTable())
 
@@ -111,13 +117,6 @@ def action():
     if conf.sqlShell:
         conf.dbmsHandler.sqlShell()
 
-    # Brute force options
-    if conf.commonTables:
-        conf.dumper.dbTables(tableExists(paths.COMMON_TABLES))
-
-    if conf.commonColumns:
-        conf.dumper.dbTableColumns(columnExists(paths.COMMON_COLUMNS))
-
     # User-defined function options
     if conf.udfInject:
         conf.dbmsHandler.udfInjectCustom()
