From: Jonathan LaCour <jonathan@cleverdevil.org>
Date: Fri, 30 Jun 2006 01:19:52 +0000 (+0000)
Subject: Put back in the foreign-key checking code in process_relationships for
X-Git-Tag: rel_0_2_5~10
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=a8c2b7b97902fa9dd69a11f696ad2d4e0bf6328b;p=thirdparty%2Fsqlalchemy%2Fsqlalchemy.git

Put back in the foreign-key checking code in process_relationships for
ActiveMapper.  It looks like it is required by at least one person, so for
now the code will stay in!
---

diff --git a/lib/sqlalchemy/ext/activemapper.py b/lib/sqlalchemy/ext/activemapper.py
index 132c93e75f..541bf78127 100644
--- a/lib/sqlalchemy/ext/activemapper.py
+++ b/lib/sqlalchemy/ext/activemapper.py
@@ -110,11 +110,6 @@ def process_relationships(klass, was_deferred=False):
     # and make sure that we can find the related tables (they do not 
     # have to be processed yet, just defined), and we defer if we are 
     # not able to find any of the related tables
-    
-    # thanks to deferred mapper compilation, this loop should no longer 
-    # be necessary -- however, I will leave it here commented out until
-    # I get the feeling that its not causing problems for people.
-    '''
     for col in klass.columns:
         if col.foreign_key is not None:
             found = False
@@ -127,7 +122,6 @@ def process_relationships(klass, was_deferred=False):
                 if not was_deferred: __deferred_classes__.add(klass)
                 defer = True
                 break
-    '''
     
     # if we are able to find all related and referred to tables, then
     # we can go ahead and assign the relationships to the class