jenkins-bot has submitted this change and it was merged.
Change subject: Fix an issue where latest post wasn't considered as potentially
topmost
......................................................................
Fix an issue where latest post wasn't considered as potentially topmost
Bug: T126264
Change-Id: I43d24cdc3a00053c971f2734b23cbbc3bb5240c6
---
M includes/Notifications/PostReplyPresentationModel.php
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
Catrope: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/Notifications/PostReplyPresentationModel.php
b/includes/Notifications/PostReplyPresentationModel.php
index 4d160ad..042d6d8 100644
--- a/includes/Notifications/PostReplyPresentationModel.php
+++ b/includes/Notifications/PostReplyPresentationModel.php
@@ -26,7 +26,9 @@
$notificationController = Container::get(
'controller.notification' );
$firstChronologicallyEvent = end( $bundledEvents );
$firstChronologicallyPostId =
$firstChronologicallyEvent->getExtraParam( 'post-id' );
- $topmostPostID =
$notificationController->getTopmostPostId( $bundledEvents );
+
+ $bundledEventsIncludingThis = array_merge( array(
$this->event ), $bundledEvents );
+ $topmostPostID =
$notificationController->getTopmostPostId( $bundledEventsIncludingThis );
} else {
$event = $this->event;
--
To view, visit https://gerrit.wikimedia.org/r/284611
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I43d24cdc3a00053c971f2734b23cbbc3bb5240c6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Mattflaschen <mflasc...@wikimedia.org>
Gerrit-Reviewer: Catrope <roan.katt...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
