From 85a2883dc13228e8d6921c135a9c58c18299303a Mon Sep 17 00:00:00 2001 From: =?utf8?q?Tim=20D=C3=BCsterhus?= Date: Wed, 1 Jun 2022 09:47:05 +0200 Subject: [PATCH] Simplify `isset()` check in ControllerMap::isDefaultController() The first parameter was a prefix of the second, thus it is implicitly checked. --- .../install/files/lib/system/request/ControllerMap.class.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/wcfsetup/install/files/lib/system/request/ControllerMap.class.php b/wcfsetup/install/files/lib/system/request/ControllerMap.class.php index 95e5d3455d..bb78285b5c 100644 --- a/wcfsetup/install/files/lib/system/request/ControllerMap.class.php +++ b/wcfsetup/install/files/lib/system/request/ControllerMap.class.php @@ -302,7 +302,7 @@ class ControllerMap extends SingletonFactory public function isDefaultController($application, $controller) { // lookup custom urls first - if (isset($this->customUrls['lookup'][$application], $this->customUrls['lookup'][$application][$controller])) { + if (isset($this->customUrls['lookup'][$application][$controller])) { $controller = $this->customUrls['lookup'][$application][$controller]; if (\preg_match('~^(?P__WCF_CMS__\d+)(?:-(?P\d+))?$~', $controller, $matches)) { if ( -- 2.20.1