diff --git a/cms/static/js/models/settings/advanced.js b/cms/static/js/models/settings/advanced.js index fe76164c61..052f8c9bd6 100644 --- a/cms/static/js/models/settings/advanced.js +++ b/cms/static/js/models/settings/advanced.js @@ -23,7 +23,6 @@ CMS.Models.Settings.Advanced = Backbone.Model.extend({ options = options ? _.clone(options) : {}; // add saveSuccess to the success var success = options.success; - var model = this; options.success = function(model, resp, options) { model.afterSave(model); if (success) success(model, resp, options); diff --git a/cms/templates/settings_advanced.html b/cms/templates/settings_advanced.html index 8c0e9c4bbf..5fe1b21a9b 100644 --- a/cms/templates/settings_advanced.html +++ b/cms/templates/settings_advanced.html @@ -22,7 +22,7 @@ from contentstore import utils // proactively populate advanced b/c it has the filtered list and doesn't really follow the model pattern var advancedModel = new CMS.Models.Settings.Advanced(${advanced_dict | n}, {parse: true}); advancedModel.blacklistKeys = ${advanced_blacklist | n}; - advancedModel.url = "${reverse('course_advanced_settings', kwargs=dict(org=context_course.location.org, course=context_course.location.course, name=context_course.location.name))}"; + advancedModel.url = "${reverse('course_advanced_settings_updates', kwargs=dict(org=context_course.location.org, course=context_course.location.course, name=context_course.location.name))}"; var editor = new CMS.Views.Settings.Advanced({ el: $('.settings-advanced'), diff --git a/cms/urls.py b/cms/urls.py index 6a805eed8d..7b7b5e9375 100644 --- a/cms/urls.py +++ b/cms/urls.py @@ -47,8 +47,10 @@ urlpatterns = ('', url(r'^(?P[^/]+)/(?P[^/]+)/settings-grading/(?P[^/]+)$', 'contentstore.views.course_config_graders_page', name='course_settings'), url(r'^(?P[^/]+)/(?P[^/]+)/settings-details/(?P[^/]+)/section/(?P
[^/]+).*$', 'contentstore.views.course_settings_updates', name='course_settings'), url(r'^(?P[^/]+)/(?P[^/]+)/settings-grading/(?P[^/]+)/(?P.*)$', 'contentstore.views.course_grader_updates', name='course_settings'), + # This is the URL to initially render the course advanced settings. url(r'^(?P[^/]+)/(?P[^/]+)/settings-advanced/(?P[^/]+)$', 'contentstore.views.course_config_advanced_page', name='course_advanced_settings'), - url(r'^(?P[^/]+)/(?P[^/]+)/settings-advanced/(?P[^/]+).*$', 'contentstore.views.course_advanced_updates', name='course_advanced_settings'), + # This is the URL used by BackBone for updating and re-fetching the model. + url(r'^(?P[^/]+)/(?P[^/]+)/settings-advanced/(?P[^/]+)/update.*$', 'contentstore.views.course_advanced_updates', name='course_advanced_settings_updates'), url(r'^(?P[^/]+)/(?P[^/]+)/(?P[^/]+)/(?P[^/]+)/gradeas.*$', 'contentstore.views.assignment_type_update', name='assignment_type_update'),