summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMike Bayer <mike_mp@zzzcomputing.com>2005-12-22 03:25:43 +0000
committerMike Bayer <mike_mp@zzzcomputing.com>2005-12-22 03:25:43 +0000
commitd22f5edcae0f29c6542e800660bc4fef9b3f12cb (patch)
tree855b9c0b469bdff9d6e14d55da3af9d3af282d77
parent9ebac6151a743b5f841e715d04b551d8995e0e61 (diff)
downloadsqlalchemy-d22f5edcae0f29c6542e800660bc4fef9b3f12cb.tar.gz
deferred property, checks for NULL primary key components and returns None
-rw-r--r--lib/sqlalchemy/mapping/properties.py5
-rw-r--r--test/mapper.py3
2 files changed, 7 insertions, 1 deletions
diff --git a/lib/sqlalchemy/mapping/properties.py b/lib/sqlalchemy/mapping/properties.py
index f3862ddd9..922aab453 100644
--- a/lib/sqlalchemy/mapping/properties.py
+++ b/lib/sqlalchemy/mapping/properties.py
@@ -80,7 +80,10 @@ class DeferredColumnProperty(ColumnProperty):
def lazyload():
clause = sql.and_()
for primary_key in self.parent.pks_by_table[self.parent.primarytable]:
- clause.clauses.append(primary_key == self.parent._getattrbycolumn(instance, primary_key))
+ attr = self.parent._getattrbycolumn(instance, primary_key)
+ if not attr:
+ return None
+ clause.clauses.append(primary_key == attr)
return sql.select([self.parent.table.c[self.key]], clause).scalar()
return lazyload
diff --git a/test/mapper.py b/test/mapper.py
index 51751de6f..bcc6de098 100644
--- a/test/mapper.py
+++ b/test/mapper.py
@@ -184,6 +184,9 @@ class DeferredTest(MapperSuperTest):
'description':deferred(orders.c.description)
})
+ o = Order()
+ self.assert_(o.description is None)
+
def go():
l = m.select()
o2 = l[2]