Skip selectSetupLanguage() and showLicense() in developer mode by using gotoNextStep()

While this adds additional HTTP requests, it cleans up the dispatcher,
specifically getting rid of the fall-through logic, making the setup cleaner
with negligible performance loss.

Merge pull request #5168 from WoltLab/wcfsetup-tmp-handling

Improve validation of tmpFilePrefix in install.php

Merge pull request #5167 from WoltLab/wcfsetup-no-directory-select

Remove directory selection from WCFSetup

Use `->prepare()` in SetupFileHandler

Co-authored-by: Alexander Ebert <ebert@woltlab.com>

Strongly validate the `tmpFilePrefix` in install.php

Previously it was possible to create directories using name that is partly
externally controllable. Now either a known prefix (i.e. known directory) needs
to be provided or a new random prefix will be generated.

Do not accept the `tmpFilePrefix` parameter from a URI parameter in WCFSetup

This parameter is never transmitted in the URI, thus we neither need, nor
should take it from the URI.

Do not accept the `step` parameter from a URI parameter in WCFSetup

Previously it was possible to jump into arbitrary locations of the set-up by a
simple link click.

Remove dedicated logFiles step from WCFSetup

As the database connection is available during unzipFiles(), we can log the
files immediately instead of writing them into a temporary file first.

Reorder WCFSetup steps to configure the DB before unzipping files

Since the directory selection has been removed, the target location of the
files is known a-priori. By reordering the steps we can validate that the
database is fine before extracting all the files. In case of an error clearing
just the database is easier than also nuking the webroot.

Remove step to configure directories from WCFSetup

Resolves #5027

Remove the registration of the `__wcfSetup_directories` session variable

Ignore the WCFSetup directory selection in PackageInstallationDispatcher

Merge pull request #5165 from WoltLab/dynamic-property-user-editor

Do not attempt to update the accessToken on a live object in UserEditor::update()

Merge pull request #5163 from SoftCreatR/bugfix/faStyleEditor

Update style editor

Update composer dependencies

Fix rendering of `wcf.acp.devtools.project.requiredPackage.requiredPackage`

see 19b96010188f4859023ea4d26116c9da81b43c36

Fix rendering of `wcf.acp.devtools.project.instruction.instruction`

This phrase is evaluated in JavaScript and thus the `{icon}` helper is not
(currently) available.

Remove non-COMPILER_TARGET_DEFAULT variants of WCF.Dropdown.Interactive

These were missed in b433eb3580d56dc41dabc029324a3cc03c6ef3f1.

Do not attempt to update the accessToken on a live object in UserEditor::update()

The purpose of this is not entirely clear, but when taking a look at the commit
that added the line 6a41a21e09894236c26daa10019d0e0a859f963b, one can see that
the update of the accessToken was added beneath an update of the (dedicated)
salt that still existed back then. Notably absent is an update of the new
*hash*, though.

No other property that is updated via an *Editor class is actively attempted to
be updated on a live object and even in this case, only a dynamic property was
created on the UserEditor, instead of actually updating the accessToken in the
underlying user object (which would be possible, because the DBOEditors inherit
from the DBO class, allowing clean access to the `->data` property).

Thus we simply remove the attempted update to fix the creation of this dynamic
property which is deprecated as of PHP 8.2.

Fixes #5164

Add proper types to various UserEditor methods

Update style editor

Replaced old FA reference

Resolves #5161

Merge pull request #5162 from WoltLab/template-add-no-default-group

Add empty default selection for templateGroupID in templateAdd.tpl

Add empty default selection for templateGroupID in templateAdd.tpl

see https://www.woltlab.com/community/thread/296343-template-kopieren-leere-vorauswahl-bei-der-templategruppe/

Merge branch '5.5'

Stop using undefined properties in SearchHandler::saveSearch()

These are not intended to be stored on the class and not used elsewhere, the
`this->` is likely a copy and paste mistake in the initial implementation.

see 11b63e8aa4e1a71425e1bffd7eb28f9db4a4bf13

Add `--watch` parameter for web component bundling

Add typings for `<fa-icon>` to webcomponents

Remove the superfluous replacement of special characters

This is a left-over from the previous implementation that worked with the raw HTML returned from the server. The new code offloads the handling to the browser’s DOM API.

Update composer dependencies

Merge branch '5.5'

Remove useless reference parameter in ArticleCategoryCache::countArticles()

This is never written to and thus does not need to be and thus should not be a
reference.

Remove unused `@var` tag from ArticleCategoryCache

fix phpdoc of `ArticleCategoryCache::countArticles()`

fix phpdoc of `SessionEditor::create()`

fix phpdoc of `Redis::info()`

fix phpdoc of `ViewableArticle::$unreadArticlesByCategory`

Merge pull request #5155 from WoltLab/css-variables

Migrate to native CSS variables

Replace inline CSS variables with global values

Fix the calculation of the ACP page content offset

Replace inline SCSS variables with native CSS variables

Replace usage of `opacify()` and `transparentize()` with `rgba()`

Replace SCSS variables with CSS variables

Export style colors as plain RGB values

Update the compiled JS artifacts

Remove the API version on upgrade

Remove the support for style API versions

Closes #5036

Merge pull request #5154 from WoltLab/desktop-notification-img-alt-text

Show alt tags of images in desktop notifications

Show alt tags of images in desktop notifications

Merge pull request #5153 from WoltLab/details-tag-in-exception-log

Use <details> HTML tag for toggleable content in exception log

Merge pull request #4963 from WoltLab/reaction-summary

Reaction summary web component

Lazily initialize the reaction summary

Migrate to the new dialog API

Use actual private properties and methods

Minor code clean-up

Fix Database::getInsertID() return type PHPDoc

see #4915

Merge pull request #5151 from SoftCreatR/bugfix/tpl

Make unfurlUrl template available in ACP

Add compiled js

Add missing definitions

Add preload for language variable

Use light dom instead of the shadow dom

Rename web component to new naming scheme

Add highlighting for selected reaction

Add web component for reaction summary

Add default styling for <summary> HTML tag

See #4612

Use <details> HTML tag for toggleable content in exception log

see #4612

Made unfurlUrl template available in ACP

See https://www.woltlab.com/community/thread/298168-artikel-erstellen-im-acp-bringt-fehler-bei-vorschau/

Merge pull request #5147 from WoltLab/search-index-inner-filter

Add experimental context filter for search backends

Mark the context aware API as experimental

Merge pull request #5149 from WoltLab/output-buffer-clear

Clean out any default output buffers

Clean out any default output buffers

Resolves #5145

Use array-shape as the return type for ISearchEngine::getInnerJoin()

Update to cuyz/valinor 1.0.x

Merge pull request #5144 from WoltLab/package-update-instructions

Fix the selection of update instructions

Add `SearchEngine::isContextAware()`

The helper method is used for runtime optimizations to avoid index rebuilds for backends that are not context aware.

Fix the use of the context filter

Add support for context filters when querying the search backend

Add prototype for a search index that is aware of a message’s context

Pass the update instructions as a parameter to buildPluginNodes()

Now the method has a single responsibility: Building the actual nodes.
Previously it both queried the instructions and built the nodes.

This also allows us to easily record the “fromversion” in the audit log which
is the main improvement of this change.

Update the pending packages after the plugin nodes were built

Previously the `buildPluginNodes()` step in PackageInstallationNodeBuilder
would attempt to find update instructions for the *target* version, instead of
the source version. We must move the update of the pending packages below this
step to fix this. It also makes sense, because only once the plugin nodes have
been built (and thus executed) the package logically is on the new version.

see 5df3e65185d4d3f1be7464b1999b949946a5c765

Slightly reorder assignments in PackageInstallationNodeBuilder::buildPluginNodes()

Move the retrieval of the instructions to the top of buildPluginNodes()

This part of the logic dues not depend on the nodes and sequence numbers, by
moving it to the top data flow becomes clearer.

Make updates/installations explicit in package audit log

Use PackageInstallationDispatcher::getPackage() in PackageUninstallationNodeBuilder

Unify the querying of the “action” in PackageInstallationNodeBuilder

Make PackageInstallationDispatcher::$archive protected

Make PackageInstallationDispatcher::$package protected

Make `PackageInstallationNodeBuilder::$pendingPackages` private

Improve formatting in PackageInstallationNodeBuilder

Add support for `<void/>` instructions to PackageManifest

Merge branch '5.5'

Merge pull request #5142 from WoltLab/php-ddl-index-validate

Fix validation of changed indices in PHP DDL

Fix validation of changed indices in PHP DDL

The validation of the to-be-performed DDL operation currently identifies
indices by their data (i.e. column list and type), whereas the actual DDL
operation uses the `->diffIndices()` operation which also takes into account
the name of the index.

This mismatch allows a package to drop a foreign index, consider the following
situation:

Package A:

    DatabaseTable::create('wcf1_test')
        ->columns([
            NotNullInt10DatabaseTableColumn::create('a'),
            NotNullInt10DatabaseTableColumn::create('b'),
        ])
        ->indices([
            DatabaseTableIndex::create('testing')
                ->columns(['a']),
        ])

The package creates a table with two columns and a named index (“testing”) that
includes one of the columns.

Now Package B:

    DatabaseTable::create('wcf1_test')
        ->indices([
            DatabaseTableIndex::create('testing')
                ->columns(['a', 'b'])
                ->drop(),
        ])

This definition drops the named index (“testing”), but with a different column
definition. Thus the validation believes the indices to be different, allowing
the operation to proceed. The actual operation however identifies the index by
its name and thus drops the “testing” index that belongs to a different
package.

Merge branch '5.5'

Release 5.5.7

Merge branch '5.5'

Update list of deleted phrases

see WoltLab/com.woltlab.wcf.conversation#180

The `$value` for `getSearchFormElement()` is nullable

See https://www.woltlab.com/community/thread/297958-fehler-bei-suche-mit-php-8-1/

Revert "`getSearchFormElement()` expects a string for the value parameter"

This reverts commit d2beab69c1aee0a33cfe8bdb7d967da2300b1691.

Merge pull request #5143 from WoltLab/require-absolute-path

Use absolute paths in require() calls during script init

Use absolute paths in require() calls during script init

Relative paths in `require()` are relative to the accessed “entrypoint” (i.e.
not the script that calls `require()`). By using absolute paths the requiring
is more resilient and more specifically the `acp/global.php` no longer depends
on an application correctly configuring `RELATIVE_WCF_DIR`, removing one of the
remaining users of the `RELATIVE_*` constants.