- tables with schemas can still be used in sqlite, firebird,

schema name just gets dropped [ticket:890]

diff --git a/CHANGES b/CHANGES
index 12e1782f8d0da1d30a6cd39fc424e52ee6eab1b9..fc71cb4ae01e80fb3d18d5a095b9579514b21073 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,9 @@
 0.3.12
 - sql
     - func. objects can be pickled/unpickled [ticket:844]
+    - tables with schemas can still be used in sqlite, firebird,
+      schema name just gets dropped [ticket:890]
+
 - orm
     - query.get() and related functions (like many-to-one lazyloading)
       generate randomly-generated bind parameter names, to prevent

diff --git a/lib/sqlalchemy/ansisql.py b/lib/sqlalchemy/ansisql.py
index e7146fb3d9fd05063781f3901ee844d779b1f9d2..a901df74f7372ab418eb7e5d63f607c2a1aecb19 100644 (file)
--- a/lib/sqlalchemy/ansisql.py
+++ b/lib/sqlalchemy/ansisql.py
@@ -1072,7 +1072,7 @@ class ANSIIdentifierPreparer(object):
         if name is None:
             name = table.name
         result = self.__generic_obj_format(table, name)
-        if use_schema and getattr(table, "schema", None):
+        if not self.omit_schema and use_schema and getattr(table, "schema", None):
             result = self.__generic_obj_format(table, table.schema) + "." + result
         return result
 

diff --git a/test/engine/reflection.py b/test/engine/reflection.py
index 5fbdf5ea9d69dd1c5c0b5c6e910b7be5cb841f8d..5c014ea466cba42ba1f817a836c28748048da437 100644 (file)
--- a/test/engine/reflection.py
+++ b/test/engine/reflection.py
@@ -595,7 +595,6 @@ class CreateDropTest(PersistTest):
 
 class SchemaTest(PersistTest):
     # this test should really be in the sql tests somewhere, not engine
-    @testbase.unsupported('sqlite')
     def testiteration(self):
         metadata = MetaData()
         table1 = Table('table1', metadata,
@@ -616,8 +615,12 @@ class SchemaTest(PersistTest):
         gen.traverse(table2)
         buf = buf.getvalue()
         print buf
-        assert buf.index("CREATE TABLE someschema.table1") > -1
-        assert buf.index("CREATE TABLE someschema.table2") > -1
+        if testbase.db.dialect.preparer().omit_schema:
+            assert buf.index("CREATE TABLE table1") > -1
+            assert buf.index("CREATE TABLE table2") > -1
+        else:
+            assert buf.index("CREATE TABLE someschema.table1") > -1
+            assert buf.index("CREATE TABLE someschema.table2") > -1
 
 
 if __name__ == "__main__":