Koodikatselmointi - Githubissues

Ohjelma ladattu 30.3 klo 22.10.

-README:n linkit keskustelunaiheisiin ja keskusteluun eivät toimineet. Muut linkit toimivat, jote toimimattomien linkkien sivuille pääsi niiden kautta. -dokumentaatio on hyvin huolellella tehty ja selkeää luettavaa. -käyttäjät eivät voi itse luoda itselleen tunnuksia käyttötapauskaavion mukaan, onko tämä hyvä keskustelufoorumille? saatat haluta että adminin ei tarvitse luoda kaikille uusille käyttäjille tunnuksia. -keskusteluiden luomisella tulee varmasti vielä painike sivulle. -kannattaa kiinnittää huomiota foorumin valmistamisessa sivutukseen jos haluat toteuttaa foorumit joissa esim. sivulle mahtuu kerralla kymmenen aihetta tai viestiä. Tämän toteuttaminen oli meille Tietokannat-kurssilla vaikeaa joten päädyimme itse käyttämään ratkaisua, jossa meillä selain-linkki näytti tältä: /keskustelu/1/sivu/2. Tämä näytti hieman rumalta, mutta pystyimme jakamaan foorumeita useammille sivuille kun se oli mielekästä. Esimerkissä siis keskustelun 1 sivun 2 tarkastelu, jossa ensimmäisellä sivulla vaikka 10 viestiä ja toisella sivulla seuraavat viisi. Nykyisessä muodossaan tietyn keskustelun kaikki viestit näkyisivät samalla sivulla kaiketi? -Siirtääkö nykyisessä koodissa oleva viestin luonti käyttäjän keskustelut-osioon? Jos näin, olisiko hyvä päätyä takaisin samaan keskusteluun jonne viesti kirjoitettiin.

public static function addViesti($id){
    $params = $_POST;
    $viesti = new Viesti(array(
      'keskustelu_id' => $id,
      'kayttaja_id' => 1,
      'sisalto' => $params['content']
    ));

    $viesti->save();
    Redirect::to('/keskustelut');   // **tämä tässä**
}

-On hyvä että keskustelunalueella lisättäessä uusi keskustelu täytyy keskustelunaloittajan lisätä myös viesti. Pystyykö admin poistamaan keskustelusta ainoan sen sisältämän viestin jättämällä tyhjän keskustelun? -Relaatiotietokantakaavio, käsitekaavio ja käyttötapausdiagrammi hyvän näköisiä :) -Kirjoitusvirheitä dokumentaatiossa: Tietokohde Viestissä: sisältä ja vastaa? "lukijalla näytetään automaattosesti"-johdannossa. -Hieno homma, mukavaa jatkoa!

JaakkoKoli / Tsoha-Bootstrap

Koodikatselmointi #1