Merge pull request #4816 from WoltLab/template-engine-psr-7

Add TemplateEngine::fetchStream() to efficiently render a template as a PSR-7 StreamInterface

Add TemplateEngine::fetchStream() to efficiently render a template as a PSR-7 StreamInterface

Merge pull request #4815 from WoltLab/wcfsetup-relative-wcf-dir

Remove the need for SetupTemplateCompiler

Move the output generation in WCFSetup::installPackages() to a more appropriate location

Remove SetupTemplateCompiler

The implementation only differed from the TemplateCompiler /
TemplateScriptingCompiler by not rejecting the use of RELATIVE_WCF_DIR. As that
usage was removed in the previous commit the SetupTemplateCompiler is obsolete.

Do not reference `RELATIVE_WCF_DIR` in stepInstallPackages.tpl

Merge pull request #4814 from WoltLab/wcfsetup-tmp

Clean up the handling of temporary files during WCFSetup

Always use install.php's __DIR__ as the temporary directory

The previous implementation of `BasicFileUtil::getTempFolder()` was unstable,
as it automatically detected `tmp/` directories in the document root. If
WoltLab Suite Core is installed into the document root, then the extraction of
the files will implicitly create a `tmp/` directory and thus change the temp
folder for the remaining installation.

Increase the randomness in `TMP_FILE_PREFIX` in install.php

Remove all extracted temporary files in WCFSetup::installPackages()

Move the package archives into WCF_DIR/tmp in WCFSetup::installPackages()

This allows for a complete cleanup of the temporary directory.

Remove error handling for broken archives from WCFSetup

Any errors are generally not recoverable at this point, making the error
handling useless.

Simplify calculation of `$processNo` in WCFSetup::installPackages()

Similarly to the previous commit we know for sure that no process numbers are
taken yet, so we can hardcode this to 1.

Remove useless check whether com.woltlab.wcf is already installed in WCFSetup::installPackages()

The database structure was just created in WCFSetup, we know for sure that the
wcf1_package table is empty.

Remove unneeded decompression of packages in WCFSetup::installPackages()

As indicated in the previous commit, the fact that this decompression did not
happen for Strato implies that it is not actually necessary in practice.

Remove Strato workaround from WCFSetup

This workaround exists since the very first commit and it's not clear why that
was needed back then. It likely was related to the temporary directory not
being writable or files not being deletable in some cases, but with the change
of WCFSetup's temporary directory anything less than full writing permissions
to the directory mean that WoltLab Suite won't actually run after installation
anyway, making this workaround obsolete.

Also the fact that decompression of the archive does not happen in all cases
implies that the decompression is not actually necessary at this stage, making
that the next candidate for removal.

Use the DOMContentLoaded event in WCFSetup

see 99e56d2929be14b1f5141e6a5136ff7c7de6f39d
see 0479cdecbc8788c5d980490af9c2fda00753cd6a

Revert "Remove unneeded `window.onload` callbacks in WCFSetup JavaScript"

This change had the undesired effect that the footer did not show. Instead the
more appropriate `DOMContentLoaded` event should be used.

This reverts commit 99e56d2929be14b1f5141e6a5136ff7c7de6f39d.

Merge pull request #4813 from WoltLab/wcfsetup-javascript

Simplify JavaScript handling in WCFSetup

Merge pull request #4812 from WoltLab/admin-panel-user-header

Simplified the admin panel‘s user header

Remove unneeded `window.onload` callbacks in WCFSetup JavaScript

Disable JavaScript relocation in WCFSetup

There is no benefit to this in WCFSetup, it just increases complexity.

Merge branch '5.5'

Fix code style in WCFACP

Apparently PSR-12 allows either brace placement, with PHP-CS-Fixer preferring
the brace on the same line and Intelephense preferring it on the next line. If
a multi-line interface definition would be used here, then the brace must be on
a dedicated line, making Intelephense slightly more right on this one.

However as WCFACP is expected to be touched seldomly, thus seldomly running the
Intelephense formatter, the pragmatic solution is to satisfy PHP-CS-Fixer.

see FriendsOfPHP/PHP-CS-Fixer#6396 for a related issue

Simplified the link to access the frontend

Remove user links expect for the logout

This is still not ideal, the header might need a slight redesign to better represent the logged-in user and the logout button.

Fix typo in WCFSetup

Merge pull request #4811 from WoltLab/wcfsetup-finalize

Clean up finalization of WCFSetup

Remove unneeded `file_exists` check for install files

With the absolute path within the constants we know for sure that these files
must exist.

Use appropriate constants to identify to-be-deleted install files

Do not suppress errors during removal of setup files in WCFSetup::installPackages()

It is unsafe if these files remain. Also it is very likely that the system not
actually capable of reliably running WoltLab Suite if something as simple as
deleting a file fails.

Remove `*Deleted' template variable assignment in WCFSetup::installPackages()

These are not actually referenced in the template.

Merge pull request #4810 from WoltLab/gzip-file-dynamic-dispatch

Remove dynamic dispatch of common cases from GZipFile

Remove the `lang` prefilter from WCFSetup's template engine

WCFSetup is not performance critical and the less logic that runs the better.

Remove list of to-be-installed packages from WCFSetup

This is an incredible performance bottleneck, because it reparses all the
archives in every request.

Remove GZipFile::$gzopen64

This property is no longer accessible since GZipFile is `final`.

Mark GZipFile as `final`

There is no good reason to inherit from this class and the performance
optimization for the commonly used method calls might break derived classes
that rely on `->__call()` calling something else.

Remove the need for dynamic dispatch for common GZipFile methods

`->read()`, `->tell()` and `->seek()` are frequently used methods when parsing
Tar archives. We know for sure that these will be mapped to the corresponding
`gz*` function. Define them statically to avoid the need for repeated
`function_exists()` checks in `__call()`.

Replace calls to `ord()` in checksum calculation of Tar::readHeader() with a table lookup

This makes Tar header processing much faster, due to the reduced number of function
calls (> 3.1 million calls to `ord()` in a WCFSetup with WoltLab Suite Forum
included).

see ec4be13babcd7e1579c08f1805412fcc9f3d65c5

Reduce the number of function calls in Tar::readHeader()

Merge pull request #4785 from WoltLab/remove-multi-domain

Remove multi-domain support

Merge branch '5.5'

Merge branch '5.5'

Unified phrases for deprecated options

Deprecated AMP support

PR #4803 accidentally had the wrong target branch.

(cherry picked from commit 8c97dca561b32b3c02f3cb30b32574a8f83b8bd5)

Fix code style

Revert "Use checkstyle output for php-cs-fixer"

This change was meant to improve the output, by properly annotating the files,
but instead it makes the output absolutely useless, as it does not include line
numbers.

This reverts commit 65a9247ac9566923cf89e4055c001833622996a6.

Simplified checks against the domain name

Redesigned the rescue mode, enforce shared domain

Added documentation of minvalue / maxvalue

Email address was not visible in listing of group members

Remove obsolete check for `->getAttributes()` method on ReflectionFunction/ReflectionMethod

Deprecate the obsolete WCF::useDesktopNotifications()

Merge branch 'master' into remove-multi-domain

Merge branch '5.5'

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

Incorrect handling of the return focus

Related https://www.woltlab.com/community/thread/295562-inhalte-k%C3%B6nnen-doppelt-gemeldet-werden/

(cherry picked from commit e9ffefb4d0db61bf8e12b3824c6515ac16fe3812)

Revert "Suppress errors when disabling the focus trap"

This commit duplicates commit e9ffefb4d0db61bf8e12b3824c6515ac16fe3812 which is
the cleaner fix, but was accidentally commited to `master` only. We revert this
one to allow for a clean cherry-picking of
e9ffefb4d0db61bf8e12b3824c6515ac16fe3812.

This reverts commit e7439c4f5fc7daf8d45d2de478c488798ddf463a.

Merge pull request #4808 from WoltLab/55-incorrect-http-204-handling

HTTP 204 does not carry a response type

Install phpcs via `shivammathur/setup-php`

Apparently the previously used actions uses an out of date phpcs.

(cherry picked from commit c5cd836751c3fb357b8ddce52a95a83f3ac46347)

Consistent evaluation of the response content type

Proper handling of a 204 response

HTTP 204 does not carry a `content-type`

See https://www.woltlab.com/community/thread/295631-importer-h%C3%A4ngt-bei-reaktionen-100/

Merge pull request #4805 from WoltLab/package-xsd

Added documentation of the void element

Use correct CodeMirror mode for SCSS inputs

`sass` is the indentation based SASS syntax.

Incorrect markup in the authentication dialog

Suppress errors when disabling the focus trap

Use CSS variable to apply the dialog offset

Revert "Dynamic width of code box line numbers"

This reverts commit d3853740d8304f9703876006db990c2e01642656.

Dynamic width of code box line numbers

This avoids indenting short code listings too much, especially on screens with limited space.

Do not wrap code boxes on mobile

Having long lines wrap multiple times is horrible, overflow scrolling is much better and also quite natural on touch devices.

Better accessibility of code box buttons

Newer iOS versions permit a click near the bottom

Prevent the "external url" indicator to wrap

See https://www.woltlab.com/community/thread/295487-einzelnes-umbrechen-des-externalurl-icons-verhindern/

Expand the active menu item by default

Merge pull request #4807 from WoltLab/55-desktop-notifications

Remove the option to disable desktop notifications

Align the mobile main menu to the right

See https://www.woltlab.com/community/thread/295627-mobil-men%C3%BC-links/

Suppress error messages when aborting `fetch()`

See https://www.woltlab.com/community/thread/295566-fetch-is-aborted-bei-suchergebnissen/

Remove the option to disable desktop notifications

Fixes #4806

Added documentation of the void element

Incorrect handling of the return focus

Related https://www.woltlab.com/community/thread/295562-inhalte-k%C3%B6nnen-doppelt-gemeldet-werden/

Merge pull request #4801 from WoltLab/template-event-javascript-head-include

Remove long-deprecated JavaScript template events from `headInclude.tpl`

Merge pull request #4800 from WoltLab/convert-encoding

Deprecate StringUtil::convertEncoding()

Merge pull request #4802 from WoltLab/sensitive-parameter

Use PHP 8.2's SensitiveParameter argument

Merge branch '5.4' into 5.5

Truncate error message in cronjob log if necessary

Support for meta tags during article import

Typo

Merge branch 'master' into remove-multi-domain

Remove accidentally commited bootstrap.php

see 4f33a332b8aaf66e6c2fb1c9919063146c328c56

Use Guzzle in PackageAction::searchForPurchasedItems()

see #4281

Use Guzzle in PackageUpdateDispatcher::getPurchasedVersions()

see #4281

Merge pull request #4803 from WoltLab/deprecate-amp

Deprecated AMP support

Deprecated AMP support

Replace `\wcf\SensitiveArgument` by `\SensitiveParameter`

Deprecate `\wcf\SensitiveArgument`

Add support for PHP 8.2's native SensitiveParameter attribute

Use checkstyle output for php-cs-fixer

Install phpcs via `shivammathur/setup-php`

Apparently the previously used actions uses an out of date phpcs.

Merge branch '5.5'