Stop faking custom URLs for controllers classes with exactly two consecutive uppercas...

This was introduced in 519f15c7700222357952e8cab41bbe960730c7fd and its purpose
is not entirely clear: Everything works identically even without this, even in
WoltLab Suite 5.5.

RoutingCacheBuilder is only ever used within ControllerMap and within
ControllerMap there are just a few locations where custom URLs are processed:

- resolve(): This enforces that renamed controllers are accessed via their
  canonical URL. Not relevant here, because we are already working with the
  canonical URL and it's not an intentional rename.
- resolveCustomController(): This is what we intent to avoid. If this method
  does not match, then `->resolve()` will correctly match.
- lookup(): `self::transformController()` will perform the correct
  transformation.
- lookupCmsPage(): Not relevant, because we do not deal with CMS pages.

diff --git a/wcfsetup/install/files/lib/system/cache/builder/RoutingCacheBuilder.class.php b/wcfsetup/install/files/lib/system/cache/builder/RoutingCacheBuilder.class.php
index 31d558c50e5a7c750c36825679216b1f2fe29d74..5516327e89e63a00e6fd47530df89d50441467a2 100644 (file)
--- a/wcfsetup/install/files/lib/system/cache/builder/RoutingCacheBuilder.class.php
+++ b/wcfsetup/install/files/lib/system/cache/builder/RoutingCacheBuilder.class.php
@@ -23,16 +23,6 @@ use wcf\util\FileUtil;
  */
 class RoutingCacheBuilder extends AbstractCacheBuilder
 {
-    /**
-     * list of controllers violating the url schema, but are
-     * supported for legacy reasons
-     * @var array
-     */
-    protected $brokenControllers = [
-        'lookup' => [],
-        'reverse' => [],
-    ];
-
     /**
      * @inheritDoc
      */
@@ -175,10 +165,8 @@ class RoutingCacheBuilder extends AbstractCacheBuilder
                             // fix for invalid pages that would cause single character fragments
                             $sanitizedParts = [];
                             $tmp = '';
-                            $isBrokenController = false;
                             foreach ($parts as $part) {
                                 if (\strlen($part) === 1) {
-                                    $isBrokenController = true;
                                     $tmp .= $part;
                                     continue;
                                 }
@@ -195,15 +183,6 @@ class RoutingCacheBuilder extends AbstractCacheBuilder
                             $className = $abbreviation . '\\' . ($libDirectory === 'lib/acp' ? 'acp\\' : '') . $pageType . '\\' . $filename;
 
                             $data[$abbreviation][$libDirectory === 'lib' ? 'frontend' : 'acp'][$ciController] = $className;
-
-                            if ($isBrokenController) {
-                                $this->brokenControllers['lookup'][$abbreviation][$ciController] = $className;
-                                $this->brokenControllers['reverse'][$abbreviation][\preg_replace(
-                                    '~(?:Page|Form|Action)$~',
-                                    '',
-                                    $filename
-                                )] = $ciController;
-                            }
                         }
                     }
                 }
@@ -275,17 +254,6 @@ class RoutingCacheBuilder extends AbstractCacheBuilder
             }
         }
 
-        // masquerade broken controllers as custom urls
-        foreach ($this->brokenControllers as $type => $brokenControllers) {
-            foreach ($brokenControllers as $application => $controllers) {
-                foreach ($controllers as $key => $value) {
-                    if (!isset($data[$type][$application][$key])) {
-                        $data[$type][$application][$key] = $value;
-                    }
-                }
-            }
-        }
-
         return $data;
     }