Anonymous wrote:

submitted by kobipe3@gmail.com

Michael Bayer (@zzzeek) wrote:

it is possible using a comparable_property mapping to your descriptor:

http://www.sqlalchemy.org/docs/05/reference/orm/mapping.html#sqlalchemy.orm.comparable_property

build a Comparator and implement the comparison methods you'd like and they will be assembled on your associationproxy.

im wondering how hard it would be to build this into associationproxy directly.

Changes by Michael Bayer (@zzzeek):

• removed labels: bug
• added labels: feature, orm
• set milestone to "0.6.xx"

Changes by Michael Bayer (@zzzeek):

• changed title from "Make it possible to query on association_proxy" to "supply a pre-built comparator on associationproxy "

Anonymous wrote:

I wanted to use the has and any functions on associationproxies, and didn't know about about comparable_property, so I added this to AssociationProxy:

import sqlalchemy.exceptions as sa_exc

#snip

    def any(self, criterion=None, **kwargs):
        if self._target_is_scalar():
            raise sa_exc.InvalidRequestError("'any()' not implemented for scalar attributes. Use has().")
        return self._get_property().comparator.any(getattr(self.target_class, self.value_attr).has(criterion, **kwargs))
    
    def has(self, criterion=None, **kwargs):
        if not self._target_is_scalar():
            raise sa_exc.InvalidRequestError("'has()' not implemented for collections.  Use any().")
        return self._get_property().comparator.has(getattr(self.target_class, self.value_attr).has(criterion, **kwargs))

I'll try using a comparable_property instead now, but if it's possible to implement a general solution for AssociationProxy, I'd be willing to try to figure out the other Comparator functions.

Anonymous wrote:

Here's a patch against 26edef4 that adds proxying of .any(), .has(), .contains(), ==, and != to association_proxies. Lots of tests are included.

Any thoughts on whether or not this implementation is appropriate?

Also, please add storborg@mit.edu to the cc list on this ticket. Thanks!

Anonymous wrote:

(original author: ged)

Michael Bayer (@zzzeek) wrote:

hey this is a great patch. We just have to work the unittest to use MappedTest, Comparable and eq_ (at the very least speeds up the test to only setup/teardown once for the whole suite) and its good to go.

Changes by Michael Bayer (@zzzeek):

• changed milestone from "0.6.xx" to "0.6.0"

Michael Bayer (@zzzeek) wrote:

Scott, you are the man. this is in 2d15d9b thanks !

Changes by Michael Bayer (@zzzeek):

• changed status to closed