Formal Technical Review
Aika: 9.11. klo 12.15-13
Paikka: Exactum, A319
Läsnä: Pirjo Tervonen, Tuija Åkerblom, Maruan Khoury, Ilkka Manner, Mikko Hakila,
Kirsi Ylänne, Sini Ruohomaa, Juha Taina

1. Kokouksen avaaminen

   Puheenjohtaja Ilkka Manner avasi kokouksen 12.17.

2. Dokumentin läpikäynti (korjauslista)

   2.1 Johdanto - Pirjo

   Ei korjattavaa/huomautettavaa.

   2.2 Järjestelmän kokonaiskuvaus - Ilkka

   Pakkauskomponentti-kohdan toinen lause (Komponentti tuottaa myös...) lisättävä
   kaikkiin komponenttiesittelyihin.

   2.3 Tietokannan rakenne - Tuija

   Taulukuvaukset epäyhtenäisiä, "Arvon tuottaa käyttäjä" -muodossa vs. "määrittää".
   Taulujen nimet johdannossa eri järjestyksessä kuin aliluvuissa.
   Integer-arvojen selitteet puuttuvat joistakin tauluista.
   Feedbackmode module-taulussa "annetaanko palautetta vai ei" ei konsistentti.

   2.4 MainServlet-komponentti - Tuija 
 
   Ei korjattavaa/huomautettavaa.

   2.5 Tietokantakyselyiden suorituskomponentti - Kirsi

   Sivu 8 kohta 5.1 tarvitaan lisätietoa siitä, mitä DatabaseAccess-luokka tekee
   SQL-lauseille.
   Sivu 9 kuvan alla "..-luokassa on kaksi metodia, " - on konstruktori ja kaksi
   metodia.
   Sekvenssikaaviossa ei eroteltu haku- ja päivityspyyntöä.
   Kuvan 5 sijainti huono.
   Kuvan 6 tekstiviittauksesta puuttuu sivunumero. 
  
   2.6 Navigointi- ja Käyttäjän syötteen validointikomponentti - Pirjo
 
 6.1.1 "metodien kuvaus" - yksikköön
  6.2.1 sama.

  2.7 Ryhmäkomponentti - Kirsi

  Sivu 12 listan 4. kohta epäselvä ("ryhmän näyttäminen omistajalle").
  Sivu 13 sekvenssikaavio: metodikutsut "yhtä jäljessä", processRequest() tulisi
  olla getHTML() jne. HTMLFilter-olion merkintä näkyviin poikkeavaa.
  Sivu 16 "Metodi tuo" -> hakee kahdessa ensimmäisessä metodikuvauksessa.
  Sivu 16 kolmas metodikuvaus passiivimuotossa, kun muualla aktiivimuodossa. Kaipaa
  tarkennusta: "samalla uuden ryhmän luoja liitetään uuteen ryhmään".
  Sivu 17 metodikuvaus kaipaa lisäselvitystä erikoistapauksesta, kun ryhmän
  viimeinen/ainoa jäsen poistuu.
  Sivu 15 yhtenäistys kuvaviittaukseen: jos kuva samalla sivulla, sivunumeroa ei
  mainita.  

  2.8 Pakkauskomponentti - Ilkka

  Sivu 18 - sekvenssikaaviossa metodinimet "yhtä jäljessä" kuten sivulla 13 kohdassa
  2.7.
  Sivu 18 kuva 12 HTMLFilter-olion merkintä näkyviin poikkeavaa.
  Sivu 21 metodikuvaus, pakkauksen/käyttäjän "nimi" tulisi olla tunniste.

  2.9 Palautelomakekomponentti - Maruan

  Kuvaviittaus yhtenäiseksi: sivulla 22 sivunumero pois, kuva samalla sivulla.
  Sivu 24 storeFeedbackin paluuarvon tyyppi outo (boolean voisi riittää).

  2.10 Palauteyhteenvetokomponentti - Maruan

  Kuva 17 pahassa paikassa.
  Sivu 24 Lause "Tarkemmat kuvaukset kysymyksistä..." ei päde, kysymykset 
  liitteessä 4.  
  Sivu 24 "sivulla" pois.
  Sivu 25 käyttöliittymäkuva selventäisi luokan kuvausta.
  Sivu 27 metodikuvaukset kaipaavat lisäselvitystä siitä, miten laskenta tapahtuu.
  Sivu 27 caculateTaskAverage module-parametri tulisi olla moduleId. 
  Sivu 27 calculateModuleAverage metodin kuvaus päättyy )-merkkiin, poistettava.

  2.11 QTI-muunnoskomponentti - Mikko

  Sivu 29 QTI-luokan kuvauskappaleen viimeinen lause ei pidä paikkaansa.
  Sivu 30 private-metodit puuttuvat luokkakaaviossa vaikka kuvattu.
  Sivu 30 "luokan sisäisiä metodeja" -> yksityisiä.
  Sivu 30 makeQti-metodin parametrinä parempi ResultSet.
  Sivu 31 makeFile-metodin parametri taskdata on parametrit-taulussa väärin, tulisi
  olla qtiTasks.

3. Dokumentin hyväksyminen / hylkääminen

   Dokumentti päätettiin hyväksyä muutoksin yksimielisesti.

4. Kokouksen päättäminen

   Puheenjohtaja päätti kokouksen 12.51.