Inline DynamicRequestRoute::getControllerName()

Explicitly implement StaticRequestRoute::isACP()

The removes dependencies on the implementation in DynamicRequestRoute, possibly
allowing StaticRequestRoute to not inherit from DynamicRequestRoute in the
future.

Remove LookupRequestRoute::setIsACP()

This method always throws and is no longer required by the interface.

Remove IRequestRoute::setIsACP()

This is not a method that can usefully be generically called, because one needs
to know how the properties of the route implementation to usefully modify it.
There are already route implementations that always throw in this method.

Improve typing in RequestHandler

Remove obsolete import

see fd99dc13a68da8f29a5c17ec5d564672304fc7dd

Inline RequestHandler::redirect()

see d8b9780a06174bee45b7e911b945ce5644259146

Merge pull request #4871 from WoltLab/route-data

Clean up default controller processing

Improve RequestHandler::buildRequest()'s control flow

Perform the default controller processing within a single block, instead of
first branching based on whether the ACP or frontend was accessed.

Unify the handling of the default controller for ACP requests

Enforce that routes return a controller name XOR the default controller

It is not useful to return both a name and pretending that it's the default
controller, as the controller name would be overriden anyway.

see 5da310cc32d5300cb393dd127ce5ec7abc9315bf

Merge pull request #4869 from WoltLab/handle-default-controller

Remove dead branch in RequestHandler::handleDefaultController()

Inline RequestHandler::handleDefaultController()

This makes the data flow easier to grasp, as one does not need to jump around
in the file.

Remove dead branch in RequestHandler::handleDefaultController()

It turns out that that the third and last remaining change of
000c0c8a26491708b2af995bdd6f0e627cc75161 is also broken.

After effectively reverting / removing the logic in the other two files
modified by that commit in e0976fcfdbeeafc80bc202f80ae2f946326d465d and
416dafddb18a3e82750c6e8ca089d4f9132b39dc, this commit cleans up the last
change.

Since 000c0c8a26491708b2af995bdd6f0e627cc75161 is is impossible for this
redirect to happen and the reason is simple:

The logic will only ever be executed for non-CMS pages, as these are the only
ones where `->lookupDefaultController()` returns an application.

The `controller` return value of `->lookupDefaultController()` will contain the
`routePart` (i.e. `board-list`). However the application overrides are keyed by
the controller name (i.e. `BoardList`), causing the lookup to always fail,
resulting in the given `$application` being returned as-is in `$override`. As
both values are identical, the branch will never be taken and the redirect will
never be executed.

Replace emptiness check of $this->routeData with strict comparison in LookupRequestRoute

Add return type to ControllerMap::resolveCustomController()

Rename local `$controller` to `$routePart` in ControllerMap::lookupDefaultController()

Make usage of ControllerMap::$landingPages easier to inspect

By removing the use of the local variable one can simple search for
$this->landingPages to find all locations.

Mark ControllerMap as final

Reinstall all composer packages

Tighten up composer constraints

Merge branch '5.5'

Merge remote-tracking branch 'origin/5.5' into 5.5

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Update Guzzle

see guzzle/guzzle@a52f0440530b54fa079ce76e8c5d196a42cad981

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Update guzzlehttp/psr7

This is a dependency for an updated Guzzle.

see guzzle/psr7@e98e3e6d4f86621a9b75f623996e6bbdeb4b9318
see guzzle/guzzle@a52f0440530b54fa079ce76e8c5d196a42cad981

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Regenerate composer files

Merge pull request #4868 from WoltLab/request-handler-early-response

Use PSR-7 responses in RequestHandler

Redirect requests to app-specific ACP landing pages using PSR-7

Use PSR-7 to generate the redirect to the canonical domain

Pass the PSR-7 RequestInterface to RequestHandler::buildRequest()

Support returning responses in RequestHandler::buildRequest()

The return type of `->buildRequest()` is a little ugly now, but this will be
better than a hard call to `exit;` whenever a redirect is desired.

This whole routing and request building logic would ideally move into a
Middleware, this can be considered a first step to clean that up.

Merge pull request #4848 from WoltLab/fix-is-default-controller

Fix ControllerMap::isDefaultController() to properly take into account overriden applications

Release 5.5.0 RC 2

Updating minified JavaScript files

Use proper placeholders in SQL queries in RoutingCacheBuilder

Fix ControllerMap::isDefaultController() to properly take into account overriden applications

Previously virtually moving a CMS page (e.g. the Dashboard) into a different
app and making that page the landing page for that app would not drop the
controller string from the URL.

Merge branch '5.5'

Add missing newline at EOF in acp/style/layout.scss

see 89763fcf8fea68d5cda6bda419a5bd8326d4db58

Merge branch '5.5'

Skip the search close handler if it was not open

Executing the callback causes some unintentional changes to the page state.

See https://www.woltlab.com/community/thread/295991-dialoge-hintergrund-scrollt/

Align the mobile ACP user drop-down to the right

See https://www.woltlab.com/community/thread/295980-acp-men%C3%BC-mobil-ausrichtung-%C3%A4ndern/

Merge pull request #4867 from WoltLab/landing-page-overriden-app

Remove RoutingCacheBuilder::handleLandingPageWithOverriddenApplication()

Merge branch '5.5' of https://github.com/WoltLab/WCF into 5.5

Fix non-functioning template scripting in language variable

Indent the closing heredoc/nowdoc marker in StyleCompiler

Remove RoutingCacheBuilder::handleLandingPageWithOverriddenApplication()

This logic was introduced in 000c0c8a26491708b2af995bdd6f0e627cc75161 which
intended to (and actually did) fix #2905.

However the commit makes more changes than are actually necessary to fix the
issue:

The change in ControllerMap caused the branch condition to become impossible.
This change might have been required to fix the issue, but the logic was
unneccessarily complex, because of the dead branch. It was subsequently removed
in commit 416dafddb18a3e82750c6e8ca089d4f9132b39dc.

Likewise the change to RoutingCacheBuilder does not have any effect on the bug
reported in #2905. After reverting the change within RoutingCacheBuilder the
bug is not reproducible. It is reproducible however when reverting the change
in RequestHandler, indicating that this change is the only relevant in that
commit.

This commit reverts the change to RoutingCacheBuilder, because it is not
neccessary and because it actually introduced intransparent behavior into the
routing system:

If the landing page of `wcf` (i.e. the page that previously was considered to
be the global landing page before removing the distinction in
64a1be56f2ef06f8b4b2bea0cccc7c8b8374588a) is a page that is assigned to
`packageID = 1` (i.e. to `wcf`), but virtually relocated into an application
(e.g. `wbb`), then the application's configured landing page will be ignored
and wcf's landing page will be used instead.

This implicit override is not exposed to the administrator and it is also not
necessary. It works perfectly fine to move wcf's landing page into a different
app where it is not considered to be the landing page: The URLs generated by
the link handler will simply include the controller name and the redirect
performed by the URL canonicalization feature will go to the right place.

Merge branch '5.5'

Merge branch '5.4' into 5.5

Match foreign keys independent of referenced column casing in PHP DDL

Early versions of MySQL 8 contained a bug that would return the column names of
the referenced table in lowercase whenever querying the FOREIGN KEY or when
showing the table's structure:

https://bugs.mysql.com/bug.php?id=88718

This issue also affects the any created database dumps and is not corrected
when importing the dump into a fixed version of MySQL:

https://bugs.mysql.com/bug.php?id=98976

Thus any instances that were ever dumped with an affected version of MySQL 8
will have foreign keys with a mismatching column case.

The FOREIGN KEY itself will be completely functional, MySQL does not care about
the casing of column names.

However the PHP DDL considers those FOREIGN KEYs to be different when
attempting to find an existing FOREIGN KEY to update, thus believing that the
FOREIGN KEY does not yet exist. As a result it attemtps to create it,
generating a name that is identical to the existing FOREIGN KEY's, leading to
conflicts when attempting to log the change into package_installation_sql_log.

Fix this issue by lowercasing the referenced column names when grabbing the
`->getDiffData()`. The PHP DDL will then find the existing FOREIGN KEY, but it
will realize that it is different when diffing the `->getData()`.

This will lead to the "broken" FOREIGN KEY being dropped and it being recreated
correctly, both avoiding the duplicate key error and also actually correcting
the column name.

Merge branch 'php8.2'

Allow PHP 8.2 in CheckSystemEnvironment

Merge branch '5.5'

Merge remote-tracking branch 'origin/5.5' into 5.5

Merge branch 'php8.2-fixes' into 5.5

Fix PHP 8.2 compatibility in TemplateEngine

> Creation of dynamic property wcf\system\template\TemplateEngine::$pluginObjects is deprecated
> Creation of dynamic property wcf\system\template\TemplateEngine::$tagStack is deprecated

Fix PHP 8.2 compatibility in HascontentPrefilterTemplatePlugin

> Use of "self" in callables is deprecated

Fix saving box for page assignments

The methods still contained code that became obsolete with the PR linked below.

see https://github.com/WoltLab/WCF/pull/4866

Merge branch '5.5'

Removed obsolete update script for box condition conversion

see https://github.com/WoltLab/WCF/pull/4866

Update composer dependencies

Merge branch '5.5'

Merge remote-tracking branch 'origin/5.5' into 5.5

Update composer dependencies

Merge pull request #4866 from WoltLab/box-visibility-conditions

Hiding boxes did not work properly

Modernized the embedded JavaScript

Hiding boxes did not work properly

Merge pull request #4865 from WoltLab/controllermap-lookup-default-controller

Remove dead branch on ControllerMap::lookupDefaultController()

Merge branch '5.5'

Add missing semicolon in Quote.ts

see cc8832d874e4da51c58b7e44503eb3701abbd7a7

Merge pull request #4864 from WoltLab/linkhandler-cms-link

Clean up data flow in LinkHandler::getCmsLink()

Remove dead branch in RequestHandler::handleDefaultController()

`->lookupDefaultController()` will not return a `redirect` key since the
previous commit.

Remove dead branch on ControllerMap::lookupDefaultController()

It is impossible for this branch to ever be taken:

To even reach this branch it is a necessary precondition that the landing page
of the accessed application is a non-system CMS page.

A non-system CMS page can either be user-created (originIsSystem = 0) or
installed by a package (e.g. the Dashboard). These two have differently with
regard to application overrides: Only package-installed CMS pages can ever have
an application override. For a user-created CMS page instead of creating an
override the page's packageID will simply be updated.

1)

Let's first look into user-created CMS pages: For the reasons outlined above
the call to `->getApplicationOverride()` will simply pass through the
`$application`, because no override can exist. `$application !== $application`
will never be true, thus this branch will never be taken.

2)

The same is true for a package-installed CMS page without an override: The
second half of the condition will not be true.

3)

Thus the remaining case to look into is a package-installed CMS page with an
override.

Let us consider the situation with 3 installed apps, `wcf`, `forum` and `blog`.
The CMS page was installed by `blog` and virtually reassigned to `forum`. The
application overrides will look like the following:

    array (size=2)
      'lookup' =>
        array (size=1)
          'wbb' =>
            array (size=1)
              'testing' => string 'blog' (length=4)
      'reverse' =>
        array (size=1)
          'blog' =>
            array (size=1)
              'testing' => string 'wbb' (length=3)

And only the `reverse` case is relevant here.

Now when accessing the `blog` the first half of the condition will be false:
The accessed application matches the actual application.

When accessing any other app, the second half of the condition will be false,
as the `reverse` lookup maps from `blog` to `wbb` which was already ruled out
in the previous paragraph. As the lookup will not match, the `$application`
will be passed through unmodified.

In none of the cases can both parts of the conditional branch ever be true at
the same time, making this a dead branch.

The logic was last modified in commit 000c0c8a26491708b2af995bdd6f0e627cc75161.
Peeking at the code as of that commit does not indicate that the behavior of
`getApplicationOverride()` was different back then, indicating that that commit
broke the `redirect` return case.

It is likely that it was intended that `$cmsPageData['application']` was to be
passed as a parameter instead of `$application`. However this code being broken
for that long of a time indicates that it wasn't important after all.

Clean up data flow in LinkHandler::getCmsLink()

Merge branch '5.5'

Mark closure in KeywordHighlighter as static

see 406348deb7debaeaa908db6fc1a724a40e82e8a3

Fix typo in PHPDoc of CheckboxFormField

Skip the highlighting of words with a length < 3

See https://www.woltlab.com/community/thread/295850-highlighter-in-suchergebnissen-hebt-zu-kurze-begriffe-hervor/

Quote links now imply the `https` protocol if not set

Default to `https` for protocol-less links

Use the same icon for quotes in the editor

Right sidebar boxes did not span the entire width

See https://www.woltlab.com/community/thread/295902-rechte-sidebardarstellung-im-userprofil-mobil-fehlerhaft/

The CSS was accidentally removed in c00bdb0694b993be21d779ecef6badc8524e8967

Merge pull request #4863 from WoltLab/thumbnail-graceful

Gracefully ignore Thumbnail generation failures in DefaultUploadFileSaveStrategy

Gracefully ignore Thumbnail generation failures in DefaultUploadFileSaveStrategy

`->generateThumbnails()` might already not do anything of the memory limit is
likely going to be exceeded, causing image upload to be thumbnail-less. Extend
this to all cases of thumbnail generation failure by catching exceptions thrown
when loading the image or when performing the resizing operation.

Use Header::splitList() instead of Header::normalize()

see guzzle/psr7#477

Merge branch '5.5'

Fix SCSS formatting

Tighten up composer version constraints

Sort dependencies in composer.json

Update composer dependencies

Merge branch '5.5'

Update composer dependencies

Especially for guzzle/guzzle@e3ff079b22820c2029d4c2a87796b6a0b8716ad8.

Merge branch '5.4' into 5.5

Update Guzzle

see guzzle/guzzle@e3ff079b22820c2029d4c2a87796b6a0b8716ad8

Merge branch '5.3' into 5.4