From d18221c542bf4d7381be2afdcd4c098698ff8eae Mon Sep 17 00:00:00 2001 From: Pierre Jaury Date: Sun, 16 Oct 2016 16:45:54 +0200 Subject: [PATCH] Remove some legacy dead code from the declared_attr logic The second 'if' condition was never called because the original condition always returns. --- lib/sqlalchemy/ext/declarative/api.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/lib/sqlalchemy/ext/declarative/api.py b/lib/sqlalchemy/ext/declarative/api.py index d02db7aa53..c3be636bb0 100644 --- a/lib/sqlalchemy/ext/declarative/api.py +++ b/lib/sqlalchemy/ext/declarative/api.py @@ -181,9 +181,6 @@ class declared_attr(interfaces._MappedAttribute, property): "non-mapped class %s" % (desc.fget.__name__, cls.__name__)) return desc.fget(cls) - - if reg is None: - return desc.fget(cls) elif desc in reg: return reg[desc] else: -- 2.47.3