jenkins-bot has submitted this change and it was merged. Change subject: Use same link for flow & flowusertalk summary notification ......................................................................
Use same link for flow & flowusertalk summary notification Per https://phabricator.wikimedia.org/T125119#2086639 Bug: T125119 Change-Id: Id504624a376110960ea103d92a56f2ee7b7670fb --- M includes/Notifications/SummaryEditedPresentationModel.php 1 file changed, 1 insertion(+), 7 deletions(-) Approvals: Sbisson: Looks good to me, approved jenkins-bot: Verified diff --git a/includes/Notifications/SummaryEditedPresentationModel.php b/includes/Notifications/SummaryEditedPresentationModel.php index 1290e74..884eedc 100644 --- a/includes/Notifications/SummaryEditedPresentationModel.php +++ b/includes/Notifications/SummaryEditedPresentationModel.php @@ -23,13 +23,7 @@ if ( $this->isBundled() ) { return array( $this->getBoardLink() ); } else { - $links = array( $this->getAgentLink() ); - if ( $this->isUserTalkPage() && $this->event->getExtraParam( 'prev-revision-id' ) !== null ) { - $links[] = $this->getDiffLink(); - } else { - $links[] = $this->getBoardLink(); - } - return $links; + return array( $this->getAgentLink(), $this->getBoardLink() ); } } -- To view, visit https://gerrit.wikimedia.org/r/275385 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id504624a376110960ea103d92a56f2ee7b7670fb Gerrit-PatchSet: 2 Gerrit-Project: mediawiki/extensions/Flow Gerrit-Branch: master Gerrit-Owner: Matthias Mullie <[email protected]> Gerrit-Reviewer: Matthias Mullie <[email protected]> Gerrit-Reviewer: Sbisson <[email protected]> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
