mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2025-07-04 17:59:13 +00:00
Code Issues 32 Releases Wiki Activity

[1.2.X] Fixed #14870 -- don't catch AttributeErrors in database router methods, instead check that the method itself exists. Thanks to jonash for the patch. Backport of [14857].

Browse Source 
git-svn-id: http://code.djangoproject.com/svn/django/branches/releases/1.2.X@14858 bcc190cf-cafb-0310-a4f2-bffc1f526a37
This commit is contained in:
Alex Gaynor 2010-12-08 18:32:20 +00:00
parent d0056051a5
commit 0f7f6a8adf
1 changed files with 17 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
django/db/utils.py
View File

@ -5,6 +5,7 @@ from django.conf import settings
from django.core.exceptions import ImproperlyConfigured
from django.utils.importlib import import_module
DEFAULT_DB_ALIAS = 'default'
# Define some exceptions that mirror the PEP249 interface.
@ -125,12 +126,15 @@ class ConnectionRouter(object):
chosen_db = None
for router in self.routers:
try:
chosen_db = getattr(router, action)(model, **hints)
if chosen_db:
return chosen_db
method = getattr(router, action)
except AttributeError:
# If the router doesn't have a method, skip to the next one.
pass
else:
chosen_db = method(model, **hints
)
if chosen_db:
return chosen_db
try:
return hints['instance']._state.db or DEFAULT_DB_ALIAS
except KeyError:
@ -143,21 +147,25 @@ class ConnectionRouter(object):
def allow_relation(self, obj1, obj2, **hints):
for router in self.routers:
try:
allow = router.allow_relation(obj1, obj2, **hints)
if allow is not None:
return allow
method = router.allow_relation
except AttributeError:
# If the router doesn't have a method, skip to the next one.
pass
else:
allow = method(obj1, obj2, **hints)
if allow is not None:
return allow
return obj1._state.db == obj2._state.db
def allow_syncdb(self, db, model):
for router in self.routers:
try:
allow = router.allow_syncdb(db, model)
if allow is not None:
return allow
method = router.allow_syncdb
except AttributeError:
# If the router doesn't have a method, skip to the next one.
pass
else:
allow = method(db, model)
if allow is not None:
return allow
return True