----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113347/ -----------------------------------------------------------
(Updated Oct. 19, 2013, 10:26 a.m.) Status ------ This change has been marked as submitted. Review request for Bodega. Repository: bodega-server Description ------- With discourse 0.9.7 the bodega users aren't able to login anymore. The patch fixes the issue. Diffs ----- sql/discourse.plsql fdc0f01 Diff: http://git.reviewboard.kde.org/r/113347/diff/ Testing ------- This one is tricky in order to reproduce it. There are 2 use cases which end up with a complete different result. ======> Use case 1 (discourse works) You used to have a discourse installation of github.com/terietor/discourse and then you updated the installation to github.com/aseigo/discourse and it works. It works because terietor's discourse had written the write values in the table users. So when some columns moved from the users table to the user_stats table your database had the *right* values. =====> Use case 2 (discourse fails) You *don't* have an existing discourse installation and you setup github.com/aseigo/discourse. You start the whole process from the begin and your users fail to login due to the "days_visited" error. Why? Unlike the first use case the users table *doesn't* contain the columns of the user_stats table so the user_stats table it never gets populated. Discourse assumes that the table contains the right values and when the code is being executed the object is nill(NULL) so it fails. Thanks, Giorgos Tsiapaliokas
_______________________________________________ Active mailing list Active@kde.org https://mail.kde.org/mailman/listinfo/active