Merge branch 'develop' into 'master'

Update option in course permissions

See merge request !114

courses_legacy/core/views.py

@@ -172,8 +172,8 @@ class EnrollCourseView(LoginRequiredMixin, RedirectView):
         course = self.get_object()
         if course.is_enrolled(self.request.user):
             return reverse_lazy('courses_legacy:resume_course', args=[course.slug])
-        if course.status == 'draft':
-            return reverse_lazy('courses_legacy:course_intro')
+        if course.status == 'draft' or (not(self.request.user.groups.all() & course.groups.all()) and not course.is_public):
+            return reverse_lazy('courses_legacy:course_intro', args=[course.slug])
         if self.request.user.accepted_terms or not settings.TERMS_ACCEPTANCE_REQUIRED:
             if course.has_started and course.first_lesson():
                 course.enroll_student(self.request.user)

courses_legacy/static/js/course-permissions/app.js

@@ -23,8 +23,8 @@
         UserLocalStorage.set('currentLanguage', defaultLanguage);
 
         // UserLocalStorage.set('currentLanguage', 'es');
-        gettextCatalog.setStrings('es', {"Access Groups":"Grupos de acceso","Actions":"Acciones","Assistant Professor":"Profesor asistente","Cancel":"Cancelar","Close":"Cerrar","Coordinating Professor":"Profesor coordinador","Course Permissions":"Permisos del curso","Email":"Email","Full name":"Nombre completo","Insert User":"Insertar usuario","Insert Users":"Insertar usuarios","Office":"Oficina","Save":"Salvar","Save course":"Guardar curso","Saved successfully!":"¡Guardado exitosamente!","Search":"Buscar","Search for a User":"Buscar un usuario","User":"Usuario"});
-        gettextCatalog.setStrings('pt_br', {"Access Groups":"Acesso a Grupos","Actions":"Ações","Assistant Professor":"Professor Assistente","Cancel":"Cancelar","Close":"Fechar","Coordinating Professor":"Professor Coordenador","Course Permissions":"Permissões de curso","Email":"Email","Full name":"Nome completo","Insert User":"Inserir usuário","Insert Users":"Inserir usuários","Office":"Cargo","Save":"Salvar","Search for a User":"Pesquisar um usuário","User":"Usuário"});
+        gettextCatalog.setStrings('es', {"Access Groups":"Grupos de acceso","Actions":"Acciones","Assistant Professor":"Profesor asistente","Cancel":"Cancelar","Close":"Cerrar","Coordinating Professor":"Profesor coordinador","Course Permissions":"Permisos del curso","Email":"Email","Full name":"Nombre completo","Insert User":"Insertar usuario","Insert Users":"Insertar usuarios","Office":"Oficina","Save":"Salvar","Save course":"Guardar curso","Saved successfully!":"¡Guardado exitosamente!","Search":"Buscar","Search for a User":"Buscar un usuario","User":"Usuario", "Is public": "Es público", "Mark if this course is public and can be accessed by anyone": "Marque si este curso es público y cualquiera puede acceder"});
+        gettextCatalog.setStrings('pt_br', {"Access Groups":"Acesso a Grupos","Actions":"Ações","Assistant Professor":"Professor Assistente","Cancel":"Cancelar","Close":"Fechar","Coordinating Professor":"Professor Coordenador","Course Permissions":"Permissões de curso","Email":"Email","Full name":"Nome completo","Insert User":"Inserir usuário","Insert Users":"Inserir usuários","Office":"Cargo","Save":"Salvar","Search for a User":"Pesquisar um usuário","User":"Usuário", "Is public": "É público", "Mark if this course is public and can be accessed by anyone": "Marque se este curso é público e pode ser accessado por qualquer pessoa"});
     }]);
 
 })(angular);

courses_legacy/static/js/course-permissions/controllers.js

@@ -12,6 +12,7 @@
             'CourseProfessor',
             'BasicCourseProfessor',
             'Groups',
+            'Course',
             function(
                 $scope,
                 $window,
@@ -22,7 +23,8 @@
                 CourseGroup,
                 CourseProfessor,
                 BasicCourseProfessor,
-                Groups
+                Groups,
+                Course
             ) {
 
             var success_save_msg = 'Alterações salvas com sucesso.';
@@ -43,6 +45,7 @@
             };
 
             $scope.courseId = /course\/([^\/]+)\/permissions/.extract(location.pathname, 1);
+            $scope.course = Course.get({'id' : $scope.courseId});
             $scope.professors = BasicCourseProfessor.query({course: $scope.courseId, has_user: true}, function(professors){
                 $scope.professors_before_changes = angular.copy(professors);
             });
@@ -103,6 +106,7 @@
                 );
 
                 $scope.course_groups.$update({id: $scope.courseId});
+                $scope.course.$save();
             };
 
             $scope.cancel_permissions_changes = function() {