X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2Frcfeed%2FIRCColourfulRCFeedFormatter.php;h=10ba83fc1980283e6158650739943265f33d86da;hb=b6b8526a04c072ed8a4b283f2c4ece8f4edda861;hp=0efcebf9a38c4758ab2b454c98b4cfc2c97ecbce;hpb=a36c567fa636af4a7c96cdeff521b64d6f291f22;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/rcfeed/IRCColourfulRCFeedFormatter.php b/includes/rcfeed/IRCColourfulRCFeedFormatter.php index 0efcebf9a3..10ba83fc19 100644 --- a/includes/rcfeed/IRCColourfulRCFeedFormatter.php +++ b/includes/rcfeed/IRCColourfulRCFeedFormatter.php @@ -33,6 +33,11 @@ class IRCColourfulRCFeedFormatter implements RCFeedFormatter { global $wgUseRCPatrol, $wgUseNPPatrol, $wgLocalInterwikis, $wgCanonicalServer, $wgScript; $attribs = $rc->getAttributes(); + if ( $attribs['rc_type'] == RC_CATEGORIZE ) { + // Don't send RC_CATEGORIZE events to IRC feed (T127360) + return null; + } + if ( $attribs['rc_type'] == RC_LOG ) { // Don't use SpecialPage::getTitleFor, backwards compatibility with // IRC API which expects "Log". @@ -56,7 +61,7 @@ class IRCColourfulRCFeedFormatter implements RCFeedFormatter { $query .= '&rcid=' . $attribs['rc_id']; } // HACK: We need this hook for WMF's secure server setup - Hooks::run( 'IRCLineURL', array( &$url, &$query, $rc ) ); + Hooks::run( 'IRCLineURL', [ &$url, &$query, $rc ] ); $url .= $query; } @@ -84,7 +89,9 @@ class IRCColourfulRCFeedFormatter implements RCFeedFormatter { ) ); $flag = $attribs['rc_log_action']; } else { - $comment = self::cleanupForIRC( $attribs['rc_comment'] ); + $comment = self::cleanupForIRC( + CommentStore::newKey( 'rc_comment' )->getComment( $attribs )->text + ); $flag = ''; if ( !$attribs['rc_patrolled'] && ( $wgUseRCPatrol || $attribs['rc_type'] == RC_NEW && $wgUseNPPatrol ) @@ -124,8 +131,8 @@ class IRCColourfulRCFeedFormatter implements RCFeedFormatter { */ public static function cleanupForIRC( $text ) { return str_replace( - array( "\n", "\r" ), - array( " ", "" ), + [ "\n", "\r" ], + [ " ", "" ], Sanitizer::decodeCharReferences( $text ) ); }