]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commitdiff
- [feature] ORM entities can be passed
authorMike Bayer <mike_mp@zzzcomputing.com>
Mon, 23 Jul 2012 22:22:06 +0000 (18:22 -0400)
committerMike Bayer <mike_mp@zzzcomputing.com>
Mon, 23 Jul 2012 22:22:06 +0000 (18:22 -0400)
to select() as well as the select_from(),
correlate(), and correlate_except()
methods, where they will be unwrapped
into selectables. [ticket:2245]

CHANGES
lib/sqlalchemy/orm/attributes.py
lib/sqlalchemy/orm/interfaces.py
lib/sqlalchemy/orm/properties.py
lib/sqlalchemy/orm/query.py
lib/sqlalchemy/orm/util.py
lib/sqlalchemy/sql/expression.py
test/orm/test_froms.py
test/orm/test_inspect.py
test/orm/test_query.py

diff --git a/CHANGES b/CHANGES
index ac0303c9c3d17a0aa6f7f0b56651605c8b08ed4d..db71d310b1ad70390bc99b2573e745fd31e790cb 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -151,6 +151,12 @@ underneath "0.7.xx".
     need autoflush w pre-attached object.
     [ticket:2464]
 
+  - [feature] ORM entities can be passed
+    to select() as well as the select_from(),
+    correlate(), and correlate_except()
+    methods, where they will be unwrapped
+    into selectables. [ticket:2245]
+
   - [feature] The registry of classes
     in declarative_base() is now a
     WeakValueDictionary.  So subclasses of
index 47349e64a8b9eec76ba764e23d1cfc01dd170b06..045a9465dc47b02d7b9b66a57f217cfbff1b79bb 100644 (file)
@@ -112,9 +112,11 @@ PASSIVE_ONLY_PERSISTENT = util.symbol("PASSIVE_ONLY_PERSISTENT",
 )
 
 
-class QueryableAttribute(interfaces.PropComparator):
+class QueryableAttribute(interfaces._InspectionAttr, interfaces.PropComparator):
     """Base class for class-bound attributes. """
 
+    is_attribute = True
+
     def __init__(self, class_, key, impl=None,
                         comparator=None, parententity=None,
                         of_type=None):
@@ -149,6 +151,10 @@ class QueryableAttribute(interfaces.PropComparator):
         # TODO: conditionally attach this method based on clause_element ?
         return self
 
+    @property
+    def expression(self):
+        return self.comparator.__clause_element__()
+
     def __clause_element__(self):
         return self.comparator.__clause_element__()
 
@@ -191,10 +197,7 @@ class QueryableAttribute(interfaces.PropComparator):
     def property(self):
         return self.comparator.property
 
-
-@inspection._inspects(QueryableAttribute)
-def _get_prop(source):
-    return source.property
+inspection._self_inspects(QueryableAttribute)
 
 class InstrumentedAttribute(QueryableAttribute):
     """Class bound instrumented attribute which adds descriptor methods."""
index 84c75525a3f18dfb0f3ce41b844904726ea86e4e..d0732b91359264127ed3de6009877e1aacdb8529 100644 (file)
@@ -62,6 +62,8 @@ class _InspectionAttr(object):
     is_instance = False
     is_mapper = False
     is_property = False
+    is_attribute = False
+    is_clause_element = False
 
 class MapperProperty(_InspectionAttr):
     """Manage the relationship of a ``Mapper`` to a single class
index 4533bbdb062539126a92f5421bc1f4e0fc607476..53ee1b5fd319edd446dc9a04defdc11d0979aa4b 100644 (file)
@@ -102,6 +102,7 @@ class ColumnProperty(StrategizedProperty):
         else:
             self.strategy_class = strategies.ColumnLoader
 
+
     @property
     def expression(self):
         """Return the primary column or expression for this ColumnProperty.
index 35c70d51ed890713c63b4d3ee3af18bda881ad12..0a345f2841811d0032cb9184979caa0910bd89a1 100644 (file)
@@ -26,10 +26,11 @@ from . import (
     )
 from .util import (
     AliasedClass, ORMAdapter, _entity_descriptor, PathRegistry,
-    _is_aliased_class, _is_mapped_class, _orm_columns, _orm_selectable,
+    _is_aliased_class, _is_mapped_class, _orm_columns,
     join as orm_join,with_parent, aliased
     )
-from .. import sql, util, log, exc as sa_exc, inspect
+from .. import sql, util, log, exc as sa_exc, inspect, inspection
+from ..sql.expression import _interpret_as_from
 from ..sql import (
         util as sql_util,
         expression, visitors
@@ -539,6 +540,9 @@ class Query(object):
 
         return self.enable_eagerloads(False).statement.as_scalar()
 
+    @property
+    def selectable(self):
+        return self.__clause_element__()
 
     def __clause_element__(self):
         return self.enable_eagerloads(False).with_labels().statement
@@ -798,7 +802,8 @@ class Query(object):
          """
 
         self._correlate = self._correlate.union(
-                                        _orm_selectable(s)
+                                        _interpret_as_from(s)
+                                        if s is not None else None
                                         for s in args)
 
     @_generative()
@@ -2672,7 +2677,6 @@ class Query(object):
             statement.append_order_by(*context.eager_order_by)
         return statement
 
-
     def _adjust_for_single_inheritance(self, context):
         """Apply single-table-inheritance filtering.
 
@@ -2696,6 +2700,7 @@ class Query(object):
     def __str__(self):
         return str(self._compile_context().statement)
 
+inspection._self_inspects(Query)
 
 class _QueryEntity(object):
     """represent an entity column returned within a Query result."""
index 8f340d36676f20ba6e539609896dd9984e6b7f53..27d9b1b69fc500454dea1aa4f6e611cff12c41cc 100644 (file)
@@ -1074,13 +1074,6 @@ def _orm_columns(entity):
     else:
         return [entity]
 
-def _orm_selectable(entity):
-    insp = inspection.inspect(entity, False)
-    if hasattr(insp, 'selectable'):
-        return insp.selectable
-    else:
-        return entity
-
 def has_identity(object):
     state = attributes.instance_state(object)
     return state.has_identity
index b9c149954ad8f3bd84542c256e7c865a72accaf8..a518852d89e9d426ee656f249d7ef35507ca58f3 100644 (file)
@@ -1426,6 +1426,19 @@ def _literal_as_text(element):
             "SQL expression object or string expected."
         )
 
+def _interpret_as_from(element):
+    insp = inspection.inspect(element, raiseerr=False)
+    if insp is None:
+        if isinstance(element, (util.NoneType, bool)):
+            return _const_expr(element)
+        elif isinstance(element, basestring):
+            return TextClause(unicode(element))
+    elif hasattr(insp, "selectable"):
+        return insp.selectable
+    else:
+        raise exc.ArgumentError("FROM expression expected")
+
+
 def _const_expr(element):
     if element is None:
         return null()
@@ -1445,12 +1458,15 @@ def _clause_element_as_expr(element):
         return element
 
 def _literal_as_column(element):
-    if isinstance(element, Visitable):
-        return element
-    elif hasattr(element, '__clause_element__'):
-        return element.__clause_element__()
-    else:
-        return literal_column(str(element))
+    insp = inspection.inspect(element, raiseerr=False)
+    if insp is not None:
+        if hasattr(insp, "expression"):
+            return insp.expression
+        elif hasattr(insp, "selectable"):
+            return insp.selectable
+        elif insp.is_clause_element:
+            return insp
+    return literal_column(str(element))
 
 def _literal_as_binds(element, name=None, type_=None):
     if hasattr(element, '__clause_element__'):
@@ -1539,6 +1555,7 @@ class ClauseElement(Visitable):
     bind = None
     _is_clone_of = None
     is_selectable = False
+    is_clause_element = True
 
     def _clone(self):
         """Create a shallow copy of this ClauseElement.
@@ -2173,6 +2190,15 @@ class ColumnElement(ClauseElement, CompareMixin):
     _key_label = None
     _alt_names = ()
 
+    @property
+    def expression(self):
+        """Return a column expression.
+
+        Part of the inspection interface; returns self.
+
+        """
+        return self
+
     @property
     def _select_iterable(self):
         return (self, )
@@ -2973,6 +2999,10 @@ class TextClause(Executable, ClauseElement):
     def _select_iterable(self):
         return (self,)
 
+    @property
+    def selectable(self):
+        return self
+
     _hide_froms = []
 
     def __init__(
@@ -5315,7 +5345,8 @@ class Select(SelectBase):
         if fromclauses and fromclauses[0] is None:
             self._correlate = ()
         else:
-            self._correlate = set(self._correlate).union(fromclauses)
+            self._correlate = set(self._correlate).union(
+                    _interpret_as_from(f) for f in fromclauses)
 
     @_generative
     def correlate_except(self, *fromclauses):
@@ -5323,15 +5354,16 @@ class Select(SelectBase):
         if fromclauses and fromclauses[0] is None:
             self._correlate_except = ()
         else:
-            self._correlate_except = set(self._correlate_except
-                                            ).union(fromclauses)
+            self._correlate_except = set(self._correlate_except).union(
+                    _interpret_as_from(f) for f in fromclauses)
 
     def append_correlation(self, fromclause):
         """append the given correlation expression to this select()
         construct."""
 
         self._should_correlate = False
-        self._correlate = set(self._correlate).union([fromclause])
+        self._correlate = set(self._correlate).union(
+                _interpret_as_from(f) for f in fromclause)
 
     def append_column(self, column):
         """append the given column expression to the columns clause of this
@@ -5387,7 +5419,7 @@ class Select(SelectBase):
 
         """
         self._reset_exported()
-        fromclause = _literal_as_text(fromclause)
+        fromclause = _interpret_as_from(fromclause)
         self._from_obj = self._from_obj.union([fromclause])
 
     def _populate_column_collection(self):
index f6037d0719b461d4b531fe72a2dd65c9cf542d09..546c66204752daddbaa95a3be851fed4d611a926 100644 (file)
@@ -65,7 +65,6 @@ class QueryTest(_fixtures.FixtureTest):
 
         configure_mappers()
 
-
 class QueryCorrelatesLikeSelect(QueryTest, AssertsCompiledSQL):
 
     query_correlated = "SELECT users.name AS users_name, " \
index 487182fac22aa7ed7c527ad0c8f0be12210ccdc6..1c5cab8a039ea2e603b5e861cb1fe4ff5f054087 100644 (file)
@@ -178,17 +178,23 @@ class TestORMInspection(_fixtures.FixtureTest):
     def test_insp_prop(self):
         User = self.classes.User
         prop = inspect(User.addresses)
-        is_(prop, User.addresses.property)
+        is_(prop, User.addresses)
+
+    def test_insp_aliased_prop(self):
+        User = self.classes.User
+        ua = aliased(User)
+        prop = inspect(ua.addresses)
+        is_(prop, ua.addresses)
 
     def test_rel_accessors(self):
         User = self.classes.User
         Address = self.classes.Address
         prop = inspect(User.addresses)
-        is_(prop.parent, class_mapper(User))
-        is_(prop.mapper, class_mapper(Address))
+        is_(prop.property.parent, class_mapper(User))
+        is_(prop.property.mapper, class_mapper(Address))
 
         assert not hasattr(prop, 'columns')
-        assert not hasattr(prop, 'expression')
+        assert hasattr(prop, 'expression')
 
 
     def test_instance_state(self):
index e8d6c0901dd6660ca58d2854ac5d589152c971c1..b80db67ebc066975f6e99671db9dd7374603bd59 100644 (file)
@@ -119,6 +119,80 @@ class RowTupleTest(QueryTest):
                 asserted
             )
 
+class RawSelectTest(QueryTest, AssertsCompiledSQL):
+    __dialect__ = 'default'
+
+    def test_select_from_entity(self):
+        User = self.classes.User
+
+        self.assert_compile(
+            select(['*']).select_from(User),
+            "SELECT * FROM users"
+        )
+
+    def test_select_from_aliased_entity(self):
+        User = self.classes.User
+        ua = aliased(User, name="ua")
+        self.assert_compile(
+            select(['*']).select_from(ua),
+            "SELECT * FROM users AS ua"
+        )
+
+    def test_correlate_entity(self):
+        User = self.classes.User
+        Address = self.classes.Address
+
+        self.assert_compile(
+            select([User]).where(User.id == Address.user_id).
+                correlate(Address),
+            "SELECT users.id, users.name FROM users "
+            "WHERE users.id = addresses.user_id"
+        )
+
+    def test_correlate_aliased_entity(self):
+        User = self.classes.User
+        Address = self.classes.Address
+        aa = aliased(Address, name="aa")
+
+        self.assert_compile(
+            select([User]).where(User.id == aa.user_id).
+                correlate(aa),
+            "SELECT users.id, users.name FROM users "
+            "WHERE users.id = aa.user_id"
+        )
+
+    def test_columns_clause_entity(self):
+        User = self.classes.User
+
+        self.assert_compile(
+            select([User]),
+            "SELECT users.id, users.name FROM users"
+        )
+
+    def test_columns_clause_columns(self):
+        User = self.classes.User
+
+        self.assert_compile(
+            select([User.id, User.name]),
+            "SELECT users.id, users.name FROM users"
+        )
+
+    def test_columns_clause_aliased_columns(self):
+        User = self.classes.User
+        ua = aliased(User, name='ua')
+        self.assert_compile(
+            select([ua.id, ua.name]),
+            "SELECT ua.id, ua.name FROM users AS ua"
+        )
+
+    def test_columns_clause_aliased_entity(self):
+        User = self.classes.User
+        ua = aliased(User, name='ua')
+        self.assert_compile(
+            select([ua]),
+            "SELECT ua.id, ua.name FROM users AS ua"
+        )
+
 class GetTest(QueryTest):
     def test_get(self):
         User = self.classes.User