Commit dfde9672333b7f03fdb8cea0791fcc9d5bbb7366
1 parent
b3fb9eed
Exists in
master
and in
5 other branches
modified notifications, fixed resource.url for post Model
Showing
3 changed files
with
17 additions
and
8 deletions
Show diff stats
core/mixins.py
... | ... | @@ -56,10 +56,13 @@ class NotificationMixin(object): |
56 | 56 | |
57 | 57 | if resource.exists(): |
58 | 58 | resource = resource[0] |
59 | + resource.url = resource_link | |
60 | + resource.save() | |
59 | 61 | else: |
60 | 62 | resource = Resource(name = resource_name, url= resource_link) |
61 | 63 | resource.save() |
62 | 64 | |
65 | + | |
63 | 66 | action_resource = Action_Resource.objects.filter(action = action, resource = resource) |
64 | 67 | |
65 | 68 | if action_resource.exists(): | ... | ... |
courses/views.py
... | ... | @@ -384,8 +384,7 @@ class TopicsView(LoginRequiredMixin, generic.ListView): |
384 | 384 | topic = get_object_or_404(Topic, slug = self.kwargs.get('slug')) |
385 | 385 | subject = topic.subject |
386 | 386 | topics_q = Topic.objects.filter(subject = subject, visible=True) |
387 | - #if (self.request.user in subject.professors.all() or has_role(self.request.user,'system_admin')): | |
388 | - #context = subject.topics.all() <- Change it By Activities | |
387 | + | |
389 | 388 | return topics_q |
390 | 389 | |
391 | 390 | def get_context_data(self, **kwargs): |
... | ... | @@ -393,14 +392,13 @@ class TopicsView(LoginRequiredMixin, generic.ListView): |
393 | 392 | context = super(TopicsView, self).get_context_data(**kwargs) |
394 | 393 | activitys = Activity.objects.filter(topic__name = topic.name) |
395 | 394 | students_activit = User.objects.filter(activities__in = Activity.objects.all()) |
396 | - # page_user = User.objects.get(id= self.kwargs['user_id']) | |
395 | + | |
397 | 396 | context['topic'] = topic |
398 | 397 | context['subject'] = topic.subject |
399 | 398 | context['activitys'] = activitys |
400 | 399 | context['students_activit'] = students_activit |
401 | 400 | context['form'] = ActivityForm |
402 | - # context['user_activity_id'] = Activity.objects.filter(students__id = self.kwargs['students_id']) | |
403 | - # context['page_user'] = page_user | |
401 | + | |
404 | 402 | return context |
405 | 403 | |
406 | 404 | |
... | ... | @@ -467,7 +465,7 @@ class UpdateTopicView(LoginRequiredMixin, HasRoleMixin, generic.UpdateView): |
467 | 465 | context['subjects'] = topic.subject.course.subjects.all() |
468 | 466 | return context |
469 | 467 | |
470 | -class CreateSubjectView(LoginRequiredMixin, HasRoleMixin, generic.edit.CreateView): | |
468 | +class CreateSubjectView(LoginRequiredMixin, HasRoleMixin, NotificationMixin, generic.edit.CreateView): | |
471 | 469 | |
472 | 470 | allowed_roles = ['professor', 'system_admin'] |
473 | 471 | login_url = reverse_lazy("core:home") |
... | ... | @@ -494,6 +492,11 @@ class CreateSubjectView(LoginRequiredMixin, HasRoleMixin, generic.edit.CreateVie |
494 | 492 | self.object.course = course |
495 | 493 | self.object.save() |
496 | 494 | self.object.professors.add(self.request.user) |
495 | + if self.object.visible: | |
496 | + print(reverse('course:view_subject', args=[self.object.slug])) | |
497 | + super(CreateSubjectView, self).createNotification( " created subject " + self.object.name, | |
498 | + resource_slug = self.object.slug, actor=self.request.user, users= self.object.course.students.all(), | |
499 | + resource_link = reverse('course:view_subject', args=[self.object.slug])) | |
497 | 500 | |
498 | 501 | return super(CreateSubjectView, self).form_valid(form) |
499 | 502 | ... | ... |
forum/views.py
... | ... | @@ -163,8 +163,10 @@ class CreatePostView(LoginRequiredMixin, generic.edit.CreateView, NotificationMi |
163 | 163 | self.object.user = self.request.user |
164 | 164 | |
165 | 165 | self.object.save() |
166 | - super(CreatePostView, self).createNotification(self.object.user.username + " posted on " + self.object.forum,name, | |
167 | - resource_slug = self.object.forum.slug, actor=self.request.user, users= self.object.forum.topic.subject.students.all()) | |
166 | + | |
167 | + super(CreatePostView, self).createNotification(" posted on " + self.object.forum.name, | |
168 | + resource_slug = self.object.forum.slug, actor=self.request.user, users= self.object.forum.topic.subject.students.all(), | |
169 | + resource_link = reverse('course:forum:view', args=[self.object.forum.slug])) | |
168 | 170 | |
169 | 171 | return super(CreatePostView, self).form_valid(form) |
170 | 172 | |
... | ... | @@ -266,6 +268,7 @@ class CreatePostAnswerView(LoginRequiredMixin, generic.edit.CreateView): |
266 | 268 | |
267 | 269 | self.object.save() |
268 | 270 | |
271 | + | |
269 | 272 | return super(CreatePostAnswerView, self).form_valid(form) |
270 | 273 | |
271 | 274 | def get_success_url(self): | ... | ... |