Renamed registered_model to has_model.
This avoids possible confusion with register_model. Thanks Marc Tamlyn for the suggestion.
This commit is contained in:
parent
65cd74be8e
commit
70c9654d45
|
@ -298,12 +298,12 @@ class AppCache(object):
|
||||||
models[model_name] = model
|
models[model_name] = model
|
||||||
self._get_models_cache.clear()
|
self._get_models_cache.clear()
|
||||||
|
|
||||||
def registered_model(self, app_label, model_name):
|
def has_model(self, app_label, model_name):
|
||||||
"""
|
"""
|
||||||
Test if a model is registered and return the model class or None.
|
Returns the model class if one is registered and None otherwise.
|
||||||
|
|
||||||
It's safe to call this method at import time, even while the app cache
|
It's safe to call this method at import time, even while the app cache
|
||||||
is being populated.
|
is being populated. It returns None for models that aren't loaded yet.
|
||||||
"""
|
"""
|
||||||
return self.all_models[app_label].get(model_name.lower())
|
return self.all_models[app_label].get(model_name.lower())
|
||||||
|
|
||||||
|
|
|
@ -151,7 +151,7 @@ class ModelBase(type):
|
||||||
new_class._base_manager = new_class._base_manager._copy_to_model(new_class)
|
new_class._base_manager = new_class._base_manager._copy_to_model(new_class)
|
||||||
|
|
||||||
# Bail out early if we have already created this class.
|
# Bail out early if we have already created this class.
|
||||||
m = new_class._meta.app_cache.registered_model(new_class._meta.app_label, name)
|
m = new_class._meta.app_cache.has_model(new_class._meta.app_label, name)
|
||||||
if m is not None:
|
if m is not None:
|
||||||
return m
|
return m
|
||||||
|
|
||||||
|
@ -278,7 +278,7 @@ class ModelBase(type):
|
||||||
# the first time this model tries to register with the framework. There
|
# the first time this model tries to register with the framework. There
|
||||||
# should only be one class for each model, so we always return the
|
# should only be one class for each model, so we always return the
|
||||||
# registered version.
|
# registered version.
|
||||||
return new_class._meta.app_cache.registered_model(new_class._meta.app_label, name)
|
return new_class._meta.app_cache.has_model(new_class._meta.app_label, name)
|
||||||
|
|
||||||
def copy_managers(cls, base_managers):
|
def copy_managers(cls, base_managers):
|
||||||
# This is in-place sorting of an Options attribute, but that's fine.
|
# This is in-place sorting of an Options attribute, but that's fine.
|
||||||
|
|
|
@ -68,7 +68,7 @@ def add_lazy_relation(cls, field, relation, operation):
|
||||||
# string right away. If get_model returns None, it means that the related
|
# string right away. If get_model returns None, it means that the related
|
||||||
# model isn't loaded yet, so we need to pend the relation until the class
|
# model isn't loaded yet, so we need to pend the relation until the class
|
||||||
# is prepared.
|
# is prepared.
|
||||||
model = cls._meta.app_cache.registered_model(app_label, model_name)
|
model = cls._meta.app_cache.has_model(app_label, model_name)
|
||||||
if model:
|
if model:
|
||||||
operation(field, model, cls)
|
operation(field, model, cls)
|
||||||
else:
|
else:
|
||||||
|
|
Loading…
Reference in New Issue