From b502b57a2a4db62dccc032e0c284b6cd9f2a7be0 Mon Sep 17 00:00:00 2001
From: Fred Pauchet <grnx@grimbox.be>
Date: Fri, 7 Jan 2022 19:28:33 +0100
Subject: [PATCH] Finalize pylint review

---
 ultron/followup/admin.py                      |  4 +-
 ultron/followup/forms.py                      |  3 +-
 ultron/followup/tests.py                      |  3 --
 ultron/followup/urls.py                       |  6 +--
 ultron/followup/views.py                      |  4 +-
 ultron/objective/views.py                     |  2 +-
 ultron/people/models.py                       | 40 ++++++++++++++-----
 ultron/people/templatetags/level_chart_bar.py |  2 +-
 ultron/people/views.py                        | 12 +++---
 ultron/planning/admin.py                      |  4 +-
 ultron/planning/forms.py                      |  2 -
 ultron/planning/views.py                      |  1 -
 ultron/profiles/forms.py                      |  2 -
 ultron/tools/templatetags/menuitems.py        |  2 +-
 14 files changed, 48 insertions(+), 39 deletions(-)
 delete mode 100644 ultron/followup/tests.py

diff --git a/ultron/followup/admin.py b/ultron/followup/admin.py
index ac719f3ae6..a4b861ca3f 100644
--- a/ultron/followup/admin.py
+++ b/ultron/followup/admin.py
@@ -1,5 +1,6 @@
 from django.contrib import admin
-from django.db.models import fields
+
+from django_extensions.admin import ForeignKeyAutocompleteAdmin
 
 from .models import (
     Chrono,
@@ -12,7 +13,6 @@ from .models import (
     HeightWeight,
     Plan
 )
-from django_extensions.admin import ForeignKeyAutocompleteAdmin
 
 
 class ChronoAdmin(ForeignKeyAutocompleteAdmin):
diff --git a/ultron/followup/forms.py b/ultron/followup/forms.py
index e14a5c3416..9e6e7e8001 100644
--- a/ultron/followup/forms.py
+++ b/ultron/followup/forms.py
@@ -1,7 +1,6 @@
 from datetime import date
 
 from django import forms
-from django.forms import widgets
 
 from .models import (
     Chrono,
@@ -234,7 +233,7 @@ class MindStateForm(forms.ModelForm):
             "informations": forms.Textarea(
                 attrs={
                     "class": "form-control",
-                    "placeholder": "Informations about the psychological state of mind : context (why, where, …), possible consequencies, …",
+                    "placeholder": "Informations about the psychological state of mind : context (why, where, …), possible consequencies, …",  # pylint: disable=line-too-long
                 }
             ),
         }
diff --git a/ultron/followup/tests.py b/ultron/followup/tests.py
deleted file mode 100644
index 7ce503c2dd..0000000000
--- a/ultron/followup/tests.py
+++ /dev/null
@@ -1,3 +0,0 @@
-from django.test import TestCase
-
-# Create your tests here.
diff --git a/ultron/followup/urls.py b/ultron/followup/urls.py
index e752a97655..be29e79eb3 100644
--- a/ultron/followup/urls.py
+++ b/ultron/followup/urls.py
@@ -1,6 +1,4 @@
-from django.urls import path, re_path
-
-from ultron.followup.models import MindState
+from django.urls import path
 
 from . import views
 
@@ -152,4 +150,4 @@ plan_urlpatterns = [
         views.plan_create_or_update,
         name="plan_update",
     ),
-]
\ No newline at end of file
+]
diff --git a/ultron/followup/views.py b/ultron/followup/views.py
index a756deb6cb..713b1312e5 100644
--- a/ultron/followup/views.py
+++ b/ultron/followup/views.py
@@ -464,8 +464,8 @@ def routinedone_create_or_update(request, routinedone_id=None, gymnast_id=None):
             return HttpResponseRedirect(
                 reverse("gymnast_details", args=(form.cleaned_data["gymnast"].id,))
             )
-        else:
-            print(form.errors)
+
+        print(form.errors)
     else:
         form = NumberOfRoutineDoneForm(instance=routinedone, initial=data)
 
diff --git a/ultron/objective/views.py b/ultron/objective/views.py
index eddade4193..e71a4be888 100644
--- a/ultron/objective/views.py
+++ b/ultron/objective/views.py
@@ -231,7 +231,7 @@ def routine_create_or_update(request, routineid=None):
             routine = form.save()
             # ici faire un FOR skill in form_skills_list:
             # record.save()  # ca sauve le record dans la table RoutineSkill
-            # something like this : http://stackoverflow.com/questions/3074938/django-m2m-form-save-through-table
+            # something like this : http://stackoverflow.com/questions/3074938/django-m2m-form-save-through-table  # pylint: disable=line-too-long
             # TO_FRED : can you help me ?
             return HttpResponseRedirect(reverse("routine_details", args=(routine.pk,)))
 
diff --git a/ultron/people/models.py b/ultron/people/models.py
index 239b43c0da..61dd2e2d24 100644
--- a/ultron/people/models.py
+++ b/ultron/people/models.py
@@ -154,8 +154,9 @@ class Gymnast(Markdownizable):
 
     def unknown_skill_lte_level(self, max_level_skill):
         """
-        Renvoie la liste des skill inférieurs ou égaux au niveau max que le gymnaste ne sait *pas* faire.
-        Liste des Skill que le gymnaste ne sait PAS faire, classé par niveau (ayant un niveau inférieur ou égal au niveau max du gym)
+        Liste des Skill que le gymnaste ne sait PAS faire/
+
+        Ces skills sont classé par niveau.
         """
         return Skill.objects.filter(level__lte=max_level_skill).exclude(
             known_by__gymnast=self.id
@@ -163,8 +164,8 @@ class Gymnast(Markdownizable):
 
     def unknown_skill_gt_level(self, max_level_skill):
         """
-        Renvoie la liste des skill inférieurs ou égaux au niveau max que le gymnaste ne sait *pas* faire.
-        Liste des Skill que le gymnaste ne sait PAS faire, classé par niveau (ayant un niveau inférieur ou égal au niveau max du gym)
+        Liste des Skill que le gymnaste ne sait PAS faire,
+        classé par niveau (ayant un niveau inférieur ou égal au niveau max du gym)
         """
         return Skill.objects.filter(level__gt=max_level_skill).exclude(
             known_by__gymnast=self.id
@@ -179,7 +180,8 @@ class Gymnast(Markdownizable):
 
     def unknown_skill_gt_rank(self, max_rank_skill):
         """
-        Liste des Skill que le gymnaste ne sais PAS faire (ayant un niveau plus grand que le niveau max du gym)
+        Liste des Skill que le gymnaste ne sais PAS faire
+        (ayant un niveau plus grand que le niveau max du gym)
         """
         return Skill.objects.filter(level__gt=max_rank_skill).exclude(
             known_by__gymnast=self.id
@@ -274,15 +276,33 @@ class Gymnast(Markdownizable):
 
         1. on va chercher le niveau maximum de skill que le gymnast sait faire
         2. on va chercher le nombre de skill par niveau que le gymnast sait faire
-            nb_known_skill_by_level = [{'level': 1, 'nb_known_skill': 1}, {'level': 2, 'nb_known_skill': 2}]
+            nb_known_skill_by_level = [
+                {
+                    'level': 1,
+                    'nb_known_skill': 1
+                },
+                {
+                    'level': 2,
+                    'nb_known_skill': 2
+                }
+            ]
         3. si le niveau max est supérieur à 0
             OUI:
-                a. on va chercher le nombre total des skills dont le niveau est inférieur ou égale au niveau max (cf. 1.)
+                a. on va chercher le nombre total des skills dont le niveau
+                    est inférieur ou égale au niveau max (cf. 1.)
                 b. on va chercher le nombre de skill qu'il y a par niveau :
-                    [{'level': 1, 'nb_skill': 1}, {'level': 2, 'nb_skill': 1}]
+                    [
+                        {
+                            'level': 1, 'nb_skill': 1
+                        },
+                        {
+                            'level': 2, 'nb_skill': 1
+                        }
+                    ]
 
                 c. Pour chaque élément da la liste obtenue en b :
-                    - si le niveau du skill est le même que le niveau courant ... je sais pas, je comprends plus...
+                    - si le niveau du skill est le même que le niveau courant ...
+                        je sais pas, je comprends plus...
                     - on incrémente j
             NON:
 
@@ -319,7 +339,7 @@ class Gymnast(Markdownizable):
         """
 
         context = {}
-        min_rank_skill = self.min_rank_skill()
+        # min_rank_skill = self.min_rank_skill()
         max_rank_skill = self.max_rank_skill()
         context["max_rank_skill"] = max_rank_skill
         nb_known_skill_by_rank = self.known_skill_by_rank()
diff --git a/ultron/people/templatetags/level_chart_bar.py b/ultron/people/templatetags/level_chart_bar.py
index 055dbc16fb..d14179e0ff 100644
--- a/ultron/people/templatetags/level_chart_bar.py
+++ b/ultron/people/templatetags/level_chart_bar.py
@@ -13,7 +13,7 @@ def generate_level_chart_bar(gymnast_id):
     """
 
     gymnast = get_object_or_404(Gymnast, pk=gymnast_id)
-    gymnast_known_skills = gymnast.known_skills.count()
+    # gymnast_known_skills = gymnast.known_skills.count()
     context = gymnast.get_informations_from_level()
 
     return context
diff --git a/ultron/people/views.py b/ultron/people/views.py
index e6052e78ec..0157f70ac0 100644
--- a/ultron/people/views.py
+++ b/ultron/people/views.py
@@ -93,7 +93,7 @@ def gymnast_details(request, gymnast_id, tab=None):
         .order_by("-score")[:1]
     )
 
-    nb_unknown_skill = nb_skill - gymnast_nb_known_skills
+    # nb_unknown_skill = nb_skill - gymnast_nb_known_skills
 
     context = {
         "gymnast": gymnast,
@@ -284,8 +284,8 @@ def gymnast_create_or_update(request, gymnast_id=None):
         if gymnast_form.is_valid():
             gymnast = gymnast_form.save()
             return HttpResponseRedirect("/gymnast/details/" + str(gymnast.id))
-        else:
-            print(gymnast_form.errors)
+
+        print(gymnast_form.errors)
 
     form = GymnastForm(instance=gymnast, initial=data)
 
@@ -296,9 +296,9 @@ def gymnast_create_or_update(request, gymnast_id=None):
 @login_required
 @require_http_methods(["GET"])
 def gymnast_display_skill(request, gymnast_id):
-    """
-    Tag affichant les statistiques de skill d'un gymnaste : le nombre de saut qu'il sait faire (total,
-    par niveau, par rank, …), calcule la complétude, …
+    """Tag affichant les statistiques de skill d'un gymnaste
+
+    Le nombre de saut qu'il sait faire (total, par niveau, par rank, …), calcule la complétude, …
 
     .. todo:: Générer UNE fois la liste de skill que le gymnaste ne sait pas faire (1 query)
     et les counts puis, dans le template on parcourt plusieurs fois cette même liste mais on
diff --git a/ultron/planning/admin.py b/ultron/planning/admin.py
index 4a9f8b9d66..6990f2efca 100644
--- a/ultron/planning/admin.py
+++ b/ultron/planning/admin.py
@@ -45,7 +45,7 @@ class EventAdmin(ForeignKeyAutocompleteAdmin):
     # filter_horizontal = ('gymnasts',)
 
 
-class Event_ParticipationAdmin(admin.ModelAdmin):
+class EventParticipationAdmin(admin.ModelAdmin):
     model = Event_Participation
 
     fields = ("event", "gymnast", "rank")
@@ -54,4 +54,4 @@ class Event_ParticipationAdmin(admin.ModelAdmin):
 
 admin.site.register(EventType, EventTypeAdmin)
 admin.site.register(Event, EventAdmin)
-admin.site.register(Event_Participation, Event_ParticipationAdmin)
+admin.site.register(Event_Participation, EventParticipationAdmin)
diff --git a/ultron/planning/forms.py b/ultron/planning/forms.py
index 5ccc51543d..276452ace8 100644
--- a/ultron/planning/forms.py
+++ b/ultron/planning/forms.py
@@ -2,8 +2,6 @@ from datetime import date
 
 from django import forms
 
-from ultron.people.models import Gymnast
-
 from .models import Event
 
 
diff --git a/ultron/planning/views.py b/ultron/planning/views.py
index 152f94d0f7..d0a82ee675 100644
--- a/ultron/planning/views.py
+++ b/ultron/planning/views.py
@@ -1,7 +1,6 @@
 from django.contrib.auth.decorators import login_required
 from django.http import HttpResponseRedirect, HttpResponse
 from django.shortcuts import render, get_object_or_404
-from django.utils import timezone
 from django.views.decorators.http import require_http_methods
 
 import simplejson
diff --git a/ultron/profiles/forms.py b/ultron/profiles/forms.py
index d0c116e498..fb5efad974 100644
--- a/ultron/profiles/forms.py
+++ b/ultron/profiles/forms.py
@@ -1,6 +1,4 @@
 
-from datetime import date
-
 from django import forms
 
 from .models import Profile
diff --git a/ultron/tools/templatetags/menuitems.py b/ultron/tools/templatetags/menuitems.py
index b5ce6fa595..ef7184c52e 100644
--- a/ultron/tools/templatetags/menuitems.py
+++ b/ultron/tools/templatetags/menuitems.py
@@ -1,6 +1,6 @@
 from django import template
 from django.utils.html import format_html
-from django.urls import resolve, reverse
+from django.urls import reverse
 
 
 register = template.Library()