Closed jkpj closed 13 years ago
Mergeän tämän huomenna, kun olet saanut nuo loput pikkumuutokset tehtyä.
Vieläkö tästä puuttuu jotain? Mahdoitko unohtaa muuttaa show-metodin nimelle index controllerissa?
Aika paljon löytyi virheitä/epäselvää:
Menin käynnissä olevan kilpailun etusivulle ja klikkasin Tulokset. Tuli nil-virhe (johtuu siitä, että series.start_time voi olla nil):
app/helpers/application_helper.rb:349:in `result_rotation_series_list'
Tästä huomasin, että tuossa metodissa on paljon turhaa koodia. Sorttauksen ja rajoituksen voisi tehdä:
race.series.order('start_time desc').limit(3)
Minua hieman huolestuttaa se, että nyt on tulossa ominaisuus, jonka lisäarvo on kyseenalainen mutta käytettävyys huono. Tosin ehkä tässä vaan on kyse jostain bugista, joka hämää pahasti.
Jep, käytettävyys oli huono, aika paljon viimeistelyä puuttui ja puuttuu varmaan jotain vieläkin. Eniten näkyvinä bugeina puutteet tilanteessa, jossa ei ole mitään näytettäväksi.
ongelma:
# FIXME sql generated is: SELECT "series".* FROM
# "series" WHERE ("series".race_id = 10)
# ORDER BY name, start_time desc LIMIT 3
# where does name come from?
race.series.order('start_time desc').limit(result_rotation_cookie.to_i).each do |s|
result_rotation_series_list << series_competitors_path(s) if s.running?
end
result_rotation_series_list
Nyt en ymmärrä, miten rotaation saa päälle. Kun cookieta ei ole aluksi asetettu (tyhjä kone), niin result_rotation_series_list-metodissa tehdään limit(0). Siitä seuraa tyhjä result_rotation_list, josta seuraa se, ettei Tuloskehä-välilehteä näy.
Totta, ei mitenkään. Korjaus autotestissä pyörimässä.
24.03.2011 23:14, karhatsu kirjoitti:
Nyt en ymmärrä, miten rotaation saa päälle. Kun cookieta ei ole aluksi asetettu (tyhjä kone), niin result_rotation_series_list-metodissa tehdään limit(0). Siitä seuraa tyhjä result_rotation_list, josta seuraa se, ettei Tuloskehä-välilehteä näy.
Nyt kun yritän käynnistää, niin tulee "Tuloskehää ei voida käynnistää, koska ei ole käynnissä olevia kilpailuja."
Tämän pitäisi olla korjattu ennen kuin sain viestisi. (Arvelen että kyseessä oli tuloskehän lopettamisesta tullut virheellinen viesti, ei käynnistämisestä)
24.03.2011 23:27, karhatsu kirjoitti:
Nyt kun yritän käynnistää, niin tulee "Tuloskehää ei voida käynnistää, koska ei ole käynnissä olevia kilpailuja."
Nyt löytyi se bugi. Se unscoped tekee myös sellaisen jutun, että race.series.where():sta tulee sama kuin Series.where(). Eli race-tieto katoaa myös. Korjaan tämän itse suoraan.
Okei, nyt toimii. Suljen tämän.
Tässä on yhdessä tuloskehä ja tuloskehä-ui eli käyttöliittymä (välilehti) jolla tuloskehän saa startattua ja stopattua. Laitan erikseen siltä varalta että haluat mergetä pala kerrallaan.