Merge branch '5.4'

Merge pull request #4442 from SoftCreatR/css-optimizations

Prefixed CSS First

Prefixed CSS First

Ensured vendor-prefixed versions of a CSS property are listed before the standardized, unprefixed version.

When multiple versions of the same CSS property are specified, the last supported one will be used due to how browsers handle fallback values. This means the order matters when using both vendor-prefixed and unprefixed versions of the same property. Specifically, the unprefixed version must be listed last to ensure standardized behavior takes precedence.

Removing inline formatting yielded empty elements and broke the selection

See https://community.woltlab.com/thread/291496-fettmarkierung-bei-einem-einzelnen-wort-kann-nicht-mehr-entfernt-werden/

Strip de fact empty paragraphs in messages

Merge branch '5.4'

Updating minified JavaScript files

Merge pull request #4440 from WoltLab/sourcemap-proxy

Add proxy_sourcemap.php

Merge branch '5.3' into 5.4

Prevent submitting an empty message to the preview

Merge pull request #4441 from WoltLab/acp-logout-to-reauth

Redirect the user to ACP reauthentication instead of frontend after logout

Add proxy_sourcemap.php

Redirect the user to ACP reauthentication instead of frontend after logout

Revert "Update Open Sans"

This reverts Open Sans back to the non-variable version, as the variable one
appears to render pretty badly on Windows.

This reverts commit 33c8866a790e9e6bc46358f6e82df76d2e32f56e.

Merge branch '5.4'

Merge branch '5.3' into 5.4

Merge branch '5.2' into 5.3

Merge pull request #4439 from xopez/5.2

Fixes Bings Sitemap Howto

Reactions were not displayed in article list when sorted by title

Mark as read doesn't removed the badge in the mobile menu

Fixes Bings Sitemap Howto

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

Merge pull request #4438 from WoltLab/5.4-fix-missing-salts

Fix having an incorrect parameter if a hash does not contains a salt

Fix having an incorrect parameter if a hash does not contains a salt

Fixes #4416

Reset the avatar cache to force the use of the WebP variants

Fixes #4424

Delete reaction type icon on delete action

Merge branch '5.2' into 5.3

Merge branch '5.4'

Run SCSS prettier

Fix typo in watchedArticleList.tpl

Merge branch '5.4'

Merge branch '5.3' into 5.4

Some external links in the admin panel did not open in a new window

Trim the preview height using traditional overflow

`-webkit-line-clamp` is broken in Firefox and Safari once complex HTML is used.

See https://community.woltlab.com/thread/291373-rich-embeds-mit-liste-fehlerhaft/

Skip the user menu initialization if there are no items

Detect `<center>` as a block element

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.

Incorrect handling of `null` values for metacode attributes

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 #4430 from WoltLab/js-legacy-inheritance

Resolved side effects from previous inheritance approach

Merge pull request #4429 from WoltLab/form-field-data-handler-es6

Support ES 6 default exports for form builder field JavaScript data handlers

Support ES 6 default exports for form builder field JavaScript data handlers

For legacy exports the value loaded by `require()` (`FormBuilderField`) will be
whatever the module author returned within the module definition. ES 6 modules
on the other hand will be passed as an object containing the exported values,
with the default export residing in the `default` key.

tslib's `__importDefault` will transform a legacy export into a ES 6 module, by
putting the exported value into the `default` key of an freshly created object.

This allows us to handle both legacy as well as ES 6 default exports identically
by simply using the `default` value of the `FormBuilderField` variable.

Merge pull request #4432 from WoltLab/package-server-optional

Do not store optional information from update servers within the database

Merge branch '5.4'

Merge branch '5.3' into 5.4

Update style for PHP CS Fixer 3.0.1

Do not store optional information from update servers within the database

These do not appear to be used anywhere and the official package servers do not
expose them either, making them effectively empty even in current versions.

see fd3511448663b86f4428e848bbefa07b93bf4bad
see #4371
see #4385

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

Resolved side effects from previous inheritance approach

This is a follow-up for db23f8af33398c4851d6ba36436592f406b35a0d which introduced a flawed change.

The iteration over the prototype chain caused the prototype itself being bound to an object on runtime, conflicting with other objects.

The root cause was that some parts of the inherited functions were still bound to `constructed`, which was attempted to be fixed by poking the prototype chain.

This new fix is a bit weird, unless one understands that the call to `Reflect.construct()` is a bit tricky because any bound call inside the constructor of `legacyClass` will be bound to `constructed`.

This change is more of a sledge hammer approach, but it works all cases that I tested, including those that were initially the cause for the previous fix as well as new issues caused by the fix.

Sneaky whitespace

Incorrect value type when using the legacy MySQL extension

Improved a11y of content items

Improved usability of footer links on mobile devices

Merge branch '5.4'

Merge branch '5.3' into 5.4

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

Merge pull request #4426 from WoltLab/label-form-field-selection

Properly show the label form field's selection if it has a value upon load

ACL values were not deleted when deleting categories

Properly show the label form field's selection if it has a value upon load

This got broken during the TypeScript migration, because the constructor's
`labelId` parameter is typed as string, but in reality it was passed an
integer. This broke a `===` comparison with the `data-label-id` property of the
list items, thus failing to properly select the label.

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

Re-added previous share button selector to keep compatibility

Missing js file

Re-added previous share button selector to keep compatibility

share button class name changed to prevent blocking by ad blockers

Foreach tag unset variables that have the value NULL

Merge branch '5.4'

Use correct English quote marks in en.xml (5.4)

Merge branch '5.3' into 5.4

Use correct English quote marks in en.xml

Merge branch '5.4'

Merge pull request #4423 from WoltLab/oauth-pkce

Add PKCE support to AbstractOauth2Action

Name “Notfallcodes“ consistently

Drop the hyphen from “Notfall-Code”.

Add PKCE support to AbstractOauth2Action

Merge branch '5.4'

Merge pull request #4421 from WoltLab/5.4-deleteDsStore

Add files/acp/update_com.woltlab.wcf_5.4.3_deleteDsStore.php

Merge pull request #4422 from WoltLab/5.4-check-search-rebuild

Refer to RebuildDataPage in the system check for search engine tables

Refer to RebuildDataPage in the system check for search engine tables

Generate preload manifest when testing style

Fix system font stack when combined with a Google Font

Perform the replacement of the system font name, before prepending the Google
font to properly match the font name without the Google font prefix.

Add files/acp/update_com.woltlab.wcf_5.4.3_deleteDsStore.php

Drop obsolete update_com.woltlab.wcf_5.4.2_deleteInvalidSessions.php

This was only required to fix a bug in 5.4.0.

Merge pull request #4420 from WoltLab/userQuit

Add notice letting the user know quitting their account is active

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

Release 5.4.2

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

Fix code style

Updating minified JavaScript files

Merge pull request #4417 from WoltLab/captcha-delete-after-use

Delete the captcha registration after retrieving data in Comment/Add and Message/Reply

Updating minified JavaScript files

TOC code was visible in search results of articles

Ignore click events that were triggered outside of the editor

This appears to haben if the click event was partially intercepted because of an open overlay that was implicitly closed.

Add notice letting the user know quitting their account is active

Allow canceling the quit after losing the permission to quit