jenkins-bot has submitted this change and it was merged.

Change subject: Add 'target-page' param to flow notifications
......................................................................


Add 'target-page' param to flow notifications

This depends on echo patch: https://gerrit.wikimedia.org/r/#/c/152808/

Change-Id: Ia8d9d795fff7f51e2f6eb9c207227f93d9307876
---
M includes/Notifications/Controller.php
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Matthias Mullie: Looks good to me, approved
  Jdlrobson: Looks good to me, but someone else must approve
  jenkins-bot: Verified



diff --git a/includes/Notifications/Controller.php 
b/includes/Notifications/Controller.php
index a982693..bb15be1 100644
--- a/includes/Notifications/Controller.php
+++ b/includes/Notifications/Controller.php
@@ -100,6 +100,7 @@
                $extraData['revision-id'] = $revision->getRevisionId();
                $extraData['post-id'] = $revision->getPostId();
                $extraData['topic-workflow'] = $topicWorkflow->getId();
+               $extraData['target-page'] = 
$topicWorkflow->getArticleTitle()->getArticleID();
 
                $newPost = null;
                switch( $eventName ) {
@@ -201,6 +202,7 @@
                                'content' => $firstPost
                                        ? Utils::htmlToPlaintext( 
$firstPost->getContent(), 200, $this->language )
                                        : null,
+                               'target-page' => 
$topicWorkflow->getArticleTitle()->getArticleID()
                        )
                ) );
 
@@ -271,6 +273,7 @@
                                        'post-id' => $newRevision ? 
$newRevision->getPostId() : null,
                                        'mentioned-users' => $mentionedUsers,
                                        'topic-workflow' => 
$topicWorkflow->getId(),
+                                       'target-page' => 
$topicWorkflow->getArticleTitle()->getArticleID(),
                                        'reply-to' => isset( $data['reply-to'] 
) ? $data['reply-to'] : null
                                ),
                                'agent' => $user,

-- 
To view, visit https://gerrit.wikimedia.org/r/152935
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia8d9d795fff7f51e2f6eb9c207227f93d9307876
Gerrit-PatchSet: 6
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Bsitu <[email protected]>
Gerrit-Reviewer: Bsitu <[email protected]>
Gerrit-Reviewer: Jdlrobson <[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

Reply via email to