fixed a bug in the save method of Question and Answer models

Author: v8ify

main/migrations/0011_auto_20210522_1726.py

@@ -0,0 +1,25 @@
+# Generated by Django 3.1.1 on 2021-05-22 11:56
+
+from django.conf import settings
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+        ('main', '0010_answerreport_questionreport'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='answerreport',
+            name='reporter',
+            field=models.ManyToManyField(blank=True, to=settings.AUTH_USER_MODEL),
+        ),
+        migrations.AlterField(
+            model_name='questionreport',
+            name='reporter',
+            field=models.ManyToManyField(blank=True, to=settings.AUTH_USER_MODEL),
+        ),
+    ]

main/models.py

@@ -21,10 +21,16 @@ class Question(models.Model):
     def __str__(self) -> str:
         return self.title[:60]
 
+    def save_model(self, first_time=True):
+        return super(Question, self).save()
+
     def save(self, *args, **kwargs):
-        super().save(*args, **kwargs)
-        QuestionVote.objects.create(question=self)
-        QuestionReport.objects.create(question=self)
+        if self.id:
+            super(Question, self).save(*args, **kwargs)
+        else:
+            super().save(*args, **kwargs)
+            QuestionVote.objects.create(question=self)
+            QuestionReport.objects.create(question=self)
 
 
 class Answer(models.Model):
@@ -46,9 +52,15 @@ def __str__(self) -> str:
         return self.content[:60]
 
     def save(self, *args, **kwargs):
-        super().save(*args, **kwargs)
-        AnswerVote.objects.create(answer=self)
-        AnswerReport.objects.create(answer=self)
+        if self.id:
+            super(Answer, self).save(*args, **kwargs)
+        else:
+            super(Answer, self).save(*args, **kwargs)
+            AnswerVote.objects.create(answer=self)
+            AnswerReport.objects.create(answer=self)
+
+    def save_model(self, first_time=True):
+        return super(Question, self).save()
 
 
 class QuestionVote(models.Model):
@@ -127,7 +139,7 @@ class Report(models.Model):
 
     # who has flagged this question
     reporter = models.ManyToManyField(
-        settings.AUTH_USER_MODEL, null=True, blank=True)
+        settings.AUTH_USER_MODEL, blank=True)
 
     # required so that django doesn't create table for this class
     class Meta:

main/templates/main/all_questions.html

@@ -12,7 +12,7 @@ <h2 class="welcome_header">All Questions</h2>
 
   <div class="questions_container">
     {% for question in page_obj %}
-      <div class="question">
+      <div class="question bg-white">
         <div class="question__votes_container">
           <div class="question__votes">
             <div class="question__votes--number">{{ question.questionvote.votes }}</div>

main/templates/main/home.html

@@ -17,7 +17,7 @@ <h2 class="welcome_header__text">Explore Answerly</h2>
 
 <div class="questions_container">
   {% for question in question_list %}
-  <div class="question">
+  <div class="question bg-white">
     <div class="question__votes_container">
       <div class="question__votes">
         <div class="question__votes--number">{{ question.questionvote.votes }}</div>

main/templates/main/navbar.html

@@ -18,6 +18,7 @@
     <div class="collapse navbar-collapse" id="navbarToggle">
       <div class="navbar-nav mr-auto">
         <a class="nav-item nav-link" href="{% url 'main:home' %}">Home</a>
+        <a class="nav-item nav-link" href="{% url 'main:all_questions' %}">All Questions</a>
       </div>
       <!-- Navbar Right Side -->
       <div class="navbar-nav">

main/views.py

@@ -20,7 +20,7 @@ def home(request):
 def all_questions(request):
     question_list = Question.objects.order_by('-created_at')
 
-    paginator = Paginator(question_list, 15)
+    paginator = Paginator(question_list, 2)
     page_number = request.GET.get('page')
     page_obj = paginator.get_page(page_number)