Remove `$pageName` from `wcf\system\request\Request`
authorTim Düsterhus <duesterhus@woltlab.com>
Fri, 20 May 2022 12:40:59 +0000 (14:40 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Fri, 20 May 2022 12:58:46 +0000 (14:58 +0200)
The value is unused and badly named, because it refers to the controller name
and not the name of some page.

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

index 64dfdabe24dcc15500b6beeb6e8ea5bb2f9a59eb..d12a6607e6027d614bdbeae69a65e3bbf8f2c861 100644 (file)
@@ -41,12 +41,6 @@ final class Request implements RequestHandlerInterface
      */
     protected $pageID;
 
-    /**
-     * page name
-     * @var string
-     */
-    protected $pageName = '';
-
     /**
      * request object
      * @var object
@@ -57,14 +51,12 @@ final class Request implements RequestHandlerInterface
      * Creates a new request object.
      *
      * @param string $className fully qualified name
-     * @param string $pageName class name
      * @param string[] $metaData additional meta data
      */
-    public function __construct($className, $pageName, array $metaData)
+    public function __construct($className, array $metaData)
     {
         $this->className = $className;
         $this->metaData = $metaData;
-        $this->pageName = $pageName;
     }
 
     /**
@@ -136,16 +128,6 @@ final class Request implements RequestHandlerInterface
         return $this->metaData;
     }
 
-    /**
-     * Returns the page name of this request.
-     *
-     * @return  string
-     */
-    public function getPageName()
-    {
-        return $this->pageName;
-    }
-
     /**
      * Returns the current request object.
      *
index 56a999f10a600bdf456429d8cdb78a387ed30b2f..2a3572ab973640a4bd5d74b9ad1717d993d611b7 100644 (file)
@@ -213,7 +213,6 @@ final class RequestHandler extends SingletonFactory
 
             $this->activeRequest = new Request(
                 $classData['className'],
-                $classData['controller'],
                 $metaData
             );