From 8f3046de298f575d826e874a8e48455622113592 Mon Sep 17 00:00:00 2001 From: Thomas Rubini <74205383+ThomasRubini@users.noreply.github.com> Date: Tue, 7 Jun 2022 22:37:34 +0200 Subject: [PATCH 1/4] Suppression de sysout inutiles --- src/main/java/fr/univ_amu/iut/dao/jpa/DAOFactoryJPA.java | 2 -- src/main/java/fr/univ_amu/iut/fResultat/TableEntry.java | 1 - src/main/java/fr/univ_amu/iut/fp/Controller.java | 1 - src/main/java/fr/univ_amu/iut/loginPrompt/Controller.java | 1 + src/main/java/fr/univ_amu/iut/view/map/AcademiePath.java | 4 ++-- 5 files changed, 3 insertions(+), 6 deletions(-) diff --git a/src/main/java/fr/univ_amu/iut/dao/jpa/DAOFactoryJPA.java b/src/main/java/fr/univ_amu/iut/dao/jpa/DAOFactoryJPA.java index 402e13e..3538aae 100644 --- a/src/main/java/fr/univ_amu/iut/dao/jpa/DAOFactoryJPA.java +++ b/src/main/java/fr/univ_amu/iut/dao/jpa/DAOFactoryJPA.java @@ -24,8 +24,6 @@ public class DAOFactoryJPA implements DAOFactory { private void insertAllHelper(DAO dao, Collection list){ if(dao.findAll().size()==0){ entityManager.getTransaction().begin(); - System.out.println("LOOP"); - System.out.flush(); for(var a : list){ entityManager.persist(a); } diff --git a/src/main/java/fr/univ_amu/iut/fResultat/TableEntry.java b/src/main/java/fr/univ_amu/iut/fResultat/TableEntry.java index aec118e..80e8a4c 100644 --- a/src/main/java/fr/univ_amu/iut/fResultat/TableEntry.java +++ b/src/main/java/fr/univ_amu/iut/fResultat/TableEntry.java @@ -16,7 +16,6 @@ public class TableEntry { nom = usage.getNom(); thematique = usage.getThematique().getNom(); discipline = usage.getDiscipline().getNom(); - System.out.println("DESC="+usage.getDescription()); description = usage.getDescription(); niveau = usage.getNiveau().getNom(); } diff --git a/src/main/java/fr/univ_amu/iut/fp/Controller.java b/src/main/java/fr/univ_amu/iut/fp/Controller.java index bf75539..dd05f56 100644 --- a/src/main/java/fr/univ_amu/iut/fp/Controller.java +++ b/src/main/java/fr/univ_amu/iut/fp/Controller.java @@ -230,7 +230,6 @@ public class Controller implements Initializable { recherche.setOnAction(handleRechercheCrieters); EventHandler loginPrompt = event ->{ - System.out.println("test"); Stage loginWindow = new Stage(); try { loginWindow.setScene(new Scene(FXMLLoader.load(getClass().getResource("/fr/univ_amu/iut/loginPrompt/Code_acces.fxml")))); diff --git a/src/main/java/fr/univ_amu/iut/loginPrompt/Controller.java b/src/main/java/fr/univ_amu/iut/loginPrompt/Controller.java index 1af678f..7203668 100644 --- a/src/main/java/fr/univ_amu/iut/loginPrompt/Controller.java +++ b/src/main/java/fr/univ_amu/iut/loginPrompt/Controller.java @@ -24,6 +24,7 @@ public class Controller implements Initializable{ public void initialize(URL url, ResourceBundle resourceBundle) { EventHandler handleLogin = event ->{ + // TODO Use SecretProvider System.out.println(password.getText()); }; log.setOnAction(handleLogin); diff --git a/src/main/java/fr/univ_amu/iut/view/map/AcademiePath.java b/src/main/java/fr/univ_amu/iut/view/map/AcademiePath.java index c690d9a..e3e7dfd 100644 --- a/src/main/java/fr/univ_amu/iut/view/map/AcademiePath.java +++ b/src/main/java/fr/univ_amu/iut/view/map/AcademiePath.java @@ -36,8 +36,8 @@ public class AcademiePath extends SVGPath { final Properties PROPERTIES = new Properties(); try (InputStream resourceStream = LOADER.getResourceAsStream(fileName)) { PROPERTIES.load(resourceStream); - } catch (IOException exception) { - System.out.println(Arrays.toString(exception.getStackTrace())); + } catch (IOException e) { + throw new RuntimeException(e); } return PROPERTIES; } From 9c59af3953eda58ee99f8069c2c85a8c87c70e95 Mon Sep 17 00:00:00 2001 From: Thomas Rubini <74205383+ThomasRubini@users.noreply.github.com> Date: Tue, 7 Jun 2022 22:48:17 +0200 Subject: [PATCH 2/4] cleanup --- src/main/java/fr/univ_amu/iut/AppMain.java | 6 +-- .../java/fr/univ_amu/iut/fp/Controller.java | 45 +++++-------------- 2 files changed, 14 insertions(+), 37 deletions(-) diff --git a/src/main/java/fr/univ_amu/iut/AppMain.java b/src/main/java/fr/univ_amu/iut/AppMain.java index 4a4d3a9..e4ebcd5 100644 --- a/src/main/java/fr/univ_amu/iut/AppMain.java +++ b/src/main/java/fr/univ_amu/iut/AppMain.java @@ -22,19 +22,19 @@ public class AppMain extends Application { testMode = res!=null&&res.equals("true"); - ScreenController.addScreen("Acceuil",FXMLLoader.load(getClass().getResource("/fr/univ_amu/iut/fp/fp.fxml"))); + ScreenController.addScreen("Accueil",FXMLLoader.load(getClass().getResource("/fr/univ_amu/iut/fp/fp.fxml"))); //TODO Ajouter les pages d'admin pour l'ajout des usages //ScreenController.addScreen("admin1",pane); //ScreenController.addScreen("admin2",pane); - ScreenController.activate("Acceuil"); + ScreenController.activate("Accueil"); stage.setTitle("Carte des académie"); stage.setScene(scene); stage.show(); } - public static void main(String[] args) throws InterruptedException { + public static void main(String[] args) { launch(args); } } diff --git a/src/main/java/fr/univ_amu/iut/fp/Controller.java b/src/main/java/fr/univ_amu/iut/fp/Controller.java index dd05f56..26003ee 100644 --- a/src/main/java/fr/univ_amu/iut/fp/Controller.java +++ b/src/main/java/fr/univ_amu/iut/fp/Controller.java @@ -2,6 +2,7 @@ package fr.univ_amu.iut.fp; import fr.univ_amu.iut.AppMain; import fr.univ_amu.iut.Donnees; +import fr.univ_amu.iut.dao.DAO; import fr.univ_amu.iut.dao.DAODiscipline; import fr.univ_amu.iut.dao.DAOThematique; import fr.univ_amu.iut.dao.DAOUsage; @@ -80,7 +81,7 @@ public class Controller implements Initializable { } private Button initButton(Object obj,int x,int y){ - String nom = ""; + String nom; EventHandler actionHandler; EventHandler onPressHandler; @@ -99,15 +100,11 @@ public class Controller implements Initializable { if (obj instanceof Discipline) { Discipline disciplineActuelle = (Discipline) obj; nom = disciplineActuelle.getNom(); - actionHandler = event -> { - Donnees.setDisciplineSelectionee(disciplineActuelle); - }; + actionHandler = event -> Donnees.setDisciplineSelectionee(disciplineActuelle); } else { Thematique thematiqueActuelle = (Thematique) obj; nom = thematiqueActuelle.getNom(); - actionHandler = event -> { - Donnees.setThematiqueSelectionee(thematiqueActuelle); - }; + actionHandler = event -> Donnees.setThematiqueSelectionee(thematiqueActuelle); } @@ -157,31 +154,15 @@ public class Controller implements Initializable { return bt; } - private Void placeButtonDiscipline() { - List disciplines = daoDiscipline.findAll(); - discipline.setMinHeight(disciplines.size() * 65); - for (int i = 0; i < disciplines.size(); ++i) { + private void placeButtons(List list, AnchorPane anchorPane) { + anchorPane.setMinHeight(list.size() * 65); + for (int i = 0; i < list.size(); ++i) { if (i % 2 == 0) { - discipline.getChildren().add(initButton(disciplines.get(i), 38, i / 2 * 130)); + anchorPane.getChildren().add(initButton(list.get(i), 38, i / 2 * 130)); } else { - discipline.getChildren().add(initButton(disciplines.get(i), 332, i / 2 * 130)); + anchorPane.getChildren().add(initButton(list.get(i), 332, i / 2 * 130)); } } - - return null; - } - - private Void placeButtonThematique() { - List thematiques = daoThematique.findAll(); - thematique.setMinHeight(thematiques.size() * 65); - for (int i = 0; i < thematiques.size(); ++i) { - if (i % 2 == 0) { - thematique.getChildren().add(initButton(thematiques.get(i), 38, i / 2 * 130)); - } else { - thematique.getChildren().add(initButton(thematiques.get(i), 332, i / 2 * 130)); - } - } - return null; } @Override @@ -195,13 +176,9 @@ public class Controller implements Initializable { daoThematique = daoFactory.createDAOThematique(); daoUsage = daoFactory.createDAOUsage(); - // a chaque fois + placeButtons(daoThematique.findAll(), thematique); + placeButtons(daoDiscipline.findAll(), discipline); - List d = daoDiscipline.findAll(); - List t = daoThematique.findAll(); - - placeButtonThematique(); - placeButtonDiscipline(); EventHandler handleRechercheTextuelle = event ->{ Donnees.setUsagesObtenus(daoUsage.findByNamePart(barreDeRecherche.getText())); From 8e0d6868ece83c98e9f113efde60ff97634972dd Mon Sep 17 00:00:00 2001 From: Thomas Rubini <74205383+ThomasRubini@users.noreply.github.com> Date: Tue, 7 Jun 2022 22:55:12 +0200 Subject: [PATCH 3/4] rajout des @Transient --- .../fr/univ_amu/iut/model/Discipline.java | 19 +++++----- .../univ_amu/iut/model/RegionAcademique.java | 37 +++++++++---------- 2 files changed, 27 insertions(+), 29 deletions(-) diff --git a/src/main/java/fr/univ_amu/iut/model/Discipline.java b/src/main/java/fr/univ_amu/iut/model/Discipline.java index 5442e16..eae23f0 100644 --- a/src/main/java/fr/univ_amu/iut/model/Discipline.java +++ b/src/main/java/fr/univ_amu/iut/model/Discipline.java @@ -15,25 +15,24 @@ public class Discipline implements Serializable { private static final List INSTANCES = new ArrayList<>(); - + @Transient public static Discipline HistoireGeographie = new Discipline("Histoire-géographie"); - + @Transient public static Discipline PhysiqueChimie = new Discipline("Physique-chimie"); - + @Transient public static Discipline ScienceEtVieDeLaTerre = new Discipline("SVT"); - + @Transient public static Discipline Technologie = new Discipline("Technologie"); - + @Transient public static Discipline Francais = new Discipline("Français"); - + @Transient public static Discipline EconomieGestion = new Discipline("Économie Gestion"); - + @Transient public static Discipline LangueVivante = new Discipline("Langue Vivante"); - + @Transient public static Discipline Mathematiques = new Discipline("Mathématiques"); - + @Transient public static Discipline Philosophie = new Discipline("Philosophie"); - @Id @GeneratedValue int id; diff --git a/src/main/java/fr/univ_amu/iut/model/RegionAcademique.java b/src/main/java/fr/univ_amu/iut/model/RegionAcademique.java index fb5d90d..35a1e15 100644 --- a/src/main/java/fr/univ_amu/iut/model/RegionAcademique.java +++ b/src/main/java/fr/univ_amu/iut/model/RegionAcademique.java @@ -14,44 +14,43 @@ import java.util.List; public class RegionAcademique implements Serializable { private static List INSTANCES = new ArrayList<>(); - + @Transient public static RegionAcademique AuvergneRhoneAlpes = new RegionAcademique("Région académique Auvergne-Rhône-Alpes"); - + @Transient public static RegionAcademique BourgogneFrancheComte = new RegionAcademique("Région académique Bourgogne-Franche-Comté"); - + @Transient public static RegionAcademique Bretagne = new RegionAcademique("Région académique Bretagne"); - + @Transient public static RegionAcademique CentreValDeLoire = new RegionAcademique("Région académique Centre-Val de Loire"); - + @Transient public static RegionAcademique Corse = new RegionAcademique("Région académique Corse"); - + @Transient public static RegionAcademique GrandEst = new RegionAcademique("Région académique Grand Est"); - + @Transient public static RegionAcademique Guadeloupe = new RegionAcademique("Région académique Guadeloupe"); - + @Transient public static RegionAcademique Guyane = new RegionAcademique("Région académique Guyane"); - + @Transient public static RegionAcademique HautsDeFrance = new RegionAcademique("Région académique Hauts-de-France"); - + @Transient public static RegionAcademique IleDeFrance = new RegionAcademique("Région académique Île-de-France"); - + @Transient public static RegionAcademique Martinique = new RegionAcademique("Région académique Martinique"); - + @Transient public static RegionAcademique Normandie = new RegionAcademique("Région académique Normandie"); - + @Transient public static RegionAcademique NouvelleAquitaine = new RegionAcademique("Région académique Nouvelle-Aquitaine"); - + @Transient public static RegionAcademique Occitanie = new RegionAcademique("Région académique Occitanie"); - + @Transient public static RegionAcademique PaysDeLaLoire = new RegionAcademique("Région académique Pays de la Loire"); - + @Transient public static RegionAcademique ProvenceAlpesCoteDAzur = new RegionAcademique("Région académique Provence-Alpes-Côte d'Azur"); - + @Transient public static RegionAcademique Reunion = new RegionAcademique("Région académique La Réunion"); - + @Transient public static RegionAcademique Mayotte = new RegionAcademique("Région académique Mayotte"); - @Id @GeneratedValue int id; From 19354fecaa9db9e582aa6278a47ad1a9e3e0fe4c Mon Sep 17 00:00:00 2001 From: Thomas Rubini <74205383+ThomasRubini@users.noreply.github.com> Date: Tue, 7 Jun 2022 22:55:26 +0200 Subject: [PATCH 4/4] Suppression de quelques warnings --- src/main/java/fr/univ_amu/iut/admin2/Controller.java | 6 +++--- src/main/java/fr/univ_amu/iut/admin2/admin2.fxml | 2 +- src/main/java/fr/univ_amu/iut/fp/Controller.java | 6 +++--- .../iut/loginPrompt/{Code_acces.fxml => Code_access.fxml} | 0 .../java/fr/univ_amu/iut/secret/LocalSecretProvider.java | 4 ++-- 5 files changed, 9 insertions(+), 9 deletions(-) rename src/main/java/fr/univ_amu/iut/loginPrompt/{Code_acces.fxml => Code_access.fxml} (100%) diff --git a/src/main/java/fr/univ_amu/iut/admin2/Controller.java b/src/main/java/fr/univ_amu/iut/admin2/Controller.java index 2a49c0e..35aa27c 100644 --- a/src/main/java/fr/univ_amu/iut/admin2/Controller.java +++ b/src/main/java/fr/univ_amu/iut/admin2/Controller.java @@ -101,8 +101,8 @@ public class Controller implements Initializable { public void initialize(URL url, ResourceBundle resourceBundle) { addActeurs.setOnAction(actionEvent -> addActeur()); addRessources.setOnAction(actionEvent -> addRessource()); - id1.setOnAction(actionEvent -> textMenuButton.setPromptText("Degrès 1")); - id2.setOnAction(actionEvent -> textMenuButton.setPromptText("Degrès 2")); - id3.setOnAction(actionEvent -> textMenuButton.setPromptText("Degrès +")); + id1.setOnAction(actionEvent -> textMenuButton.setPromptText("Degré 1")); + id2.setOnAction(actionEvent -> textMenuButton.setPromptText("Degré 2")); + id3.setOnAction(actionEvent -> textMenuButton.setPromptText("Degré +")); } } diff --git a/src/main/java/fr/univ_amu/iut/admin2/admin2.fxml b/src/main/java/fr/univ_amu/iut/admin2/admin2.fxml index 07e4ad0..1aa7c91 100644 --- a/src/main/java/fr/univ_amu/iut/admin2/admin2.fxml +++ b/src/main/java/fr/univ_amu/iut/admin2/admin2.fxml @@ -69,7 +69,7 @@ - + diff --git a/src/main/java/fr/univ_amu/iut/fp/Controller.java b/src/main/java/fr/univ_amu/iut/fp/Controller.java index 26003ee..bb152c2 100644 --- a/src/main/java/fr/univ_amu/iut/fp/Controller.java +++ b/src/main/java/fr/univ_amu/iut/fp/Controller.java @@ -194,7 +194,7 @@ public class Controller implements Initializable { rechercheTextuelle.setOnAction(handleRechercheTextuelle); - EventHandler handleRechercheCrieters = event ->{ + EventHandler handleRechercheCriterias = event ->{ Donnees.setUsagesObtenus(daoUsage.findByCriterias(Donnees.getThematiqueSelectionee(),Donnees.getDisciplineSelectionee(),Donnees.getAcademieSelectionee())); Stage resultats = new Stage(); try { @@ -204,12 +204,12 @@ public class Controller implements Initializable { e.printStackTrace(); } }; - recherche.setOnAction(handleRechercheCrieters); + recherche.setOnAction(handleRechercheCriterias); EventHandler loginPrompt = event ->{ Stage loginWindow = new Stage(); try { - loginWindow.setScene(new Scene(FXMLLoader.load(getClass().getResource("/fr/univ_amu/iut/loginPrompt/Code_acces.fxml")))); + loginWindow.setScene(new Scene(FXMLLoader.load(getClass().getResource("/fr/univ_amu/iut/loginPrompt/Code_accessa.fxml")))); loginWindow.show(); } catch (IOException e) { e.printStackTrace(); diff --git a/src/main/java/fr/univ_amu/iut/loginPrompt/Code_acces.fxml b/src/main/java/fr/univ_amu/iut/loginPrompt/Code_access.fxml similarity index 100% rename from src/main/java/fr/univ_amu/iut/loginPrompt/Code_acces.fxml rename to src/main/java/fr/univ_amu/iut/loginPrompt/Code_access.fxml diff --git a/src/main/java/fr/univ_amu/iut/secret/LocalSecretProvider.java b/src/main/java/fr/univ_amu/iut/secret/LocalSecretProvider.java index 3614732..c3eb1c1 100644 --- a/src/main/java/fr/univ_amu/iut/secret/LocalSecretProvider.java +++ b/src/main/java/fr/univ_amu/iut/secret/LocalSecretProvider.java @@ -5,8 +5,8 @@ import de.mkammerer.argon2.Argon2Factory; public class LocalSecretProvider implements SecretProvider { - private static String ARGON_HASH = "$argon2id$v=19$m=16,t=2,p=1$Tms1dkJYWEZ5OTdJTndVTg$+iBsHiILEshy1JLZsTfZFQ"; - private static Argon2 argon2 = Argon2Factory.create(Argon2Factory.Argon2Types.ARGON2id); + private final static String ARGON_HASH = "$argon2id$v=19$m=16,t=2,p=1$Tms1dkJYWEZ5OTdJTndVTg$+iBsHiILEshy1JLZsTfZFQ"; + private final static Argon2 argon2 = Argon2Factory.create(Argon2Factory.Argon2Types.ARGON2id); @Override