Fixed the consideration of the ``between()`` operator

so that it works correctly with the new relationship local/remote
system.
[ticket:1768]

diff --git a/doc/build/changelog/changelog_08.rst b/doc/build/changelog/changelog_08.rst
index 52414be6d6c4b610a513cd04e0422482ad23e9e3..95e1b44b5ee200f0dd8d749202695c892cd7b49a 100644 (file)
--- a/doc/build/changelog/changelog_08.rst
+++ b/doc/build/changelog/changelog_08.rst
@@ -6,6 +6,14 @@
 .. changelog::
     :version: 0.8.0
 
+    .. change::
+        :tags: bug, orm
+        :tickets: 1768
+
+      Fixed the consideration of the ``between()`` operator
+      so that it works correctly with the new relationship local/remote
+      system.
+
     .. change::
         :tags: bug, sql
         :tickets: 2660, 1768

diff --git a/lib/sqlalchemy/sql/operators.py b/lib/sqlalchemy/sql/operators.py
index 10d01acd626c2741a583cb6aa0234fdddd46932c..a7e6af11611173801d5b2ec122bb7da471c496ac 100644 (file)
--- a/lib/sqlalchemy/sql/operators.py
+++ b/lib/sqlalchemy/sql/operators.py
@@ -758,7 +758,7 @@ def nullslast_op(a):
 
 _commutative = set([eq, ne, add, mul])
 
-_comparison = set([eq, ne, lt, gt, ge, le])
+_comparison = set([eq, ne, lt, gt, ge, le, between_op])
 
 
 def is_comparison(op):

diff --git a/test/orm/test_relationships.py b/test/orm/test_relationships.py
index 3ecd273aa69c0a20f078cfcbb323c569dfb3dc78..fb3f4f8240a13a4cfdd390a0c661f26792d81312 100644 (file)
--- a/test/orm/test_relationships.py
+++ b/test/orm/test_relationships.py
@@ -1952,6 +1952,73 @@ class ViewOnlyComplexJoin(_RelationshipErrors, fixtures.MappedTest):
         mapper(T3, t3)
         self._assert_raises_no_local_remote(configure_mappers, "T1.t3s")
 
+class RemoteForeignBetweenColsTest(fixtures.DeclarativeMappedTest):
+    """test a complex annotation using between().
+
+    Using declarative here as an integration test for the local()
+    and remote() annotations in conjunction with already annotated
+    instrumented attributes, etc.
+
+    """
+    @classmethod
+    def setup_classes(cls):
+        Base = cls.DeclarativeBasic
+
+        class Network(fixtures.ComparableEntity, Base):
+            __tablename__ = "network"
+
+            id = Column(sa.Integer, primary_key=True)
+            ip_net_addr = Column(Integer)
+            ip_broadcast_addr = Column(Integer)
+
+            addresses = relationship("Address",
+                    primaryjoin="remote(foreign(Address.ip_addr)).between("
+                                "Network.ip_net_addr,"
+                                "Network.ip_broadcast_addr)",
+                    viewonly=True
+        )
+
+        class Address(fixtures.ComparableEntity, Base):
+            __tablename__ = "address"
+
+            ip_addr = Column(Integer, primary_key=True)
+
+
+    @classmethod
+    def insert_data(cls):
+        Network, Address = cls.classes.Network, cls.classes.Address
+        s = Session(testing.db)
+
+        s.add_all([
+            Network(ip_net_addr=5, ip_broadcast_addr=10),
+            Network(ip_net_addr=15, ip_broadcast_addr=25),
+            Network(ip_net_addr=30, ip_broadcast_addr=35),
+            Address(ip_addr=17), Address(ip_addr=18), Address(ip_addr=9),
+            Address(ip_addr=27)
+        ])
+        s.commit()
+
+    def test_col_query(self):
+        Network, Address = self.classes.Network, self.classes.Address
+
+        session = Session(testing.db)
+        eq_(
+            session.query(Address.ip_addr).\
+                select_from(Network).\
+                join(Network.addresses).\
+                filter(Network.ip_net_addr == 15).\
+                all(),
+            [(17, ), (18, )]
+        )
+
+    def test_lazyload(self):
+        Network, Address = self.classes.Network, self.classes.Address
+
+        session = Session(testing.db)
+
+        n3 = session.query(Network).filter(Network.ip_net_addr == 5).one()
+        eq_([a.ip_addr for a in n3.addresses], [9])
+
 
 class ExplicitLocalRemoteTest(fixtures.MappedTest):