Fix lint errors

Author: ihalaij1

course/models.py

@@ -337,6 +337,7 @@ def check_and_tag_retaking(sender, instance, created, **kwargs): # pylint: disab
             course_instance=instance.course_instance,
         )
 
+
 post_save.connect(create_enrollment_code, sender=Enrollment)
 post_save.connect(create_anon_id, sender=Enrollment)
 post_save.connect(pseudonymize, sender=Enrollment)

course/templatetags/course.py

@@ -1,10 +1,8 @@
 from typing import Any, Dict, List, Optional, Union
-import re
 
 from django import template
 from django.db import models
 from django.utils.safestring import mark_safe
-from django.utils.html import strip_tags
 from django.utils.translation import get_language
 
 from course.models import CourseInstance, UserTagging

edit_course/operations/configure.py

@@ -241,7 +241,7 @@ def lobject_class(config: dict) -> Type[LearningObject]:
     return CourseChapter
 
 
-def update_learning_objects( # pylint: disable=too-many-locals # noqa: MC0001
+def update_learning_objects( # pylint: disable=too-many-locals too-many-branches too-many-statements  # noqa: MC0001
         category_map: Dict[str, LearningObjectCategory],
         configs: Dict[str, Dict[str, Any]],
         learning_objects: Dict[str, LearningObject],

exercise/api/csv/submission_sheet.py

@@ -1,13 +1,10 @@
-import json
-
 from collections import OrderedDict
 from typing import Any, Dict, Iterable, List, Set, Tuple
 
 from rest_framework.request import Request
 from rest_framework.reverse import reverse
 
 from ...models import BaseExercise, Submission
-from course.models import SubmissionTag
 
 
 def filter_best_submissions(
@@ -44,7 +41,7 @@ def filter_best_submissions(
             filtered.append(submissions[i])
     return filtered
 
-def submissions_sheet( # pylint: disable=too-many-locals # noqa: MC0001
+def submissions_sheet( # pylint: disable=too-many-locals too-many-branches # noqa: MC0001
         request: Request,
         submissions: Iterable[Submission],
         revealed_ids: Set[int],

exercise/api/views.py

@@ -336,7 +336,7 @@ def submit(self, request, *args, **kwargs):
         url_name='zip',
         methods=['get'],
     )
-    def zip(self, request, exercise_id, *args, **kwargs): # pylint: disable=too-many-locals # noqa: MC0001
+    def zip(self, request, exercise_id, *args, **kwargs): # pylint: disable=too-many-locals too-many-statements # noqa: MC0001
         if not self.instance.is_course_staff(request.user):
             return Response(
                 'Only course staff can download submissions via this API',
@@ -656,7 +656,7 @@ def manage_tag(self, request, *args, **kwargs):
             SubmissionTagging.objects.create(submission=self.submission, tag=tag)
             return Response({'detail': 'Tag added successfully'}, status=status.HTTP_201_CREATED)
 
-        elif request.method == 'DELETE':
+        if request.method == 'DELETE':
             try:
                 # Get the tagging object
                 tagging = SubmissionTagging.objects.get(submission=self.submission, tag=tag)

exercise/async_views.py

@@ -16,7 +16,7 @@
 logger = logging.getLogger('aplus.exercise')
 
 
-def _post_async_submission(request, exercise, submission, errors=None):
+def _post_async_submission(request, exercise, submission, errors=None): # pylint: disable=too-many-branches
     """
     Creates or grades a submission.
 
@@ -52,7 +52,7 @@ def _post_async_submission(request, exercise, submission, errors=None):
         }
 
     # Grade the submission.
-    try:
+    try: # pylint: disable=too-many-nested-blocks
         submission.set_points(form.cleaned_data["points"],
                               form.cleaned_data["max_points"])
         submission.feedback = form.cleaned_data["feedback"]

exercise/cache/points.py

@@ -647,7 +647,7 @@ def post_build(self, precreated: ProxyManager):
                     submission.feedback_revealed = True
 
     # pylint: disable-next=too-many-locals
-    def _generate_data( # noqa: MC0001
+    def _generate_data( # pylint: disable=too-many-branches too-many-statements # noqa: MC0001
             self,
             precreated: ProxyManager,
             prefetched_data: ExercisePoints.DBCLS,

exercise/exercise_models.py

@@ -898,7 +898,8 @@ def check_submission_allowed(self, profile, request=None):
             alerts[key] = [str(msg) for msg in alerts[key]]
         return success, alerts, students
 
-    def _check_submission_allowed(self, profile, request=None): # pylint: disable=too-many-locals # noqa: MC0001
+    # pylint: disable-next=too-many-locals too-many-branches too-many-statements
+    def _check_submission_allowed(self, profile, request=None): # noqa: MC0001
         students = [profile]
         alerts = {
             'error_messages' : [],

exercise/management/commands/export_submissions.py

@@ -110,7 +110,8 @@ def parse_comma_list_file(self, file_path):
         except OSError as e:
             raise CommandError(f'Error in reading the file "{file_path}".') from e
 
-    def handle(self, *args, **options): # pylint: disable=too-many-locals # noqa: MC0001
+    # pylint: disable-next=too-many-locals too-many-branches too-many-statements
+    def handle(self, *args, **options): # noqa: MC0001
         course_instance_ids = options['course_instance_id']
 
         exercise_file_path = options['exercise_output_file']

exercise/management/commands/list_exercises_services.py

@@ -25,7 +25,8 @@ def add_arguments(self, parser):
             help="Limit results to courses, which are currently visible "
                  "to students")
 
-    def handle(self, *args, **options): # pylint: disable=too-many-locals # noqa: MC0001
+    # pylint: disable-next=too-many-locals too-many-branches too-many-statements
+    def handle(self, *args, **options): # noqa: MC0001
         services = {}
         content_types = {}