From 9e4c51c76d337e1f8c757cd2f99a4eed615687b4 Mon Sep 17 00:00:00 2001
From: Denise sur Lya <sekhmet@lya>
Date: Fri, 19 Jun 2020 16:27:40 +0200
Subject: [PATCH] nettoyage du code, suite

---
 app.py                  | 10 +++++++---
 configuration.py        | 36 +++++++++++++++++++++++-------------
 data/changelog_data.txt |  5 +++++
 gestion_donnees.py      |  5 +++--
 trace_courbe.py         |  7 +++++--
 5 files changed, 43 insertions(+), 20 deletions(-)

diff --git a/app.py b/app.py
index 93c5b30..30afa1c 100644
--- a/app.py
+++ b/app.py
@@ -3,6 +3,10 @@
 
 import flask
 import io
+import matplotlib.pyplot as plt
+import base64
+from matplotlib.backends.backend_agg import FigureCanvasAgg as FigureCanvas
+
 from configuration import *
 from gestion_erreurs import *
 from gestion_donnees import *
@@ -10,11 +14,8 @@ from gestion_unites import *
 from trace_courbe import *
 from faq import *
 
-import matplotlib.pyplot as plt
 
-import base64
 
-from matplotlib.backends.backend_agg import FigureCanvasAgg as FigureCanvas
 
 app = flask.Flask(__name__)
 
@@ -59,10 +60,13 @@ def courbe_image(ext):
     # créer la figure
     try:
         fig = cree_figure(config,l_jours,l_poids,liste_err)
+        #warning("Jusque là ça va, on a créé la figure",liste_err)    
         result = "success"
         output = io.BytesIO()
         #warning("Là ça va",liste_err)
+        #warning("Jusque là ça va",liste_err)
         FigureCanvas(fig).print_png(output)
+ 
         plt.close(fig)
     except:
         result = "fail"
diff --git a/configuration.py b/configuration.py
index 3e65255..c2595c2 100644
--- a/configuration.py
+++ b/configuration.py
@@ -23,19 +23,28 @@ liste_data_choisie_p.sort(reverse=True)
 liste_data_choisie_z = [(2,-3),(3,-2),(4,-1),(5,0),(6,1),(7,2),(8,3)]
 liste_data_choisie_z.sort(reverse=True)
 
-
+# config, côté python
 CONFIG = {}
 
-liste_unites_valides = ["jours", "semaines", "mois", "années"]
+# les unités acceptées
+CONFIG["liste_unites"] = ["jours", "semaines", "mois", "années"]
 
 # nombre de jours dans autre chose
 jours_dans_mois = 30.4375
 jours_dans_annee = 365.25
 jours_dans_semaine = 7
+CONFIG["jours_dans_mois"] = 30.4375
+CONFIG["jours_dans_annee"] = 365.25
+CONFIG["jours_dans_semaine"] = 7
+
 
 ## Maxi et mini de "sécurité"
-# jours maxi
-jours_maxi_courbe = 5.5*jours_dans_annee
+# jours maxi et mini
+jours_maxi_courbe = 5.5*CONFIG["jours_dans_annee"]
+CONFIG["jours_maxi_courbe"] = 5.5*CONFIG["jours_dans_annee"]
+jours_mini_courbe = 10
+CONFIG["jours_mini_courbe"] = 10
+jours_defaut_donneesvides = 6*CONFIG["jours_dans_mois"]
 # poids max (protection)
 poids_maxi = 80
 # taille max du nom
@@ -48,9 +57,17 @@ largeur_graphique_min = 3
 hauteur_graphique_max = 40
 hauteur_graphique_min = 2
 
+CONFIG["couleurs"] = {
+        "courbe1" : (0,0,1),
+        "courbe2" : (0,1,0),
+        "courbe3" : (1,0,0),
+        "fond": (1,1,1),
+        "cadretxt": (0,0,0),
+        "grille": (0.5,0.5,0.5)
+        }
 
 
-## Configuration "par défaut" du graphique, tel qu'exporté ?
+## Configuration "par défaut" du graphique, tel qu'exporté/importé
 DEFAUT = {}
 
 ### Taille du graphique par défaut (pouces, en 80 points par pouce)
@@ -71,14 +88,7 @@ DEFAUT["couleurs"]["grille"] = "#7f7f7f" # gris
 #couleur_defaut_1 = "#0000FF" # bleu
 #couleur_defaut_2 = "#00FF00" # vert
 #couleur_defaut_3 = "#FF0000" # rouge
-CONFIG["couleurs"] = {
-        "courbe1" : (0,0,1),
-        "courbe2" : (0,1,0),
-        "courbe3" : (1,0,0),
-        "fond": (1,1,1),
-        "cadretxt": (0,0,0),
-        "grille": (0.5,0.5,0.5)
-        }
+
 #couleur_defaut_1_tuple = (0,0,1)
 #couleur_defaut_2_tuple = (0,1,0)
 #couleur_defaut_3_tuple = (1,0,0)
diff --git a/data/changelog_data.txt b/data/changelog_data.txt
index c10f53e..9dce153 100644
--- a/data/changelog_data.txt
+++ b/data/changelog_data.txt
@@ -1,3 +1,8 @@
+"Version 0.5","19/06/2020","<ul>
+<li>Amélioration du code (ça se voit pas mais...)</li>
+<li>Des changements dans le format d'export des données. Les anciens fichiers json vont probablement bugguer un peu.</li>
+</ul>"
+
 "Version 0.4","18/06/2020","<ul>
 <li>Mis certaines sections en masquées par défaut, on peut cliquer pour les afficher</li>
 <li>Encore des ajouts dans la FAQ</li>
diff --git a/gestion_donnees.py b/gestion_donnees.py
index 12afd5a..228a7cb 100644
--- a/gestion_donnees.py
+++ b/gestion_donnees.py
@@ -172,11 +172,12 @@ def gere_configuration(data,liste_err):
     
     # unité
     unite = data.get("unite","")
-    if not (unite in liste_unites_valides):
+
+    if not (unite in CONFIG["liste_unites"]):
         unite = ""
         #warning("L'unité "+unite+" n'est pas reconnue !",liste_err)
     configuration["unite"] = unite
-    
+
     # grille
     grille = data.get("grille","")
     if grille != "on":
diff --git a/trace_courbe.py b/trace_courbe.py
index fccb657..4ccc6eb 100644
--- a/trace_courbe.py
+++ b/trace_courbe.py
@@ -24,7 +24,10 @@ def cree_figure(conf,l_jours,l_poids,liste_err):
     else:
         jour_maxi = conf["maxi"]+1
     
-    jour_maxi = min(jour_maxi,jours_maxi_courbe)
+    # On s'assure que c'est bien compris dans les bornes
+    #warning("bla"+str(jour_maxi),liste_err)
+    jour_maxi = max(CONFIG["jours_mini_courbe"],min(jour_maxi,CONFIG["jours_maxi_courbe"]))
+    #warning("bla"+str(jour_maxi),liste_err)
     
     if conf["unite"] == "":
         conf["unite"] = choix_unite(jour_maxi)
@@ -36,7 +39,7 @@ def cree_figure(conf,l_jours,l_poids,liste_err):
 
     titre = "Courbe de poids OMS"
     
-#    warning("cree_figure : ça va jusque là ",liste_err)
+    #warning("cree_figure : ça va jusque là ",liste_err)
     #warning("bli"+str(conf),liste_err)
     #warninf("coucou")
     
-- 
2.41.0