Add Buck to spiderList.xml

User-Agent: `Buck/2.2; (+https://app.hypefactors.com/media-monitoring/about.html)`

see #3111

Add ZoominfoBot to spiderList.xml

User-Agent: `ZoominfoBot (zoominfobot at zoominfo dot com)`

see #3111

Fix SCSS code style

Merge branch '5.3'

Logical separation of attachments in messages and signatures

Fixes #3835

Use consistent title for flood control delay time

Close #3801

Consistently use "entered" in English phrases instead of "given"

See #3801

Add missing url attribute for message share buttons' container

See #4108

Hide the quote tooltip while dragging the touch selection

Improved the behavior of page action buttons on mobile

Work-around for the scroll position when inserting quotes on iOS

Detection of pasted blob uris in iOS

Block the form submit if one or more editors are still in source code mode

Detection of repeated mentions with the first mention at index 0

Work-around for the HTML entity in notifications for reactions

Revert "Do not encode link in `Ui/Message/Share/Dialog`"

This reverts commit e1fc63b48550f147b2c331949bf1bc438adf8dbb.

Do not encode link in `Ui/Message/Share/Dialog`

It is already encoded in the template.

Add `Ui/Message/Share/Dialog` as reusable share dialog module (#4108)

* Add `Ui/Message/Share/Dialog` as reusable share dialog module

Close  #4026

* Apply suggestions from code review

* Apply suggestions from code review

Avoid the use of gradients involving the `transparent` keyboard

Use a separate shadow link for embedded content

Avoid conflicts with complex HTML inside the excerpt similar to how articles work.

Implement `_dialogSubmit()` for confirmation dialogs

See #3234

Merge pull request #4110 from WoltLab/xars-oof

Set X-Auto-Response-Suppress: OOF header on all mail

Set X-Auto-Response-Suppress: OOF header on all mail

I was not able to think of a case where out-of-office notifications would be
desired in response to an email sent by WoltLab Suite, so onto all emails it
goes.

Resolves #4104

Consistently use "The entered X" in English phrases (#4109)

See #3801

Merge branch '5.3'

Merge pull request #4103 from WoltLab/acp-user-email-search

Add acp user email search

Fix codestyle

Co-authored-by: Tim Düsterhus <duesterhus@woltlab.com>

Remove German language tag for spider URL

See 2f4ab6f10ee34601a2931307cc6cc1601943fcd7

Merge branch '5.3'

Merge branch '5.2' into 5.3

Lack of filtering of content from ignored users

Merge branch '5.3'

Merge pull request #4105 from WoltLab/notification-markAllAsRead

Attempt to prevent deadlocks involving marking all notifications as read

Attempt to prevent deadlocks involving marking all notifications as read

The previous implementation, UPDATEing all rows with a specific userID, needed
to create pretty coarse locks on the `userID` INDEX. Specifically it also
created gap locks, preventing concurrent *creation* of INDEX records that would
have been matched.

My current understanding of MySQL's locking behavior is that the `confirmTime`
being part of the INDEX is what caused the issue:

The `userID` INDEX includes the columns userID, eventID, objectID and
confirmTime. Now consider the following:

Thread 1: Marks all notifications for userID = A as read.
Thread 2: Marks objects X, Y and Z for userID A as read.

Thread 1 will lock all existing notifications for userID = A as well as all
insertions into the `userID` INDEX with that specific userID. This includes
marking notifications as read, because this will delete the old index record
and insert a new index record with an updated confirmTime.

Thread 2 will lock the specific notifications for userID = A as well as all
insertions into the `userID` INDEX with that specific userID and objectIDs.
This includes marking notifications as read, because this will delete the old
index record and insert a new index record with an updated confirmTime.

Now consider the following timeline:

T1: Locks the gaps for userID = A.
T2: Locks the gaps for userID = A, objects X, Y, Z.

These locks are allowed to coexist:

> Gap locks in InnoDB are “purely inhibitive”, which means that their only
> purpose is to prevent other transactions from inserting to the gap.
(https://dev.mysql.com/doc/refman/8.0/en/innodb-locking.html)

T1: Wants to UPDATE the confirmTime for userID = A, object X.

-> This is prevented by the gap lock held by T2, thus T1 waits.

T2: Wants to UPDATE the confirmTime for userID = A, object X.

-> This is prevented by the gap lock held by T1, thus T2 waits.

Now we have a deadlock.

As the current query needs to UPDATE a large number of rows it is fairly slow,
holding the locks for a long time and also needing to update the rows
one-by-one. This gives other threads enough opportunity to run in-between,
wreaking havoc.

Fix this issue by first selecting the exact notifications we need to mark as
read using a simple SELECT. This SELECT should not be able to deadlock with
concurrent write statements.

Afterwards we update the notifications based off a condition matching specific
rows within the PRIMARY KEY. As these must match existing rows only, no gap
locks will be needed, thus reducing the chance to block concurrent threads.

Additionally we only need to update a very small number of rows (should be less
than 50 in the vast majority of cases), reducing the time spent in the query,
further closing the window for concurrent requests and possibly making the
process faster due to less rows being updated (and thus needing to be written
to disk).

Update to Guzzle 7.3

Merge pull request #4098 from netzhuffle/patch-4

Fix param type for handleException

Add acp user email search

Add meta tags for trophy page

Add PetalBot

Add Googlebot-Video

Add WoltLab Suite bot

Add meta tags for CategoryArticleListPage

Fix code style

Fix code style

Merge branch '5.3'

Preview images and cover photos were exported twice

Remove/replace nbsp when copying code to the clipboard

Merge branch '5.3'

Cover photo deletion was not saved properly

Always fetch package information using the default language

Improve `Ui/Object/Action` implementation (#4101)

* Improve `Ui/Object/Action` implementation

1. The `objectAction` callback of `Ui/Object/Action/Handler` no gets the whole `data: ObjectActionData` data structure instead of each element separately.
2. `ObjectActionData` now also contains the container element.
3. Add support for `data-object-action-handler` for action buttons. The default actions ignore actions triggered by buttons with such attributes. Instead, custom handlers can react to specific actions by checking for the value of this attribute.

* Use `insertAdjacentElement` in `Ui/Object/Action/Delete`

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

Fixed missing language variables for error messages

Removed obsolete language variables

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Update the user rank after users change their profile

Fix identifier of DOM change listener in `Ui/Object/Action`

Removed obsolete Chrome workaround for dialogs with scrollbars

Removed obsolete whitespaces

Delete trophy images when deleting tropies

Fixed wrong parameter type

Missing margin for trophies with images

Hide comment counter when comments are disabled

Made sure ids in toc are unique

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Strip rtl override char

Merge pull request #4093 from WoltLab/redirect-guard

Add RedirectGuard

Permit the use of `array_diff` in templates

If only developers would stop moving business logic into templates…

Merge pull request #4080 from WoltLab/ts_dbo_action

Add global `WCF.Action.(Delete|Toggle)` replacement modules

Merge branch 'master' into ts_dbo_action

Avoid using template compiler in `ObjectActionFunctionTemplatePlugin`

Language items in `wcf.global.` do not support template scripting by default. Instead of "manually" adding support for template scripting for this specific case, the relevant language item has been moved to a different language item category.

Replace use of `WCF.Search.User` with `Ui/User/Search/Input` (#4096)

* Replace use of `WCF.Search.User` with `Ui/User/Search/Input`

See #3876

* Remove obsolete TODO

See 9d7cfda776e5c3809babfeb003d1212c29413619

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

Fix param type for handleException

WCF::handleException incorrectly stated it would receive only \Exception, while it is registered to receive any \Throwable.

Incorrect position of the suggestion drop-down

Related issue: #4096

Remove duplicate colon

Remove trailing whitespace in ObjectActionFunctionTemplatePlugin documentation

Extend documentation of `ObjectActionFunctionTemplatePlugin`

Make `wcf.global.button.delete.confirmMessage` globally available in JS code

Update obsolete comment in `Media/Manager/Base`

Deprecate `WCF.Action.*` classes

Use `Ui/Object/Action/Delete` for attachments

Add Applebot as spider

Remove obsolete import

Replace use of `WCF.Action.SimpleProxy` with `Ui/Object/Action`

Remove obsolete JavaScript function

Use `Ui/Object/Action` for media files

Use `Ui/Object/Action/Delete` in more places in ACP

Support nested structures in `Ui/Object/Action/Delete`

Convert last uses of `WCF.Action.Toggle` to `Ui/Action/Toggle`

Add support for `(enable|disable)Title` for toggling in `{objectAction}`

Fix code formatting in ObjectActionFunctionTemplatePlugin

Use `Ui/Object/Action/Delete` in frontend

Use ObjectActionFunctionTemplatePlugin in more places in ACP

Add and use ObjectActionFunctionTemplatePlugin

Use new templates for object action delete and toggle icons

Add templates for object action delete and toggle icons

Start using `Ui/Object/Action/*` in ACP

Apply suggestions from code review

Move `Controller/ClipboardData` to `Controller/Clipboard/Data`