Catrope has uploaded a new change for review.
https://gerrit.wikimedia.org/r/296343
Change subject: Include parameters when performing &workflow= redirects
......................................................................
Include parameters when performing &workflow= redirects
Not doing this breaks index.php?workflow=X&markasread=Y
because the markasread parameter is not processed and
was dropped by the redirect. Thanks did this for flow-thanks
notifications.
Change-Id: I6bb2c92702e239f1a049d4260db15d795a8e6b71
---
M includes/Actions/Action.php
1 file changed, 3 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Flow
refs/changes/43/296343/1
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: newchange
Gerrit-Change-Id: I6bb2c92702e239f1a049d4260db15d795a8e6b71
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Catrope <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
