Commits

Mike Bayer committed 987759a

- Improved an error message which would occur if a query() were made
against a non-selectable, such as a :func:`.literal_column`, and then
an attempt was made to use :meth:`.Query.join` such that the "left"
side would be determined as ``None`` and then fail. This condition
is now detected explicitly.

Comments (0)

Files changed (3)

doc/build/changelog/changelog_08.rst

     :version: 0.8.6
 
     .. change::
+        :tags: bug, orm
+        :versions: 0.9.4
+
+        Improved an error message which would occur if a query() were made
+        against a non-selectable, such as a :func:`.literal_column`, and then
+        an attempt was made to use :meth:`.Query.join` such that the "left"
+        side would be determined as ``None`` and then fail.  This condition
+        is now detected explicitly.
+
+    .. change::
         :tags: bug, sql
         :versions: 0.9.4
         :tickets: 2977

lib/sqlalchemy/orm/query.py

             elif self._entities:
                 left = self._entities[0].entity_zero_or_selectable
 
+        if left is None:
+            raise sa_exc.InvalidRequestError(
+                            "Don't know how to join from %s; please use "
+                            "select_from() to establish the left "
+                            "entity/selectable of this join" % self._entities[0])
+
         if left is right and \
                 not create_aliases:
             raise sa_exc.InvalidRequestError(

test/orm/test_joins.py

             "ON addresses.id = orders.address_id"
         )
 
+    def test_left_is_none(self):
+        User = self.classes.User
+        Address = self.classes.Address
+
+        sess = create_session()
+
+        assert_raises_message(
+            sa_exc.InvalidRequestError,
+            "Don't know how to join from x; please use select_from() to "
+            "establish the left entity/selectable of this join",
+            sess.query(literal_column('x'), User).join, Address
+        )
+
     def test_join_on_synonym(self):
 
         class User(object):