]> git.ipfire.org Git - thirdparty/sqlalchemy/alembic.git/commitdiff
- Adjusted the rendering for index expressions such that a :class:`.Column`
authorMike Bayer <mike_mp@zzzcomputing.com>
Fri, 30 Oct 2015 16:12:07 +0000 (12:12 -0400)
committerMike Bayer <mike_mp@zzzcomputing.com>
Fri, 30 Oct 2015 16:12:07 +0000 (12:12 -0400)
object present in the source :class:`.Index` will not be rendered
as table-qualified; e.g. the column name will be rendered alone.
Table-qualified names here were failing on systems such as Postgresql.
fixes #337

alembic/__init__.py
alembic/autogenerate/render.py
docs/build/changelog.rst
tests/test_autogen_render.py

index bbd3e47fc1dbf1e29f0bee3c7a099d51aac687ec..56f6f5e970b2d2e845bcdedf06803386a260001a 100644 (file)
@@ -1,6 +1,6 @@
 from os import path
 
-__version__ = '0.8.3'
+__version__ = '0.8.4'
 
 package_dir = path.abspath(path.dirname(__file__))
 
index 9a64641004af708bfd84a176a50e003e37e4d33c..342330784f545a203a44d8f2beca40a64652ed42 100644 (file)
@@ -418,7 +418,8 @@ def _ident(name):
 def _render_potential_expr(value, autogen_context, wrap_in_text=True):
     if isinstance(value, sql.ClauseElement):
         if compat.sqla_08:
-            compile_kw = dict(compile_kwargs={'literal_binds': True})
+            compile_kw = dict(compile_kwargs={
+                'literal_binds': True, "include_table": False})
         else:
             compile_kw = {}
 
index 04ca31579de5327bedcacc983c1c25eaf8047a51..187e7246ffb9852f9ec6059ee77efc4354fb14fe 100644 (file)
@@ -3,6 +3,18 @@
 Changelog
 ==========
 
+.. changelog::
+    :version: 0.8.4
+
+    .. change::
+      :tags: bug, autogenerate
+      :tickets: 337
+
+      Adjusted the rendering for index expressions such that a :class:`.Column`
+      object present in the source :class:`.Index` will not be rendered
+      as table-qualified; e.g. the column name will be rendered alone.
+      Table-qualified names here were failing on systems such as Postgresql.
+
 .. changelog::
     :version: 0.8.3
     :released: October 16, 2015
index c113a233ec724af1fceaa68a5a751bd49092c194..cd4b1eb433ec25b6ed81a4bd695e8920c04ea38c 100644 (file)
@@ -154,7 +154,7 @@ class AutogenRenderTest(TestBase):
                 autogenerate.render_op_text(autogen_context, op_obj),
                 """op.create_index('foo_idx', 't', \
 ['x', 'y'], unique=False, """
-                """postgresql_where=sa.text(!U"t.y = 'something'"))"""
+                """postgresql_where=sa.text(!U"y = 'something'"))"""
             )
         else:
             eq_ignore_whitespace(
@@ -178,7 +178,7 @@ unique=False, """
         eq_ignore_whitespace(
             autogenerate.render_op_text(self.autogen_context, op_obj),
             "op.create_index('test_lower_code_idx', 'test', "
-            "[sa.text(!U'lower(test.code)')], unique=False)"
+            "[sa.text(!U'lower(code)')], unique=False)"
         )
 
     @config.requirements.fail_before_sqla_080
@@ -210,7 +210,7 @@ unique=False, """
         eq_ignore_whitespace(
             autogenerate.render_op_text(self.autogen_context, op_obj),
             "op.create_index('test_desc_code_idx', 'test', "
-            "[sa.text(!U'test.code DESC')], unique=False)"
+            "[sa.text(!U'code DESC')], unique=False)"
         )
 
     def test_drop_index(self):