Remove obsolete code

Fix indentation

Apply suggestions from code review

Remove obsolete guest dialog code

Made use of the new backend methods

Add typescript methods for requesting the new backend

Add helper method to get a comment manager by id

Remove/deprecate obsolete code

Add RPC controller

Add commands

Add PSR-14 events

Add a check that no attachment files have been deleted

No longer add the description with the limits for uploading via the description. This information is already available in the template

It Was only necessary for testing

Merge branch '6.0'

Do not focus the editor when resetting it

See https://www.woltlab.com/community/thread/306677-js-fehler-beim-antworten-auf-kommentare/

Fixes the problem if `maxHeight` and or `maxWidth` have the value `-1`. In this case, the images were scaled anyway.

Fix wrong object type when deleting comment responses

Merge pull request #5943 from WoltLab/bugfix/wysiwyg-form-field-attachment

Bugfix with WysiwygAttachmentFormField and FileProcessor

Merge pull request #5942 from WoltLab/upload-pipeline-update-database

Add missing database columns for `wcf1_attachment`

Simplify the usage of the helper function

Fixes the problem that when using the WysiwygFormField with attachments, the `objectID` can be `null` if it is not an EditForm.

Add missing database columns for `wcf1_attachment`, that are installed by the database pip

Use a helper function instead of a property

Using a property was a dumb idea because it breaks the promise of having the entire request data be made available through the parameters passed to `__invoke()`.

Fix the deletion of attachments through the file API

Add a helper attribute for object hydration

Remove superfluous columns

Those columns were incorrectly copied and should not exist on `wcf1_file_temporary`.

Improve the handling of server errors

Migrate the important workers to the linear rebuild worker

Enforce a consistent processing by id

Merge pull request #5939 from WoltLab/rebuild-html-performance

Runtime and rebuild performance improvements

Merge pull request #5940 from WoltLab/guest-token

Guest tokens

Guest tokens

Merge branch '6.0'

Merge branch '5.5' into 6.0

Merge pull request #5932 from SoftCreatR/bugfix/log-recursive

Disable log directory recursion

Change template name to shared version

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

Remove obsolete jquery code for the captcha registration

Convert `captchaQuestion` to a shared template

Inline the call to `DatabaseObject::__get()`

See eaf3ce8100824d6c805b6d588b67d02bb58a18b7

Improve the runtime performance of `|date`

See 75f848c4f07ef00dfa0b191665f40d8a81fa2d0c

Inline the call to `DatabaseObject::__get()`

`User::__get()` is called very often on each request, easily stacking up thousands of calls.

On a well populated board list this can easily account for up to 1% of runtime. Inlining the check against `$data` cuts down the time spent inside by up to 2/3.

Reuse the `\IntlDateFormatter`

Constructing the object every time is a rather expensive process.

The test for a page with 96 calls to the `|time` modifier showed an average of 8ms spent inside the `execute()` method.

The optimized version showed a consistent runtime of around 1.3ms.

Improve the rebuild of users

The regular query using `LIMIT` and `OFFSET` is painfully slow with large datasets.

Rebuilding 500k users previously took about 3 hours, afterwards the same action can be carried out in about 20 minutes.

Improve the rebuild of user activity points

The previous implementation relied on `LIMIT … OFFSET …` which has an incredibly poor performance on large data sets. Changing this to `BETWEEN` yields a consistently low latency.

In the previous implementation, up to 92% of the request was spent fetching the target rows.

For a test with 500k records this cut down the time to rebuild from 2:30h down to 20m.

Cache the list of source and block BBCodes

These methods exist in the hot path for the BBCode processing and are called often, but are known to have a static result at runtime.

Caching the result shows latency improvements of up to 3%.

Cache the list of known HTML tag handlers

The `\class_exists()` call for handlers that do not exist always go through the autoloaders.

For large sets of documents this change improves the request latency by up to 2%.

Improve the parsing performance of HTML 5

LibXML handles HTML5 documents poorly and raises a lot of errors that are propagated to PHP and eventually get suppressed by the `@` operator.

This change will prevent LibXML from propagating the errors, allowing us to discard them without going through PHP.

For medium to large size messages this speeds up the `loadHTML()` call by 30-60%. For large sets of messages this can yield a latency reduction for the request of up to 5%.

Merge pull request #5905 from WoltLab/upload-pipeline-v2

Upload Pipeline v2

Mark the legacy file system methods as deprecated

Add support for copying files

Remove legacy methods for attachment handling

Replace references to the old JS for attachments

Add the SQL structure to the DDL PIP

Create a WebP variant of the source file

Improve the eTag handling of file downloads

Migrate attachments to the new file upload API

Add a new worker implementation with a linear, predictable runtime

Add an option to limit the maximum file size

Validate the file size before querying the server

Fix the ETag handling

Increase the length of the secret to 32 characzers

Fix the `expires` header and force lowercased http headers

Move the ETag handling and switch to weak comparisons

Add support for limiting the number of uploaded files

Fix the call to a removed helper function

Add checksums for thumbnails and a rebuild worker

The checksum is computed based on the configuration of the thumbnail, allowing to detect when a thumbnail needs to be regenerated. If the checksum matches, the file hash is checked instead to detect damaged files.

Delete orphaned files

Add caching support to file downloads

Clean up files on delete

Add support for download tracking

This partially reverts the changes made in bde83e6b8fc090654deaac827b342cea0b69695d

Minor code improvements

Add the documentation for the file processor interface

Add a default implementation for file processors

Add basic error messages

Add button phrases and simplify the button creation

Clean up the error and editor id handling

Move the attachment logic into a separate file

Reorganize the action buttons for attachments

Initialize the image viewer for dynamic attachments

Use `objectTypeID` instead of `typeName`

Add support for the image viewer for attachments

Implement a cleanup on file delete

Remove `downloads` and `lastDownloadTime`

These have been meaningless metrics for most of the time and also do not play nicely with the efficient distribution of files.

Improve the UI of uploaded attachments

Add some basic styling for uploaded attachments

Add support for uploads through drag and drop on the editor

Block drag & drop for illegal file extensions

Add the link to the full version to files

Initialize existing attachments

Bind uploaded attachments to the current user

Add an automated resizing for images exceeding the limits

Prevent the generation of thumbnails for small images

Unify the styling of attachments below a message

Fix the loading of the files of attachments

The list of files (plus their thumbnails) is already fetched in the base implementation `AttachmentList`.

Add a secret to upload files

The placement of files depends on the file extension, allowing files to be efficiently served by the web server instead of blocking a PHP worker.

Use file extensions other than `.bin` for safe types

Skip thumbnails for non-image files

Use `File::getSourceName()` to avoid accidental leaks