mirror of
https://github.com/ansible/awx.git
synced 2026-05-07 17:37:37 -02:30
refactor code
This commit is contained in:
@@ -9,6 +9,11 @@ def copy_session_settings(apps, schema_editor):
|
|||||||
_rename_setting.rename_setting(apps, schema_editor, old_key='AUTH_TOKEN_EXPIRATION', new_key='SESSIONS_PER_USER')
|
_rename_setting.rename_setting(apps, schema_editor, old_key='AUTH_TOKEN_EXPIRATION', new_key='SESSIONS_PER_USER')
|
||||||
|
|
||||||
|
|
||||||
|
def reverse_copy_session_settings(apps, schema_editor):
|
||||||
|
_rename_setting.rename_setting(apps, schema_editor, old_key='SESSION_COOKIE_AGE', new_key='AUTH_TOKEN_PER_USER')
|
||||||
|
_rename_setting.rename_setting(apps, schema_editor, old_key='SESSIONS_PER_USER', new_key='SESSIONS_PER_USER''AUTH_TOKEN_EXPIRATION')
|
||||||
|
|
||||||
|
|
||||||
class Migration(migrations.Migration):
|
class Migration(migrations.Migration):
|
||||||
|
|
||||||
dependencies = [
|
dependencies = [
|
||||||
@@ -16,6 +21,6 @@ class Migration(migrations.Migration):
|
|||||||
]
|
]
|
||||||
|
|
||||||
operations = [
|
operations = [
|
||||||
migrations.RunPython(copy_session_settings),
|
migrations.RunPython(copy_session_settings, reverse_copy_session_settings),
|
||||||
]
|
]
|
||||||
|
|
||||||
@@ -1,6 +1,7 @@
|
|||||||
# -*- coding: utf-8 -*-
|
# -*- coding: utf-8 -*-
|
||||||
from __future__ import unicode_literals
|
from __future__ import unicode_literals
|
||||||
import logging
|
import logging
|
||||||
|
from django.utils.timezone import now
|
||||||
|
|
||||||
logger = logging.getLogger('awx.conf.settings')
|
logger = logging.getLogger('awx.conf.settings')
|
||||||
|
|
||||||
@@ -12,26 +13,16 @@ def rename_setting(apps, schema_editor, old_key, new_key):
|
|||||||
Setting = apps.get_model('conf', 'Setting')
|
Setting = apps.get_model('conf', 'Setting')
|
||||||
|
|
||||||
if Setting.objects.filter(key=new_key).exists():
|
if Setting.objects.filter(key=new_key).exists():
|
||||||
logger.error('Setting', new_key, 'unexpectedly exists before this migration, \
|
logger.info('Setting ' + new_key + ' unexpectedly exists before this migration, it will be replaced by the value of the ' + old_key + ' setting.')
|
||||||
it will be replaced by the value of the', old_key, 'setting.')
|
|
||||||
Setting.objects.filter(key=new_key).delete()
|
Setting.objects.filter(key=new_key).delete()
|
||||||
|
|
||||||
old_setting = Setting.objects.filter(key=old_key).first()
|
old_setting = Setting.objects.filter(key=old_key).first()
|
||||||
if old_setting is not None:
|
if old_setting is not None:
|
||||||
Setting.objects.create(key=new_key, value=old_setting.value)
|
Setting.objects.create(key=new_key,
|
||||||
|
value=old_setting.value,
|
||||||
|
created=now(),
|
||||||
|
modified=now()
|
||||||
|
)
|
||||||
Setting.objects.filter(key=old_key).delete()
|
Setting.objects.filter(key=old_key).delete()
|
||||||
|
|
||||||
def reverse_rename_setting(apps, schema_editor, old_key, new_key):
|
|
||||||
|
|
||||||
Setting = apps.get_model('conf', 'Setting')
|
|
||||||
|
|
||||||
if Setting.objects.filter(key=old_key).exists():
|
|
||||||
logger.error('Setting', old_key, 'unexpectedly exists before this migration, \
|
|
||||||
it will be replaced by the value of the', new_key, 'setting.')
|
|
||||||
Setting.objects.filter(key=old_key).delete()
|
|
||||||
|
|
||||||
new_setting = Setting.objects.filter(key=new_key).first()
|
|
||||||
if new_setting is not None:
|
|
||||||
Setting.objects.create(key=old_key, value=new_setting.value)
|
|
||||||
Setting.objects.filter(key=new_key).delete()
|
|
||||||
|
|
||||||
Reference in New Issue
Block a user