diff options
| -rw-r--r-- | doc/build/changelog/unreleased_14/5781.rst | 9 | ||||
| -rw-r--r-- | lib/sqlalchemy/orm/context.py | 2 | ||||
| -rw-r--r-- | test/orm/inheritance/test_polymorphic_rel.py | 10 | ||||
| -rw-r--r-- | test/orm/inheritance/test_single.py | 18 |
4 files changed, 37 insertions, 2 deletions
diff --git a/doc/build/changelog/unreleased_14/5781.rst b/doc/build/changelog/unreleased_14/5781.rst new file mode 100644 index 000000000..ba1ccec7b --- /dev/null +++ b/doc/build/changelog/unreleased_14/5781.rst @@ -0,0 +1,9 @@ +.. change:: + :tags: bug, orm + :tickets: 5781 + + Fixed 1.4 regression where the use of :meth:`_orm.Query.having` in + conjunction with queries with internally adapted SQL elements (common in + inheritance scenarios) would fail due to an incorrect function call. Pull + request courtesy esoh. + diff --git a/lib/sqlalchemy/orm/context.py b/lib/sqlalchemy/orm/context.py index cd654ed3d..41b2146a3 100644 --- a/lib/sqlalchemy/orm/context.py +++ b/lib/sqlalchemy/orm/context.py @@ -585,7 +585,7 @@ class ORMSelectCompileState(ORMCompileState, SelectState): if query._having_criteria: self._having_criteria = tuple( - current_adapter(crit, True, True) if current_adapter else crit + current_adapter(crit, True) if current_adapter else crit for crit in query._having_criteria ) diff --git a/test/orm/inheritance/test_polymorphic_rel.py b/test/orm/inheritance/test_polymorphic_rel.py index 51277943f..84ef22d52 100644 --- a/test/orm/inheritance/test_polymorphic_rel.py +++ b/test/orm/inheritance/test_polymorphic_rel.py @@ -2360,4 +2360,12 @@ class PolymorphicAliasedJoinsTest( class PolymorphicJoinsTest(_PolymorphicTestBase, _PolymorphicJoins): - pass + def test_having_group_by(self): + sess = create_session() + eq_( + sess.query(Person.name) + .group_by(Person.name) + .having(Person.name == "dilbert") + .all(), + [("dilbert",)], + ) diff --git a/test/orm/inheritance/test_single.py b/test/orm/inheritance/test_single.py index aa17f4300..cbe6bd238 100644 --- a/test/orm/inheritance/test_single.py +++ b/test/orm/inheritance/test_single.py @@ -416,6 +416,24 @@ class SingleInheritanceTest(testing.AssertsCompiledSQL, fixtures.MappedTest): }, ) + def test_having(self): + + Engineer, Manager = self.classes("Engineer", "Manager") + + sess = create_session() + + self.assert_compile( + sess.query(Engineer) + .group_by(Engineer.employee_id) + .having(Engineer.name == "js"), + "SELECT employees.employee_id AS employees_employee_id, " + "employees.name AS employees_name, employees.manager_data " + "AS employees_manager_data, employees.engineer_info " + "AS employees_engineer_info, employees.type AS employees_type " + "FROM employees WHERE employees.type IN ([POSTCOMPILE_type_1]) " + "GROUP BY employees.employee_id HAVING employees.name = :name_1", + ) + def test_from_self_count(self): Engineer = self.classes.Engineer |