fix: Champs requis sur les teachers and classes

This commit is contained in:
N3WT DE COMPET
2025-02-07 16:43:29 +01:00
parent 1a8ef26f58
commit 42b4c99be8
2 changed files with 35 additions and 30 deletions

View File

@ -161,7 +161,7 @@ const ClassesSection = ({ classes, setClasses, teachers, handleCreate, handleEdi
};
const handleSaveNewClass = () => {
if (newClass.atmosphere_name) {
if (newClass.atmosphere_name && newClass.age_range && newClass.levels.length > 0 && newClass.number_of_students && newClass.school_year) {
handleCreate(newClass)
.then((createdClass) => {
setClasses((prevClasses) => [createdClass, ...classes]);
@ -178,21 +178,21 @@ const ClassesSection = ({ classes, setClasses, teachers, handleCreate, handleEdi
};
const handleUpdateClass = (id, updatedData) => {
if (!updatedData.atmosphere_name) {
setLocalErrors({ atmosphere_name: 'Le nom d\'ambiance est requis.' });
return;
if (updatedData.atmosphere_name && updatedData.age_range && updatedData.levels.length > 0 && updatedData.number_of_students && updatedData.school_year) {
handleEdit(id, updatedData)
.then((updatedClass) => {
setClasses((prevClasses) => prevClasses.map((classe) => (classe.id === id ? updatedClass : classe)));
setEditingClass(null);
setFormData({});
setLocalErrors({});
})
.catch((error) => {
console.error(error);
});
} else {
setPopupMessage("Tous les champs doivent être remplis et valides");
setPopupVisible(true);
}
handleEdit(id, updatedData)
.then((updatedClass) => {
setClasses((prevClasses) => prevClasses.map((classe) => (classe.id === id ? updatedClass : classe)));
setEditingClass(null);
setFormData({});
setLocalErrors({});
})
.catch((error) => {
console.error(error);
});
};
const handleTeachersChange = (selectedTeachers) => {