wikimedia/mediawiki-extensions-DiscussionTools
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/DiscussionTools synced 2024-11-24 08:23:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Simplify handling of sections in bundled notification links"

Browse source
This commit is contained in:
jenkins-bot 2022-01-24 23:21:54 +00:00 committed by Gerrit Code Review
parent ea54d68f2d d2405cc11c
commit 9892008135
2 changed files with 9 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
includes/Notifications/DiscussionToolsEventTrait.php
View file

@ -18,6 +18,7 @@ use WikiMap;
*
* @property \EchoEvent $event
* @property \Language $language
* @property \EchoPresentationModelSection $section
*/
trait DiscussionToolsEventTrait {
@ -51,7 +52,6 @@ trait DiscussionToolsEventTrait {
if ( !$this->userCan( RevisionRecord::DELETED_TEXT ) ) {
return null;
}
$title = $this->event->getTitle();
if ( !$this->isBundled() ) {
// For a single-comment notification, make a pretty(ish) direct link to the comment.
// The browser scrolls and we highlight it client-side.
@ -59,6 +59,7 @@ trait DiscussionToolsEventTrait {
if ( !$commentId ) {
return null;
}
$title = $this->event->getTitle();
return $title->createFragmentTarget( $commentId )->getFullURL();
} else {
// For a multi-comment notification, we can't make a direct link, because we don't know
@ -72,23 +73,21 @@ trait DiscussionToolsEventTrait {
// * Mention notifications are *never* bundled.
// Code below tries to avoid assumptions in case this behavior changes.
$sectionTitles = [ $this->event->getExtraParam( 'section-title' ) ];
$commentIds = [ $this->event->getExtraParam( 'comment-id' ) ];
foreach ( $this->getBundledEvents() as $event ) {
$sectionTitles[] = $event->getExtraParam( 'section-title' );
$commentIds[] = $event->getExtraParam( 'comment-id' );
}
$commentIds = array_values( array_filter( $commentIds ) );
$sectionTitles = array_values( array_unique( array_filter( $sectionTitles ) ) );
if ( !$commentIds ) {
return null;
}
$params = [ 'dtnewcomments' => implode( '|', $commentIds ) ];
if ( count( $sectionTitles ) === 1 ) {
return $title->createFragmentTarget( $sectionTitles[0] )->getFullURL( $params );
} else {
return $title->getFullURL( $params );
}
// This may or may not have a fragment identifier, depending on whether it was recorded for
// the first one of the bundled events. It's usually not needed because we handle scrolling
// client-side, but we can keep it for no-JS users, and to reduce the jump when scrolling.
$titleWithOptionalSection = $this->section->getTitleWithSection();
return $titleWithOptionalSection->getFullURL( $params );
}
}

2
includes/Notifications/SubscribedNewCommentPresentationModel.php
View file

@ -25,7 +25,7 @@ class SubscribedNewCommentPresentationModel extends EchoEventPresentationModel {
/**
* @var EchoPresentationModelSection
*/
private $section;
protected $section;
/**
* @inheritDoc