Conditions | 8 |
Total Lines | 52 |
Lines | 0 |
Ratio | 0 % |
Changes | 1 | ||
Bugs | 0 | Features | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
1 | from __future__ import unicode_literals |
||
40 | @register(Tags.database, Tags.compatibility) |
||
41 | def check_databases_compatibility(app_configs, **kwargs): |
||
42 | errors = [] |
||
43 | databases = settings.DATABASES |
||
44 | original_enabled_databases = getattr(settings, 'CACHALOT_DATABASES', |
||
45 | SUPPORTED_ONLY) |
||
46 | enabled_databases = cachalot_settings.CACHALOT_DATABASES |
||
47 | if original_enabled_databases == SUPPORTED_ONLY: |
||
48 | if not cachalot_settings.CACHALOT_DATABASES: |
||
49 | errors.append(Warning( |
||
50 | 'None of the configured databases are supported ' |
||
51 | 'by django-cachalot.', |
||
52 | hint='Use a supported database, or remove django-cachalot, or ' |
||
53 | 'put at least one database alias in `CACHALOT_DATABASES` ' |
||
54 | 'to force django-cachalot to use it.', |
||
55 | id='cachalot.W002' |
||
56 | )) |
||
57 | elif enabled_databases.__class__ in ITERABLES: |
||
58 | for db_alias in enabled_databases: |
||
59 | if db_alias in databases: |
||
60 | engine = databases[db_alias]['ENGINE'] |
||
61 | if engine not in SUPPORTED_DATABASE_ENGINES: |
||
62 | errors.append(Warning( |
||
63 | 'Database engine %r is not supported ' |
||
64 | 'by django-cachalot.' % engine, |
||
65 | hint='Switch to a supported database engine.', |
||
66 | id='cachalot.W003' |
||
67 | )) |
||
68 | else: |
||
69 | errors.append(Error( |
||
70 | 'Database alias %r from `CACHALOT_DATABASES` ' |
||
71 | 'is not defined in `DATABASES`.' % db_alias, |
||
72 | hint='Change `CACHALOT_DATABASES` to be compliant with' |
||
73 | '`CACHALOT_DATABASES`', |
||
74 | id='cachalot.E001', |
||
75 | )) |
||
76 | |||
77 | if not enabled_databases: |
||
78 | errors.append(Warning( |
||
79 | 'Django-cachalot is useless because no database ' |
||
80 | 'is configured in `CACHALOT_DATABASES`.', |
||
81 | hint='Reconfigure django-cachalot or remove it.', |
||
82 | id='cachalot.W004' |
||
83 | )) |
||
84 | else: |
||
85 | errors.append(Error( |
||
86 | "`CACHALOT_DATABASES` must be either %r or a list, tuple, " |
||
87 | "frozenset or set of database aliases." % SUPPORTED_ONLY, |
||
88 | hint='Remove `CACHALOT_DATABASES` or change it.', |
||
89 | id='cachalot.E002', |
||
90 | )) |
||
91 | return errors |
||
92 | |||
99 |