Commit 856e9eba by Asad Azam Committed by AsadAzam

Fixed instructor not saving on edit run

parent 6fce2dad
......@@ -2848,6 +2848,25 @@ class CourseRunEditViewTests(SiteMixin, TestCase):
self.assertEqual(response.status_code, 200)
self.assertNotContains(response, '<div id="SeatPriceBlock" class="col col-6 hidden" style="display: block;">')
def test_get_context_data_with_pc(self):
"""
Verify that the get function returns empty list of organization_ids in the context
if the logged in user is project coordinator
"""
pc_user = UserFactory()
pc_user.groups.add(self.group_project_coordinator)
pc_user.groups.add(self.organization_extension.group)
assign_perm(
OrganizationExtension.EDIT_COURSE_RUN, self.organization_extension.group, self.organization_extension
)
self.client.logout()
self.client.login(username=pc_user.username, password=USER_PASSWORD)
self.edit_page_url = reverse('publisher:publisher_course_runs_edit', kwargs={'pk': self.course_run.id})
response = self.client.get(self.edit_page_url)
self.assertEqual(response.status_code, 200)
self.assertEqual(response.context['organizations_ids'], [])
def _post_data(self, data, course, course_run):
course_dict = model_to_dict(course)
course_dict.update(**data)
......
......@@ -711,9 +711,11 @@ class CourseRunEditView(mixins.LoginRequiredMixin, mixins.PublisherPermissionMix
course_run = context.get('course_run')
course = course_run.course
course_user_role = course.get_user_role(user=self.request.user)
context['organizations_ids'] = []
if course_user_role == PublisherUserRole.CourseTeam:
context['organizations_ids'] = list(mixins.get_user_organizations(self.request.user).
values_list('id', flat=True))
context['organizations_ids'] = list(
mixins.get_user_organizations(self.request.user).values_list('id', flat=True)
)
context['course_user_role'] = course_user_role
context['run_form'] = self.run_form(
......
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