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

Change subject: Fix regression requiring flow-create-board for non-Flow moves
......................................................................


Fix regression requiring flow-create-board for non-Flow moves

Caused by d10e4402e05a48ae012479a70536302773a0e5f0

Change-Id: I3e41190f50086b6368e91f940acbe0d3fd1a5dd2
---
M Hooks.php
1 file changed, 5 insertions(+), 0 deletions(-)

Approvals:
  Legoktm: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/Hooks.php b/Hooks.php
index 6edbeda..d1f3854 100644
--- a/Hooks.php
+++ b/Hooks.php
@@ -1295,6 +1295,11 @@
         * @return true to continue, false to abort the hook
         */
        public static function onMovePageCheckPermissions( Title $oldTitle, 
Title $newTitle, User $user, $reason, Status $status ) {
+               // Only affect moves if the source has Flow content model
+               if ( $oldTitle->getContentModel() !== CONTENT_MODEL_FLOW_BOARD 
) {
+                       return true;
+               }
+
                $occupationController = self::getOccupationController();
 
                $permissionStatus = 
$occupationController->checkIfUserHasPermission(

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3e41190f50086b6368e91f940acbe0d3fd1a5dd2
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Mattflaschen <[email protected]>
Gerrit-Reviewer: Catrope <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Matthias Mullie <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to