[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 James Forrester jforres...@wikimedia.org changed: What|Removed |Added Status|PATCH_TO_REVIEW |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 --- Comment #7 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 119204 merged by jenkins-bot: Show blockedtext message in edit notices https://gerrit.wikimedia.org/r/119204 -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 James Forrester jforres...@wikimedia.org changed: What|Removed |Added Target Milestone|--- |VE-deploy-2014-03-27 -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 --- Comment #4 from Roan Kattouw roan.katt...@gmail.com --- (In reply to Alex Monk from comment #3) James suggested I tackle this bug. I messed around with the API for a little bit and assuming I didn't miss anything: I think that if we tried to fill out the 'blockedtext' message properly on the client we'd have to: * Query the API for the user's blockinfo to determine whether or not they're blocked. If they are: ** Query the API for more info about this block which meta=userinfouiprop=blockinfo doesn't provide but we need to substitute into the message (expiry, user, timestamp). ** Query the API to parse the blockedtext message because it can't be done on the client (even the default message includes stuff mw.msg can't handle) ** Substitute the data we have into the message (and we still won't have $3 - the current IP) *** Mess around with dates/times because they won't be in the right format. Another way we could do this is return the full warning from action=visualeditorpaction=parse (therefore adding no extra API requests) which would cause the client to stop loading. I don't like this idea because obviously that paction is not intended for checking this kind of thing... Or we could ignore the core blockedtext message and use our own Your username or IP address has been blocked, possibly with the info that just meta=userinfouiprop=blockinfo provides (block ID, performer, reason). Or we could add a new API module/paction that can be queried once and provides all the relevant info. I don't like this idea either but it seems the nicest. We already put random junk in the paction=parse response, I wouldn't be opposed to adding blocked-ness to that. Another paction would be OK too, but then we'd have to make two API requests. There are significant speed benefits in combining everything into one API request. I think we should stop kidding ourselves that paction=parse is a clean, only-does-one-thing action, and perhaps rename it if we want, rather than trying to split it into separate actions, because the latter would just make client-side performance worse. -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 --- Comment #5 from Alex Monk kren...@gmail.com --- (In reply to Roan Kattouw from comment #4) We already put random junk in the paction=parse response, I wouldn't be opposed to adding blocked-ness to that. Another paction would be OK too, but then we'd have to make two API requests. There are significant speed benefits in combining everything into one API request. I think we should stop kidding ourselves that paction=parse is a clean, only-does-one-thing action, and perhaps rename it if we want, rather than trying to split it into separate actions, because the latter would just make client-side performance worse. Yes, since writing that comment I decided to just add it in to paction=parse, especially since talking to James - he wants this to be an edit notice like the others. I probably should have left a new comment about it. The problem now is coming up with a nice way to fit the autoblockedtext message (blockedtext is okay) in to the edit notices popup. -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 Alex Monk kren...@gmail.com changed: What|Removed |Added Assignee|jforrester+veteambztickets@ |kren...@gmail.com |wikimedia.org | -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 --- Comment #6 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 119204 had a related patch set uploaded by Alex Monk: Show blockedtext message in edit notices https://gerrit.wikimedia.org/r/119204 -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 Gerrit Notification Bot gerritad...@wikimedia.org changed: What|Removed |Added Status|ASSIGNED|PATCH_TO_REVIEW -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 James Forrester jforres...@wikimedia.org changed: What|Removed |Added Assignee|krinklem...@gmail.com |jforrester+veteambztickets@ ||wikimedia.org -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 Alex Monk kren...@gmail.com changed: What|Removed |Added CC||kren...@gmail.com --- Comment #3 from Alex Monk kren...@gmail.com --- James suggested I tackle this bug. I messed around with the API for a little bit and assuming I didn't miss anything: I think that if we tried to fill out the 'blockedtext' message properly on the client we'd have to: * Query the API for the user's blockinfo to determine whether or not they're blocked. If they are: ** Query the API for more info about this block which meta=userinfouiprop=blockinfo doesn't provide but we need to substitute into the message (expiry, user, timestamp). ** Query the API to parse the blockedtext message because it can't be done on the client (even the default message includes stuff mw.msg can't handle) ** Substitute the data we have into the message (and we still won't have $3 - the current IP) *** Mess around with dates/times because they won't be in the right format. Another way we could do this is return the full warning from action=visualeditorpaction=parse (therefore adding no extra API requests) which would cause the client to stop loading. I don't like this idea because obviously that paction is not intended for checking this kind of thing... Or we could ignore the core blockedtext message and use our own Your username or IP address has been blocked, possibly with the info that just meta=userinfouiprop=blockinfo provides (block ID, performer, reason). Or we could add a new API module/paction that can be queried once and provides all the relevant info. I don't like this idea either but it seems the nicest. Thoughts? -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 James Forrester jforres...@wikimedia.org changed: What|Removed |Added See Also||https://bugzilla.wikimedia. ||org/show_bug.cgi?id=53009 -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 52004] VisualEditor: Blocked users are not blocked from using VE (just from saving); should instead show Blockedtext with log entry
https://bugzilla.wikimedia.org/show_bug.cgi?id=52004 James Forrester jforres...@wikimedia.org changed: What|Removed |Added Status|NEW |ASSIGNED Summary|VisualEditor: Block log |VisualEditor: Blocked users |entry is not shown to |are not blocked from using |blocked users trying to |VE (just from saving); |edit. Editing appears to|should instead show |work until they try to save |Blockedtext with log ||entry -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l