AkademiskaSangforeningen / Akademiska-Sangforeningens-intranat

Anmälningssystem, kalender, you name it.
5 stars 1 forks source link

KK funkar inte mera! #71

Closed ef-akdm closed 11 years ago

ef-akdm commented 11 years ago

Jag har 105€ saldo, men när jag försöker anmäla mej säger den att jag bara har -50€. Jag antar att det är blocken mot att använda KK med negativt saldo som inte fanns tidigare som orsakar buggen.

Bosse upplevde något motsvarande, den sa att han har 0€ fast han borde ha mer än det.

ef-akdm commented 11 years ago

Dvs. #67

Var det så att KK-tabbens felaktiga totalsaldo är fel bara pga att sidan inte är klar?

@KSundstrom

KSundstrom commented 11 years ago

Det var flera (alla?) som upplevde det här när vi testade just under mötet. Det är alltså i praktiken omöjligt att använda KK!

Jag är ledsen, men det här är riktigt hög prio - vi kan inte lansera förrän det är fixat på sätt eller annat...

brunnsbe commented 11 years ago

Jag har hittat felet, har det fixat inom 5-10 min.

Den 23 januari 2013 23:44 skrev Kasper Sundström notifications@github.com:

Det var flera (alla?) som upplevde det här när vi testade just under mötet. Det är alltså i praktiken omöjligt att använda KK!

Jag är ledsen, men det här är riktigt hög prio - vi kan inte lansera förrän det är fixat på sätt eller annat...

— Reply to this email directly or view it on GitHubhttps://github.com/AkademiskaSangforeningen/Akademiska-Sangforeningens-intranat/issues/71#issuecomment-12624174.

KSundstrom commented 11 years ago

Niice!

Den 24 januari 2013 00:02 skrev André Brunnsberg notifications@github.com:

Jag har hittat felet, har det fixat inom 5-10 min.

Den 23 januari 2013 23:44 skrev Kasper Sundström notifications@github.com:

Det var flera (alla?) som upplevde det här när vi testade just under mötet. Det är alltså i praktiken omöjligt att använda KK!

Jag är ledsen, men det här är riktigt hög prio - vi kan inte lansera förrän det är fixat på sätt eller annat...

— Reply to this email directly or view it on GitHub< https://github.com/AkademiskaSangforeningen/Akademiska-Sangforeningens-intranat/issues/71#issuecomment-12624174>.

— Reply to this email directly or view it on GitHubhttps://github.com/AkademiskaSangforeningen/Akademiska-Sangforeningens-intranat/issues/71#issuecomment-12625116.

Kasper Sundström Romgatan 5 A 3 FI–00560 HELSINGFORS

Tel: +358–40–747 16 40 Skype: kaspersundstrom E-mail: kasper.sundstrom@gmail.com

brunnsbe commented 11 years ago

Fixat!

Störande bug, hade att göra med hur databasen jämför NULL värden. Följande kod returnerar ingenting från databasen om EventId är NULL i tabellen fastän där finns rader: SELECT * FROM Transaction WHERE EventId NOT IN ('abc', 'def');

Det beror på att sql:en konverteras till följande: SELECT * FROM Transaction WHERE EventId != 'abc' AND EventId != 'def';

Och som default så returnerar jämförelser mot NULL alltid false (fastän man skulle tycka att NULL är olika 'abc-def').

Så det rätta sättet är att skriva sql:en så här:

SELECT * FROM Transaction WHERE (EventId NOT IN ('abc', 'def') OR EventId IS NULL);