- Fixed regression from 0.8.3 as a result of :ticket:`2818`

where :meth:`.Query.exists` wouldn't work on a query that only
had a :meth:`.Query.select_from` entry but no other entities.
re: #2818 fixes #2995

diff --git a/doc/build/changelog/changelog_08.rst b/doc/build/changelog/changelog_08.rst
index 899be4d0eb0b25c2976f5168aa20f251771e5864..79295f960503b06b313600dec676fae135dbca11 100644 (file)
--- a/doc/build/changelog/changelog_08.rst
+++ b/doc/build/changelog/changelog_08.rst
@@ -11,6 +11,15 @@
 .. changelog::
     :version: 0.8.6
 
+    .. change::
+        :tags: bug, orm
+        :tickets: 2995,
+        :versions: 0.9.4
+
+        Fixed regression from 0.8.3 as a result of :ticket:`2818`
+        where :meth:`.Query.exists` wouldn't work on a query that only
+        had a :meth:`.Query.select_from` entry but no other entities.
+
     .. change::
         :tags: bug, general
         :tickets: 2986

diff --git a/lib/sqlalchemy/orm/query.py b/lib/sqlalchemy/orm/query.py
index 81f49980e0d7632d5e92b2de720b8de6f6808b61..3bd6add8f126fcaa676c441c2367a0b68f5615b8 100644 (file)
--- a/lib/sqlalchemy/orm/query.py
+++ b/lib/sqlalchemy/orm/query.py
@@ -2505,7 +2505,14 @@ class Query(object):
         .. versionadded:: 0.8.1
 
         """
-        return sql.exists(self.with_labels().statement.with_only_columns(['1']))
+
+        # .add_columns() for the case that we are a query().select_from(X),
+        # so that ".statement" can be produced (#2995) but also without
+        # omitting the FROM clause from a query(X) (#2818);
+        # .with_only_columns() after we have a core select() so that
+        # we get just "SELECT 1" without any entities.
+        return sql.exists(self.add_columns('1').with_labels().
+                                statement.with_only_columns(['1']))
 
     def count(self):
         """Return a count of rows this Query would return.

diff --git a/test/orm/test_query.py b/test/orm/test_query.py
index 52f32bd9c53cf7c8aa768bf31c0e55ccf7166e30..8cba3558d18b5981b1491c6518e89531ee67c52f 100644 (file)
--- a/test/orm/test_query.py
+++ b/test/orm/test_query.py
@@ -1779,6 +1779,17 @@ class ExistsTest(QueryTest, AssertsCompiledSQL):
             ') AS anon_1'
         )
 
+    def test_exists_w_select_from(self):
+        User = self.classes.User
+        sess = create_session()
+
+        q1 = sess.query().select_from(User).exists()
+        self.assert_compile(sess.query(q1),
+            'SELECT EXISTS ('
+                'SELECT 1 FROM users'
+            ') AS anon_1'
+        )
+
 
 class CountTest(QueryTest):
     def test_basic(self):