diff options
author | Mike Bayer <mike_mp@zzzcomputing.com> | 2010-11-20 12:55:46 -0500 |
---|---|---|
committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2010-11-20 12:55:46 -0500 |
commit | c5442b3adb56cae15607aad32b4219ac11ae809e (patch) | |
tree | 0502f97a14672dd089adab6184681cb7a0f3bfea /lib/sqlalchemy/ext/declarative.py | |
parent | d505ea71aed44ecae718052131dc0a2fb2c9dd99 (diff) | |
parent | 6dbf2c3314a797a39624f1e68569bfbbb2b6ac87 (diff) | |
download | sqlalchemy-c5442b3adb56cae15607aad32b4219ac11ae809e.tar.gz |
- merge hybrid attributes branch, [ticket:1903]
Diffstat (limited to 'lib/sqlalchemy/ext/declarative.py')
-rwxr-xr-x | lib/sqlalchemy/ext/declarative.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/lib/sqlalchemy/ext/declarative.py b/lib/sqlalchemy/ext/declarative.py index 3c6cab59a..8381e5ee1 100755 --- a/lib/sqlalchemy/ext/declarative.py +++ b/lib/sqlalchemy/ext/declarative.py @@ -1265,12 +1265,13 @@ class _GetColumns(object): mapper = class_mapper(self.cls, compile=False) if mapper: - prop = mapper.get_property(key, raiseerr=False) - if prop is None: + if not mapper.has_property(key): raise exceptions.InvalidRequestError( "Class %r does not have a mapped column named %r" % (self.cls, key)) - elif not isinstance(prop, ColumnProperty): + + prop = mapper.get_property(key) + if not isinstance(prop, ColumnProperty): raise exceptions.InvalidRequestError( "Property %r is not an instance of" " ColumnProperty (i.e. does not correspond" |