enabled question delete

Author: arielfayol37

deimos/static/deimos/css/style.css

@@ -737,6 +737,8 @@ more than assignment display */
 
 .original-size {
   font-size: initial;
+  width: auto;
+  height: auto;
 }
 @keyframes blink {
   0%, 100% {

deimos/templates/deimos/answer_question.html

@@ -230,9 +230,9 @@ <h3 class="original-size">Question {{question_dict.question.number}}</h3>
                   {% elif question_dict.questtype == 'mp' %}
                       <input type="hidden" value="mp" class="question-type"/>
                       {% if not question_dict.question_student.success %}
-                        <div class="suggestion"> Match the following pairs </div>
+                        <div class="suggestion" style="width: 100%;"> Match the following pairs </div>
                       {% else %}
-                        <div class="suggestion"> You correctly matched the pairs. </div>
+                        <div class="suggestion" style="width: 100%;"> You correctly matched the pairs. </div>
                       {% endif %}
                       <div class="mp-section">
                           <!--Yet to be passed attempts -->

deimos/templates/deimos/course_management.html

@@ -10,7 +10,7 @@
         <li><a href="{% url 'deimos:index' %}">Home</a></li>
         <li><a href="{% url 'astros:course_info' course_id=course.id %}">Course Info</a></li>
         <li><a href="{% url 'deimos:search_question' %}">Search Question</a></li>
-        <li><a href="{% url 'deimos:practice_test_settings' course_id=course.id %}">Practice Test</a></li>
+        <!--<li><a href="{% url 'deimos:practice_test_settings' course_id=course.id %}">Practice Test</a></li>-->
     </ul>
 </div>
 

phobos/static/phobos/css/style.css

@@ -799,8 +799,8 @@ more than assignment display */
 
 .original-size {
   font-size: initial;
-  width: initial;
-  height: initial;
+  width: auto;
+  height: auto;
 }
 
 

phobos/templates/phobos/question_view.html

@@ -13,11 +13,17 @@
         <li><a href="{% url 'phobos:course_management' course_id=question.assignment.course.id %}">Assignments</a></li>
         {% endif %}
         <li><a href="{% url 'phobos:search_question' %}">Search Question</a></li>
-        <li><a href="{% url 'phobos:edit_question' question_id=question.id %}">Edit Question</a></li>
-        <li id="btn-export_question"> <a>Export this question</a> </li>
+        <!--<li><a href="{% url 'phobos:edit_question' question_id=question.id %}">Edit Question</a></li>-->
     </ul>
   </div>
 
+<nav class="navbar-custom">
+    <a class="nav-link home-btn" href="{% url 'phobos:index' %}">Home </a>
+    <a class="nav-link" href="{% url 'phobos:assignment_management' assignment_id=question.assignment.id course_id=question.assignment.course.id %}">Questions</a>
+    <a class="nav-link" href="{% url 'phobos:course_management' course_id=question.assignment.course.id %}">Assignments</a>
+    <a class="nav-link" id="btn-export_question" style="color:white">Export</a>
+    <a class="nav-link" href="{% url 'phobos:delete_question' question_id=question.id %}">Delete</a>
+</nav>
 
 <div class="export_question">
     <div class="form-container" style="display: none;"> 

phobos/urls.py

@@ -19,6 +19,7 @@
     path('edit_question/<int:question_id>',views.edit_question,
          name='edit_question'
     ),
+    path('delete_question/<int:question_id>', views.delete_question, name='delete_question'),
     path('question_bank', views.question_bank, name='question_bank'),
     path('courses/<int:course_id>', views.course_management, name='course_management'),
     path('courses/<int:course_id>/create_assignment',\

phobos/views.py

@@ -605,6 +605,21 @@ def create_question(request, assignment_id=None, question_nums_types=None):
         'question_difficulties': question_difficulties
     })  
 
+@transaction.atomic
+@login_required(login_url='astros:login')
+def delete_question(request, question_id):
+    # Making sure the request is done by a professor.
+    professor = get_object_or_404(Professor, pk=request.user.id)
+    question = get_object_or_404(Question, pk=question_id)
+    if not question.assignment.course.professors.filter(pk=request.user.pk).exists():
+        return HttpResponseForbidden('You are not authorized delete this question.')
+    # delete question
+    question.delete()
+    # redirect to assignment management
+    return HttpResponseRedirect(reverse("phobos:assignment_management",\
+                                            kwargs={'course_id':question.assignment.course.id,\
+                                                    'assignment_id':question.assignment.id}))
+
 # NOTE: The function below was to be useD for a better front end design of the export question functionality.
 # The function was to enable the prof select a course then select an assignment in that course.
 #this function raises an ATTRIBUTE ERROR. WHY ???