Do not take X locks in read-only methods of the MFA Setup class

This was a fun one. Apparently even a query locking exactly a single row by
leveraging an UNIQUE KEY outside of a transaction can deadlock with another
transaction.

Sending two requests to regenerate MFA backup codes at the same time caused the
following to happen within the database:

T1: Within a transaction locks the Setup by using Setup::lock(), putting an X
    onto the PRIMARY KEY.
T2: Outside of a transaction reads a Setup using Setup::find(), hitting exactly
    a single entry within the `userID` UNIQUE KEY (i.e. a specific
    `(userID, objectTypeID)` tuple).
    -> This puts an X onto the `userID` UNIQUE KEY.
    -> This wants to also put an X onto the corresponding PRIMARY KEY.
    -> The PRIMARY KEY is already locked by T1.
    -> This query (and thus this transaction) waits within this specific query
       for the lock to be granted.
T1: Within the transaction calls Setup::find() with the same parameters as T2.
    -> This needs to put the X onto the same row in the `userID` UNIQUE KEY.
    -> This row is locked by T2.
    -> This transaction needs to wait for that lock to be granted.

Now T1 needs to wait for T2 which already waits for T1 and we're experiencing a
deadlock.

Fix this issue by not taking an X within Setup::find() (i.e. removing the `FOR
UPDATE`). I've verified that nothing calls Setup::find() within a transaction
(without locking the Setup by a different means). Thus this change does not
result in a difference with regard to lock safety. Everything that needs to
modify a Setup already calls Setup::lock() which locks the PRIMARY KEY only.

Fixes #3874

Merge remote-tracking branch 'origin/master'

Merge branch '5.3'

- Dropped update_com.woltlab.wcf_5.3.3_style.php

Merge pull request #3865 from WoltLab/fix-style-preview

Fix the style preview images

Merge pull request #3872 from WoltLab/codemirror-media

Convert `Acp/Ui/CodeMirror/Media` to TypeScript

Unified the exception type for GD and Imagick

Use an `enum` for the InsertType

Convert `Acp/Ui/CodeMirror/Media` to TypeScript

Use DomUtil for element visibility in form builder modules

Convert `WCF.Message.Quote.Handler` to TypeScript (#3860)

* Convert `WCF.Message.Quote.Handler` to TypeScript

* Export the class separately

Image adapters now support saving to GIF/JPG/PNG/WebP (#3869)

* Image adapters now support saving to GIF/JPG/PNG/WebP

* Adjusted the usage of exception

* Missing remark on the version support

Merge branch '5.3'

Fix check for deleted notifications in NotificationEmailDeliveryBackgroundJob

Support for WebP images (#3861)

* Support for WebP images

* Inconsistent quote style

Merge pull request #3849 from WoltLab/media_typescript

Convert media-related JavaScript to TypeScript

Fix multiple queries for user online list stats

Merge pull request #3855 from WoltLab/rank-image-upload

Allow upload of rank images via admin panel

Add rank image upload migration script

Add user rank image upload

Merge branch '5.3'

Merge pull request #3867 from WoltLab/smtp-auth-fail

Fail SMTP auth if credentials are configured but all mechanisms fail

Properly detect EOF in SmtpEmailTransport::read()

Disconnect SMTP session for all types of Exception during connect / auth

Fail SMTP auth if credentials are configured but all mechanisms fail

Merge pull request #3853 from WoltLab/oauth-refactor

Refactor OAuth 2 login

Fix the style preview images

The update 5.3.1 -> 5.3.2 detached all the style preview images in the
database, while leaving the actual image files in the file system. This new
update script *should* fix this situation again, by scanning the asset folder
and reattaching the newest image.

Check for a logged in user in ReauthenticationForm

needsReauthentication() may only be called for logged in users.

Remove MultifactorAuthenticationAbort from ACP guest controller whitelist

Fix eslint issue

Remove obsolete code

Only `insertType = "separate"` was ever supported.

Apply suggestions from code review

Merge pull request #3854 from WoltLab/typescript-no-implicit-any

Improve TypeScript safety

Simplify formatter selection in Date/Picker.ts

Fix typing in TabMenu.ts

Improve type safety of Plural.ts

Various TypeScript fixes detected using noImplicitAny

Apply suggestions from code review

Merge pull request #3857 from WoltLab/commit-message-check

Add pull_request workflow

Add pull_request workflow

Add proper error handling to OAuth2 client

Specify timeout for OAuth 2 requests

Send 403 to spiders accessing the social login

Implement the OAuth 2 providers using AbstractOauth2Action.

Add AbstractOauth2Action

Configure a 60 second timeout for Guzzle by default

This is acting as a safety net, because the default timeout is infinite.

Add dir for rank images

Merge pull request #3813 from WoltLab/session-cookie

Session Cookie Updates

Merge branch '5.3'

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Merge pull request #3844 from WoltLab/fix-failing-cronjobs

Fix failing cronjobs

Incorrect year

Old habits die hard…

Merge branch '5.3'

Release 5.3.2

Merge pull request #3814 from WoltLab/delete-content-multiple-users

Delete content from multiple users simultaneously

Merge pull request #3852 from WoltLab/3rdParty-remove-profileFields

Stop filling in the user's profile on social registration

Updating minified JavaScript files

Stop filling in the user's profile on social registration

The behavior of this was very inconsistent across providers, required a custom
implementation for each of those and over the years less and less data was
provided (most notably Facebook drastically limited the available fields).

The benefit of filling in the gender and avatar here and there is almost nil,
nothing of real value is lost when removing this and it keeps the code simpler.

Simplify retrieving objectTypes for delete content types

Simplify delete logic by using a flat array

Rename class variable `$user` to `$users`

Add clipboard action to delete user content

Make `UserAction::prepareRemoveContent()` work with multiple users

Make UserContentRemoveWorker work with multiple userIDs

Merge pull request #3851 from WoltLab/scssphp

Update to scssphp/scssphp 1.4

Update use of scssphp

Setting output formatters is deprecated as of scssphp/scssphp 1.4.

Update to scssphp/scssphp 1.4

Mark lib/system/api/** as linguist-vendored

Merge pull request #3850 from WoltLab/intl

Recommend intl during WCFSetup

Recommend intl during WCFSetup

Resolves #3811

Update update_com.woltlab.wcf_5.4_session_1_cookies for new cookie format

Fix username suggestion in ACP's LoginForm

Resolves #3725

Refresh the session cookie only once it aged a bit

Resolves #3611

Encode additional data into the session cookie

The cookie now stores:
- sessionID
- userID
- an compact representation of the last update that repeats every 64 days.

Merge branch '5.3'

Merge branch '5.2' into 5.3

Fix VarbinaryDatabaseTableColumn

> Call to undefined method wcf\system\database\table\column\VarbinaryDatabaseTableColumn::traitGetLength()

see 4a65b1e27cc85fc9fb9bbcdf6a6d7a5f9cc76187

Remove obsolete import from LoginForm

Add media-related TypeScript interfaces

Convert `Controller/Media/List` to TypeScript

Convert `Media/Manager/Select` to TypeScript

Convert `Media/Manager/Search` to TypeScript

Convert `Media/Manager/Editor` to TypeScript

Convert `Media/Manager/Base` to TypeScript

Convert `Media/List/Upload` to TypeScript

Convert `Media/Clipboard` to TypeScript

Convert `Media/Editor` to TypeScript

Convert `Media/Replace` to TypeScript

Convert `Media/Upload` to TypeScript

Mark second parameter of `FileUtil.formatFilesize()` as optional

Add missing methods to `RedactorEditor` interface

Fix visibility of `Upload.hasPendingUploads()`

Properly handle `null` parameters in `Upload.appendFormData()`

Make clipboard's `pageObjectId` optional

Merge pull request #3848 from WoltLab/typescript-prism

Convert Prism usage to TypeScript

Use async function for event listener in `Core/Bbcode/Code`

Make CHUNK_SIZE a static class member in `Core/Bbcode/Code`

Convert Prism/Helper#splitIntoLines into a generator

This allow for a much cleaner control flow and also reduces the number of
useless intermediate objects (e.g. the DocumentFragment).

Convert `Bbcode/Code` to TypeScript

Add WoltLabSuite/Core/Prism/Helper