search

karhatsu / hirviurheilu

Hirviurheilu - tulospalvelu Metsästäjäliiton urheilulajeille
http://www.hirviurheilu.com
4 stars 1 forks source link

Latauksessa virhe: lähtöerien väli (sekuntia) ei ole luku. #40

Closed jkpj closed 13 years ago

jkpj commented 13 years ago

Yritin ladata Jupon SM-esikisat omalle koneelle, tuli virheilmoitus "Lähtöerien väli (sekuntia) ei ole luku."

Saattaa olla oman koneen migraatioissa virhe, mutta kirjaan siltä varalta että on jotain muuta.

karhatsu commented 13 years ago

Huomasin saman. Tämä johtuu siitä batch-muutoksesta, jonka migraatiotiedosto oli vähän virheellinen. Tietokantaan luotiin uusi sarake, jolle ei sanottu :null => false. Toisaalta kuitenkin model-tasolla tehtiin validaatio, että kenttä on pakollinen. Tästä seuraa se, että vanhojen tietojen kanssa tulee ongelmia.

Sarake olisi pitänyt lisätä näin: add_column :races, :batch_sarake, :integer, :null => false, :default => jokin_numero

Tuon voisi vielä muuttaa tekemällä uuden migraation, ChangeBatchSarakeInRaces: change_column :races, :batch_sarake, :integer, :null => false, :default => jokin_numero

Jos mietitään vähän tarkemmin, niin parempi itse asiassa olisi, että sarake saisi olla null. Nykyinen kilpailun lisäämisen käyttöliittymä on huonompi kuin aiemmin, koska toimitsija joutuu arpomaan jotain lukua batch-kenttiin, vaikka ei niitä oikeasti tarvitsisi ollenkaan. Täytyy jälleen muistaa se, että tämä ei ole pelkästään isojen kisojen tulospalveluksi tarkoitettu. Parempi olisi, että kentät olisivat vapaaehtoiset ja validointi tarkastaisi, että jos toinen batch-luku on syötetty, niin sitten pitää olla myös toinen. Näin päästäisiin eroon myös kömpelöstä "syötä 0 jos ..."-ohjeesta.

Joka tapauksessa Jupon erän saat korjattua siten, että tallennat tuotantokantaan batch-arvon.

jkpj commented 13 years ago

Ei toimitsijan tarvitse uutta kilpailua perustaessa arpoa lukuja eräasetuksiin, vaan ohjelma laittaa oletuksena 0 (ei eriä käytössä) ja 180 (3 minuutin erien välit). Tyhjä nollan sijasta merkitsemään "erät eivät ole käytössä" ei olisi sekään mitenkään itsestäänselvä, minusta sekin pitäisi selittää.

Eräasetusten tarjoaminen heti on kyllä hiukan luotaantyöntävää ja huononnus niiden kannalta jotka eivät taukoja tarvitse, tulee lomakekammoisille halu juosta karkuun kun silmille lävähtää iso määrä kenttiä.

Käyttöliittymän kannalta yksi tapa parantaa asiaa olisi lisätä nappi "Pidä taukoja lähtöajoissa" tms. joka toisi eräasetukset esiin.

Tarve syöttää itse 0 ja erien väli johon viittaat on tullut esille noiden vanhojen kilpailujen kohdalla, ja siitä pääsee eroon tuolla ehdottamallasi uudella migraatiolla, pitänee varmaan tehdä se.

karhatsu commented 13 years ago

Toki siellä on ne oletusarvot, mutta koko tuo eräkokoasia voi hieman hämmentää joitakin käyttäjiä. Minä voin miettiä tätä joskus myöhemmin lisää.

Nyt voidaan hoitaa tämä sillä uudella migraatiolla eikä muuta tarvitse tehdä. Minä voin tehdä suoraan sen migraation, jos otat sitten pull:n ja testaat.

karhatsu commented 13 years ago

Pushattu mutta ei tätä tarvitse enempää testata. Varmistin, että kannassa arvot muuttuivat ja se riittää.

jkpj commented 13 years ago

Jos teet vielä jossain vaiheessa migraation tuotantoon niin saan imaistua tuon esikisa-aineiston ym. aineistoja paikalliseen testejä varten.

Yksi tapa pitää perusasetukset yksinkertaisina olisi jakaa asetukset perus- ja lisäasetuksiin.