Release 5.4.8 dev 2

Merge pull request #4516 from WoltLab/xsrf-token-error

Improve phrasing for XSRF token error messages

Improve phrasing in wcf.ajax.error.sessionExpired

see #4501

Improve phrasing in wcf.global.form.error.securityToken

see #4501

Release 5.4.8 dev 1

Updating minified JavaScript files

Fix typo in setup_en.xml

Incorrect type comparison when the legacy mysql extension is been used

Merge branch '5.3' into 5.4

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

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

Take the array key into account when checking whether an unnamed KEY matches in DatabaseTableChangeProcessor

The reproducer effectively matches d7f721d6f920d66f75102723b504d89e57a8c9ff, except that the KEY
is unnamed.

Previously the update would silently fail to do anything. Now the update fails
loudly, because it attempts to create another index with an existing name. This
is no different behavior compared to an INDEX collision of two unnamed indices
`(a, b)`, `(a, c)`. The developer will be clearly alerted of this issue and can
take appropriate measures to avoid it, e.g. by using explicit names.

see #4434

Skip desktop notifications on Android

Notifications are not supported outside of the context of service workers.

See https://community.woltlab.com/thread/292374-chrome-android-failed-to-construct-notification-illegal-constructor/

Do not error during validation of TOTP codes if an invalid device is selected

Do not pass `null` to `|encodeJS`

This breaks in PHP 8.1.

Fix TypeScript code style

Fix SCSS code style

Use well-specified node.js for Prettier jobs in GitHub Actions

Update GitHub Actions to node.js 16

Skip bogus selection changes

Skip the check for the caret position if the selection is invalid

Merge branch '5.3' into 5.4

Incorrect gradient value in Safari

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

Update npm dependencies

Fix SCSS code style

Apparently the replacement of 100% by 1 caused the line to be sufficiently
short to rewrap.

Fix use of transparentize() in *.scss

The function expects a unitless number between 0 and 1 as the second parameter.
The updated SCSS compiler in 5.5+ complains about this misuse.

Fix return type for SeekableIterator::* implementations

The `mixed` type is not available with our current minimum PHP version, thus
using ReturnTypeWillChange for these.

Move scssphp to the WoltLab fork

This is for PHP 8.1 compatibility.

Merge branch '5.4' of https://github.com/WoltLab/WCF into 5.4

Enable `X-Frame-Options` for the WCFSetup

This has the side effect of suppressing `SameSite=none` for the cookies, which fails on insecure connections because this attribute value is valid for secure cookies only.

Resolves #4499
Follow up for 2a9d48c4badc4de2e0f2d2fc73c3af2bee39cce8

Fix return type of `count()` for the remaining classes implementing \Countable

see 3f6b343d10b044ab08d41fec525f69ffe0a95e49

Fix PHP 8.1 compatibility in DatabaseObjectList

> Return type of wcf\data\DatabaseObjectList::count() should either be
> compatible with Countable::count(): int, or the #[ReturnTypeWillChange]
> attribute should be used to temporarily suppress the notice […]

Enable `X-Frame-Options` for the WCFSetup

This has the side effect of suppressing `SameSite=none` for the cookies, which fails on insecure connections because this attribute value is valid for secure cookies only.

Resolves #4499

Skip the default cover photo when rebuilding users

Fixes #4500

Set the XSRF-Token cookie to SameSite=lax

As it turns out, `strict` is too strict for some use cases of the average user,
as it might suppress the cookie when the user researches something while
writing a post and ultimately comes back to the community via an external link.

This request will not have the XSRF-Token cookie attached due to violating the
`strict` policy, resulting in WoltLab Suite sending a fresh cookie in response.
This will then invalidate the token stored in the form where the user is in the
process of writing their post, ultimately resulting in an error message.

The `SameSite` value is meant as a defense in depth measure to protect the user
even if they current token leaked in some way. Reducing the strictness does not
reduce the security in a measurable way.

Typo

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 branch '5.3' into 5.4

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

Incorrect code style

Adjusted the `tab_width` for *.css/*.scss

Explicitly reduce the indent size of *.css/*.scss

Merge branch '5.3' into 5.4

Merge branch '5.2' into 5.3

Merge branch '5.3' into 5.4

Merge pull request #4495 from Fabii547/patch-79

Fix writing `runStandalone` attribute to package.xml

Merge pull request #4493 from WoltLab/5.3-aclformfieldcleanup

Reset ACL field values within form cleanup

Fix writing `runStandalone` attribute to package.xml

Release 5.4.7

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

Release 5.4.7 RC 2

Replaced a hard-wired color value, Safari fix

Release 5.4.7 RC 1

Merge branch '5.4' of https://github.com/WoltLab/WCF into 5.4

Updating minified JavaScript files

Removed the upgrade instructions for 5.3.*

This greatly reduces the size of the update packages. Upgrade from 5.3 will still work, but will upgrade to 5.4.6 only and then apply the remaining updates.

Check against the current menu item during editing only

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`.

Merge branch '5.3' into 5.4

New link format for Facebook videos

`exif` was not listed as a required extension

Typo (missing comma)

Merge pull request #4487 from WoltLab/npm

Upgrade npm dependencies

Release 5.4.6

Release 5.4.5

Release 5.4.5

Merge branch '5.3' into 5.4

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

Satisfy eslint

Run prettier

Upgrade TypeScript

Update all npm dependendencies except TypeScript itself

Release 5.4.5 RC 1

Updating minified JavaScript files

Add placeholder to the 'path' input in DevtoolsProjectAddForm

Add description to the path input in DevtoolsProjectAddForm

Resolves #4479

Revert "Remove erroneous DatabasePackageInstallationPlugin::getDefaultFilename()"

This had the unintended side-effect of no longer listing the database PIP
within the “Sync” view of dev tools. This will need to be revisited (#4480).

This reverts commit d5b180155d5805bda7e3132df2f4fde6627a49db.

Merge pull request #4478 from ilouHD/patch-2

Update de.xml

Update de.xml

fixed some typos when creating a new project using the devtools-gui in the german language.

Regarding the item `wcf.acp.devtools.project.isApplication.description` I'm unsure, why template-folders are mentioned explicitly. As I know they're the same as in plugins/projects, which are no applications.

Incorrect alignment of positioned elements when they are hidden

The calculation did not consider the `display` attribute, causing the elements dimensions to be considered as `0x0` for the purpose of the calculation.

See https://community.woltlab.com/thread/291896-beitragsoptionen-verschoben/

Reset the floating code box header when the sticky page header is hidden

See https://community.woltlab.com/thread/291977-sticky-code-header-w%C3%A4hrend-antwort-unsch%C3%B6n/

Drop obsolete update scripts for 5.4.2 -> 5.4.3

Remove erroneous DatabasePackageInstallationPlugin::getDefaultFilename()

Merge branch '5.3' into 5.4

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Fix return type comment for AbstractDatabaseObjectAction::getSingleObject()

Clean up control and data flow in UserFormField::validate()

Fix typo in en.xml

Fix error message for nonExistent user in UserFormField