Release 5.3.15

Updating minified JavaScript files

Merge branch '5.2' into 5.3

Release 5.2.16

Updating minified JavaScript files

Merge branch '3.1' into 5.2

Updating minified JavaScript files

Fixed suffix declaration in XSD

As of now, a suffix is only allowed for option categories, which is wrong,
because a suffix can only be applied to an option itself.

Resolves #4596

Release 3.1.24

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Incorrect reset of the timer to show the loading indicator

The missing reset caused repeated calls to `show()` being effectively ignored, because the check `_timeoutShow === null` would be always true if the callback was cancelled before.

See https://community.woltlab.com/thread/293232-ajaxstatus-wird-nach-einem-fehlerhaften-request-nicht-wieder-angezeigt/

Merge branch '5.2' into 5.3

Fix error message for `foreignColumnChange` in PHP DDL API

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Merge pull request #4574 from WoltLab/supportexpiry-31

Notify users of the expiring support

Notify users of the expiring support

This effectively is 14a6db14f1ac63da83098dc648288ae07df62e80 combined with
637c1ca28225918a1a07680b7759c02565cd4c50.

Regenerate composer autoloader

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Fix EmailNewActivationCodeForm

This got broken, because it inherits from RegisterNewActivationForm and the “is
already enabled” validation was moved into a dedicated method within there. This
is a perfect example of why one should never inherit from controllers …

see f394421c0cc7e8879007092e40e540b2fd1118c1

Merge pull request #4531 from WoltLab/http-request-timeout

Configure emergency timeout in HTTPRequest

Cast the Redis port to int

The `Redis::connect()` method expects the `$port` parameter to be an integer.
PHP will automatically cast numeric strings to an integer, but error out with
an TypeError if the string is not a well-formed number. This TypeError will not
be caught in an `catch(\Exception $e)` block, because TypeError does not
inherit Exception.

Perform an explicit cast to ensure the fallback to DiskCacheSource works.

Configure emergency timeout in HTTPRequest

The connect and read timeouts might not reliably trigger in all cases.
Configure a large overall timeout to ensure PHP workers will terminate
eventually.

see 2dbd5654cb9faff45bb51df9a2f3834bd320cc00

Merge pull request #4497 from max-m/patch-categoryMultiSelectOptionType

Make `categoryMultiSelectOptionType.tpl` behave like `categoryOptionList.tpl`

Incorrect gradient value in Safari

https://community.woltlab.com/thread/292475-mainmenushowprevious-mainmenushownext-safari-farbunterschied-fehler/

Make `categoryMultiSelectOptionType.tpl` behave like `categoryOptionList.tpl`

Currently the `categoryMultiSelectOptionType.tpl` outputs nodes of depth 0 and depth 1,
but article categories for example can be nested deeper than that.
The `categoryOptionList.tpl` as used by the category add forms handles arbitrary nesting levels already,
so I’ve ported the template logic over to the multi select option template.

The maximum nesting depth in the `AbstractCategoryMultiSelectOptionType.class.php` has been changed
to the default value (-1) to allow for infinite nesting depths.

Merge pull request #4496 from mutec/tagfieldfdp1

fix id of `CustomFormDataProcessor` in `TagFormField`

fix id of `CustomFormDataProcessor` in `TagFormField`

the id was `acl` which seems to be a copy paste-mistake

Merge branch '5.2' into 5.3

Merge pull request #4493 from WoltLab/5.3-aclformfieldcleanup

Reset ACL field values within form cleanup

Release 5.3.14

Incorrect use of spaces for indentation in <5.4

Sandbox `foreachVars` in templates

Nesting the same template inside a `foreach` loop that is also accessed inside the nested call will overwrite the values from the outer template due to identical identifiers being used.

The sandbox did not protected `$this->foreachVars` despite being stateful.

See #4431
Fixes #4444

Reset ACL field values within form cleanup

Remove empty lines

Merge pull request #4491 from Fighter456/devtools-language-fix

Fix miswording in language variable of devtools

Fix miswording in language variable of devtools

The german language variable `wcf.acp.pip.page.name.description` uses the word `Boxname` but is used in the `PagePackageInstallationPlugin`.

New link format for Facebook videos

Release 5.3.13

Updating minified JavaScript files

Merge branch '5.2' into 5.3

Release 5.2.15

Merge branch '3.1' into 5.2

Release 3.1.23

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Fix return type comment for AbstractDatabaseObjectAction::getSingleObject()

Merge branch '5.2' into 5.3

Correctly handle null values in UserFormField::validate()

Closes #4471

Co-authored-by: Fabii547 <Fabii547@users.noreply.github.com>

Updating minified JavaScript files

Merge branch '5.2' into 5.3

Merge pull request #4459 from SoftCreatR/bugfix/custom-errors

Unify error handling in several templates

Unified error handling in several templates

Add missing call to ->loadVariables() before downloading Google Font during style import

Merge branch '5.2' into 5.3

Merge pull request #4453 from WoltLab/iformbutton-is-available

Check whether an IFormButton is available before rendering

Check whether an IFormButton is available before rendering

Prevent submitting an empty message to the preview

Merge branch '5.2' into 5.3

Merge pull request #4439 from xopez/5.2

Fixes Bings Sitemap Howto

Fixes Bings Sitemap Howto

This includes the correct links if the Bing Webmaster Tools help & how-to for sitemaps.

Delete reaction type icon on delete action

Merge branch '5.2' into 5.3

Fix typo in watchedArticleList.tpl

Merge branch '5.2' into 5.3

Merge pull request #4435 from WoltLab/php-ddl-diff

PHP DDL Fixes

Take the array key into account when checking whether a column is up to date in DatabaseTableChangeProcessor

Previously updating a column that looks like this:

    column VARCHAR(1) NOT NULL

to:

    column VARCHAR(1) NULL

would not do anything.

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

    [ 'default' => null
    , 'notNull' => 1
    , 'type' => 'VARCHAR'
    , 'length' => 1
    ]

and

    [ 'default' => null
    , 'notNull' => 0
    , 'type' => 'VARCHAR'
    , 'length' => 1
    ]

respectively.

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

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

Take the array key into account when checking whether a KEY is up to date in DatabaseTableChangeProcessor

Previously updating a (pretty contrived) KEY that looks like this:

    […] UNIQUE KEY someIndex (`UNIQUE`)

to:

    […] KEY someIndex (`UNIQUE`)

would not do anything.

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

    [ 'columns' => 'UNIQUE'
    , 'type' => 'UNIQUE'
    ]

and

    [ 'columns' => 'UNIQUE'
    , 'type' => null
    ]

respectively.

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

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

Merge branch '5.2' into 5.3

Merge pull request #4433 from WoltLab/php-ddl-diff

PHP DDL Fixes

Take the array key into account when checking whether a FOREIGN KEY is up to date in DatabaseTableChangeProcessor

Previously updating a FOREIGN KEY that looks like this:

    […] FOREIGN KEY (fooID) REFERENCES wcf1_foo (fooID) ON DELETE SET NULL ON UPDATE CASCADE

to:

    […] FOREIGN KEY (fooID) REFERENCES wcf1_foo (fooID) ON DELETE CASCADE ON UPDATE SET NULL

would not do anything.

Converted into the `getData()` representation of the PHP DDL API these would
look identical when looking at the values only. Both span the same column and
reference the same column on the same table. The `ON …` actions are one
`CASCADE` and one `SET NULL` for both.

For this reason the diff is empty, believing that the FOREIGN KEY already
matches the expected configuration.

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

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 #4431 from WoltLab/foreach-empty

Fix `{foreach}` loops when the to-be-iterated value is empty

Fix `{foreach}` loops when the to-be-iterated value is empty

The restoring of the `item` value failed, because the necessary data in
`foreachVars` was only being filled when the `foreach()` loop was actually
entered. Move this saving of the old value up to ensure it always happens.

see 75ce18bc18904d1215c7d021ac0ac18c0a7a5d42
see #4425

Merge branch '5.2' into 5.3

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 pull request #4425 from WoltLab/template-engine-foreach-variable-unset-fix

Foreach tag unset variables that have the value NULL

Foreach tag unset variables that have the value NULL

Use correct English quote marks in en.xml

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Encode HTML in smileyTitle in Smiley::getHtml()

Release 5.3.12

Improved the explanation for missing system requirements

Release 5.3.11

Removed reference to earlier evaluation versions

Enable the upgrade notice for 5.4 by default

Use `emailConfirmed` status to exclude unconfirmed emails

With 5.3 we introduced the new `emailConfirmed` status to separate the status of the email and the activation status of the user. Therefore, we want to send emails to users, which emails are enabled. The user activation status is not relevant anymore for sending emails.

Merge branch '5.2'

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>

Set new activation code on rerequesting them

Fix reading value of `CheckboxFormField`

In contrast to `BooleanFormField` for which an explicit "no" input field exists, a normal checkbox does not submit any data if no checked.

Remove obsolete style logo upload field descriptions

See #3456

Add missing encoding of `$family` in font download requests

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2