Little corrections

Ce merge enverra dans main : le timeout des token, la limite de 5 token par user, les nouvelles tables pour les demandes d'inscriptions

Co-authored-by: Anthony Debucquoy <debucquoy.anthony@gmail.com>
Reviewed-on: #82
Reviewed-by: Debucquoy Anthony <d.tonitch@gmail.com>
Reviewed-by: Maxime <231026@umons.ac.be>
Reviewed-by: Wal <karpinskiwal@gmail.com>
Co-authored-by: LeoMoulin <leomoulin125@gmail.com>
Co-committed-by: LeoMoulin <leomoulin125@gmail.com>

backend/src/main/java/ovh/herisson/Clyde/EndPoints/MockController.java

@@ -7,12 +7,14 @@ import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.RestController;
 import ovh.herisson.Clyde.Repositories.TokenRepository;
 import ovh.herisson.Clyde.Repositories.UserRepository;
+import ovh.herisson.Clyde.Services.TokenService;
 import ovh.herisson.Clyde.Tables.Role;
 import ovh.herisson.Clyde.Tables.Token;
 import ovh.herisson.Clyde.Tables.User;
 
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Calendar;
 import java.util.Date;
 
 @RestController
@@ -23,13 +25,14 @@ public class MockController {
 
     public final UserRepository userRepo;
     public final TokenRepository tokenRepo;
-
+    public final TokenService tokenService;
     ArrayList<User> mockUsers;
 
 
-    public MockController(UserRepository userRepo, TokenRepository tokenRepo){
+    public MockController(UserRepository userRepo, TokenRepository tokenRepo, TokenService tokenService){
         this.tokenRepo = tokenRepo;
         this.userRepo = userRepo;
+        this.tokenService = tokenService;
     }
 
     /** Saves an example of each user type by :
@@ -41,10 +44,10 @@ public class MockController {
     @PostMapping("/mock")
     public void postMock(){
 
-        User herobrine = new User("brine","hero","admin@admin.com","in your WalLs","ShadowsLand",new Date(0), "none",Role.Admin,passwordEncoder.encode("admin"));
+        User herobrine = new User("brine","hero","admin@admin.com","in your WalLs","ShadowsLand",new Date(0), null,Role.Admin,passwordEncoder.encode("admin"));
-        User joe = new User("Mama","Joe","student@student.com","roundabout","DaWarudo",new Date(0), "None",Role.Student,passwordEncoder.encode("student"));
+        User joe = new User("Mama","Joe","student@student.com","roundabout","DaWarudo",new Date(0), null,Role.Student,passwordEncoder.encode("student"));
-        User meh = new User("Inspiration","lackOf","secretary@secretary.com","a Box","the street",new Date(0),"none", Role.Teacher,passwordEncoder.encode("secretary"));
+        User meh = new User("Inspiration","lackOf","secretary@secretary.com","a Box","the street",new Date(0), null,Role.Teacher,passwordEncoder.encode("secretary"));
-        User joke = new User("CthemBalls","Lemme","teacher@teacher.com","lab","faculty",new Date(0), "none",Role.Teacher,passwordEncoder.encode("teacher"));
+        User joke = new User("CthemBalls","Lemme","teacher@teacher.com","lab","faculty",new Date(0), null,Role.Teacher,passwordEncoder.encode("teacher"));
 
         mockUsers = new ArrayList<User>(Arrays.asList(herobrine,joe,meh,joke));
 

backend/src/main/java/ovh/herisson/Clyde/EndPoints/StorageController.java

@@ -0,0 +1,32 @@
+package ovh.herisson.Clyde.EndPoints;
+
+import org.springframework.http.HttpHeaders;
+import org.springframework.http.HttpStatus;
+import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.annotation.*;
+import org.springframework.web.multipart.MultipartFile;
+import ovh.herisson.Clyde.Services.StorageService;
+import org.springframework.core.io.Resource;
+import ovh.herisson.Clyde.Tables.FileType;
+
+@RestController
+@CrossOrigin(origins = "http://localhost:5173")
+public class StorageController {
+
+    private final StorageService storageServ;
+
+    public StorageController(StorageService storageServ){
+        this.storageServ= storageServ;
+    }
+
+
+    @PostMapping("/upload/{fileType}")
+    public ResponseEntity<String> handleFileUpload(@RequestParam("file") MultipartFile file, @PathVariable FileType fileType) {
+
+        String path = storageServ.store(file,fileType);
+
+        if (path == null) return new ResponseEntity<>("issue with the file storage", HttpStatus.BAD_REQUEST);
+
+        return  new ResponseEntity<>(path, HttpStatus.OK);
+    }
+}

backend/src/main/java/ovh/herisson/Clyde/EndPoints/UserController.java

@@ -31,7 +31,7 @@ public class UserController {
         return new ResponseEntity<>(user, HttpStatus.OK);
     }
 
-    @PostMapping("/user")
+    @PostMapping("/user") //todo check role
     public ResponseEntity<String> postUser(@RequestBody User user){
         userService.save(user);
         return new ResponseEntity<>(String.format("Account created with ID:%s",user.getRegNo()),HttpStatus.CREATED);

backend/src/main/java/ovh/herisson/Clyde/JdbcConfig.java

@@ -6,8 +6,10 @@ import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 import org.springframework.context.annotation.Profile;
 import org.springframework.jdbc.datasource.DriverManagerDataSource;
+import org.springframework.scheduling.annotation.EnableScheduling;
 
 @Configuration
+@EnableScheduling
 public class JdbcConfig {
 
 	@Bean

backend/src/main/java/ovh/herisson/Clyde/Repositories/FileRepository.java

@@ -0,0 +1,9 @@
+package ovh.herisson.Clyde.Repositories;
+
+import org.springframework.data.repository.CrudRepository;
+
+import ovh.herisson.Clyde.Tables.StorageFile;
+
+
+public interface FileRepository extends CrudRepository<StorageFile,Long> {
+}

backend/src/main/java/ovh/herisson/Clyde/Repositories/TokenRepository.java

@@ -4,10 +4,13 @@ import org.springframework.data.repository.CrudRepository;
 import ovh.herisson.Clyde.Tables.Token;
 import ovh.herisson.Clyde.Tables.User;
 
+import java.util.ArrayList;
+
 public interface TokenRepository extends CrudRepository<Token,Long> {
 
     Token getByToken(String token);
 
     Iterable<Token> getByUser(User user);
 
+    ArrayList <Token> getByUserOrderByExpirationDate(User user);
 }

backend/src/main/java/ovh/herisson/Clyde/Services/AuthenticatorService.java

@@ -1,6 +1,7 @@
 package ovh.herisson.Clyde.Services;
 
 import org.springframework.stereotype.Service;
+import ovh.herisson.Clyde.Tables.Token;
 import ovh.herisson.Clyde.Tables.User;
 
 import java.util.Date;
@@ -26,7 +27,7 @@ public class AuthenticatorService {
         if (user == null){return null;}
         if (!userService.checkPassword(user,password)){return null;}
         String token = tokenService.generateNewToken();
-        tokenService.saveToken(token,user,expirationDate);
+        tokenService.saveToken(new Token(user, token,expirationDate));
         return token;
     }
 }

backend/src/main/java/ovh/herisson/Clyde/Services/StorageService.java

@@ -0,0 +1,56 @@
+package ovh.herisson.Clyde.Services;
+
+import org.springframework.core.io.UrlResource;
+import org.springframework.stereotype.Service;
+import org.springframework.web.multipart.MultipartFile;
+import ovh.herisson.Clyde.Repositories.FileRepository;
+import ovh.herisson.Clyde.Tables.FileType;
+import ovh.herisson.Clyde.Tables.StorageFile;
+import java.io.IOException;
+import org.springframework.core.io.Resource;
+import java.nio.file.Files;
+import java.nio.file.Path;
+import java.nio.file.Paths;
+import java.nio.file.StandardCopyOption;
+import java.util.Objects;
+import java.util.UUID;
+
+@Service
+public class StorageService {
+
+
+    private final Path rootLocation = Paths.get("cdn/");
+    private final FileRepository fileRepo;
+
+    public StorageService(FileRepository filerepo){
+        this.fileRepo = filerepo;
+    }
+
+
+    public String store(MultipartFile file, FileType fileType) {
+
+        if (file.getOriginalFilename().isEmpty()){return null;}
+
+        UUID uuid = UUID.randomUUID();
+        String stringUuid = uuid + "." + file.getOriginalFilename().split("\\.",2)[1];
+        try {
+            if (file.isEmpty()) {
+                return null;
+            }
+
+            Path destinationFile = this.rootLocation.resolve(Paths.get(stringUuid));
+
+            Files.copy(file.getInputStream(), destinationFile,StandardCopyOption.REPLACE_EXISTING);
+        }
+        catch (IOException e) {
+            return null;
+        }
+
+        String url = this.rootLocation.resolve(Paths.get(Objects.requireNonNull(stringUuid)))
+                .normalize().toString();
+
+        fileRepo.save(new StorageFile(file.getName(),url, fileType));
+
+        return url;
+    }
+}

backend/src/main/java/ovh/herisson/Clyde/Services/TokenService.java

@@ -1,19 +1,19 @@
 package ovh.herisson.Clyde.Services;
 
-
+import org.springframework.scheduling.annotation.Scheduled;
 import org.springframework.stereotype.Service;
 import ovh.herisson.Clyde.Repositories.TokenRepository;
 import ovh.herisson.Clyde.Tables.Token;
 import ovh.herisson.Clyde.Tables.User;
 
 import java.io.UnsupportedEncodingException;
-import java.nio.charset.StandardCharsets;
 import java.security.SecureRandom;
+import java.util.ArrayList;
+import java.util.Calendar;
 import java.util.Date;
 
 @Service
 public class TokenService {
-
     TokenRepository tokenRepo;
 
     public TokenService(TokenRepository tokenRepo){
@@ -48,7 +48,28 @@ public class TokenService {
         return tokenRep.getUser();
     }
 
-    public void saveToken(String token, User user, Date expirationDate){// todo faire qlq chose de l'expDate
+    public void saveToken(Token token){
-        tokenRepo.save(new Token(user,token));
+        //Si l'utilisateur a déja 5 token delete celui qui devait expirer le plus vite
+        ArrayList<Token> tokenList = tokenRepo.getByUserOrderByExpirationDate(token.getUser());
+        while(tokenList.size() >= 5){
+            tokenRepo.delete(tokenList.get(0));
+            tokenList.remove(tokenList.get(0));
         }
+        tokenRepo.save(token);
+    }
+
+    //Tous les jours a minuit
+    @Scheduled(cron = "0 0 0 * * ?")
+    public void autoDeleteToken() {
+        for (Token t: tokenRepo.findAll()){
+            Calendar cal = Calendar.getInstance();
+
+            Calendar cal2 = Calendar.getInstance();
+            cal2.setTime(t.getExpirationDate());
+
+            if (cal.compareTo(cal2) >= 0){
+                tokenRepo.delete(t);
+            }
+        }
+    };
 }

backend/src/main/java/ovh/herisson/Clyde/Tables/FileType.java

@@ -0,0 +1,8 @@
+package ovh.herisson.Clyde.Tables;
+
+public enum FileType {
+
+    ProfilePicture,
+
+    EducationCertificate
+}

backend/src/main/java/ovh/herisson/Clyde/Tables/InscriptionRequest.java

@@ -0,0 +1,108 @@
+package ovh.herisson.Clyde.Tables;
+
+import jakarta.persistence.*;
+
+import java.util.Date;
+
+public class InscriptionRequest {
+    @GeneratedValue(strategy = GenerationType.AUTO)
+    private int id;
+    private String firstName;
+    private String lastName;
+    private String adress;
+    private String email;
+    private String country;
+    private Date birthDate;
+
+    @ManyToOne
+    @JoinColumn(name="Cursus")
+    private Cursus cursus;
+    private RequestState state;
+    private String profilePicture;
+    public InscriptionRequest(){}
+    public InscriptionRequest(String lastName, String firstName, String adress, String email, String country, Date birthDate, RequestState state, String profilePicture){
+        this.lastName = lastName;
+        this.firstName = firstName;
+        this.adress = adress;
+        this.email = email;
+        this.country = country;
+        this.birthDate = birthDate;
+        this.state = state;
+    }
+
+    public int getId() {
+        return id;
+    }
+
+    public String getFirstName() {
+        return firstName;
+    }
+
+    public void setFirstName(String firstName) {
+        this.firstName = firstName;
+    }
+
+    public String getLastName() {
+        return lastName;
+    }
+
+    public void setLastName(String lastName) {
+        this.lastName = lastName;
+    }
+
+    public String getAdress() {
+        return adress;
+    }
+
+    public void setAdress(String adress) {
+        this.adress = adress;
+    }
+
+    public String getEmail() {
+        return email;
+    }
+
+    public void setEmail(String email) {
+        this.email = email;
+    }
+
+    public String getCountry() {
+        return country;
+    }
+
+    public void setCountry(String country) {
+        this.country = country;
+    }
+
+    public Date getBirthDate() {
+        return birthDate;
+    }
+
+    public void setBirthDate(Date birthDate) {
+        this.birthDate = birthDate;
+    }
+
+    public Cursus getCursus() {
+        return cursus;
+    }
+
+    public void setCursus(Cursus cursus) {
+        this.cursus = cursus;
+    }
+
+    public RequestState getState() {
+        return state;
+    }
+
+    public void setState(RequestState state) {
+        this.state = state;
+    }
+
+    public String getProfilePicture() {
+        return profilePicture;
+    }
+
+    public void setProfilePicture(String profilePicture) {
+        this.profilePicture = profilePicture;
+    }
+}

backend/src/main/java/ovh/herisson/Clyde/Tables/ReinscriptionRequest.java

@@ -0,0 +1,69 @@
+package ovh.herisson.Clyde.Tables;
+
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.GenerationType;
+import jakarta.persistence.JoinColumn;
+import jakarta.persistence.ManyToOne;
+
+public class ReinscriptionRequest {
+    @GeneratedValue(strategy = GenerationType.AUTO)
+    private int id;
+
+    @ManyToOne
+    @JoinColumn(name = "User")
+    private User user;
+
+    @ManyToOne
+    @JoinColumn(name = "Cursus")
+    private Cursus newCursus;
+    private RequestState state;
+
+    //Permet de différencier les demandes de changement et une réinscription dans le même cursus
+    //Pour la réinscription on va le mettre a 0
+    private boolean type;
+
+    public ReinscriptionRequest(){}
+
+    public ReinscriptionRequest(User user, Cursus newCursus, RequestState state, boolean type){
+        this.user = user;
+        this.newCursus = newCursus;
+        this.state = state;
+        this.type = type;
+    }
+
+    public int getId() {
+        return id;
+    }
+
+    public User getUser() {
+        return user;
+    }
+
+    public void setUser(User user) {
+        this.user = user;
+    }
+
+    public Cursus getNewCursus() {
+        return newCursus;
+    }
+
+    public void setNewCursus(Cursus newCursus) {
+        this.newCursus = newCursus;
+    }
+
+    public RequestState getState() {
+        return state;
+    }
+
+    public void setState(RequestState state) {
+        this.state = state;
+    }
+
+    public boolean isType() {
+        return type;
+    }
+
+    public void setType(boolean type) {
+        this.type = type;
+    }
+}

backend/src/main/java/ovh/herisson/Clyde/Tables/RequestState.java

@@ -0,0 +1,7 @@
+package ovh.herisson.Clyde.Tables;
+
+public enum RequestState {
+    Accepted,
+    Refused,
+    Pending;
+}

backend/src/main/java/ovh/herisson/Clyde/Tables/StorageFile.java

@@ -0,0 +1,63 @@
+package ovh.herisson.Clyde.Tables;
+
+import jakarta.persistence.Entity;
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.GenerationType;
+import jakarta.persistence.Id;
+
+
+@Entity
+public class StorageFile {
+
+    @Id
+    @GeneratedValue(strategy = GenerationType.AUTO)
+    private Long id;
+
+    private String name;
+
+    private String url;
+
+    private FileType fileType;
+
+
+    public StorageFile(String name, String url, FileType fileType){
+        this.name = name;
+        this.url = url;
+        this.fileType = fileType;
+    }
+
+    public StorageFile(){}
+
+
+    public void setId(Long id) {
+        this.id = id;
+    }
+
+    public Long getId() {
+        return id;
+    }
+
+    public String getName() {
+        return name;
+    }
+
+    public void setName(String name) {
+        this.name = name;
+    }
+
+    public String getUrl() {
+        return url;
+    }
+
+    public void setUrl(String url) {
+        this.url = url;
+    }
+
+    public FileType getFileType() {
+        return fileType;
+    }
+
+    public void setFileType(FileType fileType) {
+        this.fileType = fileType;
+    }
+}

backend/src/main/java/ovh/herisson/Clyde/Tables/Token.java

@@ -1,6 +1,10 @@
 package ovh.herisson.Clyde.Tables;
 
 import jakarta.persistence.*;
+import org.springframework.scheduling.annotation.Scheduled;
+import ovh.herisson.Clyde.Repositories.TokenRepository;
+
+import java.util.Date;
 
 @Entity
 public class Token {
@@ -12,13 +16,16 @@ public class Token {
     @JoinColumn(name ="Users")
     private User user;
     private String token;
+    private Date expirationDate;
 
-    public Token(User user, String token){
+    public Token(User user, String token, Date expirationDate){
         this.user = user;
         this.token = token;
+        this.expirationDate = expirationDate;
     }
 
     public Token(){}
+
     public int getId() {
         return id;
     }
@@ -37,4 +44,12 @@ public class Token {
     public void setToken(String data) {
         this.token = data;
     }
+
+    public void setExpirationDate(Date date){
+        this.expirationDate = date;
+    }
+
+    public Date getExpirationDate(){
+        return expirationDate;
+    }
 }

backend/src/main/java/ovh/herisson/Clyde/Tables/User.java

@@ -16,6 +16,7 @@ public class User {
     private int regNo;
     private String lastName;
     private String firstName;
+    @Column(unique = true)
     private String email;
     private String address;
     private String country;
@@ -23,7 +24,9 @@ public class User {
     private String profilePictureUrl;
     private ovh.herisson.Clyde.Tables.Role role;
     private String password;
-    public User(String lastName, String firstName, String email, String address, String country, Date birthDate, String profilePictureUrl, Role role, String password){
+    public User(String lastName, String firstName, String email, String address,
+                String country, Date birthDate, String profilePictureUrl, Role role, String password)
+    {
         this.lastName = lastName;
         this.firstName = firstName;
         this.email = email;

frontend/src/App.vue

@@ -56,7 +56,7 @@
         <li @click="active=!active"  class="option"style="float: right;" title=settings>
             <a class="icon" >
                 <div  class="fa-solid fa-gear"  style="margin-top: 7px; margin-bottom: 3px;"></div>
-                <div v-if="active"class="dropdown">
+                <div v-if="active" class="dropdown">
                   <div class="dropdown-content">Langage</div>
                   <ul style="list-style-type:none;">
                     <li style=" margin-bottom:10px;margin-right:20px;float:left; font-size:10px; color:black;">

frontend/src/Apps/Profil.vue

@@ -1,6 +1,6 @@
 <script setup>
   import {reactive, ref} from 'vue'
-  import {getUser,register} from '../rest/Users.js'
+  import {getUser} from '../rest/Users.js'
  /* 
   const user = getUser(); 
 */
@@ -11,14 +11,15 @@
   role:"student",
   address: "Radiator Springs",
   email:"ClydeGhost@gmail.com",
-  cursus:{
+  cursus:[
-  cours1:{  "id": 12,
+  {
+  "id": 12,
   "name": "Math pour l'info",
   "credits": 11,
   "faculty": "science",
   "teacher": 42,
   "Assistants": []},
-  cours2:{ 
+  {
   "id": 42,
   "name": "Fonctionnement des ordinateurs",
   "credits": 11,
@@ -26,7 +27,7 @@
   "teacher": 42,
   "Assistants": []},
 
-  },
+  ],
   option:"IT",
   degree:"BAC1",
   password:"CeciEstUnMotDePasse123",
@@ -41,14 +42,14 @@
   role:"teacher",
   address: "Radiator Springs",
   email:"ClydeGhost@gmail.com",
-  coursesOwned:{
+  coursesOwned:[
-  cours1:{  
+  {  
   "id": 12,
   "name": "Math pour l'info",
   "faculty": "science",
   "teacher": 42,
   "Assistants": []},
-  cours2:{ 
+  { 
   "id": 42,
   "name": "Fonctionnement des ordinateurs",
   "credits": 11,
@@ -56,7 +57,7 @@
   "teacher": 42,
   "Assistants": []},
 
-  },
+  ],
   faculty:"Science",
   })*/