Merge pull request #5426 from WoltLab/update-safety-check

Add package update safety check with regard to version numbers

Fix the recognition of non namespaced variable values when editing a style

See #5376

Fixed the compilation of styles when editing them

Fixes #5376

Add package update safety check with regard to version numbers

During a multi-package update, e.g. when upgrading to a new major release, the
entire installation plan is built upfront. Do to so the package system needs to
keep track of the to-be-installed packages and package versions to properly
resolve update instructions and dependencies.

This is done using the `PackageInstallationNodeBuilder::$pendingPackages`
variable which needs to be updated at exactly the right locations. Historically
there were a few issues with incorrect versions being stored in that variable
which in turn might cause the installation to use the wrong instructions based
on the wrong assumed version number.

To ensure that such a broken installation plan does not cause havoc, the
assumed version is now stored within the start marker node. When it's time to
actually install the package this assumed version is compared to the actual
version. If there's a mismatch, the update is aborted at a safe point: The
previous package is fully installed. For the current package with the wrongly
selected instructions no actual installation logic was performed yet. The admin
can retry the update and now the installation process will (hopefully) select
correct instructions when starting from a clean state.

see #5409

Update composer dependencies

Slightly increase the spacing between a category and its first item

Use CSS classes for text alignment

Fixes WoltLab/editor#31

Update fileDelete.xml

Align the generated HTML of quotes with the editor’s expectations

Fixes WoltLab/editor#32

Add proper types to various RouteHandler symbols

Change `protected` to `private` in RouteHandler

The RouteHandler is final anyway.

Fix the use of route values using a PSR request

Fix submit button in editHistory.tpl

Fix PHP codestyle

Move the “exception escapes from middleware” detection further out

Since abd060c56250c1a340732b8180c90efd49727ee9 there should not be such an
exception even during routing, not just after the middleware stack terminates.

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

Remove AMP relict

Merge branch '5.5'

Merge pull request #5425 from WoltLab/farewell-amp

Drop AMP support

Drop support for AMP

Redirect amp page to normal page

Release 5.5.11 dev 1

Merge pull request #5418 from WoltLab/middleware-error-handling

Stop using Exceptions to return error responses outside of a controller

Merge pull request #5424 from WoltLab/wcf-node-identifier-generation

Improve performance of `<wcfNode-*>` identifier generation

Remove the final IllegalLinkException from RequestHandler::handle()

Create the PSR request before checking if any route matches in RequestHandler::handle()

Leverage the NotFoundHandler in RequestHandler::buildRequest()

Prefer JSON error responses over HTML

Send the exception in JSON response in error controllers

Accept `exception` in JSON error responses

This is a replacement for the separate file, line, stacktrace and previous
fields and matches the HTML version of these error responses.

Stop wrapping selected exceptions into an AJAXException

These exceptions are converted into clean responses using the HandleExceptions
middleware.

Do not require the `previous` array in JSON error responses

Revert "Add FixAcceptHeader middleware"

This does not appear to be generally useful. The JavaScript was modified to
send appropriate headers now.

This reverts commit fcdb3da3976359f8077c2f3a5ef2af5ebdd2d89c.

Set explicit `Accept` header in Ajax/Request.ts

Set explicit `accept` header in DboAction.ts

Add a dummy exception in ErrorDetail::fromMessage()

Detect exceptions that erroneously escape the middleware stack

Add FixAcceptHeader middleware

Add XsrfValidationFailedHandler

Merge pull request #5423 from WoltLab/wcf-node-autoloader

Stop needlessly invoking the autoloader to `<wcfNode-*>` tags

Merge pull request #5422 from WoltLab/script-pip-directory

Unify package directory calculation of database and script PIP with file pip

Remove the legacy jQuery based comment implementation

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

Add disabled appearance for icon inside disabled elements

Remove legacy `-o` suffix from icon name in UploadFile::getIconName()

Fixes #5406

Improve performance of `<wcfNode-*>` identifier generation

StringUtil::getRandomID() leverages the CSPRNG and constant time encoder to
generate IDs which is the secure, but slow, default choice.

For generation of the identifiers of `<wcfNode-*>` it is not required that the
values are absolutely unguessable, instead uniqueness and reasonable
unpredictability is sufficient, especially as the values are regenerated for
every request.

For content with a large number of BBCodes the overhead of a secure ID
generation can add up:

For a post with 300 BBCodes that is rendered 50 times the old ID generator
resulted in ~3300ms total rendering time, whereas the new ID generator with PHP
8.2 (thus leveraging xoshiro256**) is down to ~2500ms, a 25% reduction. However
even with PHP 8.1 the new generator will be faster, because it does not use the
constant time encoder. It will become even faster if PHP 8.2 is required and
the extra `$engine` closure can be removed.

Stop needlessly invoking the autoloader to `<wcfNode-*>` tags

These node names are randomly generated and will never match an existing class.
In fact the tag name transformation did not even result in valid classnames in
the majority of cases, because the randomly generated ID might start with a
number, thus including a hyphen in the classname.

Stop needlessly invoking the autoloader, and thus accessing the disk, for these
tags.

Unify package directory calculation of database and script PIP with file pip

Fixes #5392

Merge branch '5.5'

Merge pull request #5420 from darkwood-studios/5.5

added new articleLikeButtons template event to article template

Replace the use of Exceptions in middlewares

This is problematic, because Exceptions will skip the response direction of the
middleware stack. Instead appropriate responses are either generated directly
or forwarded to an appropriate request handler.

Add HandleExceptions middleware with request handlers to generate error responses

This allows controllers to still leverage Exception, but prevents the
Exceptions from skipping the response direction of the middleware stack.

Fix JavaScript relocation in HTML responses in middlewares

Add HeaderUtil::parseOutputStream()

Merge branch '5.5'

Re-randomize cronjobs when upgrading to 6.0

see #5202
see #4678

added new articleLikeButtons template event to article template

Updating minified JavaScript files

Merge pull request #5410 from WoltLab/ckeditor5-translations

Add the localization files for CKEditor 5

Improve the readability of the code for CKEditor localizations

Explicitly force the editor language

CKEditor implicitly detects an available translation, but only if it is defined first. Explicitly setting the language avoids any race conditions that could take place.

Rebuild the CKEditor 5 bundle to match the exported localization

Load the matching localization file on demand

Export the localizations for CKEditor except for 'en'

Merge pull request #5415 from WoltLab/amd-unset

Do not interact with `require()` while `define.amd` is unset

Merge pull request #5416 from WoltLab/plural-template

Improve error reporting for `value=null` in `{plural}` template function

Merge pull request #5417 from WoltLab/ckeditor-37

Migrate to proper CKEditor 37 types

Migrate to proper CKEditor 37 types

Improve error reporting for `value=null` in `{plural}` template function

Accidental `null`s are a common issue when dealing with database objects,
explicitly detect a `null` to not misleadingly report that the `value`
attribute is missing.

see https://www.woltlab.com/community/thread/299515-fatal-error-nach-update-missing-attribute-value/

Do not interact with `require()` while `define.amd` is unset

During development I noticed that regularly `focus-trap` failed to load
properly, likely due to being pulled in by some module while `define.amd` was
unset and thus failing to register itself as a module due to the UMD header.

Fix this by moving the `define.amd` unsetting and resetting into dedicated
`<script>` tags that wrap jQuery + WCF.js and nothing else.

Merge remote-tracking branch 'origin/master'

Transmit checkbox data as string in `Checked` form field

This makes the “on-the-wire” representation consistent with regular forms.

see #5412

Merge branch '5.5'

Fix reading boolean field values in form builder dialogs

For normal forms, the value of `BooleanFormField` is passed as a string. In
form builder dialogs, however, it is passed as an int.

Resolves #5412

Fixed JSON encoding in IconPreFilterTemplatePlugin (#5413)

see 50589568f191c4a3b2c015b60974378b22ccd24c

Merge branch '5.5'

Update composer dependencies

Merge pull request #5308 from WoltLab/article-mark-as-read

Migrate 'article mark as read' to new ajax api

Merge pull request #5374 from SoftCreatR/bugfix/userBBCodeTag-sync

Add `userBBCodeTag` to the shared templates

Fix the mismatching icon sizes of the new desktop user panel

Fixes #5408

Update composer dependencies

Disable the image upload when attachments are disabled

Fixes WoltLab/editor#27

Improve the visuals of the placeholder to add a comment

Closes #5395

Fix the visuals of the users online list

Fixes #5397

Preserve the `Image` plugin

This provides the general handling of image type content, such as smilies or media.

Fix the removal of the loading indicator when editing messages

Fix the position of the error message for replies

Fixes #5401

Strip the `border-radius` from the page search input

Fixes #5394

Fix the visuals of next/previous article without images

Fixes #5403

Fix the stacking of notification messages

Fixes #5396

Enforce the default box-sizing of buttons

The `inherit` value does not work well with `all: unset`, causing it to inherit the default value from the UA style sheet which is `content-box` instead of `border-box`.

Fixes #5402

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

Simplify implementation of the report function

Remove obsolete language variable

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

Improve the collection of message meta data

Fixes #5393

Merge branch '5.5'

Fix redirect after submitting ContactForm

The empty string is an invalid controller name. The landing page link is
requested by either passing `null` or leaving out all parameters.

This misuse will throw an Exception in WoltLab Suite 6.0.

Fixes #5407

Revert "Inline DynamicRequestRoute::getControllerName()"

The inlining of the method resulted in an unintentional and silent breaking
change for classes that register custom routes that, due to the missing
`final`, derive from DynamicRequestRoute and overwrite the method with a custom
implementation.

This reverts commit f9a440759dcf65edd108046b53babde404853ee8.

Add `individualScssDarkMode` to update_com.woltlab.wcf_6.0_styleVariables.php

see c292cfc3763a07281deb3b0767a5d84a6021b9a9
see 65bfb75589155b9ac1cbe8308cd7a79b07d699b7