added unittest for orm-persisted insert without a postfetch, tweak to engine to only signal postfetch if the passivedefault columns received None/NULL for their parameter (since they dont exec otherwise)

1 files changed, 2 insertions, 2 deletions

diff --git a/lib/sqlalchemy/engine.py b/lib/sqlalchemy/engine.py
index 5f681d39e..518ea8a15 100644
--- a/lib/sqlalchemy/engine.py
+++ b/lib/sqlalchemy/engine.py
@@ -468,9 +468,9 @@ class SQLEngine(schema.SchemaEngine):
                 last_inserted_ids = []
                 need_lastrowid=False
                 for c in compiled.statement.table.c:
-                    if isinstance(c.default, schema.PassiveDefault):
-                        self.context.lastrow_has_defaults = True
                     if not param.has_key(c.name) or param[c.name] is None:
+                        if isinstance(c.default, schema.PassiveDefault):
+                            self.context.lastrow_has_defaults = True
                         newid = drunner.get_column_default(c)
                         if newid is not None:
                             param[c.name] = newid