Move call to `checkAppEvaluation()` into `handle()`
authorTim Düsterhus <duesterhus@woltlab.com>
Tue, 6 Jul 2021 09:34:17 +0000 (11:34 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Tue, 6 Jul 2021 09:34:17 +0000 (11:34 +0200)
It fits better here to cleanly separate the handling from the building. No
behavioral change was found.

wcfsetup/install/files/lib/system/request/RequestHandler.class.php

index 079d50424c8b5ab2d64ed5672ae3b7fdacbd7605..2f493a18d6d85e6a1b890abb0bfc4ac32befdd8f 100644 (file)
@@ -86,6 +86,8 @@ class RequestHandler extends SingletonFactory
                 throw new IllegalLinkException();
             }
 
+            $this->checkAppEvaluation();
+
             $this->checkOfflineMode();
 
             // start request
@@ -210,8 +212,6 @@ class RequestHandler extends SingletonFactory
                 $metaData
             );
 
-            $this->checkAppEvaluation();
-
             if (!$this->isACPRequest()) {
                 // determine if current request matches the landing page
                 if (ControllerMap::getInstance()->isLandingPage($classData, $metaData)) {