[05:21:08] git worktree [22:51:41] TimStarling: actually, the db is wrong, msg_text is empty for me (I was looking at the wrong entity at first) [22:52:11] should have been "Ccc", so it the creation path is broken [22:52:16] yeah I actually have two elections, one I created before screen sharing and one after, and one of them is wrong and one is right [22:52:42] I'm looking at the creation code now, there are some suspicious things [22:54:04] the HTML of the creation form is like this: https://paste.tstarling.com/p/IxtyJz.html [22:54:30] see the missing data attribute on the third td? I'm looking into that [23:06:53] editing the poll fixes the db, but the msg still won't show [23:08:35] see HTMLFormRadioRangeColumnLabels [23:16:25] there is a JS problem going on with me [23:17:15] TimStarling: try turning off JS [23:17:44] * AaronSchulz looks at ext.securepoll.htmlform [23:18:20] interesting [23:18:53] I should have spotted this since I already did a curl fetch of the create page and noticed that it was different to what I was seeing in the browser [23:42:14] there's a cells["+1"] but changeHandler looks for cells[1] [23:47:12] TimStarling: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/SecurePoll/+/644362 seems to work [23:47:25] ok, I should do other stuff now :) [23:47:46] eh, if it works, why is it WIP? [23:48:16] I tested it with an edit and a new poll, but quickly...someone should look more thoroughly [23:48:25] I guess I can remove WIP though [23:50:02] in the PHP-generated HTML, there is +1, but JS always shows 1. I was working on fixing that with an addSign() function in JS [23:51:03] I think I'll combine that idea with your patch [23:51:10] I'll probably approve your patch though if it works