add url reverse; add new view

Author: LeonardBongard

frontend/templates/frontend/content/detail.html

@@ -104,8 +104,7 @@ <h5><a name="rating">{% trans "Rating" %}</a></h5>
         {% for i in 5|rev_range %}
             <input type="radio" id="star{{ i }}" name="rating" value="{{ i }}"
                    {% if i <= user_rate %}class="active" {% endif %}/>
-            {# TODO Replace with POST action and use proper url reversing #}
-            <label for="star{{ i }}" onclick="window.location.href ='rate/{{ i }}/#rating'"></label>
+            <label for="star{{ i }}" onclick="window.location.href='{% url 'frontend:rating' course.id topic.id content.id i %}'"></label>
         {% endfor %}
     </div>
 

frontend/urls.py

@@ -22,6 +22,8 @@
         path('<int:course_id>/topic/<int:topic_id>/content/', include([
             path('<int:content_id>/comment/<int:pk>/delete/', views.DeleteComment.as_view(), name='comment-delete'),
             path('<int:content_id>/comment/<int:pk>/edit/', views.EditComment.as_view(), name='comment-edit'),
+            path('<int:content_id>/rate/<int:pk>/', views.rate_content, name='rating'),
+
             path('<pk>/', views.ContentView.as_view(), name='content'),
         ])),
         path('add/', views.AddCourseView.as_view(), name='add-course'),

frontend/views/__init__.py

@@ -4,3 +4,4 @@
 from .course import CourseView, AddCourseView, CourseDeleteView
 from .content import ContentView
 from .comment import EditComment, DeleteComment
+from .content import RateContentView, rate_content

frontend/views/content.py

@@ -5,7 +5,8 @@
 from django.utils.translation import gettext_lazy as _
 from django.views.generic import DetailView
 
-from base.models import Content, Comment, Course, Topic, Favorite
+from base.models import Content, Comment, Course, Topic, Favorite, Rating, Profile
+from base.utils import get_user
 from frontend.forms import CommentForm, TranslateForm
 
 
@@ -15,7 +16,8 @@ class ContentView(DetailView):  # pylint: disable=too-many-ancestors
     """
     model = Content
     template_name = "frontend/content/detail.html"
-    #form_class = Comment
+
+    # form_class = Comment
 
     def post(self, request, *args, **kwargs):  # pylint: disable=unused-argument
         """
@@ -147,7 +149,8 @@ def get_context_data(self, **kwargs):
         topic = Topic.objects.get(pk=topic_id)
         if self.request.GET.get('coursebook'):
             course = get_object_or_404(Course, {"pk": self.kwargs['course_id']})
-            contents = [f.content for f in Favorite.objects.filter(course=course, user=self.request.user.profile)] #models.get_coursebook_flat(get_user(self.request), course)
+            contents = [f.content for f in Favorite.objects.filter(course=course,
+                                                                   user=self.request.user.profile)]  # models.get_coursebook_flat(get_user(self.request), course)
         else:
             contents = topic.get_contents(self.request.GET.get('s'), self.request.GET.get('f'))
 
@@ -169,3 +172,53 @@ def get_context_data(self, **kwargs):
             context['ending'] = '?s=' + self.request.GET.get('s') + "&f=" + \
                                 self.request.GET.get('f')
         return context
+
+
+class RateContentView(DetailView):
+    model = Content
+
+    #def get_context_data(self, **kwargs):
+        #None
+
+    #def dispatch(self, request, *args, **kwargs):
+        #None
+
+    def post(self, request, *args, **kwargs):  # pylint: disable=unused-argument
+        #None
+        form = RateForm(request.POST)
+        if form.is_valid():
+            post = form.save(commit=False)
+
+
+    #def get(self, request, *args, **kwargs):
+        #form = RateForm()
+
+
+def rate_content(request, course_id, topic_id, content_id, pk):
+    """
+    Let's the user rate content
+    :param int topic_id: id of the topic
+    :param HttpRequest request: request
+    :param int course_id: course id
+    :param int content_id: id of the content which gets rated
+    :param int pk: the user rating (should be in [ 1, 2, 3, 4, 5])
+    :return: redirect to content page
+    :rtype: HttpResponse
+    """
+    # check if rating is valid
+    rating = pk
+    print(course_id,topic_id,content_id,rating)
+    print(type(get_user(request)))
+    content = get_object_or_404(Content, pk=content_id)
+    #print(Content.ratings.filter(user_id=user.pk, content_id=content.pk))
+    profile = get_user(request)
+    Rating.objects.filter(user_id=profile, content_id=content_id).delete()
+    rating_obj = Rating.objects.create(user=profile, content=content, rating=rating)  # user = profile
+    content.ratings.add(rating_obj)
+    # check if content already has rating
+    #content.rate_content()
+    content.save()
+
+    return HttpResponseRedirect(
+        reverse_lazy('frontend:content', args=(course_id, topic_id, content_id,))
+        + '#rating')