Korjaa virheviestit opettajia listessä
This commit is contained in:
parent
80f235ad57
commit
502ff75f0f
|
@ -27,16 +27,21 @@ document.getElementById('lisää-opettaja').addEventListener('submit', (tapahtum
|
||||||
lyhenne = lyhenneElementti.placeholder;
|
lyhenne = lyhenneElementti.placeholder;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (opettajat.lyhenne(lyhenne) === undefined) {
|
let toinenNimi = opettajat.lyhenne(lyhenne);
|
||||||
|
if (toinenNimi === undefined) {
|
||||||
suorita(tapahtumaTyypit.lisääOpettaja, lyhenne, nimi);
|
suorita(tapahtumaTyypit.lisääOpettaja, lyhenne, nimi);
|
||||||
nimiElementti.value = '';
|
nimiElementti.value = '';
|
||||||
lyhenneElementti.value = '';
|
lyhenneElementti.value = '';
|
||||||
lyhenneElementti.placeholder = '';
|
lyhenneElementti.placeholder = '';
|
||||||
virheElementti.classList.add('piilotettu');
|
virheElementti.classList.add('piilotettu');
|
||||||
piirräOpettajat();
|
piirräOpettajat();
|
||||||
|
} else if (toinenNimi === nimi) {
|
||||||
|
virheElementti.firstChild.textContent =
|
||||||
|
`Opettaja ${nimi} on jo listassa`;
|
||||||
|
virheElementti.classList.remove('piilotettu');
|
||||||
} else {
|
} else {
|
||||||
virheElementti.firstChild.textContent =
|
virheElementti.firstChild.textContent =
|
||||||
`Jokaisella opettajalla tulee olla oma lyhenne. Lyhenne ${lyhenne} on jo käytössä opettajalla ${nimi}`;
|
`Jokaisella opettajalla tulee olla oma lyhenne. Lyhenne ${lyhenne} on jo käytössä opettajalla ${toinenNimi}`;
|
||||||
virheElementti.classList.remove('piilotettu');
|
virheElementti.classList.remove('piilotettu');
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
Loading…
Reference in New Issue