From: Mike Bayer <mike_mp@zzzcomputing.com>
Date: Tue, 21 Nov 2006 21:55:15 +0000 (+0000)
Subject: [ticket:374] [ticket:377] [ticket:375], small fix to mutable types unit test
X-Git-Tag: rel_0_3_2~46
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cf8e9d1db33630b413ac2ea3174ab05a32c3218c;p=thirdparty%2Fsqlalchemy%2Fsqlalchemy.git

[ticket:374] [ticket:377] [ticket:375], small fix to mutable types unit test
---

diff --git a/lib/sqlalchemy/engine/strategies.py b/lib/sqlalchemy/engine/strategies.py
index d48412160d..a7f2cc0036 100644
--- a/lib/sqlalchemy/engine/strategies.py
+++ b/lib/sqlalchemy/engine/strategies.py
@@ -67,7 +67,7 @@ class DefaultEngineStrategy(EngineStrategy):
             pool_args['use_threadlocal'] = self.pool_threadlocal()
             pool = poolclass(creator, **pool_args)
         else:
-            if isinstance(pool, poollib.DBProxy):
+            if isinstance(pool, poollib._DBProxy):
                 pool = pool.get_pool(*cargs, **cparams)
             else:
                 pool = pool
diff --git a/lib/sqlalchemy/orm/mapper.py b/lib/sqlalchemy/orm/mapper.py
index e07f691f41..5c1a3b5647 100644
--- a/lib/sqlalchemy/orm/mapper.py
+++ b/lib/sqlalchemy/orm/mapper.py
@@ -1149,8 +1149,8 @@ class Mapper(object):
             if populate_existing or context.session.is_expired(instance, unexpire=True):
                 if not context.identity_map.has_key(identitykey):
                     context.identity_map[identitykey] = instance
-		if self.extension.populate_instance(self, context, row, instance, identitykey, True) is EXT_PASS:
-		    self.populate_instance(context, instance, row, identitykey, True)
+                if self.extension.populate_instance(self, context, row, instance, identitykey, True) is EXT_PASS:
+                    self.populate_instance(context, instance, row, identitykey, True)
             if self.extension.append_result(self, context, row, instance, identitykey, result, isnew) is EXT_PASS:
                 if result is not None:
                     result.append(instance)
diff --git a/lib/sqlalchemy/schema.py b/lib/sqlalchemy/schema.py
index 68022f70b6..fb6894f0bc 100644
--- a/lib/sqlalchemy/schema.py
+++ b/lib/sqlalchemy/schema.py
@@ -389,7 +389,6 @@ class Column(SchemaItem, sql._ColumnClause):
                 return self.table.name + "." + self.name
             else:
                 return self.name
-            return self.name
         else:
             return self.name
     
diff --git a/test/orm/unitofwork.py b/test/orm/unitofwork.py
index 9f4e56dd09..e6a1060aa1 100644
--- a/test/orm/unitofwork.py
+++ b/test/orm/unitofwork.py
@@ -279,7 +279,7 @@ class MutableTypesTest(UnitOfWorkTest):
         self.assert_sql(db, lambda: ctx.current.flush(), [
             (
                 "UPDATE mutabletest SET value=:value WHERE mutabletest.id = :mutabletest_id",
-                {'mutabletest_id': 1, 'value': u'someothervalue'}
+                {'mutabletest_id': f1.id, 'value': u'someothervalue'}
             ),
         ])
         f1.value = unicode('hi')
@@ -287,7 +287,7 @@ class MutableTypesTest(UnitOfWorkTest):
         self.assert_sql(db, lambda: ctx.current.flush(), [
             (
                 "UPDATE mutabletest SET data=:data, value=:value WHERE mutabletest.id = :mutabletest_id",
-                {'mutabletest_id': 1, 'value': u'hi', 'data':f1.data}
+                {'mutabletest_id': f1.id, 'value': u'hi', 'data':f1.data}
             ),
         ])