Fix validation of form builder node attributes

Close #3827

Throw exception for VARBINARY columns without length

Close #3827

Validate column names in indices when using PHP DB API (#3832)

Close  #3828

Fix case of non-existing table in DatabaseTableChangeProcessor::getColumnByName()

Close #3829

Consider package during event listeners' name validation in PIP GUI

Close #3831

Fix DatabaseTableChangeProcessor::getColumnByName() for WSC < 5.4

See #3792

Fix missing variable error in DatabaseTableChangeProcessor

See #3792

Fix updating action of foreign key

See c64a33b40a7e601a6005f7ca247f7d2ecb52a393

Fix hiding/showing tab menu entries due to form builder dependencies

Excempt the admin panel from application overrides

Fixes #3762

Merge pull request #3799 from WoltLab/nonnull_in_primary_key

Validate primary indices with null columns in PHP DB API

Fix doubled validation errors for CaptchaFormField

Validate primary indices with null columns in PHP DB API

Close #3792

Merge branch '3.1' into 5.2

Force the use of the attachment thumbnail if the original is inaccessible

Merge pull request #3793 from WoltLab/style-import-permissions

Validate package installation permissions in StyleImportForm

Validate package installation permissions in StyleImportForm

Merge pull request #3786 from WoltLab/like-rebuild

Optimize LikeRebuildDataWorker performance

Fix dependencies for single media form field

Simplify selection of existing like objects in LikeRebuildDataWorker

Since the previous commit it is expected that only a single objectTypeID is hit
per worker execution in the usual case, because all objects for a single
objectTypeID are processed before proceeding to the next objectTypeID. This
patch optimizes this case by sending a separate query per objectTypeID to fetch
the existing like objects instead of combining all the objects in a single
query.

This should help MySQL in optimizing the query, because the objectTypeID (which
is the first column in the UNIQUE KEY on wcf1_like_object) is fixed for a
single query and this is immediately obvious without needing to scan all the
tuples passed to the IN(…) condition.

In the general case the number of queries does not change. In the iterations
where the last objectID for an objectTypeID is reached two queries will be sent
instead of a single one. Those two combined should be more efficient that the
single old query, though.

Allow LikeRebuildDataWorker to leverage the wcf1_like.objectTypeID key for sorting

Ignore invalid file ids during deleting uploaded files
There can be several reasons why the files are no longer known. We can ignore this and simply return a success message so that it doesn't pop up in the frontend when unknown files are deleted.

Fix detecting images with capitalised file extensions

Merge branch '3.1' into 5.2

Incorrect version comparison for accessible packages

Update the orphanedComments remark in package.xml

Missing check for empty object ids

Release 5.2.10

Merge branch '3.1' into 5.2

Release 3.1.18

Whitelist `iterator_count` in enterprise mode

Merge branch '3.1' into 5.2

Merge pull request #3744 from Krymonota/patch-18

Add missing JSDoc to `EmailSmtpTest`

Merge pull request #3745 from Krymonota/patch-19

Fix JSDoc module typo in `RewriteGenerator`

Fix point calculation for removing reactions from an object

Fix JSDoc module typo in `RewriteGenerator`

Add missing JSDoc to `EmailSmtpTest`

Updating minified JavaScript files

Merge branch '3.1' into 5.2

Updating minified JavaScript files

Workaround for a Chromium bug on Windows

Clean up comments when removing moderation queues (#3740)

* Delete comments when moderation queues are being removed

See #3707

* Clean-up script for orphaned comments

* Use the `CommentHandler` to remove comments

* Apply suggestions from code review

Co-authored-by: Tim Düsterhus <duesterhus@woltlab.com>
Co-authored-by: Tim Düsterhus <duesterhus@woltlab.com>

Replace the selector for the font-family in email_html.tpl

The Emogrifier version used in 5.2 does not fully support all features
of the `:not()` selector. Depending on the environment the use of
`:not()` results in slightly different output. Specifically the number
of `:not()`s chained there caused the result to have a very large
specificity, possibly overriding the font-size of the headline tags.

Use `body, body *` instead, it will not put any useless CSS onto heading
tags and still catch all relevant elements. The biggest difference is
that the `<br>` tag will carry a `font-family` and `font-size`.

Merge branch '3.1' into 5.2

Mutable variable access inside a callback

Merge branch '3.1' into 5.2

Fix ACL buttons when editing media files successively

Replaces 3a7740a3704339c79d08fd2f5094fa1a1973318b by backporting and using parts of 6ee58cd2865980cd7fa16a1f3ebc660131fc7fd6.

Revert "Fix ACL buttons when editing media files successively"

This reverts commit 3a7740a3704339c79d08fd2f5094fa1a1973318b.

Fix ACL buttons when editing media files successively

Fix file icon update after upload on media list page

Encode HTML special chars in form builder's action

Workaround for nested paragraphs after pasting in Firefox

Disallow changing time support after setting value in `DateFormField`

Delete FK logs after removing FKs due to column removal

Merge branch '3.1' into 5.2

Fix BBCodeHandler::setDisallowedBBCodes(...) PHPDoc (#3722)

Fix deleting obsolete nodes in form builder dependeny manager

Merge branch '3.1' into 5.2

Obsolete exclusion of the old ACP catpcha

This route does not exist anymore since WoltLab Suite 3.0.

Fix deleting obsolete nodes in form builder dependeny manager

Use native click event if available

Merge branch '3.1' into 5.2

Block search keyword list if permissions for the search function are denied

Closes #3681

Fix `Ui/Poll/Editor._reset()`

Fix de.xml

This fixes up commit 70746ece9789d3b2256495a788ae4875d01e95bc.

Fix typo in de.xml

Fix polls in AJAX form builder forms

Add data and options getters in poll editor js module

Fire event if field is registered in form builder manager

Merge branch '3.1' into 5.2

Fix GDPR export if invalid user id is given

… and stop using deprecated method `UserProfile::getUserProfile()`.

Whitelist `version_compare` in enterprise mode

Expired session during 3rd party auth led to a SystemException

(cherry picked from commit da8b0e89fa9da82fc7965955e7c0639a65d6ac00)

Expired session during google auth led to a SystemException

(cherry picked from commit 740ece8da10177354ab904f9aeb6ea914741c025)

Remove obsolete TODO

Fix reading medias' `captionEnableHtml`

See a1b75f8b9b9d6137434945bb8bea9518eddb48d3

Merge branch '3.1' into 5.2

Fix class name in UserRegistrationUtil::__construct() PHPDoc (#3652)

Merge pull request #3648 from WoltLab/actions

Refactor Workflows

Add javascript workflow

Rename php-syntax workflow to php

Adjust human readable name for CodeSniffer check

Merge pull request #3644 from Krymonota/patch-14

Add like support check to comment responses

Add like support check to comment responses

If the provided comment manager doesn't support likes, it's not possible to like comments, as can be seen here:

- https://github.com/WoltLab/WCF/blob/6747fe291c584d7c8295912f8d94297588d142b2/com.woltlab.wcf/templates/commentList.tpl#L59
- https://github.com/WoltLab/WCF/blob/6747fe291c584d7c8295912f8d94297588d142b2/com.woltlab.wcf/templates/commentList.tpl#L59

However, it's never checked for comment responses whether the comment manager supports likes. This results in the like button being displayed for comment responses, even though likes aren't supported.

Merge pull request #3612 from WoltLab/csrf-token-template-backport

Backport CsrfTokenFunctionTemplatePlugin

Whitelist `get_class` in enterprise mode

Merge branch '3.1' into 5.2

Ignore top-level node_modules/

Limit workflows to release branches

Features branches will be tested using a PR.

Merge pull request #3622 from WoltLab/workflow-style

Add codestyle workflow

Add codestyle workflow

Add .phpcs.xml

Merge pull request #3617 from WoltLab/formbuilder-php8

Fix PHP 8 support in form builder

Tighten up the code style a bit

Merge pull request #3620 from WoltLab/php-syntax-action

Add php-syntax workflow

Pin the setup-php action to v2 instead of 2.7.0

Ignore errors in HTMLPurifier*.autoload-legacy.php

Fix PHP 8 syntax in DirectoryUtil

Remove PHP syntax check from Travis CI

Suppress "No Syntax errors detected" output