[Bug 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Andre Klapper changed: What|Removed |Added See Also||https://bugzilla.wikimedia. ||org/show_bug.cgi?id=62435, ||https://bugzilla.wikimedia. ||org/show_bug.cgi?id=71534 -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Kunal Mehta (Legoktm) changed: What|Removed |Added CC||federicol...@tiscali.it --- Comment #17 from Kunal Mehta (Legoktm) --- *** Bug 70384 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 --- Comment #16 from bs...@wikimedia.org --- It would say 'username removed' for deleted/suppressed users. We could just hide the notifications completely in these two cases -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Molly White changed: What|Removed |Added CC||molly.whi...@gmail.com --- Comment #15 from Molly White --- Created attachment 15853 --> https://bugzilla.wikimedia.org/attachment.cgi?id=15853&action=edit Screenshot of notification in oversighted edit Just noting that this also affects oversighted edits on pages that have not been deleted. It does not appear that any data is leaking, at least not when only the edit summary/edit itself are oversighted, but it is pretty ugly. I'm not sure what would happen if the username is also suppressed. -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 --- Comment #14 from Kunal Mehta (Legoktm) --- (In reply to Redrose64 from comment #13) > This problem also occurs if the page was deleted, and selected revisions > undeleted. This is because whenever you delete a page and undelete it the page id changes, which is what Echo stores internally, so it thinks the page no longer exists. -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Redrose64 changed: What|Removed |Added CC||m...@redrose64.plus.com --- Comment #13 from Redrose64 --- This problem also occurs if the page was deleted, and selected revisions undeleted. If you have admin rights on en.wp, have a look at [1] which is shown in notifications as "Your edit on [No page] has been reverted by Thecounciloflions. [No page]". The page concerned was [[en:Cat Creek]] [1] http://en.wikipedia.org/w/index.php?title=Special:Undelete&target=Cat+Creek×tamp=20140522185355&diff=prev -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Andre Klapper changed: What|Removed |Added Blocks||62435 -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Kunal Mehta (Legoktm) changed: What|Removed |Added CC||sybga...@gmail.com --- Comment #12 from Kunal Mehta (Legoktm) --- *** Bug 61426 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Kunal Mehta (Legoktm) changed: What|Removed |Added CC||legoktm.wikipe...@gmail.com --- Comment #11 from Kunal Mehta (Legoktm) --- (In reply to Andrew Green from comment #9) > > We could also just access the archive table directly, but doing that in a > notification formatter would be ugly. The formatter wouldn't need to do it, it would be done by EchoEvent::getTitle() (In reply to Matthew Flaschen from comment #10) > Also note that this may have security ramifications. RevisionDelete allows > suppressing log entries (which I believe, but am not positive), includes the > deletion log. See https://www.mediawiki.org/wiki/RevisionDelete It does. Though what's most common is to just delete the page at a suppression level which doesn't create a delete log entry/ -- I think Echo should just store the page title+namespace along with page id, and fallback upon that if the page no longer exists. We would need to implement something to ensure the page wasn't suppressed, but I don't think that would be super difficult. This would handle deletion and undeletion gracefully as well as the general case of the page was just deleted. -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Matthew Flaschen changed: What|Removed |Added CC||rkald...@wikimedia.org --- Comment #10 from Matthew Flaschen --- (In reply to Andrew Green from comment #9) > - Implement in core a simple interface for getting the last title of a > deleted article from the pageId. (Apologies if it's already there and I > didn't see it.) Then add code to Echo for fetching a title object or title > text if the page represented by the 'title' parameter has been deleted. Also note that this may have security ramifications. RevisionDelete allows suppressing log entries (which I believe, but am not positive), includes the deletion log. See https://www.mediawiki.org/wiki/RevisionDelete You should talk to someone familiar with RevisionDelete and oversight to make sure any solution to this does not cause any undesired data leaks. -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 --- Comment #9 from Andrew Green --- Just to elaborate a bit on what Sage said ^ , this issue came up with regard to a notification in the Education Program extension. See: https://git.wikimedia.org/blob/mediawiki%2Fextensions%2FEducationProgram/520430baeb85384bab46442ef654ec043a4c0b5b/includes%2Fnotifications%2FCourseTalkNotification.php and https://git.wikimedia.org/blob/mediawiki%2Fextensions%2FEducationProgram/520430baeb85384bab46442ef654ec043a4c0b5b/includes%2Fnotifications%2FCourseTalkFormatter.php In a nutshell, when the notification is triggered, the extension *does* send a revid in the 'extras' parameter. Maybe I'm missing something, but the only place I see in core to get to the title of a deleted article from the revid is ApiQueryDeletedrevs. We could call that API using DerivativeRequest, but in addition that API checks for the "deletedhistory" user right, not something a notification should try to hack around. We could also just access the archive table directly, but doing that in a notification formatter would be ugly. As far as I can see, there are two real options: - Implement in core a simple interface for getting the last title of a deleted article from the pageId. (Apologies if it's already there and I didn't see it.) Then add code to Echo for fetching a title object or title text if the page represented by the 'title' parameter has been deleted. - Make Echo store the title text somewhere in the notification, so that it can be retrieved by a formatter if the page has been deleted. These options have different implications with regard to performance and what the user sees if a page is moved. It's true we could implement workarounds, but I think a proper fix is Echo's domain, with changes possibly needed in core depending on the approach. -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Sage Ross changed: What|Removed |Added Priority|Normal |High Severity|normal |major --- Comment #8 from Sage Ross --- I'm bumping this up to High again, because Echo's inability to handle deleted pages is a potential problem for every other extension that tries to implement Notifications. As I understand it, Echo typically stores the pageID rather than the plain title of a page when it records a Notification, and if a page gets deleted then it doesn't have permission to access the properties of the deleted page, such as the title. Finding a way to let Echo access the titles for deleted pageIDs might be a solution. -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Sage Ross changed: What|Removed |Added CC||andrew.green...@gmail.com, ||rages...@gmail.com See Also||https://bugzilla.wikimedia. ||org/show_bug.cgi?id=58234 -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 --- Comment #7 from John Mark Vandenberg --- (In reply to comment #5) > Changing priority to normal, lowering severity to normal since that's not > that > annoying a bug (though I've just experienced it). I disagree. I see this regularly, and it is infuriating to know there is a deleted page, but need to go poking around in the watchlist of days ago to find it. In the case of Wikidata, after finding which Q it was from the watchlist, you need to use whatlinkshere to guess what the Q's label was. -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 MZMcBride changed: What|Removed |Added CC||b...@mzmcbride.com --- Comment #6 from MZMcBride --- I just noticed this on the English Wikipedia ("Salinger (film) was linked from [No page]."). -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Tomasz W. Kozlowski changed: What|Removed |Added Priority|Unprioritized |Normal Severity|major |normal --- Comment #5 from Tomasz W. Kozlowski --- Changing priority to normal, lowering severity to normal since that's not that annoying a bug (though I've just experienced it). -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Kunal Mehta (Legoktm) changed: What|Removed |Added CC||swall...@wikimedia.org --- Comment #4 from Kunal Mehta (Legoktm) --- *** Bug 58759 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Quiddity changed: What|Removed |Added CC||jay...@gmail.com --- Comment #3 from Quiddity --- *** Bug 54525 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 --- Comment #2 from sp...@wikimedia.org --- Prioritization and scheduling of this bug is tracked on Mingle card https://mingle.corp.wikimedia.org/projects/flow/cards/223 -- You are receiving this mail because: You are the assignee for the bug. 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 50829] Echo notifications show [No page] instead of pagename if the page was deleted
https://bugzilla.wikimedia.org/show_bug.cgi?id=50829 Bartosz Dziewoński changed: What|Removed |Added Status|UNCONFIRMED |NEW CC||matma@gmail.com Ever confirmed|0 |1 Severity|normal |major --- Comment #1 from Bartosz Dziewoński --- This affects other notifications as well, I've seen it happen to link notifs. -- You are receiving this mail because: You are the assignee for the bug. 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