Commit 56dc861e by attiyaishaque Committed by Attiya Ishaque

Fix course staff/instructor ordering.

parent 53d8bab3
...@@ -220,7 +220,7 @@ ...@@ -220,7 +220,7 @@
<label class="field-label "> <label class="field-label ">
{% trans "Search for Instructor:" %} {% trans "Search for Instructor:" %}
</label> </label>
<select name="staff" data-autocomplete-light-function="select2" <select name="instructor" data-autocomplete-light-function="select2"
multiple="multiple" data-minimum-input-length="2" multiple="multiple" data-minimum-input-length="2"
data-autocomplete-light-url="/admin/course_metadata/person-autocomplete/" data-autocomplete-light-url="/admin/course_metadata/person-autocomplete/"
id="id_staff" data-html="true" class="field-input"> id="id_staff" data-html="true" class="field-input">
......
...@@ -1020,7 +1020,7 @@ class CourseRunEditView(mixins.LoginRequiredMixin, mixins.PublisherPermissionMix ...@@ -1020,7 +1020,7 @@ class CourseRunEditView(mixins.LoginRequiredMixin, mixins.PublisherPermissionMix
def post(self, request, *args, **kwargs): def post(self, request, *args, **kwargs):
user = request.user user = request.user
staff = request.POST.getlist('staff')
context = self.get_context_data() context = self.get_context_data()
course_run = context.get('course_run') course_run = context.get('course_run')
lms_course_id = course_run.lms_course_id lms_course_id = course_run.lms_course_id
...@@ -1044,7 +1044,8 @@ class CourseRunEditView(mixins.LoginRequiredMixin, mixins.PublisherPermissionMix ...@@ -1044,7 +1044,8 @@ class CourseRunEditView(mixins.LoginRequiredMixin, mixins.PublisherPermissionMix
try: try:
with transaction.atomic(): with transaction.atomic():
course_run = run_form.save(changed_by=user) course_run = run_form.save(changed_by=user)
run_form.save_m2m() course_run.staff.clear()
course_run.staff.add(*staff)
# If price-type comes with request then save the seat object. # If price-type comes with request then save the seat object.
if seat_form and request.POST.get('type'): if seat_form and request.POST.get('type'):
......
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