renamed ActionRouter::checkPermissions to checkPreconditions

As discussed in #1933

_test/tests/inc/Action/general.test.php

@@ -133,7 +133,7 @@ class action_general extends DokuWikiTest {
         $conf['subscribers'] = 1;
 
         try {
-            $class->checkPermissions();
+            $class->checkPreconditions();
         } catch(\Exception $e) {
             $this->assertNotSame(ActionAclRequiredException::class, get_class($e));
         }
@@ -141,7 +141,7 @@ class action_general extends DokuWikiTest {
         $conf['useacl'] = 0;
 
         try {
-            $class->checkPermissions();
+            $class->checkPreconditions();
         } catch(\Exception $e) {
             $this->assertSame(ActionAclRequiredException::class, get_class($e));
         }
@@ -166,7 +166,7 @@ class action_general extends DokuWikiTest {
         $_SERVER['REMOTE_USER'] = 'test';
 
         try {
-            $class->checkPermissions();
+            $class->checkPreconditions();
         } catch(\Exception $e) {
             $this->assertNotSame(ActionUserRequiredException::class, get_class($e));
         }
@@ -174,7 +174,7 @@ class action_general extends DokuWikiTest {
         unset($_SERVER['REMOTE_USER']);
 
         try {
-            $class->checkPermissions();
+            $class->checkPreconditions();
         } catch(\Exception $e) {
             $this->assertSame(ActionUserRequiredException::class, get_class($e));
         }

inc/Action/AbstractAclAction.php

@@ -14,8 +14,8 @@ use dokuwiki\Action\Exception\ActionAclRequiredException;
 abstract class AbstractAclAction extends AbstractAction {
 
     /** @inheritdoc */
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
         global $conf;
         global $auth;
         if(!$conf['useacl']) throw new ActionAclRequiredException();

inc/Action/AbstractAction.php

@@ -44,12 +44,12 @@ abstract class AbstractAction {
     abstract public function minimumPermission();
 
     /**
-     * Check permissions are correct to run this action
+     * Check conditions are met to run this action
      *
      * @throws ActionException
      * @return void
      */
-    public function checkPermissions() {
+    public function checkPreconditions() {
     }
 
     /**

inc/Action/AbstractUserAction.php

@@ -14,8 +14,8 @@ use dokuwiki\Action\Exception\ActionUserRequiredException;
 abstract class AbstractUserAction extends AbstractAclAction {
 
     /** @inheritdoc */
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
         global $INPUT;
         if(!$INPUT->server->str('REMOTE_USER')) {
             throw new ActionUserRequiredException();

inc/Action/Admin.php

@@ -24,8 +24,8 @@ class Admin extends AbstractUserAction {
         }
     }
 
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
 
         global $INFO;
         if(!$INFO['ismanager']) {

inc/Action/Draft.php

@@ -25,8 +25,8 @@ class Draft extends AbstractAction {
     }
 
     /** @inheritdoc */
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
         global $INFO;
         if(!file_exists($INFO['draft'])) throw new ActionException('edit');
     }

inc/Action/Edit.php

@@ -26,8 +26,8 @@ class Edit extends AbstractAction {
     /**
      * @inheritdoc falls back to 'source' if page not writable
      */
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
         global $INFO;
 
         // no edit permission? view source

inc/Action/Login.php

@@ -19,9 +19,9 @@ class Login extends AbstractAclAction {
     }
 
     /** @inheritdoc */
-    public function checkPermissions() {
+    public function checkPreconditions() {
         global $INPUT;
-        parent::checkPermissions();
+        parent::checkPreconditions();
         if($INPUT->server->has('REMOTE_USER')) {
             // nothing to do
             throw new ActionException();

inc/Action/Logout.php

@@ -20,8 +20,8 @@ class Logout extends AbstractUserAction {
     }
 
     /** @inheritdoc */
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
 
         /** @var \DokuWiki_Auth_Plugin $auth */
         global $auth;

inc/Action/Profile.php

@@ -20,8 +20,8 @@ class Profile extends AbstractUserAction {
     }
 
     /** @inheritdoc */
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
 
         /** @var \DokuWiki_Auth_Plugin $auth */
         global $auth;

inc/Action/ProfileDelete.php

@@ -20,8 +20,8 @@ class ProfileDelete extends AbstractUserAction {
     }
 
     /** @inheritdoc */
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
 
         /** @var \DokuWiki_Auth_Plugin $auth */
         global $auth;

inc/Action/Register.php

@@ -20,8 +20,8 @@ class Register extends AbstractAclAction {
     }
 
     /** @inheritdoc */
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
 
         /** @var \DokuWiki_Auth_Plugin $auth */
         global $auth;

inc/Action/Resendpwd.php

@@ -20,8 +20,8 @@ class Resendpwd extends AbstractAclAction {
     }
 
     /** @inheritdoc */
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
 
         /** @var \DokuWiki_Auth_Plugin $auth */
         global $auth;

inc/Action/Search.php

@@ -27,8 +27,8 @@ class Search extends AbstractAction {
      *
      * @inheritdoc
      */
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
     }
 
     public function preProcess()

inc/Action/Subscribe.php

@@ -20,8 +20,8 @@ class Subscribe extends AbstractUserAction {
     }
 
     /** @inheritdoc */
-    public function checkPermissions() {
-        parent::checkPermissions();
+    public function checkPreconditions() {
+        parent::checkPreconditions();
 
         global $conf;
         if(isset($conf['subscribers']) && !$conf['subscribers']) throw new ActionDisabledException();

inc/ActionRouter.php

@@ -204,7 +204,7 @@ class ActionRouter {
             throw new ActionDisabledException();
         }
 
-        $action->checkPermissions();
+        $action->checkPreconditions();
 
         if(isset($INFO)) {
             $perm = $INFO['perm'];