Merge pull request #3900 from WoltLab/hasOwnerAccess

Fix User::hasOwnerAccess()

Check owner access after checking controller blacklist in RequestHandler

This ensures that the check only happens when absolute required.

Remove caching from User::hasOwnerAccess()

The current caching logic is buggy as reported in issue #3899. This patch
removes this caching, as this method already is quite fast and it also is
seldomly called. Within the frontend it is only called when the user is banned.

Fixes #3899

Whitelist `array_fill` in enterprise mode

Updating minified JavaScript files

Merge branch '3.1' into 5.2

Permissions for media were not saved correctly

Fix double html encoding in trophy activity events

Merge branch '3.1' into 5.2

Set explicit value for invalid select options

Support WysiwygFormContainer as children of ITabMenuFormContainer

Fix displaying error messages for devtool projects' instructions

Remove incorrect devtools form field description

Improve variable name

Merge branch '3.1' into 5.2

Merge pull request #3844 from WoltLab/fix-failing-cronjobs

Fix failing cronjobs

Fix VarbinaryDatabaseTableColumn

> Call to undefined method wcf\system\database\table\column\VarbinaryDatabaseTableColumn::traitGetLength()

see 4a65b1e27cc85fc9fb9bbcdf6a6d7a5f9cc76187

Hide article-related notification events if articles are disabled

Close #3845

fixup! Disable disabling of failed system cronjobs Fixes #3841

Disable disabling of failed system cronjobs
Fixes #3841

Prevent failing cronjobs from blocking other cronjobs by executing them last

Fix validation of required single media form fields

Improve readability of form field template code

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