Matěj Suchánek has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/326272 )

Change subject: Don't show red diff link
......................................................................

Don't show red diff link

If the log entry refers to a non-existing page, the link is red. In
this case, the diff link only duplicates the main link.

Change-Id: I3f0e6e7dfbaa8cccbac8450162df97f4a58cccdc
---
M special/SpecialAbuseLog.php
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/AbuseFilter 
refs/changes/72/326272/2

diff --git a/special/SpecialAbuseLog.php b/special/SpecialAbuseLog.php
index 69ac5ec..44f9ce9 100644
--- a/special/SpecialAbuseLog.php
+++ b/special/SpecialAbuseLog.php
@@ -464,8 +464,9 @@
 
                if ( !$row->afl_wiki ) {
                        $pageLink = $linkRenderer->makeLink( $title );
-                       if ( $row->afl_rev_id ) {
-                               $diffLink = $linkRenderer->makeLink( $title,
+                       if ( $row->afl_rev_id && $title->exists() ) {
+                               $diffLink = $linkRenderer->makeKnownLink(
+                                       $title,
                                        new HtmlArmor( $this->msg( 
'abusefilter-log-diff' )->parse() ),
                                        array(),
                                        array( 'diff' => 'prev', 'oldid' => 
$row->afl_rev_id ) );

-- 
To view, visit https://gerrit.wikimedia.org/r/326272
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3f0e6e7dfbaa8cccbac8450162df97f4a58cccdc
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/AbuseFilter
Gerrit-Branch: master
Gerrit-Owner: Matěj Suchánek <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to