Koodikatselmointi #1

[Granigan]

Koodikatselmointi #1

Projekti haettu 11:30 13.08.2018.

Palaute

Projekti on hyvällä mallilla ja toteuttaa annettua hyvän rakenteen ja koodauksen mallia erinomaisesti. Applikaation käyttö on selkeää ja suoraviivaista. Toivottavasti tehtävien yhdistäminen prosesseiksi onnistuu jatkossa yhtä hyvin! :)

Pikku vikoja, helppo korjata

• Prosessi-sivulla sarakkeen nimenä on 'Viimeksi muokattu', vaikka kyseessä taitaa olla prosessin alku- ja loppuajat. Sarakkeet ovat kuitenkin helppo muokata /templates/prosessi/list.html -tiedostosta, ja kyseessä on vain pieni käyttisasia.

• Ylläolevaan liittyen, osassa näkymistä on ehkä hieman ylimääräisesti näkyvissä luontipäivä ja muokkaustietoa. Jos ne eivät ole olennaisia näkymälle, ehkä esim. juuri työprosessilistauksessa, ne tekevät sivusta helposti tarpeettoman sekavan. Tämä on kuitenkin pitkälti kiinni käyttötarpeesta, ja sittenkin makuasia. :)

• /auth/forms.py tiedoston kolmannesta luokasta (class KayttajaLisaysLomake(FlaskForm)) puuttuu csrf = False -määrittely.

• Käyttäjänlisäys-sivulla syötteen validointi ei toimi, esim. käyttäjänimi voi olla vain yhden merkin mittainen. Validointisäännöt on määritelty, mutta /auth/views.py -tiedostosta puuttuu kaavakkeen tietojen validointikutsu. Tämä on kuitenkin tehty oikein esim. /tehtavat/views.py -tiedostossa, joten eikun sieltä mallaamalla korjaamaan.

• Myöskään työprosessin nimeä ei validoida - joskin tämä on nähtävästi tarkoituksella kommentoitu ulos /prosessi/views.py -tiedostossa riveillä 61 ja 62..? Oletettavasti toiminnallisuuden haluttu toteutus on siis tuossa kohden hieman kesken - parempi näin että kokonaisuus kuitenkin toimii.

• Ylimääräiset init.py -tiedostot jokaisessa olioalihakemistossa kummastuttavat. Nämä voinee siivota pois huoletta.

Loppukommentti

Projekti on edennyt vaaditun mukaisesti ja tästä on varmasti hyvä jatkaa eteenpäin. Tsemppiä, menestystä ja hyvää lukuvuoden alkua!