jenkins-bot has submitted this change and it was merged.
Change subject: Update wl_notificationtimestamp when updating Flow page
......................................................................
Update wl_notificationtimestamp when updating Flow page
Requires Iae213b87706c447b880244711e7747954423bb69 to be merged
in core first.
There may still be interaction issues with if we want to be doing
the watchlist for the Topic or the Board, but that can be worked
out separately.
Bug: 66876
Change-Id: I14320b747d16b7be652da05bda59fdf4e8eece81
---
M Hooks.php
1 file changed, 3 insertions(+), 0 deletions(-)
Approvals:
Bsitu: Looks good to me, but someone else must approve
Matthias Mullie: Looks good to me, but someone else must approve
EBernhardson: Looks good to me, approved
jenkins-bot: Verified
diff --git a/Hooks.php b/Hooks.php
index 9aa31a5..f1d09df 100644
--- a/Hooks.php
+++ b/Hooks.php
@@ -609,6 +609,9 @@
*/
public static function onAbortEmailNotification( $editor, $title ) {
if ( self::$occupationController->isTalkpageOccupied( $title )
) {
+ // Since we are aborting the notification we need to
manually update the watchlist
+ EmailNotification::updateWatchlistTimestamp( $editor,
$title, wfTimestampNow() );
+
return false;
}
--
To view, visit https://gerrit.wikimedia.org/r/159400
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I14320b747d16b7be652da05bda59fdf4e8eece81
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: EBernhardson <[email protected]>
Gerrit-Reviewer: Bsitu <[email protected]>
Gerrit-Reviewer: EBernhardson <[email protected]>
Gerrit-Reviewer: Matthias Mullie <[email protected]>
Gerrit-Reviewer: SG <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
