ZIP 14.4.14 n. klo 22
MVC näyttää olevan vielä vähän kesken, ja nyt esim. sivuissa on 2 kertaan alkurimpsut (<html> -tagit jne), kuten tässä (kuvan alapuolella) sekä tiedostoissa on kontrollerin ja näkymän asioita samassa (itselläkin oli/on vieläkin).
Mallit (& CRUD) on mielestäni suht hyvällä mallilla 👍
Näkymistä sen verran että Luokkaview.php on tyhjä? Tulossa jotain?
Ei varsinaisesti toiminnallisuuteen vaikuttava, mutta itseäni hieman häiritsevä on se että kirjautum.&rekister. salasana kenttä on normaali tekstikenttä eikä password -tyyppinen, jolloin kirjaimet olisivat palluroita..
KirjauduUlos.php:ssä voisi myös olla Meta Refresh, joka siirtäisi käyttäjän esim. 5 sekunnin jälkeen takaisin etusivulle.
Luokkienhallinta toiminnallisuus on toimivaa, mutta sallii syötteen sellaisenaan (XSS -aukko), esim. <b>testi</b> näyttää tekstin lihavoituna jne, tämän voisi esim. htmlspecialchars -functiolla korjata.
Koodia voisi siistiä ja MVC:n mukaisuutta parannella, niin ihan hyvä siitä tulee ;)
ZIP 14.4.14 n. klo 22
MVC näyttää olevan vielä vähän kesken, ja nyt esim. sivuissa on 2 kertaan alkurimpsut (
<html>-tagit jne), kuten tässä (kuvan alapuolella) sekä tiedostoissa on kontrollerin ja näkymän asioita samassa (itselläkin oli/on vieläkin).Mallit (& CRUD) on mielestäni suht hyvällä mallilla 👍
Näkymistä sen verran että Luokkaview.php on tyhjä? Tulossa jotain?
Ei varsinaisesti toiminnallisuuteen vaikuttava, mutta itseäni hieman häiritsevä on se että kirjautum.&rekister. salasana kenttä on normaali tekstikenttä eikä password -tyyppinen, jolloin kirjaimet olisivat palluroita..
KirjauduUlos.php:ssä voisi myös olla Meta Refresh, joka siirtäisi käyttäjän esim. 5 sekunnin jälkeen takaisin etusivulle.
Luokkienhallinta toiminnallisuus on toimivaa, mutta sallii syötteen sellaisenaan (XSS -aukko), esim.
<b>testi</b>näyttää tekstin lihavoituna jne, tämän voisi esim. htmlspecialchars -functiolla korjata.Koodia voisi siistiä ja MVC:n mukaisuutta parannella, niin ihan hyvä siitä tulee ;)