Rename values in SiteConfiguration/History (3/3)

common/djangoapps/student/admin.py

@@ -460,7 +460,7 @@ def clean_email(self):
         if not allowed_site_email_domain:
             raise forms.ValidationError(
                 _("Please add a key/value 'THIRD_PARTY_AUTH_ONLY_DOMAIN/{site_email_domain}' in SiteConfiguration "
-                  "model's values field.")
+                  "model's site_values field.")
             )
         elif email_domain != allowed_site_email_domain:
             raise forms.ValidationError(

common/djangoapps/student/tests/test_admin_views.py

@@ -454,7 +454,7 @@ def setUpClass(cls):
 
     def _update_site_configuration(self):
         """ Updates the site's configuration """
-        self.site.configuration.values = {'THIRD_PARTY_AUTH_ONLY_DOMAIN': self.email_domain_name}
+        self.site.configuration.site_values = {'THIRD_PARTY_AUTH_ONLY_DOMAIN': self.email_domain_name}
         self.site.configuration.save()
 
     def _assert_form(self, site, email, is_valid_form=False):
@@ -480,7 +480,7 @@ def test_form_with_invalid_site_configuration(self):
         self.assertEqual(
             error,
             "Please add a key/value 'THIRD_PARTY_AUTH_ONLY_DOMAIN/{site_email_domain}' in SiteConfiguration "
-            "model's values field."
+            "model's site_values field."
         )
 
     def test_form_with_invalid_domain_name(self):

common/djangoapps/util/tests/test_db.py

@@ -221,7 +221,6 @@ class MigrationTests(TestCase):
     """
     Tests for migrations.
     """
-
     @override_settings(MIGRATION_MODULES={})
     def test_migrations_are_in_sync(self):
         """

lms/djangoapps/branding/tests/test_views.py

@@ -297,7 +297,7 @@ def test_index_redirects_to_marketing_site_with_site_override(self):
         response = self.client.get(reverse("root"))
         self.assertRedirects(
             response,
-            self.site_configuration_other.values["MKTG_URLS"]["ROOT"],
+            self.site_configuration_other.site_values["MKTG_URLS"]["ROOT"],
             fetch_redirect_response=False
         )
 
@@ -309,4 +309,4 @@ def test_header_logo_links_to_marketing_site_with_site_override(self):
         self.use_site(self.site_other)
         self.client.login(username=self.user.username, password="password")
         response = self.client.get(reverse("dashboard"))
-        self.assertIn(self.site_configuration_other.values["MKTG_URLS"]["ROOT"], response.content.decode('utf-8'))
+        self.assertIn(self.site_configuration_other.site_values["MKTG_URLS"]["ROOT"], response.content.decode('utf-8'))

lms/djangoapps/discussion/tests/test_signals.py

@@ -28,7 +28,8 @@ def setUp(self):
     @mock.patch('lms.djangoapps.discussion.signals.handlers.send_message')
     def test_comment_created_signal_sends_message(self, mock_send_message, mock_get_current_site):
         site_config = SiteConfigurationFactory.create(site=self.site)
-        site_config.values[ENABLE_FORUM_NOTIFICATIONS_FOR_SITE_KEY] = True
+        enable_notifications_cfg = {ENABLE_FORUM_NOTIFICATIONS_FOR_SITE_KEY: True}
+        site_config.site_values = enable_notifications_cfg
         site_config.save()
         mock_get_current_site.return_value = self.site
         signals.comment_created.send(sender=self.sender, user=self.user, post=self.post)
@@ -56,7 +57,8 @@ def test_comment_created_signal_msg_not_sent_with_site_config_disabled(
             self, mock_send_message, mock_get_current_site
     ):
         site_config = SiteConfigurationFactory.create(site=self.site)
-        site_config.values[ENABLE_FORUM_NOTIFICATIONS_FOR_SITE_KEY] = False
+        enable_notifications_cfg = {ENABLE_FORUM_NOTIFICATIONS_FOR_SITE_KEY: False}
+        site_config.site_values = enable_notifications_cfg
         site_config.save()
         mock_get_current_site.return_value = self.site
         signals.comment_created.send(sender=self.sender, user=self.user, post=self.post)

lms/djangoapps/discussion/tests/test_tasks.py

@@ -193,7 +193,7 @@ def test_send_discussion_email_notification(self, user_subscribed):
         comment = cc.Comment.find(id=self.comment['id']).retrieve()
         site = Site.objects.get_current()
         site_config = SiteConfigurationFactory.create(site=site)
-        site_config.values[ENABLE_FORUM_NOTIFICATIONS_FOR_SITE_KEY] = True
+        site_config.site_values[ENABLE_FORUM_NOTIFICATIONS_FOR_SITE_KEY] = True
         site_config.save()
         with mock.patch('lms.djangoapps.discussion.signals.handlers.get_current_site', return_value=site):
             comment_created.send(sender=None, user=user, post=comment)

lms/djangoapps/instructor/tests/test_api.py

@@ -3990,8 +3990,8 @@ def test_send_email_no_message(self):
         self.assertEqual(response.status_code, 400)
 
     def test_send_email_with_site_template_and_from_addr(self):
-        site_email = self.site_configuration.values.get('course_email_from_addr')
-        site_template = self.site_configuration.values.get('course_email_template_name')
+        site_email = self.site_configuration.site_values.get('course_email_from_addr')
+        site_template = self.site_configuration.site_values.get('course_email_template_name')
         CourseEmailTemplate.objects.create(name=site_template)
         url = reverse('send_email', kwargs={'course_id': text_type(self.course.id)})
         response = self.client.post(url, self.full_test_message)
@@ -4009,7 +4009,7 @@ def test_send_email_with_org_template_and_from_addr(self):
         org_email = 'fake_org@example.com'
         org_template = 'fake_org_email_template'
         CourseEmailTemplate.objects.create(name=org_template)
-        self.site_configuration.values.update({
+        self.site_configuration.site_values.update({
             'course_email_from_addr': {self.course.id.org: org_email},
             'course_email_template_name': {self.course.id.org: org_template}
         })

lms/djangoapps/instructor/tests/views/test_instructor_dashboard.py

@@ -168,7 +168,11 @@ def test_membership_reason_field_visibility(self, enbale_reason_field):
             "ENABLE_MANUAL_ENROLLMENT_REASON_FIELD": enbale_reason_field
         }
         site = Site.objects.first()
-        SiteConfiguration.objects.create(site=site, values=configuration_values, enabled=True)
+        SiteConfiguration.objects.create(
+            site=site,
+            site_values=configuration_values,
+            enabled=True
+        )
 
         url = reverse(
             'instructor_dashboard',
@@ -196,7 +200,11 @@ def test_membership_site_configuration_role(self):
             ]
         }
         site = Site.objects.first()
-        SiteConfiguration.objects.create(site=site, values=configuration_values, enabled=True)
+        SiteConfiguration.objects.create(
+            site=site,
+            site_values=configuration_values,
+            enabled=True
+        )
         url = reverse(
             'instructor_dashboard',
             kwargs={

openedx/core/djangoapps/ace_common/tests/test_tracking.py

@@ -122,7 +122,7 @@ def test_missing_settings(self):
     @override_settings(GOOGLE_ANALYTICS_TRACKING_ID='UA-123456-1')
     def test_site_config_override(self):
         site_config = SiteConfigurationFactory.create(
-            values=dict(
+            site_values=dict(
                 GOOGLE_ANALYTICS_ACCOUNT='UA-654321-1'
             )
         )

openedx/core/djangoapps/catalog/views.py

@@ -27,7 +27,7 @@ def cache_programs(request):
         SiteConfiguration.objects.create(
             site=request.site,
             enabled=True,
-            values={"COURSE_CATALOG_API_URL": "{catalog_url}/api/v1/".format(catalog_url=CATALOG_STUB_URL)}
+            site_values={"COURSE_CATALOG_API_URL": "{catalog_url}/api/v1/".format(catalog_url=CATALOG_STUB_URL)}
         )
 
     if settings.FEATURES.get('EXPOSE_CACHE_PROGRAMS_ENDPOINT'):

openedx/core/djangoapps/config_model_utils/models.py

@@ -244,7 +244,7 @@ def all_current_course_configs(cls):
         """
         all_courses = CourseOverview.objects.all()
         all_site_configs = SiteConfiguration.objects.filter(
-            values__contains='course_org_filter', enabled=True
+            site_values__contains='course_org_filter', enabled=True
         ).select_related('site')
 
         try:
@@ -254,7 +254,7 @@ def all_current_course_configs(cls):
 
         sites_by_org = defaultdict(lambda: default_site)
         site_cfg_org_filters = (
-            (site_cfg.site, site_cfg.values['course_org_filter'])
+            (site_cfg.site, site_cfg.site_values['course_org_filter'])
             for site_cfg in all_site_configs
         )
         sites_by_org.update({

openedx/core/djangoapps/credentials/management/commands/tests/test_notify_credentials.py

@@ -127,7 +127,7 @@ def test_page_size(self):
     @mock.patch(COMMAND_MODULE + '.handle_cert_change')
     def test_site(self, mock_grade_interesting, mock_cert_change):
         site_config = SiteConfigurationFactory.create(
-            values={'course_org_filter': ['testX']},
+            site_values={'course_org_filter': ['testX']}
         )
 
         call_command(Command(), '--site', site_config.site.domain, '--start-date', '2017-01-01')

openedx/core/djangoapps/credentials/tests/test_signals.py

@@ -110,7 +110,7 @@ def test_send_grade_without_issuance_enabled(self, _mock_is_course_run_in_a_prog
     def test_send_grade_records_enabled(self, _mock_is_course_run_in_a_program, mock_send_grade_to_credentials,
                                         _mock_is_learner_issuance_enabled):
         site_config = SiteConfigurationFactory.create(
-            values={'course_org_filter': [self.key.org]},
+            site_values={'course_org_filter': [self.key.org]}
         )
 
         # Correctly sent
@@ -119,7 +119,7 @@ def test_send_grade_records_enabled(self, _mock_is_course_run_in_a_program, mock
         mock_send_grade_to_credentials.delay.reset_mock()
 
         # Correctly not sent
-        site_config.values['ENABLE_LEARNER_RECORDS'] = False
+        site_config.site_values['ENABLE_LEARNER_RECORDS'] = False
         site_config.save()
         send_grade_if_interesting(self.user, self.key, 'verified', 'downloadable', None, None)
         self.assertFalse(mock_send_grade_to_credentials.delay.called)

openedx/core/djangoapps/programs/tasks/v1/tests/test_tasks.py

@@ -196,7 +196,7 @@ def test_awarding_certs_with_skip_program_certificate(
         mock_get_certified_programs.return_value = [1]
 
         # programs to be skipped
-        self.site_configuration.values = {
+        self.site_configuration.site_values = {
             "programs_without_certificates": [2]
         }
         self.site_configuration.save()

openedx/core/djangoapps/programs/tests/test_signals.py

@@ -146,7 +146,7 @@ def test_records_enabled(self, mock_is_learner_issuance_enabled, mock_task):
         mock_is_learner_issuance_enabled.return_value = True
 
         site_config = SiteConfigurationFactory.create(
-            values={'course_org_filter': ['edX']},
+            site_values={'course_org_filter': ['edX']}
         )
 
         # Correctly sent
@@ -155,7 +155,7 @@ def test_records_enabled(self, mock_is_learner_issuance_enabled, mock_task):
         mock_task.reset_mock()
 
         # Correctly not sent
-        site_config.values['ENABLE_LEARNER_RECORDS'] = False
+        site_config.site_values['ENABLE_LEARNER_RECORDS'] = False
         site_config.save()
         handle_course_cert_changed(**self.signal_kwargs)
         self.assertFalse(mock_task.called)

openedx/core/djangoapps/schedules/management/commands/tests/send_email_base.py

@@ -306,8 +306,12 @@ def test_enqueue_config(self, is_enabled):
     def test_site_config(self, this_org_list, other_org_list, expected_message_count, mock_ace):
         filtered_org = 'filtered_org'
         unfiltered_org = 'unfiltered_org'
-        this_config = SiteConfigurationFactory.create(values={'course_org_filter': this_org_list})
-        other_config = SiteConfigurationFactory.create(values={'course_org_filter': other_org_list})
+        this_config = SiteConfigurationFactory.create(
+            site_values={'course_org_filter': this_org_list}
+        )
+        other_config = SiteConfigurationFactory.create(
+            site_values={'course_org_filter': other_org_list}
+        )
 
         for config in (this_config, other_config):
             ScheduleConfigFactory.create(site=config.site)

openedx/core/djangoapps/schedules/tests/test_resolvers.py

@@ -61,7 +61,7 @@ def setUp(self):
         'course1'
     )
     def test_get_course_org_filter_equal(self, course_org_filter):
-        self.site_config.values['course_org_filter'] = course_org_filter
+        self.site_config.site_values['course_org_filter'] = course_org_filter
         self.site_config.save()
         mock_query = Mock()
         result = self.resolver.filter_by_org(mock_query)
@@ -73,7 +73,7 @@ def test_get_course_org_filter_equal(self, course_org_filter):
         (['course1', 'course2'], ['course1', 'course2'])
     )
     def test_get_course_org_filter_include__in(self, course_org_filter, expected_org_list):
-        self.site_config.values['course_org_filter'] = course_org_filter
+        self.site_config.site_values['course_org_filter'] = course_org_filter
         self.site_config.save()
         mock_query = Mock()
         result = self.resolver.filter_by_org(mock_query)
@@ -88,7 +88,7 @@ def test_get_course_org_filter_include__in(self, course_org_filter, expected_org
     )
     def test_get_course_org_filter_exclude__in(self, course_org_filter, expected_org_list):
         SiteConfigurationFactory.create(
-            values={'course_org_filter': course_org_filter},
+            site_values={'course_org_filter': course_org_filter}
         )
         mock_query = Mock()
         result = self.resolver.filter_by_org(mock_query)

openedx/core/djangoapps/site_configuration/admin.py

@@ -12,8 +12,8 @@ class SiteConfigurationAdmin(admin.ModelAdmin):
     """
     Admin interface for the SiteConfiguration object.
     """
-    list_display = ('site', 'enabled', 'values')
-    search_fields = ('site__domain', 'values')
+    list_display = ('site', 'enabled', 'site_values')
+    search_fields = ('site__domain', 'site_values')
 
     class Meta(object):
         """
@@ -29,7 +29,7 @@ class SiteConfigurationHistoryAdmin(admin.ModelAdmin):
     Admin interface for the SiteConfigurationHistory object.
     """
     list_display = ('site', 'enabled', 'created', 'modified')
-    search_fields = ('site__domain', 'values', 'created', 'modified')
+    search_fields = ('site__domain', 'site_values', 'created', 'modified')
 
     ordering = ['-created']
 

openedx/core/djangoapps/site_configuration/helpers.py

@@ -53,7 +53,7 @@ def has_configuration_override(name):
         (bool): True if given key is present in the configuration.
     """
     configuration = get_current_site_configuration()
-    if configuration and name in configuration.values:
+    if configuration and name in configuration.site_values:
         return True
     return False
 

openedx/core/djangoapps/site_configuration/management/commands/create_site_configuration.py

@@ -47,4 +47,8 @@ def handle(self, *args, **options):
             LOG.info(u"Found existing site for '{site_name}'".format(site_name=domain))
 
         LOG.info(u"Creating '{site_name}' SiteConfiguration".format(site_name=domain))
-        SiteConfiguration.objects.create(site=site, values=configuration, enabled=True)
+        SiteConfiguration.objects.create(
+            site=site,
+            site_values=configuration,
+            enabled=True
+        )

openedx/core/djangoapps/site_configuration/management/commands/tests/test_create_site_configuration.py

@@ -21,7 +21,7 @@ def setUp(self):
 
     def _validate_site_configuration(self, site):
         site_configuration = SiteConfiguration.objects.get(site_id=site.id)
-        self.assertDictEqual(site_configuration.values, self.input_configuration)
+        self.assertDictEqual(site_configuration.site_values, self.input_configuration)
 
     def test_create_site_and_config(self):
         call_command(

openedx/core/djangoapps/site_configuration/migrations/0003_add_site_values_field.py

@@ -0,0 +1,26 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.27 on 2020-01-02 22:05
+from __future__ import unicode_literals
+
+from django.db import migrations
+import jsonfield.fields
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('site_configuration', '0002_auto_20160720_0231'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='siteconfiguration',
+            name='site_values',
+            field=jsonfield.fields.JSONField(blank=True),
+        ),
+        migrations.AddField(
+            model_name='siteconfigurationhistory',
+            name='site_values',
+            field=jsonfield.fields.JSONField(blank=True),
+        ),
+    ]

openedx/core/djangoapps/site_configuration/migrations/0004_copy_values_to_site_values.py

@@ -0,0 +1,29 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.27 on 2020-01-03 20:57
+from __future__ import unicode_literals
+
+from django.db import migrations
+
+
+def copy_column_values(apps, schema_editor):
+    """
+    Copy the values field into the site_values field.
+    """
+    SiteConfiguration = apps.get_model('site_configuration', 'SiteConfiguration')
+    for site_configuration in SiteConfiguration.objects.all():
+        site_configuration.site_values = site_configuration.values
+        site_configuration.save()
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('site_configuration', '0003_add_site_values_field'),
+    ]
+
+    operations = [
+        migrations.RunPython(
+            copy_column_values,
+            reverse_code=migrations.RunPython.noop,  # Allow reverse migrations, but make it a no-op.
+        ),
+    ]

openedx/core/djangoapps/site_configuration/migrations/0005_remove_values_field.py

@@ -0,0 +1,23 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.27 on 2020-01-07 21:26
+from __future__ import unicode_literals
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('site_configuration', '0004_copy_values_to_site_values'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='siteconfiguration',
+            name='values',
+        ),
+        migrations.RemoveField(
+            model_name='siteconfigurationhistory',
+            name='values',
+        ),
+    ]