Werdna has uploaded a new change for review.
https://gerrit.wikimedia.org/r/104909
Change subject: Fix issue with change 102154, one call to isAllowed was left.
......................................................................
Fix issue with change 102154, one call to isAllowed was left.
Bug: 59195
Change-Id: Ieaac35f5afc0893bde49e5e760a85f423078f76e
---
M includes/Notifications/Controller.php
1 file changed, 4 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Flow
refs/changes/09/104909/1
diff --git a/includes/Notifications/Controller.php
b/includes/Notifications/Controller.php
index 614425c..cf14eec 100644
--- a/includes/Notifications/Controller.php
+++ b/includes/Notifications/Controller.php
@@ -408,7 +408,10 @@
$post = reset( $post );
- if ( $post && $post->isAllowed() ) {
+ $user = new User;
+ $actionPermissions = new RevisionActionPermissions(
Container::get( 'flow_actions' ), $user );
+
+ if ( $post && $actionPermissions->isAllowed( $post,
'view' ) ) {
$userid = $post->getCreatorId();
if ( $userid ) {
$users[$userid] = User::newFromId(
$userid );
--
To view, visit https://gerrit.wikimedia.org/r/104909
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ieaac35f5afc0893bde49e5e760a85f423078f76e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Werdna <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
