Rename dependencies to dependent_objects

This commit is contained in:
Konsta Vesterinen
2014-05-07 14:55:07 +03:00
parent 2193d27d7b
commit 134f112697
5 changed files with 21 additions and 20 deletions

View File

@@ -4,10 +4,10 @@ Model helpers
.. module:: sqlalchemy_utils.functions .. module:: sqlalchemy_utils.functions
dependencies dependent_objects
^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^
.. autofunction:: dependencies .. autofunction:: dependent_objects
escape_like escape_like

View File

@@ -8,7 +8,7 @@ from .functions import (
create_mock_engine, create_mock_engine,
database_exists, database_exists,
defer_except, defer_except,
dependencies, dependent_objects,
drop_database, drop_database,
escape_like, escape_like,
get_columns, get_columns,
@@ -75,7 +75,7 @@ __all__ = (
create_mock_engine, create_mock_engine,
database_exists, database_exists,
defer_except, defer_except,
dependencies, dependent_objects,
drop_database, drop_database,
escape_like, escape_like,
force_auto_coercion, force_auto_coercion,

View File

@@ -12,7 +12,7 @@ from .database import (
non_indexed_foreign_keys, non_indexed_foreign_keys,
) )
from .orm import ( from .orm import (
dependencies, dependent_objects,
get_columns, get_columns,
get_declarative_base, get_declarative_base,
get_primary_keys, get_primary_keys,
@@ -31,6 +31,7 @@ __all__ = (
'create_mock_engine', 'create_mock_engine',
'database_exists', 'database_exists',
'defer_except', 'defer_except',
'dependent_objects',
'drop_database', 'drop_database',
'escape_like', 'escape_like',
'get_columns', 'get_columns',

View File

@@ -17,7 +17,7 @@ from sqlalchemy.orm.util import AliasedInsp
from ..query_chain import QueryChain from ..query_chain import QueryChain
def dependencies(obj, foreign_keys=None): def dependent_objects(obj, foreign_keys=None):
""" """
Return a QueryChain that iterates through all dependent objects for given Return a QueryChain that iterates through all dependent objects for given
SQLAlchemy object. SQLAlchemy object.
@@ -27,17 +27,17 @@ def dependencies(obj, foreign_keys=None):
:: ::
from sqlalchemy_utils import dependencies from sqlalchemy_utils import dependent_objects
dependencies(user) dependent_objects(user)
If you expect an object to have lots of dependencies it might be good to If you expect an object to have lots of dependent_objects it might be good to
limit the results:: limit the results::
dependencies(user).limit(5) dependent_objects(user).limit(5)
The common use case is checking for all dependent objects before deleting The common use case is checking for all dependent objects before deleting
@@ -46,7 +46,7 @@ def dependencies(obj, foreign_keys=None):
it will lead to nasty IntegrityErrors being raised. it will lead to nasty IntegrityErrors being raised.
In the following example we delete given user if it doesn't have any In the following example we delete given user if it doesn't have any
foreign key restricted dependencies. foreign key restricted dependent_objects.
:: ::
@@ -58,7 +58,7 @@ def dependencies(obj, foreign_keys=None):
deps = list( deps = list(
dependencies( dependent_objects(
user, user,
( (
fk for fk in get_referencing_foreign_keys(User) fk for fk in get_referencing_foreign_keys(User)
@@ -78,7 +78,7 @@ def dependencies(obj, foreign_keys=None):
:param obj: SQLAlchemy declarative model object :param obj: SQLAlchemy declarative model object
:param foreign_keys: :param foreign_keys:
A sequence of foreign keys to use for searching the dependencies for A sequence of foreign keys to use for searching the dependent_objects for
given object. By default this is None, indicating that all foreign keys given object. By default this is None, indicating that all foreign keys
referencing the object will be used. referencing the object will be used.

View File

@@ -1,9 +1,9 @@
import sqlalchemy as sa import sqlalchemy as sa
from sqlalchemy_utils import dependencies, get_referencing_foreign_keys from sqlalchemy_utils import dependent_objects, get_referencing_foreign_keys
from tests import TestCase from tests import TestCase
class TestDependencies(TestCase): class TestDependentObjects(TestCase):
def create_models(self): def create_models(self):
class User(self.Base): class User(self.Base):
__tablename__ = 'user' __tablename__ = 'user'
@@ -46,7 +46,7 @@ class TestDependencies(TestCase):
self.session.add_all(articles) self.session.add_all(articles)
self.session.commit() self.session.commit()
deps = list(dependencies(user)) deps = list(dependent_objects(user))
assert len(deps) == 3 assert len(deps) == 3
assert articles[0] in deps assert articles[0] in deps
assert articles[2] in deps assert articles[2] in deps
@@ -65,7 +65,7 @@ class TestDependencies(TestCase):
self.session.commit() self.session.commit()
deps = list( deps = list(
dependencies( dependent_objects(
user, user,
( (
fk for fk in get_referencing_foreign_keys(self.User) fk for fk in get_referencing_foreign_keys(self.User)
@@ -78,7 +78,7 @@ class TestDependencies(TestCase):
assert objects[3] in deps assert objects[3] in deps
class TestDependenciesWithCompositeKeys(TestCase): class TestDependentObjectsWithCompositeKeys(TestCase):
def create_models(self): def create_models(self):
class User(self.Base): class User(self.Base):
__tablename__ = 'user' __tablename__ = 'user'
@@ -113,7 +113,7 @@ class TestDependenciesWithCompositeKeys(TestCase):
self.session.add_all(articles) self.session.add_all(articles)
self.session.commit() self.session.commit()
deps = list(dependencies(user)) deps = list(dependent_objects(user))
assert len(deps) == 2 assert len(deps) == 2
assert articles[0] in deps assert articles[0] in deps
assert articles[3] in deps assert articles[3] in deps