diff options
| -rw-r--r-- | doc/build/changelog/changelog_11.rst | 8 | ||||
| -rw-r--r-- | lib/sqlalchemy/sql/crud.py | 2 | ||||
| -rw-r--r-- | test/sql/test_insert.py | 12 |
3 files changed, 21 insertions, 1 deletions
diff --git a/doc/build/changelog/changelog_11.rst b/doc/build/changelog/changelog_11.rst index 1554987c1..704962e42 100644 --- a/doc/build/changelog/changelog_11.rst +++ b/doc/build/changelog/changelog_11.rst @@ -52,6 +52,14 @@ sqlalchemy.sql.expression, due to mis-spelled "any_" and "all_" functions. + .. change:: 3880 + :tags: bg, sql + :tickets: 3880 + + Fixed bug where literal_binds compiler flag was not honored by the + :class:`.Insert` construct for the "multiple values" feature; the + subsequent values are now rendered as literals. + .. change:: 3877 :tags: bug, oracle, postgresql :tickets: 3877 diff --git a/lib/sqlalchemy/sql/crud.py b/lib/sqlalchemy/sql/crud.py index 42e22f93d..5739c22f9 100644 --- a/lib/sqlalchemy/sql/crud.py +++ b/lib/sqlalchemy/sql/crud.py @@ -603,7 +603,7 @@ def _extend_values_for_multiparams(compiler, stmt, values, kw): c, (_create_bind_param( compiler, c, row[c.key], - name="%s_m%d" % (c.key, i + 1) + name="%s_m%d" % (c.key, i + 1), **kw ) if elements._is_literal(row[c.key]) else compiler.process( row[c.key].self_group(), **kw)) diff --git a/test/sql/test_insert.py b/test/sql/test_insert.py index 73731e952..e23ab520d 100644 --- a/test/sql/test_insert.py +++ b/test/sql/test_insert.py @@ -683,6 +683,18 @@ class InsertImplicitReturningTest( 'othername_m0': 'foo'} ) + def test_insert_multiple_values_literal_binds(self): + ins = self.tables.myothertable.insert().values([ + {"othername": "foo"}, + {"othername": "bar"}, + ]) + self.assert_compile( + ins, + "INSERT INTO myothertable (othername) VALUES ('foo'), ('bar')", + checkparams={}, + literal_binds=True + ) + def test_insert_multiple_values_return_defaults(self): # TODO: not sure if this should raise an # error or what |