Make renaming columns with PHP DB API idempotent (#4367)

An error message should only be shown if the neither a column with the old name, nor with the name name exists so that after the rename, a second rename is a no-op.

Close #4362
See #3765

Merge branch '5.4'

Merge branch '5.3' into 5.4

Also applied the second change from #4366.

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

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Fix typo in de.xml

Closes #4366

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

Add missing `@since` to AbstractOauth2Action

Make AbstractOauth2Action::getHttpClient() non-final

It might be desirable to be able to add custom headers, specifically a custom
`user-agent`, for non-stock IDPs.

Replace use of `WCF.Clipboard` with `Controller/Clipboard`

See #3876

Merge branch '5.4'

Merge pull request #4356 from WoltLab/user-logged-in

Add UserLoggedIn event and cancel lost password requests upon login

Add cancelLostPasswordRequest to UserAction

Cancel lost password requests when the user logs in

Resolves #3922

Implement the UserLoggedIn event

Add a UserLoggedIn event

Merge pull request #4353 from WoltLab/ColorFormField

Modernize color picker and add `ColorFormField`

Support wildcard for category name in `AclFormField` (#4355)

Fix TOTP QR code in dark styles

Release 5.3.10

Merge branch '5.4'

Merge branch '5.3' into 5.4

Merge pull request #4354 from WoltLab/systemcheck-performance

Improve performance of SystemCheckPage::validateWritableDirectories()

Improve performance of SystemCheckPage::validateWritableDirectories()

The previous implementation resulted in a syscall hell for large attachment or
image directories. RecursiveDirectoryIterator requires two syscalls for
directory entry just for iteration and that does not yet include any custom
logic.

Replace the implementation by a simple `glob()`. This is a small behavior
change, as a `*` will only check the files immediately below the directory in
question instead of recursing all the way down. However this likely is what was
intended anyway.

Updating minified JavaScript files

Add `ColorFormField`

Add new phrases to JavaScript for existing uses of color pickers

Deprecate `WCF.ColorPicker` in favor of `Ui/Color/Picker`

Update `Ui/Color/Picker` to use native TypeScript-based implementation

Instead of having to implement a color picker ourselves, we delegate the task to the browser by using a `input[type=color]` element. Because this color input does not support transparency, there is an additional `input[type=range]` element for transparency.

Modified update instructions

Add `ColorUtil.stringToRgba()`

Add `ColorUtil.isValidColor()`

Add `ColorUtil.rgbaToString()`

Add `ColorUtil.rgbaToHex()`

Use `.toString(16)` for `ColorUtil.rgbToHex()`

Export color interfaces from `ColorUtil`

Merge pull request #4352 from WoltLab/twitter-auth

Modernize Twitter Auth

Merge branch '5.4'

Handle Facebook not providing an email address

Fixes #4241

Correctly handle Twitter not providing an email address

Migrate Twitter authentication to Guzzle

Improve readability of OAuth header generation in TwitterAuthAction

Use `getControllerLink()` to generate TwitterAuth callback URL

Use constant time b64 encoder in TwitterAuthAction

Use `\hash_equals()` for token validation in TwitterAuthAction

Fix use of unused parameter in `AbstractOauth2Action::codeToAccessToken()`

Make `failure` parameter of `Popover.ajaxApi()` optional

There is no reason to make the custom failure callback a required parameter here because in most cases, there is no need fur such a custom handler.

Merge remote-tracking branch 'origin/master'

Merge branch '5.4'

Improve speed of PIPs to delete files

… by fetching all existing database records at once and deleting obsolete records in one transaction.

See #4269
See #4289

Register reauthentication after MFA in ACP only

It was not desired that a login within the frontend also registers a
reauthentication, as the actions requiring a reauthentication are very rarely
used and can benefit from this additional password check. In the ACP however
each request requires a reauthentication, thus it makes sense to register it
during login.

see 4624ceeda0e19109245d19e883dc6fd388e95519

Merge branch '5.4'

Merge pull request #4350 from WoltLab/update-server-db

Adjust the URL of the update servers in install.sql

Merge branch '5.4'

Merge branch '5.3' into 5.4

Sorting in article category had no effect

Adjust the URL of the update servers in install.sql

This is technically unnecessary as the proper URLs will be inserted during
runtime, but might improve debugging, as it is clear which version was used to
install the software.

Merge branch '5.4'

Add acpTemplate PIP to update instructions from 5.4.0 Beta 2

Use RejectEverythingFormField in PackageEnableUpgradeOverrideForm

Clear WCF::AVAILABLE_UPGRADE_VERSION

Merge branch '5.3' into 5.4

Add support for version upgrades (#4160)

* Add WCF::AVAILABLE_UPGRADE_VERSION

* Expose the available upgrade version to packageList.tpl

* Show notice in packageList if an upgrade is available

* Add upgrade override support to PackageUpdateServer

* Show notice about enabled upgrade override in packageList and packageUpdateServerList

* Add PackageEnableUpgradeOverrideForm

* Support disabling the upgrade override if there are issues

* Add version checks from update_com.woltlab.wcf_5.4_checkSystemRequirements.php to PackageEnableUpgradeOverrideForm

* Add language items for WoltLab Suite upgrades

* Clear package lists when toggling the upgrade override

* Add requested-woltlab-suite-version header to package list requests

* Improved the messages and restrict the upgrade to the maintenance mode

* Restrict the experimental upgrade to the maintenance mode

* Removed the upgrade notice from the package server list

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

Merge branch '5.4'

Release 5.4.0 RC 1

Updating minified JavaScript files

Add modified PIPs to 5.3 -> 5.4 instructions

Revert "Removed the upgrade support from 5.3.*"

This reverts commit 466ac360246fc568e0e3f6e1527c218f70bf158e.

Remove obsolete file deletions from `fileDelete.xml`

These files were re-added with 704cb1bfbe3b7c6088d3895bd1e561167ebe59e1 / #4277.

Remove unused `DevicesContainer::$devices` property

Merge remote-tracking branch 'origin/master'

Merge branch '5.4'

Type hint instead of a type cast

The type hint is only meant to narrow the type, not replace it.

Incorrect access of an element that is not always present

Merge pull request #4345 from WoltLab/scss-1.5

Update to scssphp/scssphp 1.5

Fix scssphp deprecations in StyleCompiler

Update to scssphp/scssphp 1.5

Resolves #4274

Merge pull request #4343 from WoltLab/leafo-alias

Remove autoloader compatibility layer mapping Leafo to Scssphp

Merge pull request #4341 from WoltLab/unfurl-download-fail-no-log

Prevent unfurling DownloadFailed exceptions from ending up in the error log

Merge branch '5.4'

Fix code style issue

Merge branch '5.4'

Encode non-UTF-8 parameters as hexadecimal in Benchmark

Some database queries, such as the new Flood Control framework, contain binary
fields that might break proper rendering or further processing.

Encode all parameters that are not valid UTF-8 as hexadecimal and wrap them
into a call of `UNHEX()` to maintain copyability.

see 048aaf0c318f01d11da4f6b8c01b6fdd1acbcb7a

Remove autoloader compatibility layer mapping Leafo to Scssphp

It's unlikely that anything depends on this and as outlined in the commit
9bfa7303627983c13607536aae839e7e0ebb0968 that initially added it, the
compatibility is incomplete as it cannot map Exceptions.

With scssphp 1.5 (WoltLab/WCF#4274) further changes will be introduced by the
upstream library that definitely will require adjustments in consumers. So
let's clean this up.

Merge branch '5.4'

Merge branch '5.3' into 5.4

Fix poll insertion after message edit if no poll exists

Previously, an empty `div.jsInlineEditorHideContent` element was added.

Enable Unicode support for SCSS Compiler

The large performance difference measured back when this was introduced appears
to be gone. On PHP 8 both encodings appear to be equally fast. On PHP 7.4 the
UTF-8 version is ~25% slower, which is certainly not ideal, but not as bad as a
7× increase. In any case the UTF-8 version completes in way less than 2 seconds
which I consider to be the line that the CSS compiler should not cross.

Resolves #4339
see #4274

Decreased button size in share dialog

Prevent unfurling DownloadFailed exceptions from ending up in the error log

Resolves #4237

Add element ID to form builder exception messages (#4327)

Delete `ModificationLogHandler` (#4340)

This class has been deprecated for many years since 28854f57445a35894483db28e250f94b444962a0.

Replace `Response` by `ResponseInterface` in `UnfurlUrlBackgroundJob`

Merge branch '5.4'

Make return types in HttpFactory more readable

see f5b3800800e74296ca1411b25496a6b964c0600e
see #4337

Merge branch '5.4'

Improve return types in HttpFactory

Resolves #4337

Delete `acp/dereferrer.php`

Use `HeaderUtil::sendNoCacheHeaders()` in `UserExportGdprAction`

Deprecate \wcf\system\io\FTP

Resolves #4332

Force non-cursive font for code listings inside quotes