jenkins-bot has submitted this change and it was merged.
Change subject: Include parameters when performing &workflow= redirects
......................................................................
Include parameters when performing &workflow= redirects
Not doing this broke index.php?workflow=X&markasread=Y
because the markasread parameter was not processed and
was dropped by the redirect. Thanks does this for flow-thanks
notifications.
Change-Id: I6bb2c92702e239f1a049d4260db15d795a8e6b71
---
M includes/Actions/Action.php
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
Sbisson: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/Actions/Action.php b/includes/Actions/Action.php
index ae81034..e6c4668 100644
--- a/includes/Actions/Action.php
+++ b/includes/Actions/Action.php
@@ -149,6 +149,8 @@
}
// We need to redirect
- return $redirTitle->getLinkURL();
+ return $redirTitle->getLinkURL(
+ array_diff_key( $request->getValues(), array( 'title'
=> '', 'workflow' => '' ) )
+ );
}
}
--
To view, visit https://gerrit.wikimedia.org/r/296343
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6bb2c92702e239f1a049d4260db15d795a8e6b71
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Catrope <[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
