Take the array key into account when matching up FOREIGN KEYs in DatabaseTableChangeProcessor

Previously a FOREIGN KEY within the database that looks like this:

    […] FOREIGN KEY (someOtherUserID) REFERENCES wcf1_user (userID) […]

would match up a FOREIGN KEY definition like the following:

    […] FOREIGN KEY (userID) REFERENCES wcf1_user (userID) […]

Converted into the `getDiffData()` representation of the PHP DDL API these
would like:

    [ 'columns'           => 'someOtherUserID'
    , 'referencedColumns' => 'userID'
    , 'referencedTable'   => 'wcf1_user'
    ]

and

    [ 'columns'           => 'userID'
    , 'referencedColumns' => 'userID'
    , 'referencedTable'   => 'wcf1_user'
    ]

respectively.

Now taking the diff of the second array against the first array (subtracting
the first from the second) will remove *both* 'userID' values, resulting in an
empty difference, thus believing both FOREIGN KEYs are identical

Fix this issue by using `array_diff_assoc` which will also take the key into
account.

Merge pull request #4427 from WoltLab/label-form-field-dependency

Add missing check for dependencies in LabelFormField processor

Add missing check for dependencies in LabelFormField processor

Merge branch '3.1' into 5.2

Encode HTML in smileyTitle in Smiley::getHtml()

Merge branch '3.1'

Don't leak an account's activation status (#4365)

* Don't leak an account's activation status

* Update wcfsetup/install/files/lib/form/RegisterNewActivationCodeForm.class.php

Co-authored-by: Tim Düsterhus <timwolla@googlemail.com>
Co-authored-by: Tim Düsterhus <timwolla@googlemail.com>

Merge branch '3.1' into 5.2

Remove duplicate `class` attribute in cacheList.tpl

Merge branch '3.1' into 5.2

Fix typo in de.xml

Closes #4366

Co-authored-by: Sascha Greuel <sascha@softcreatr.de>

Merge branch '3.1' into 5.2

Fix typo in de.xml

Use “Moderation” instead of “Moderatoren” in de.xml

Merge branch '3.1' into 5.2

Set 'accept-encoding: identity' by default in HTTPRequest

RFC 7231#5.3.4 allows a server to use an arbitrary 'content-encoding' when no
'accept-encoding' request header is sent.

Make it clear that no 'content-encoding' is acceptable by only accepting the
'identity' encoding.

Merge branch '3.1' into 5.2

Fix 'to its full extent' in en.xml

Add missing `wcf.acp.devtools.pip.error.className` phrase

Close #4256

Merge branch '3.1' into 5.2

Merge pull request #4209 from WoltLab/comment_moderation_queue

Delete moderation queue entries after deleting comments/comment responses

Delete moderation queue entries after deleting comments/comment responses

Close #4208

Add missing `$definitionName` value for comment response moderation queue handlers

Preserve article sorting when using pagination

Merge branch '3.1' into 5.2

Fix ignoring the disable state of an article category

Deactivated categories are currently only hidden in the overview of categories in the frontend. However, articles in these categories (and the category itself, via the direct link) were still accessible. This patch solves the problem by hiding the category for all users and making articles and the category itself no longer accessible for any user.

Fix error messages of numeric form fields for minimum/maximum

Add missing `license` element in `package.xsd`

Merge branch '3.1' into 5.2

Reset language form field in language import form after saving

Merge branch '3.1' into 5.2

Fix detection of nested tables in messages

Close #4175

Fix table borders when cells with `rowspan` are visually in the last row (#4176)

See https://community.woltlab.com/thread/289598

Add missing return value of `readValue()` of DevTools project form fields

Do not write empty i18n package information in `package.xml`

Merge branch '3.1' into 5.2

Merge pull request #4143 from WoltLab/3.1-fix-sitemap

Fix generating sitemaps with factor 250 objects sitemaps

Fix generating sitemaps with factor 250 objects

Only validate minimum text length if any text is given

The case of no text having been entered and the field being required is already handled separately.

Merge pull request #4133 from WoltLab/article-update-outdated

Ensure that the creating of related rows uses up to date article fields

Ensure that the creating of related rows uses up to date article fields

Previously the code would store outdated data in the search index and recent
activities when changing the article's timestamp while publishing the article
or when changing the article's author.

Merge branch '3.1' into 5.2

Update the number of responses that will be loaded

Updating minified JavaScript files

Release 5.2.14

Merge branch '3.1' into 5.2

Release 3.1.22

Add missing permissions check when removing moderated content

Fix `reloadPageOnSuccess` from different clipboard actions for same object type

If there are multiple clipboard action classes for the same object type, for example in case a plugin adds further clipboard actions, only the `reloadPageOnSuccess` data of the first clipboard action instance is used instead of collecting the data from all relevant clipboard action instances.

See #2584

Merge branch '3.1' into 5.2

Updated Google's address in the privacy policy

Merge pull request #4112 from WoltLab/user-option-title

Use UserOption::getTitle() where possible

Use UserOption::getTitle() in userOptionList.tpl

This one was missed in the previous commit.

Use UserOption::getTitle() where possible

Not all places could be adjusted for compatibility reasons, as the `Option`
super class does not have the `getTitle()` method and as the prefix is given
explicitly.

Helper methods for the description should also be added.

see #4107

Merge branch '3.1' into 5.2

Reset the assignment cache on group deletion

Fixes #4045

Merge pull request #4100 from SoftCreatR/patch-2

Add support for youtube shorts

Lack of filtering of content from ignored users

Merge pull request #4098 from netzhuffle/patch-4

Fix param type for handleException

Merge branch '3.1' into 5.2

Update the user rank after users change their profile

Added support for youtube shorts

Merge branch '3.1' into 5.2

Strip rtl override char

Fix param type for handleException

WCF::handleException incorrectly stated it would receive only \Exception, while it is registered to receive any \Throwable.

Fix reading ACL values in non-Ajax form builder forms (#4060)

The wrong data source was used in `AclFormField` (the whole `$_POST` array instead of the dedicated entry) and the data was always stored in `aclValues` instead of a dedicated entry per form field.

Fix Ajax user form fields with pre-set values

`values[i].objectId` is only set for users added manually via the UI. For pre-existing usernames, only `values[i].value` exists.

Release 5.2.13

Merge branch '3.1' into 5.2

Release 3.1.21

Merge branch '3.0' into 3.1

Add missing permission check to ModerationQueueAction::validateAssignUser()

This only affected registered users. Guests never could trigger this action,
because it is not listed in `$allowGuestAccess`.

Merge branch '3.1' into 5.2

Merge branch '3.0' into 3.1

Release 3.0.27

Adjust upgrade notice

see f4a8700ae34fa3120576c5ab4e2096f514cffdc5
see 14a6db14f1ac63da83098dc648288ae07df62e80

Add upgrade instructions for 2.1.24 pl 3

Properly handle empty logfiles in ExceptionLogUtil::splitLog()

Properly handle corrupted or huge log files in exception mailer

Merge branch '3.1' into 5.2

Match all prefixes for store.woltlab.com in PackageAction::validateSearchForPurchasedItems()

see #3910

Merge branch '3.1' into 5.2

Merge branch '3.0' into 3.1

Merge pull request #3997 from WoltLab/major-upgrade-no-upload

Reject major com.woltlab.wcf upgrades via package upload

Reject major com.woltlab.wcf upgrades via package upload

Fixes twitch clips which contain a dash char

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

Resolves #3996

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.

Additional template event in articleAdd template

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

… and improve the preceding comment.

See #3926

Merge branch '3.1' into 5.2

Merge branch '3.0' into 3.1

Fix typo in phrases

Ignore name of primary keys during creation (#3921)

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 '3.1' into 5.2

Merge branch '3.0' into 3.1