Page MenuHomePhabricator

Don't add a history link for log entries
Closed, ResolvedPublic1 Estimated Story Points

Description

The page might not exist (eg if the log entry was for deletion or something that doesn't affect pages at all, like rights/rights)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 661103 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/GlobalWatchlist@master] Don't add a history link for log entries

https://gerrit.wikimedia.org/r/661103

DannyS712 set the point value for this task to 1.

This would match what core does (I got sucked into looking through the history to find when this was introduced - in the "Initial revision" commit at d82c14fb4fbac288b42ca5918b0a72f33ecb1e69 it was already the case, see if ( !$e->islog ) { in Skin::recentChangesBlockGroup - https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/core/+/d82c14fb4fbac288b42ca5918b0a72f33ecb1e69/includes/Skin.php#1398 )

Change 661103 merged by jenkins-bot:

[mediawiki/extensions/GlobalWatchlist@master] Don't add a history link for log entries

https://gerrit.wikimedia.org/r/661103

DannyS712 moved this task from In progress to Done on the MediaWiki-extensions-GlobalWatchlist board.
DannyS712 set Final Story Points to 2.