Merge branch '5.2' into 5.3

Fixes twitch clips which contain a dash char

Twitch clips can contain a dash (´-`).
Example: https://clips.twitch.tv/NeighborlyManlyBaconMikeHogu-_9zfXbRK47vik1WO

Resolves #3996

Merge branch '5.2' into 5.3

Fix deleting an empty list of objects

It can happen that content that was previously counted no longer exists and therefore fewer runs are required than initially assumed. This can happen if, for example, a user's posts and threads are deleted and the user has only written posts in his or her own threads (the posts are then deleted along with the threads). This may have led to errors before because an empty list of objects was passed.

Merge pull request #3992 from WoltLab/export-mail-address-permission

Bind exportMailAddress bulk process to `canEditMailAddress` instead of `canMailUser`

Bind exportMailAddress bulk process to `canEditMailAddress` instead of `canMailUser`

Unfortunately the `permissions` form a logical disjunction, not a conjunction,
thus we can't check both permissions. The `canEditMailAddress` is more fitting,
because even without `canMailUser` the admin could simply look up all users
manually if they are allowed to edit (and thus see) mail addresses. It's just a
larger effort.

Resolves #3963

Deprecate wcf\system\exception\HTTP*Exception

see 5ff8745012d43b1bcdc5df3c74a19532379d182f
see 33893f8e7201269f326117c3b1eed1af31106c06

Deprecate \wcf\util\exception\HTTPException

see 5ff8745012d43b1bcdc5df3c74a19532379d182f

Merge branch '5.2' into 5.3

Additional template event in articleAdd template

Merge branch '5.2' into 5.3

Fix mixed boolean operators in condition in `TI18nFormField::validate()`

… and improve the preceding comment.

See #3926

Fix name of English language item

See 324aa486d32e0d1e7021385657fd1388d00403ca

Download Google fonts before validating the individual SCSS

The individual SCSS will compile the style and this compiled CSS will later be
used for the style itself.

When first downloading a new Google font the font will not yet be available
during this test, thus the necessary CSS will not be included which in turn
will cause the font to fail to display.

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Merge branch '3.0' into 3.1

Fix typo in phrases

Merge branch '5.2' into 5.3

Ignore name of primary keys during creation (#3921)

Long file name caused SQL error

Release 5.3.4

Merge branch '5.2' into 5.3

Release 5.2.12

Merge branch '3.1' into 5.2

Release 3.1.20

Merge branch '3.0' into 3.1

Release 3.0.26

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Merge branch '3.0' into 3.1

Missing check for accessible user groups

Merge branch '3.0' into 3.1

Notify users of the expiring support

Merge branch '3.0' into 3.1

Ignore top-level node_modules/

Merge pull request #3920 from WoltLab/http-request-proxy

Fix HTTPRequest with proxies

Stream HTTPRequest responses only of a maximum length is desired

cURL is not supported for streaming responses. As we process the full response
body for requests without a maximum length there is no benefit to streaming the
response, because we lose cURL support.

Update Guzzle

see guzzle/guzzle#2848
see guzzle/guzzle#2850

Merge branch '5.2' into 5.3

Fix which index object is used when dropping indices with PHP API

Only `$matchingExistingIndex` is guaranteed to have the correct index name.

Merge branch '5.2' into 5.3

Fix checked property when adding indices to `DatabaseTable`

Merge branch '5.2' into 5.3

Release 5.2.11

Merge branch '3.1' into 5.2

Release 3.1.19

Updating minified JavaScript files

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Fix converting float value to integer

Release 5.3.3

Object edit link led to the admin panel

Make update_com.woltlab.wcf_5.3_packageServer.php compatible with WCF_N != 1

Merge branch '5.2' into 5.3

Merge pull request #3908 from WoltLab/pr_build_package

Add wcfsetup workflow

Add wcfsetup workflow
This workflow generates the WCFSetup and stores it as an artifact.

Improved message for rejected credentials on paid packages (#3903)

Merge branch '5.2' into 5.3

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

Fix bad revert

see 6e01c38f9218c5763130f19814f3d5b94cca26be

Reformat update_com.woltlab.wcf_5.3.3_db.php to use Tabs

Code style for 5.3 expects tabs, not spaces. PHP CodeSniffer complains.

Add PSR-12 reformatting commits for 5.3 to .git-blame-ignore-revs

see 6e01c38f9218c5763130f19814f3d5b94cca26be
see 9faac4ad4b5d27f9159e531b645ed3f4088adb3a

Revert "Reformat EventListenerPackageInstallationPlugin to PSR-12"

This reverts commit 9faac4ad4b5d27f9159e531b645ed3f4088adb3a.

Merge branch '5.2' into 5.3

Whitelist `array_fill` in enterprise mode

Support `environment=all` in event listener PIP GUI

Reformat EventListenerPackageInstallationPlugin to PSR-12

Support `environment=all` for event listeners in database

See #3145

Merge pull request #3884 from WoltLab/password-phpbb-combined

Add support for phpBB's combined hash

Add support for phpBB's combined hash

Co-authored-by: Alexander Ebert <ebert@woltlab.com>

Set 'stream' to `true` for Guzzle in `HTTPRequest`

This is required to properly support the 'maxLength' option on endless streams
(such as a web radio). Without setting 'stream' to `true`, Guzzle attempts to
download the entire response body before returning from `->send()`.

Merge pull request #3873 from WoltLab/user-online-performance

Fix performance of user online list

Fix performance of user online list
If the UserStorageHandler has to load all users that are online, this is quite resource intensive in larger communities and the query is very slow. The UserStorageHandler must actually be loaded so that the permissions of the users can be checked to see if they can make themselves invisible. We switch off this check with this commit and assume that users who cannot change this setting are always online.

Updating minified JavaScript files

Merge branch '5.2' into 5.3

Updating minified JavaScript files

Unify reaction count button style with notification style

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Permissions for media were not saved correctly

Add spacing beetween reaction icon in notifications

Fix double html encoding in trophy activity events

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Set explicit value for invalid select options

Merge branch '5.2' into 5.3

Support WysiwygFormContainer as children of ITabMenuFormContainer

Merge branch '5.2' into 5.3

Fix displaying error messages for devtool projects' instructions

Remove incorrect devtools form field description

Improve variable name

Merge pull request #3865 from WoltLab/fix-style-preview

Fix the style preview images

Fix check for deleted notifications in NotificationEmailDeliveryBackgroundJob

Fix multiple queries for user online list stats

Merge pull request #3867 from WoltLab/smtp-auth-fail

Fail SMTP auth if credentials are configured but all mechanisms fail

Properly detect EOF in SmtpEmailTransport::read()

Disconnect SMTP session for all types of Exception during connect / auth