Download as
Browse Code »

Commit ae0d1c79d4e305b68bbe5816721ef5360583d258

Authored by Zambom
1 parent 048c9d41
Exists in master and in 5 other branches amadeus_univasf, dev, image-crop, pdf_file_html, refactoring

Adding log to other crud functions [Issue: #203]

Showing 1 changed file with 60 additions and 4 deletions   Show diff stats
courses/views.py
  Diff comments   View file @ae0d1c7
@@ -141,7 +141,11 @@ class AllCoursesView(LoginRequiredMixin, NotificationMixin, generic.ListView): @@ -141,7 +141,11 @@ class AllCoursesView(LoginRequiredMixin, NotificationMixin, generic.ListView):
141 141
142 return context 142 return context
143 143
144 -class CreateCourseView(LoginRequiredMixin, HasRoleMixin, NotificationMixin,generic.edit.CreateView): 144 +class CreateCourseView(LoginRequiredMixin, HasRoleMixin, LogMixin, NotificationMixin, generic.edit.CreateView):
  145 + log_component = "course"
  146 + log_resource = "course"
  147 + log_action = "create"
  148 + log_context = {}
145 149
146 allowed_roles = ['professor', 'system_admin'] 150 allowed_roles = ['professor', 'system_admin']
147 login_url = reverse_lazy("core:home") 151 login_url = reverse_lazy("core:home")
@@ -153,6 +157,15 @@ class CreateCourseView(LoginRequiredMixin, HasRoleMixin, NotificationMixin,gener @@ -153,6 +157,15 @@ class CreateCourseView(LoginRequiredMixin, HasRoleMixin, NotificationMixin,gener
153 def form_valid(self, form): 157 def form_valid(self, form):
154 self.object = form.save() 158 self.object = form.save()
155 self.object.professors.add(self.request.user) 159 self.object.professors.add(self.request.user)
  160 +
  161 + self.log_context['course_id'] = self.object.id
  162 + self.log_context['course_name'] = self.object.name
  163 + self.log_context['course_slug'] = self.object.slug
  164 + self.log_context['course_category_id'] = self.object.category.id
  165 + self.log_context['course_category_name'] = self.object.category.name
  166 +
  167 + super(CreateCourseView, self).createLog(self.request.user, self.log_component, self.log_action, self.log_resource, self.log_context)
  168 +
156 return super(CreateCourseView, self).form_valid(form) 169 return super(CreateCourseView, self).form_valid(form)
157 170
158 def get_context_data(self, **kwargs): 171 def get_context_data(self, **kwargs):
@@ -166,7 +179,11 @@ class CreateCourseView(LoginRequiredMixin, HasRoleMixin, NotificationMixin,gener @@ -166,7 +179,11 @@ class CreateCourseView(LoginRequiredMixin, HasRoleMixin, NotificationMixin,gener
166 context['now'] = date.today() 179 context['now'] = date.today()
167 return context 180 return context
168 181
169 -class ReplicateCourseView(LoginRequiredMixin, HasRoleMixin, NotificationMixin,generic.edit.CreateView): 182 +class ReplicateCourseView(LoginRequiredMixin, HasRoleMixin, LogMixin, NotificationMixin,generic.edit.CreateView):
  183 + log_component = "courses"
  184 + log_action = "replicate"
  185 + log_resource = "course"
  186 + log_context = {}
170 187
171 allowed_roles = ['professor', 'system_admin'] 188 allowed_roles = ['professor', 'system_admin']
172 login_url = reverse_lazy("core:home") 189 login_url = reverse_lazy("core:home")
@@ -188,6 +205,15 @@ class ReplicateCourseView(LoginRequiredMixin, HasRoleMixin, NotificationMixin,ge @@ -188,6 +205,15 @@ class ReplicateCourseView(LoginRequiredMixin, HasRoleMixin, NotificationMixin,ge
188 elif has_role(self.request.user,'professor'): 205 elif has_role(self.request.user,'professor'):
189 courses = self.request.user.courses_professors.all() 206 courses = self.request.user.courses_professors.all()
190 categorys_courses = CourseCategory.objects.all() 207 categorys_courses = CourseCategory.objects.all()
  208 +
  209 + self.log_context['course_id'] = course.id
  210 + self.log_context['course_name'] = course.name
  211 + self.log_context['course_slug'] = course.slug
  212 + self.log_context['course_category_id'] = course.category.id
  213 + self.log_context['course_category_name'] = course.category.name
  214 +
  215 + super(ReplicateCourseView, self).createLog(self.request.user, self.log_component, self.log_action, self.log_resource, self.log_context)
  216 +
191 context['courses'] = courses 217 context['courses'] = courses
192 context['course'] = course 218 context['course'] = course
193 context['categorys_courses'] = categorys_courses 219 context['categorys_courses'] = categorys_courses
@@ -198,7 +224,11 @@ class ReplicateCourseView(LoginRequiredMixin, HasRoleMixin, NotificationMixin,ge @@ -198,7 +224,11 @@ class ReplicateCourseView(LoginRequiredMixin, HasRoleMixin, NotificationMixin,ge
198 def get_success_url(self): 224 def get_success_url(self):
199 return reverse_lazy('course:view', kwargs={'slug' : self.object.slug}) 225 return reverse_lazy('course:view', kwargs={'slug' : self.object.slug})
200 226
201 -class UpdateCourseView(LoginRequiredMixin, HasRoleMixin, generic.UpdateView): 227 +class UpdateCourseView(LoginRequiredMixin, HasRoleMixin, LogMixin, generic.UpdateView):
  228 + log_component = "courses"
  229 + log_action = "update"
  230 + log_resource = "course"
  231 + log_context = {}
202 232
203 allowed_roles = ['professor', 'system_admin'] 233 allowed_roles = ['professor', 'system_admin']
204 login_url = reverse_lazy("core:home") 234 login_url = reverse_lazy("core:home")
@@ -213,6 +243,19 @@ class UpdateCourseView(LoginRequiredMixin, HasRoleMixin, generic.UpdateView): @@ -213,6 +243,19 @@ class UpdateCourseView(LoginRequiredMixin, HasRoleMixin, generic.UpdateView):
213 return self.handle_no_permission() 243 return self.handle_no_permission()
214 return super(UpdateCourseView, self).dispatch(*args, **kwargs) 244 return super(UpdateCourseView, self).dispatch(*args, **kwargs)
215 245
  246 + def form_valid(self, form):
  247 + self.object = form.save()
  248 +
  249 + self.log_context['course_id'] = self.object.id
  250 + self.log_context['course_name'] = self.object.name
  251 + self.log_context['course_slug'] = self.object.slug
  252 + self.log_context['course_category_id'] = self.object.category.id
  253 + self.log_context['course_category_name'] = self.object.category.name
  254 +
  255 + super(UpdateCourseView, self).createLog(self.request.user, self.log_component, self.log_action, self.log_resource, self.log_context)
  256 +
  257 + return super(UpdateCourseView, self).form_valid(form)
  258 +
216 def get_context_data(self, **kwargs): 259 def get_context_data(self, **kwargs):
217 context = super(UpdateCourseView, self).get_context_data(**kwargs) 260 context = super(UpdateCourseView, self).get_context_data(**kwargs)
218 course = get_object_or_404(Course, slug = self.kwargs.get('slug')) 261 course = get_object_or_404(Course, slug = self.kwargs.get('slug'))
@@ -229,7 +272,11 @@ class UpdateCourseView(LoginRequiredMixin, HasRoleMixin, generic.UpdateView): @@ -229,7 +272,11 @@ class UpdateCourseView(LoginRequiredMixin, HasRoleMixin, generic.UpdateView):
229 def get_success_url(self): 272 def get_success_url(self):
230 return reverse_lazy('course:view', kwargs={'slug' : self.object.slug}) 273 return reverse_lazy('course:view', kwargs={'slug' : self.object.slug})
231 274
232 -class DeleteCourseView(LoginRequiredMixin, HasRoleMixin, generic.DeleteView): 275 +class DeleteCourseView(LoginRequiredMixin, HasRoleMixin, LogMixin, generic.DeleteView):
  276 + log_component = "courses"
  277 + log_action = "delete"
  278 + log_resource = "course"
  279 + log_context = {}
233 280
234 allowed_roles = ['professor', 'system_admin'] 281 allowed_roles = ['professor', 'system_admin']
235 login_url = reverse_lazy("core:home") 282 login_url = reverse_lazy("core:home")
@@ -242,6 +289,15 @@ class DeleteCourseView(LoginRequiredMixin, HasRoleMixin, generic.DeleteView): @@ -242,6 +289,15 @@ class DeleteCourseView(LoginRequiredMixin, HasRoleMixin, generic.DeleteView):
242 course = get_object_or_404(Course, slug = self.kwargs.get('slug')) 289 course = get_object_or_404(Course, slug = self.kwargs.get('slug'))
243 if(not has_object_permission('delete_course', self.request.user, course)): 290 if(not has_object_permission('delete_course', self.request.user, course)):
244 return self.handle_no_permission() 291 return self.handle_no_permission()
  292 +
  293 + self.log_context['course_id'] = course.id
  294 + self.log_context['course_name'] = course.name
  295 + self.log_context['course_slug'] = course.slug
  296 + self.log_context['course_category_id'] = course.category.id
  297 + self.log_context['course_category_name'] = course.category.name
  298 +
  299 + super(DeleteCourseView, self).createLog(self.request.user, self.log_component, self.log_action, self.log_resource, self.log_context)
  300 +
245 return super(DeleteCourseView, self).dispatch(*args, **kwargs) 301 return super(DeleteCourseView, self).dispatch(*args, **kwargs)
246 302
247 def get_context_data(self, **kwargs): 303 def get_context_data(self, **kwargs):