PGL/Clyde
3
2
Fork 3
Code Issues 13 Pull Requests Actions Projects 3 Releases 2 Wiki Activity

Leo/Backend #82

Merged
Maxime merged 11 commits from Leo/Backend into master 2024-03-11 19:16:20 +01:00
Conversation 8 Commits 11 Files Changed 10 +547 -244
30 changed files with 547 additions and 244 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit dcf5b26b05 - Show all commits

3
backend/src/main/java/ovh/herisson/Clyde/EndPoints/MockController.java
View File

@ -53,9 +53,6 @@ public class MockController {
userRepo.saveAll(mockUsers); userRepo.saveAll(mockUsers);
Calendar c = Calendar.getInstance();
c.setTime(new Date());
for (User user: mockUsers){ for (User user: mockUsers){
tokenService.saveToken(new Token(user,user.getPassword(), new Date())); tokenService.saveToken(new Token(user,user.getPassword(), new Date()));
} }

1
backend/src/main/java/ovh/herisson/Clyde/Services/TokenService.java
View File

@ -51,7 +51,6 @@ public class TokenService {
//Tous les jours a minuit //Tous les jours a minuit
tonitch commented 2024-03-09 14:49:38 +01:00
Outdated
Review
Copy Link

Calendar.getInstance retourne déjà l'instant ou il est crée
https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/Calendar.html

Calendar.getInstance retourne déjà l'instant ou il est crée https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/Calendar.html
👍 1
LeoMoulin commented 2024-03-09 19:30:07 +01:00
Outdated
Review
Copy Link

C'est bien vu ca

C'est bien vu ca
@Scheduled(cron = "0 0 0 * * ?") @Scheduled(cron = "0 0 0 * * ?")
public void autoDeleteToken() { public void autoDeleteToken() {
System.out.println("triggered");
for (Token t: tokenRepo.findAll()){ for (Token t: tokenRepo.findAll()){
Calendar cal = Calendar.getInstance(); Calendar cal = Calendar.getInstance();
Maxime marked this conversation as resolved Outdated
tonitch commented 2024-03-09 14:52:04 +01:00
Outdated
Review
Copy Link

peut être faire des >= au lieu des == pour éviter que le programme ne passe au dessus si imaginons le serveur est fermé pendant 2 jours

Mais encore mieux tu pourrais comparer les deux Calendar avec la fonction ´.compareTo(Calendar)`
https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/Calendar.html#compareTo(java.util.Calendar) et juste voir si le résultat est négatif

peut être faire des >= au lieu des == pour éviter que le programme ne passe au dessus si imaginons le serveur est fermé pendant 2 jours Mais encore mieux tu pourrais comparer les deux `Calendar` avec la fonction ´.compareTo(Calendar)` https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/util/Calendar.html#compareTo(java.util.Calendar) et juste voir si le résultat est négatif
👍 1
LeoMoulin commented 2024-03-09 19:30:35 +01:00
Outdated
Review
Copy Link

Excellente remarque ! Ca sera changé dans mon prochain commit !

Excellente remarque ! Ca sera changé dans mon prochain commit !