Commit fadf3020 by Justin Riley

Remove fix extensions button from extensions dashboard

This partially reverts commit d7d6f54c
by removing the UI from the extensions dashboard but leaving
fix_extensions method in instructor.views.tools. This is ultimately a
hack to fix a deeper platform issue so no need to give it an official UI
within the extensions dashboard. Removing in favor of an XSift that runs
the reapply_extensions management command.
parent 47f3d6ce
...@@ -67,7 +67,6 @@ from .tools import ( ...@@ -67,7 +67,6 @@ from .tools import (
set_due_date_extension, set_due_date_extension,
strip_if_string, strip_if_string,
bulk_email_is_enabled_for_course, bulk_email_is_enabled_for_course,
fix_missing_extensions,
) )
from xmodule.modulestore import Location from xmodule.modulestore import Location
...@@ -1234,19 +1233,6 @@ def reset_due_date(request, course_id): ...@@ -1234,19 +1233,6 @@ def reset_due_date(request, course_id):
@ensure_csrf_cookie @ensure_csrf_cookie
@cache_control(no_cache=True, no_store=True, must_revalidate=True) @cache_control(no_cache=True, no_store=True, must_revalidate=True)
@require_level('staff') @require_level('staff')
def fix_extensions(request, course_id):
"""
Fix any <problems> missing their parent's 'extended_due'
"""
course = get_course_by_id(course_id)
fix_missing_extensions(course)
return JsonResponse(_('Successfully repaired extensions'))
@handle_dashboard_error
@ensure_csrf_cookie
@cache_control(no_cache=True, no_store=True, must_revalidate=True)
@require_level('staff')
@require_query_params('url') @require_query_params('url')
def show_unit_extensions(request, course_id): def show_unit_extensions(request, course_id):
""" """
......
...@@ -47,8 +47,6 @@ urlpatterns = patterns('', # nopep8 ...@@ -47,8 +47,6 @@ urlpatterns = patterns('', # nopep8
name='show_unit_extensions'), name='show_unit_extensions'),
url(r'^show_student_extensions$', 'instructor.views.api.show_student_extensions', url(r'^show_student_extensions$', 'instructor.views.api.show_student_extensions',
name='show_student_extensions'), name='show_student_extensions'),
url(r'^fix_extensions$', 'instructor.views.api.fix_extensions',
name='fix_extensions'),
# Grade downloads... # Grade downloads...
url(r'^list_report_downloads$', url(r'^list_report_downloads$',
......
...@@ -190,7 +190,6 @@ def _section_extensions(course): ...@@ -190,7 +190,6 @@ def _section_extensions(course):
'reset_due_date_url': reverse('reset_due_date', kwargs={'course_id': course.id}), 'reset_due_date_url': reverse('reset_due_date', kwargs={'course_id': course.id}),
'show_unit_extensions_url': reverse('show_unit_extensions', kwargs={'course_id': course.id}), 'show_unit_extensions_url': reverse('show_unit_extensions', kwargs={'course_id': course.id}),
'show_student_extensions_url': reverse('show_student_extensions', kwargs={'course_id': course.id}), 'show_student_extensions_url': reverse('show_student_extensions', kwargs={'course_id': course.id}),
'fix_extensions_url': reverse('fix_extensions', kwargs={'course_id': course.id}),
} }
return section_data return section_data
......
...@@ -23,7 +23,6 @@ class Extensions ...@@ -23,7 +23,6 @@ class Extensions
@$reset_due_date = @$section.find("input[name='reset-due-date']") @$reset_due_date = @$section.find("input[name='reset-due-date']")
@$show_unit_extensions = @$section.find("input[name='show-unit-extensions']") @$show_unit_extensions = @$section.find("input[name='show-unit-extensions']")
@$show_student_extensions = @$section.find("input[name='show-student-extensions']") @$show_student_extensions = @$section.find("input[name='show-student-extensions']")
@$fix_extensions = @$section.find("input[name='fix-extensions']")
# Gather notification areas # Gather notification areas
@$section.find(".request-response").hide() @$section.find(".request-response").hide()
...@@ -67,16 +66,6 @@ class Extensions ...@@ -67,16 +66,6 @@ class Extensions
success: (data) => @display_response "reset-extension", data success: (data) => @display_response "reset-extension", data
error: (xhr) => @fail_with_error "reset-extension", "Error reseting due date", xhr error: (xhr) => @fail_with_error "reset-extension", "Error reseting due date", xhr
@$fix_extensions.click =>
@clear_display()
send_data = {}
$.ajax
dataType: 'json'
url: @$fix_extensions.data 'endpoint'
data: send_data
success: (data) => @display_response "fix-extensions", data
error: (xhr) => @fail_with_error "fix-extensions", "Error fixing extensions", xhr
@$show_unit_extensions.click => @$show_unit_extensions.click =>
@clear_display() @clear_display()
@$grid_table.text 'Loading...' @$grid_table.text 'Loading...'
......
...@@ -110,18 +110,3 @@ ...@@ -110,18 +110,3 @@
</p> </p>
</div> </div>
<hr/>
<div id="fix-extensions">
<h2>${_("Fix extensions")}</h2>
<p>
${_("Newly added problems currently do not inherit the extended_due "
"property from it's parent. The button below fixes this issue.")}
</p>
<p class="request-response"></p>
<p class="request-response-error"></p>
<p>
<input type="button" name="fix-extensions"
value="${_("Fix extensions")}"
data-endpoint="${section_data['fix_extensions_url']}">
</p>
</div>
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment