summaryrefslogtreecommitdiff
path: root/lib/sqlalchemy/orm/loading.py
diff options
context:
space:
mode:
authorMike Bayer <mike_mp@zzzcomputing.com>2013-01-08 10:55:46 -0500
committerMike Bayer <mike_mp@zzzcomputing.com>2013-01-08 10:55:46 -0500
commit06ddce3215d771e3345e19c5d3a1857b23db69f1 (patch)
tree2445237779db7eaac4196213b8a2ce2d33db5145 /lib/sqlalchemy/orm/loading.py
parentac1ee45fcc749de4d75283f0b343be4b4c6c31ff (diff)
downloadsqlalchemy-06ddce3215d771e3345e19c5d3a1857b23db69f1.tar.gz
:meth:`.Query.merge_result` can now load rows from an outer join
where an entity may be ``None`` without throwing an error. [ticket:2640]
Diffstat (limited to 'lib/sqlalchemy/orm/loading.py')
-rw-r--r--lib/sqlalchemy/orm/loading.py9
1 files changed, 5 insertions, 4 deletions
diff --git a/lib/sqlalchemy/orm/loading.py b/lib/sqlalchemy/orm/loading.py
index baff076eb..5937197fd 100644
--- a/lib/sqlalchemy/orm/loading.py
+++ b/lib/sqlalchemy/orm/loading.py
@@ -129,10 +129,11 @@ def merge_result(query, iterator, load=True):
for row in iterator:
newrow = list(row)
for i in mapped_entities:
- newrow[i] = session._merge(
- attributes.instance_state(newrow[i]),
- attributes.instance_dict(newrow[i]),
- load=load, _recursive={})
+ if newrow[i] is not None:
+ newrow[i] = session._merge(
+ attributes.instance_state(newrow[i]),
+ attributes.instance_dict(newrow[i]),
+ load=load, _recursive={})
result.append(util.KeyedTuple(newrow, keys))
return iter(result)