From 17ab2412508fb7da0ca5e2a92d0682571b2797a8 Mon Sep 17 00:00:00 2001
From: Anthony Debucquoy <debucquoy.anthony@gmail.com>
Date: Sat, 16 Mar 2024 16:57:52 +0100
Subject: [PATCH 1/3] show map by role

---
 .../EndPoints/ApplicationsController.java     | 10 ++-
 frontend/src/App.vue                          | 62 ++++---------------
 frontend/src/rest/apps.js                     | 55 +++++++++++++++-
 frontend/src/rest/restConsumer.js             |  1 -
 4 files changed, 75 insertions(+), 53 deletions(-)

diff --git a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/ApplicationsController.java b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/ApplicationsController.java
index 1629baa..77b2f3e 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/ApplicationsController.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/ApplicationsController.java
@@ -2,6 +2,7 @@ package ovh.herisson.Clyde.EndPoints;
 
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.annotation.CrossOrigin;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.RequestHeader;
@@ -9,10 +10,12 @@ import org.springframework.web.bind.annotation.RestController;
 import ovh.herisson.Clyde.Services.AuthenticatorService;
 import ovh.herisson.Clyde.Tables.Applications;
 import ovh.herisson.Clyde.Tables.Role;
+import ovh.herisson.Clyde.Tables.User;
 
 import java.util.ArrayList;
 
 @RestController
+@CrossOrigin(originPatterns = "*", allowCredentials = "true")
 public class ApplicationsController {
 
     AuthenticatorService authServ;
@@ -41,12 +44,17 @@ public class ApplicationsController {
     }
 
     public ArrayList<Applications> getAuthorizedApplications(String token){
-        Role posterRole = authServ.getUserFromToken(token).getRole();
         ArrayList<Applications> authorizedApps = new ArrayList<>();
 
         authorizedApps.add(Applications.Login);
         authorizedApps.add(Applications.Profile);
 
+		User user = authServ.getUserFromToken(token);
+		if(user == null)
+			return authorizedApps;
+
+		Role posterRole = user.getRole();
+
         if (posterRole == Role.Teacher || posterRole == Role.Student || posterRole == Role.Admin){
             authorizedApps.add(Applications.Msg);
             authorizedApps.add(Applications.Forum);
diff --git a/frontend/src/App.vue b/frontend/src/App.vue
index 1163b03..030c38d 100644
--- a/frontend/src/App.vue
+++ b/frontend/src/App.vue
@@ -1,32 +1,10 @@
 <script setup>
   import { toast } from 'vue3-toastify';
-  import { ref, computed } from 'vue'
+  import { ref } from 'vue'
   import i18n, { setLang } from './i18n.js'
   import { isLogged } from '@/rest/Users.js'
 
-
-  // Liste des apps
-  import LoginPage from './Apps/Login.vue'
-  import Inscription from "./Apps/Inscription.vue"
-  import Profil from "./Apps/Profil.vue"
-  import Courses from "./Apps/ManageCourses.vue" 
-
-  const apps = {
-  	'/login': LoginPage,
-	'/inscription': Inscription,
-  '/profil': Profil,
-  '/manage-courses' : Courses,
-  }
-  const currentPath = ref(window.location.hash)
-
-  window.addEventListener('hashchange', () => {
-	Logged.value = isLogged();
-    currentPath.value = window.location.hash
-  })
-
-  const currentView = computed(() => {
-    return apps[currentPath.value.slice(1) || '/']
-  })
+  import { appList, currentView } from '@/rest/apps.js'
 
   const home=ref(i18n("app.home"))
   const notifications=ref(i18n("app.notifications"))
@@ -36,6 +14,9 @@
 
   const Logged = ref(isLogged());
 
+	const apps = ref([])
+	appList().then(e => apps.value = e)
+
 </script>
 
 <template>
@@ -84,31 +65,12 @@
 
     <div class="leftBar">
     <ul class="vertical">
-        <li style="margin-top: 25px;" >
-          <a href="#Messages">
-            <div class="fa-solid fa-comment" style="font-size: 40px;"></div>
-            <div class="text">{{i18n("app.messages")}}</div>
-          </a></li>
-        <li >
-          <a href="#Notifications">
-            <div class="fa-solid fa-bell" style="font-size: 40px;" ></div>
-            <div class="text">{{i18n("app.notifications")}}</div>
-          </a></li>
-        <li >
-          <a href="#Schedule">
-            <div class="fa-solid fa-calendar-days" style="font-size: 40px;"></div>
-            <div class="text">{{i18n("app.schedules")}}</div>
-          </a></li>
-        <li ><a href="#Forum">
-            <div class="fa-solid fa-envelope" style="font-size: 40px;" ></div>
-            <div class="text">{{i18n("app.forum")}}</div></a></li>
-        <li><a href="#/inscription">
-            <div class="fa-solid fa-users" style="align-self:center;font-size: 40px;"></div>
-            <div class="text" style="top:0;">{{i18n("app.inscription.requests")}}</div></a></li>
-
-        <li><a href="#/manage-courses">
-            <div class="fa-solid fa-book" style="align-self:center;font-size: 40px;overflow:none;"></div>
-            <div class="text">{{i18n("app.manage.courses")}}</div></a></li>
+        <li v-for="app in apps">
+						<a href="app.path">
+            <div class="fa-solid" :class="app.icon" style="font-size: 40px;"></div>
+            <div class="text">{{app.text}}</div>
+						</a>
+				</li>
     </ul>
 
     </div>
@@ -277,3 +239,5 @@
 	  }
     
 </style>
+
+<!-- vim:set noet sts=0 sw=4 ts=2: --> 
diff --git a/frontend/src/rest/apps.js b/frontend/src/rest/apps.js
index 61fb716..7517762 100644
--- a/frontend/src/rest/apps.js
+++ b/frontend/src/rest/apps.js
@@ -1,9 +1,60 @@
 import { restGet } from './restConsumer.js' 
+import { ref, computed } from 'vue'
+import i18n from '@/i18n.js'
 
+// Liste des apps
+import LoginPage from '@/Apps/Login.vue'
+import Inscription from "@/Apps/Inscription.vue"
+import Profil from "@/Apps/Profil.vue"
+import Courses from "@/Apps/ManageCourses.vue" 
+
+const apps = {
+		'/login': LoginPage,
+		'/inscription': Inscription,
+		'/profil': Profil,
+		'/manage-courses' : Courses,
+}
+
+const appsList = {
+		'Msg': { path: '#/msg', icon: 'fa-comment', text: i18n("app.messages") },
+		'Notification': { path: '#/notifs', icon: 'fa-bell', text: i18n("app.notifications") },
+		'Forum': { path: '#/forum', icon: 'fa-envelope', text: i18n("app.forum") },
+		'Schedule': { path: '#/schedule', icon: 'fa-calendar-days', text: i18n("app.schedules") },
+		'Inscription': { path: '#/inscription', icon: 'fa-users', text: i18n("app.inscription.requests") },
+		'ManageCourses': { path: '#/manage-courses', icon: 'fa-book', text: i18n("app.manage.courses") },
+}
+
+const currentPath = ref(window.location.hash)
+
+export const currentView = computed(() => {
+		return apps[currentPath.value.slice(1) || '/']
+})
+
+/**
+ * Return the list of app accesible by a logged (or not)
+ * user.
+ */
 export async function appList(){
-	return restGet("/apps")
+		let ret = [];
+		let userAppList = await restGet("/apps");
+		for (let userapp in userAppList) {
+				if(appsList[userAppList[userapp]] != null){
+						ret.push(appsList[userAppList[userapp]])
+				}
+		}
+		return ret;
 }
 
+/**
+ * Check if the specified page is authorized for the
+ * user
+ */
 export async function checkPage(page){
-	return restGet("/apps/" + page)
+		return restGet("/apps/" + page)
 }
+
+window.addEventListener('hashchange', () => {
+		currentPath.value = window.location.hash
+})
+
+// vim:set noet sts=0 sw=4 ts=2 tw=2:
diff --git a/frontend/src/rest/restConsumer.js b/frontend/src/rest/restConsumer.js
index fca6e65..9680ac8 100644
--- a/frontend/src/rest/restConsumer.js
+++ b/frontend/src/rest/restConsumer.js
@@ -46,7 +46,6 @@ async function _rest(endPoint, config){
 			pending: config['pending'] != null ? config['pending'] : 'pending',
 			error: config['error'] != null ? config['error'] : 'Network Failure...',
 			success: config['success'] != null ? config['success'] : {render(res){
-				console.log(res);
 				return res.data.ok ? "Success" : "error";
 			}},
 		})

From d5f6656e2bba2ba52e1d36c892401d6a23347c67 Mon Sep 17 00:00:00 2001
From: Bartha Maxime <231026@umons.ac.be>
Date: Sat, 16 Mar 2024 17:16:46 +0100
Subject: [PATCH 2/3] added the GET /teachers (#130)

had to fix the mock and the UserController isAdminOrSecretary

Reviewed-on: https://git.herisson.ovh/PGL/Clyde/pulls/130
Reviewed-by: Wal <karpinskiwal@gmail.com>
Reviewed-by: Debucquoy Anthony <d.tonitch@gmail.com>
Co-authored-by: Bartha Maxime <231026@umons.ac.be>
Co-committed-by: Bartha Maxime <231026@umons.ac.be>
---
 .../Clyde/EndPoints/MockController.java       |  2 +-
 .../Clyde/EndPoints/UserController.java       | 19 +++++++++++++++++++
 .../Clyde/Repositories/UserRepository.java    |  4 ++++
 .../herisson/Clyde/Services/UserService.java  |  4 ++++
 4 files changed, 28 insertions(+), 1 deletion(-)

diff --git a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/MockController.java b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/MockController.java
index 608a450..089da2d 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/MockController.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/MockController.java
@@ -49,7 +49,7 @@ public class MockController {
 
         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), null,Role.Student,passwordEncoder.encode("student"));
-        User meh = new User("Inspiration","lackOf","secretary@secretary.com","a Box","the street",new Date(0), null,Role.Teacher,passwordEncoder.encode("secretary"));
+        User meh = new User("Inspiration","lackOf","secretary@secretary.com","a Box","the street",new Date(0), null,Role.Secretary,passwordEncoder.encode("secretary"));
         User joke = new User("CthemBalls","Lemme","teacher@teacher.com","lab","faculty",new Date(0), null,Role.Teacher,passwordEncoder.encode("teacher"));
         User lena = new User("Louille","Lena","inscriptionService@InscriptionService.com","no","yes",new Date(0), null,Role.Teacher,passwordEncoder.encode("inscriptionService"));
         mockUsers = new ArrayList<>(Arrays.asList(herobrine,joe,meh,joke));
diff --git a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/UserController.java b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/UserController.java
index 412ba80..8437b81 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/UserController.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/UserController.java
@@ -9,6 +9,8 @@ import ovh.herisson.Clyde.Services.AuthenticatorService;
 import ovh.herisson.Clyde.Services.UserService;
 import ovh.herisson.Clyde.Tables.Role;
 import ovh.herisson.Clyde.Tables.User;
+
+import java.security.Key;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.Map;
@@ -72,6 +74,23 @@ public class UserController {
 
         return new ResponseEntity<>("data modified", HttpStatus.OK);
     }
+
+    @GetMapping("/teachers")
+    public ResponseEntity<Iterable<HashMap<String,Object>>> getAllTeachers(@RequestHeader("Authorization") String token){
+        if (authServ.getUserFromToken(token) == null)
+            return new UnauthorizedResponse<>(null);
+        Iterable<User> teachers = userService.getAllTeachers();
+        ArrayList<HashMap<String, Object>> withoutPassword = new ArrayList<>();
+
+        for (User t: teachers){
+            withoutPassword.add(userWithoutPassword(t));
+        }
+
+        return new ResponseEntity<>(withoutPassword, HttpStatus.OK);
+    }
+
+
+
         /** return user's data except password
          * @param user the user to return
          * @return all the user data without the password
diff --git a/backend/src/main/java/ovh/herisson/Clyde/Repositories/UserRepository.java b/backend/src/main/java/ovh/herisson/Clyde/Repositories/UserRepository.java
index af2bd08..b2643e0 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/Repositories/UserRepository.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/Repositories/UserRepository.java
@@ -15,4 +15,8 @@ public interface UserRepository extends CrudRepository<User, Long> {
     /**
     @Query(value = "select a.* from Users a ",nativeQuery = true)
     Iterable<User> findAllUsers();**/
+
+    @Query("select u from User u where u.role = ovh.herisson.Clyde.Tables.Role.Teacher")
+    Iterable<User> findAllTeachers();
+
 }
\ No newline at end of file
diff --git a/backend/src/main/java/ovh/herisson/Clyde/Services/UserService.java b/backend/src/main/java/ovh/herisson/Clyde/Services/UserService.java
index 55a2f92..a561512 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/Services/UserService.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/Services/UserService.java
@@ -103,4 +103,8 @@ public class UserService {
     public Iterable<User> getAll(){
         return userRepo.findAll();
     }
+
+
+
+    public Iterable<User> getAllTeachers (){return userRepo.findAllTeachers();}
 }
\ No newline at end of file

From 069466ef5faa9ee89042ddf0c823f493640a7fdb Mon Sep 17 00:00:00 2001
From: Bartha Maxime <231026@umons.ac.be>
Date: Sat, 16 Mar 2024 17:17:07 +0100
Subject: [PATCH 3/3] Max/Backend/CoursesEndpoints (#129)

PLZ FIX

Reviewed-on: https://git.herisson.ovh/PGL/Clyde/pulls/129
Reviewed-by: Wal <karpinskiwal@gmail.com>
Reviewed-by: Debucquoy Anthony <d.tonitch@gmail.com>
Co-authored-by: Bartha Maxime <231026@umons.ac.be>
Co-committed-by: Bartha Maxime <231026@umons.ac.be>
---
 .../Clyde/EndPoints/CourseController.java     | 80 +++++++++++++++++++
 .../Clyde/EndPoints/CurriculumController.java | 14 ++++
 .../Clyde/EndPoints/LoginController.java      |  2 +-
 .../Clyde/EndPoints/MockController.java       |  8 +-
 .../Repositories/TeacherCourseRepository.java |  8 ++
 .../Clyde/Services/AuthenticatorService.java  | 13 +++
 .../Clyde/Services/CourseService.java         | 36 ++++++++-
 .../Clyde/Services/TeacherCourseService.java  | 39 +++++++++
 .../ovh/herisson/Clyde/Tables/Course.java     | 30 +++----
 ...herGivenCourse.java => TeacherCourse.java} | 21 ++---
 10 files changed, 213 insertions(+), 38 deletions(-)
 create mode 100644 backend/src/main/java/ovh/herisson/Clyde/EndPoints/CourseController.java
 create mode 100644 backend/src/main/java/ovh/herisson/Clyde/Repositories/TeacherCourseRepository.java
 create mode 100644 backend/src/main/java/ovh/herisson/Clyde/Services/TeacherCourseService.java
 rename backend/src/main/java/ovh/herisson/Clyde/Tables/{TeacherGivenCourse.java => TeacherCourse.java} (56%)

diff --git a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/CourseController.java b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/CourseController.java
new file mode 100644
index 0000000..011689f
--- /dev/null
+++ b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/CourseController.java
@@ -0,0 +1,80 @@
+package ovh.herisson.Clyde.EndPoints;
+
+import org.springframework.http.HttpStatus;
+import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.annotation.*;
+import ovh.herisson.Clyde.Responses.UnauthorizedResponse;
+import ovh.herisson.Clyde.Services.AuthenticatorService;
+import ovh.herisson.Clyde.Services.CourseService;
+import ovh.herisson.Clyde.Services.TeacherCourseService;
+import ovh.herisson.Clyde.Tables.Course;
+import ovh.herisson.Clyde.Tables.Role;
+import ovh.herisson.Clyde.Tables.TeacherCourse;
+import ovh.herisson.Clyde.Tables.User;
+
+import java.util.ArrayList;
+import java.util.Map;
+
+@RestController
+@CrossOrigin(originPatterns = "*", allowCredentials = "true")
+public class CourseController {
+
+    private final CourseService courseServ;
+
+    private final TeacherCourseService teacherCourseServ;
+
+    private final AuthenticatorService authServ;
+
+    public CourseController(CourseService courseServ, TeacherCourseService teacherCourseServ, AuthenticatorService authServ) {
+        this.courseServ = courseServ;
+        this.teacherCourseServ = teacherCourseServ;
+        this.authServ = authServ;
+    }
+
+    @GetMapping("/course/{id}")
+    public ResponseEntity<Course> getCourse(@RequestHeader("Authorization") String token, @PathVariable long id){
+        if (authServ.getUserFromToken(token) == null)
+            return new UnauthorizedResponse<>(null);
+
+        return new ResponseEntity<>(courseServ.findById(id), HttpStatus.OK);
+    }
+
+
+    @PostMapping("/course")
+    public ResponseEntity<Course> postCourse(@RequestHeader("Authorization") String token, @RequestBody Course course){
+        if (authServ.isNotSecretaryOrAdmin(token))
+            return new UnauthorizedResponse<>(null);
+
+        return new ResponseEntity<>(courseServ.save(course), HttpStatus.CREATED);
+    }
+
+
+    @PatchMapping("/course/{id}")
+    public ResponseEntity<Course> patchCourse(@RequestHeader("Authorization") String token,
+                                              @RequestBody Map<String,Object> updates,
+                                              @PathVariable long id)
+    {
+
+        if (authServ.IsNotIn(new Role[]{Role.Admin,Role.Teacher,Role.Secretary}, token)){
+            return new UnauthorizedResponse<>(null);
+        }
+
+        return new ResponseEntity<>(courseServ.modifyData(id, updates, authServ.getUserFromToken(token).getRole()), HttpStatus.OK);
+    }
+
+    @PostMapping("/course/{id}")
+    public ResponseEntity<String> postTeachers(@RequestHeader("Authorization") String token,
+                                               @RequestBody Iterable<Long> teacherIds,
+                                               @PathVariable Long id)
+    {
+        if (authServ.IsNotIn(new Role[]{Role.Admin,Role.Secretary}, token))
+            return new UnauthorizedResponse<>(null);
+
+
+
+        teacherCourseServ.saveAll(teacherIds,courseServ.findById(id));
+        return new ResponseEntity<>(HttpStatus.OK);
+
+    }
+
+}
diff --git a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/CurriculumController.java b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/CurriculumController.java
index 8e9b256..1892d6c 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/CurriculumController.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/CurriculumController.java
@@ -15,6 +15,7 @@ import ovh.herisson.Clyde.Tables.User;
 import java.util.Map;
 
 @RestController
+@CrossOrigin(originPatterns = "*", allowCredentials = "true")
 public class CurriculumController {
 
 
@@ -43,4 +44,17 @@ public class CurriculumController {
     public ResponseEntity<Iterable<CurriculumCourse>> findAll(){
         return new ResponseEntity<>(curriculumCourseServ.findAll(),HttpStatus.OK);
     }
+
+    /**@PostMapping("/curriculum") //todo now
+    public ResponseEntity<String> postCurriculum(@RequestHeader("Authorization") String token,@RequestBody Curriculum curriculum){
+
+        if (!isSecretaryOrAdmin(token)){
+            return new UnauthorizedResponse<>("you're not allowed to post a Curriculum");
+        }
+
+        CurriculumServ.save(Curriculum);
+
+        return new ResponseEntity<>("created !",HttpStatus.CREATED);
+    }**/
+
 }
diff --git a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/LoginController.java b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/LoginController.java
index 1e761ec..2be125d 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/LoginController.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/LoginController.java
@@ -47,6 +47,6 @@ public class LoginController {
     @PostMapping("/request/register")
     public ResponseEntity<String> register(@RequestBody InscriptionRequest inscriptionRequest){
         authServ.register(inscriptionRequest);
-        return new ResponseEntity<>("Is OK", HttpStatus.OK);
+        return new ResponseEntity<>("Is OK", HttpStatus.CREATED);
     }
 }
diff --git a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/MockController.java b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/MockController.java
index 089da2d..1750889 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/EndPoints/MockController.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/EndPoints/MockController.java
@@ -68,10 +68,10 @@ public class MockController {
         curriculumService.save(psychologyBab1);
 
 
-        Course progra1 = new Course(5,"Programmation et algorithimque 1","TODO DELETE");
-        Course chemistry1 = new Course(12, "Thermochimie","TODO DELETE");
-        Course psycho1 = new Course(21, "rien faire t'as cru c'est psycho", "TODO DELETE");
-        Course commun = new Course(2, "cours commun","TODO DELETE");
+        Course progra1 = new Course(5,"Programmation et algorithimque 1",joke);
+        Course chemistry1 = new Course(12, "Thermochimie",joke);
+        Course psycho1 = new Course(21, "rien faire t'as cru c'est psycho",joke);
+        Course commun = new Course(2, "cours commun",joke);
 
         courseService.save(progra1);
         courseService.save(chemistry1);
diff --git a/backend/src/main/java/ovh/herisson/Clyde/Repositories/TeacherCourseRepository.java b/backend/src/main/java/ovh/herisson/Clyde/Repositories/TeacherCourseRepository.java
new file mode 100644
index 0000000..ffe654a
--- /dev/null
+++ b/backend/src/main/java/ovh/herisson/Clyde/Repositories/TeacherCourseRepository.java
@@ -0,0 +1,8 @@
+package ovh.herisson.Clyde.Repositories;
+
+
+import org.springframework.data.repository.CrudRepository;
+import ovh.herisson.Clyde.Tables.TeacherCourse;
+
+public interface TeacherCourseRepository extends CrudRepository<TeacherCourse, Long> {
+}
diff --git a/backend/src/main/java/ovh/herisson/Clyde/Services/AuthenticatorService.java b/backend/src/main/java/ovh/herisson/Clyde/Services/AuthenticatorService.java
index a3301c1..63ef3c1 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/Services/AuthenticatorService.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/Services/AuthenticatorService.java
@@ -50,5 +50,18 @@ public class AuthenticatorService {
         return poster.getRole() != Role.Secretary || poster.getRole() != Role.Admin;
     }
 
+    public boolean IsNotIn(Role[] roles, String token){
+        if (token == null)
+            return true;
+
+        User poster = getUserFromToken(token);
+        if (poster == null) return true;
+
+        for (Role r:roles){
+            if (poster.getRole() == r)
+                return false;
+        }
+        return true;
+    }
 }
 
diff --git a/backend/src/main/java/ovh/herisson/Clyde/Services/CourseService.java b/backend/src/main/java/ovh/herisson/Clyde/Services/CourseService.java
index 368dbb2..8278bbe 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/Services/CourseService.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/Services/CourseService.java
@@ -3,6 +3,10 @@ package ovh.herisson.Clyde.Services;
 import org.springframework.stereotype.Service;
 import ovh.herisson.Clyde.Repositories.CourseRepository;
 import ovh.herisson.Clyde.Tables.Course;
+import ovh.herisson.Clyde.Tables.Role;
+import ovh.herisson.Clyde.Tables.User;
+
+import java.util.Map;
 
 @Service
 public class CourseService {
@@ -13,11 +17,39 @@ public class CourseService {
         this.courseRepo = courseRepo;
     }
 
-    public void save(Course course){
-        courseRepo.save(course);
+    public Course save(Course course){
+        return courseRepo.save(course);
     }
 
     public Course findById(long id){
         return courseRepo.findById(id);
     }
+
+    public Course modifyData(long id, Map<String, Object> updates, Role role) {
+        Course target = courseRepo.findById(id);
+
+        if (role == Role.Teacher){
+            for (Map.Entry<String, Object> entry : updates.entrySet()){
+                if (entry.getKey().equals("title")){
+                    target.setTitle((String) entry.getValue());
+                    return courseRepo.save(target);
+                }
+            }
+        }
+
+        for (Map.Entry<String ,Object> entry: updates.entrySet()){
+            switch (entry.getKey()){
+                case "title":
+                    target.setTitle((String) entry.getValue());
+                    break;
+                case "credits":
+                    target.setCredits((Integer) entry.getValue());
+                    break;
+                case "owner":
+                    target.setOwner((User) entry.getValue()); //todo check if is a teacher !
+                    break;
+            }
+        }
+        return courseRepo.save(target);
+    }
 }
diff --git a/backend/src/main/java/ovh/herisson/Clyde/Services/TeacherCourseService.java b/backend/src/main/java/ovh/herisson/Clyde/Services/TeacherCourseService.java
new file mode 100644
index 0000000..0996adf
--- /dev/null
+++ b/backend/src/main/java/ovh/herisson/Clyde/Services/TeacherCourseService.java
@@ -0,0 +1,39 @@
+package ovh.herisson.Clyde.Services;
+
+import org.springframework.stereotype.Controller;
+import ovh.herisson.Clyde.Repositories.TeacherCourseRepository;
+import ovh.herisson.Clyde.Repositories.UserRepository;
+import ovh.herisson.Clyde.Tables.Course;
+import ovh.herisson.Clyde.Tables.TeacherCourse;
+import ovh.herisson.Clyde.Tables.User;
+
+import java.util.ArrayList;
+
+@Controller
+public class TeacherCourseService {
+    private final TeacherCourseRepository teacherCourseRepo;
+
+    private final UserRepository userRepo;
+
+    public TeacherCourseService(TeacherCourseRepository teacherCourseRepo, UserRepository userRepo) {
+        this.teacherCourseRepo = teacherCourseRepo;
+        this.userRepo = userRepo;
+    }
+
+    public boolean saveAll(Iterable<Long> teacherIds, Course course){
+
+        ArrayList<Long> addedIds = new ArrayList<>();
+        for (Long teacherId : teacherIds){
+            User teacher = userRepo.findById((long) teacherId);
+            if ( teacher== null){
+                return false;
+            }
+            if (!addedIds.contains(teacherId))
+            {
+                teacherCourseRepo.save(new TeacherCourse(teacher,course));
+                addedIds.add(teacherId);
+            }
+        }
+        return true;
+    }
+}
diff --git a/backend/src/main/java/ovh/herisson/Clyde/Tables/Course.java b/backend/src/main/java/ovh/herisson/Clyde/Tables/Course.java
index 54e167a..e338d7d 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/Tables/Course.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/Tables/Course.java
@@ -1,9 +1,6 @@
 package ovh.herisson.Clyde.Tables;
 
-import jakarta.persistence.Entity;
-import jakarta.persistence.GeneratedValue;
-import jakarta.persistence.GenerationType;
-import jakarta.persistence.Id;
+import jakarta.persistence.*;
 
 @Entity
 public class Course {
@@ -12,12 +9,15 @@ public class Course {
     private int courseID;
     private int credits;
     private String title;
-    private String faculty;
 
-    public Course(int credits, String title, String faculty){
+    @ManyToOne(fetch = FetchType.EAGER)
+    @JoinColumn(name = "Users")
+    private User owner;
+
+    public Course(int credits, String title, User owner){
         this.credits = credits;
         this.title = title;
-        this.faculty = faculty;
+        this.owner = owner;
     }
 
     public Course() {}
@@ -34,14 +34,6 @@ public class Course {
         this.credits = credits;
     }
 
-    public String getFaculty() {
-        return faculty;
-    }
-
-    public void setFaculty(String faculty){
-        this.faculty = faculty;
-    }
-
     public String getTitle() {
         return title;
     }
@@ -49,4 +41,12 @@ public class Course {
     public void setTitle(String title){
         this.title = title;
     }
+
+    public User getOwner() {
+        return owner;
+    }
+
+    public void setOwner(User owner) {
+        this.owner = owner;
+    }
 }
diff --git a/backend/src/main/java/ovh/herisson/Clyde/Tables/TeacherGivenCourse.java b/backend/src/main/java/ovh/herisson/Clyde/Tables/TeacherCourse.java
similarity index 56%
rename from backend/src/main/java/ovh/herisson/Clyde/Tables/TeacherGivenCourse.java
rename to backend/src/main/java/ovh/herisson/Clyde/Tables/TeacherCourse.java
index a1ee138..3392c72 100644
--- a/backend/src/main/java/ovh/herisson/Clyde/Tables/TeacherGivenCourse.java
+++ b/backend/src/main/java/ovh/herisson/Clyde/Tables/TeacherCourse.java
@@ -3,30 +3,26 @@ package ovh.herisson.Clyde.Tables;
 import jakarta.persistence.*;
 
 @Entity
-public class TeacherGivenCourse {
+public class TeacherCourse {
     @Id
     @GeneratedValue(strategy = GenerationType.AUTO)
     private int id;
 
-    @ManyToOne(fetch = FetchType.LAZY)
+    @ManyToOne(fetch = FetchType.EAGER)
     @JoinColumn(name = "Users")
     private User user;
 
 
-    @ManyToOne(fetch = FetchType.LAZY)
+    @ManyToOne(fetch = FetchType.EAGER)
     @JoinColumn(name = "Course")
     private Course course;
 
-    //This flag helps make the difference between an assistant or a Teacher (who owns the course)
-    private boolean owned;
-
-    public TeacherGivenCourse(User user, Course course, boolean owned){
+    public TeacherCourse(User user, Course course){
         this.user = user;
         this.course = course;
-        this.owned = owned;
     }
 
-    public TeacherGivenCourse() {}
+    public TeacherCourse() {}
 
     public int getId() {
         return id;
@@ -48,11 +44,4 @@ public class TeacherGivenCourse {
         this.course = course;
     }
 
-    public boolean isOwned() {
-        return owned;
-    }
-
-    public void setOwned(boolean owned) {
-        this.owned = owned;
-    }
 }