Improve user experience when accessing the ACP with an unauthorized user

Specifically the logout link is available now.

Fix missing consideration of the ad position when calculating the show order

Fix typo in de.xml

Merge pull request #5099 from WoltLab/articlelist-comments-column

Remove comments column from ACP's ArticleListPage

Merge pull request #5100 from WoltLab/notification-email-unconfirmed

Discard notification emails if the recipient’s email address is unconfirmed

Discard notification emails if the recipient’s email address is unconfirmed

This change discards notification emails if the recipient’s email address is
unconfirmed after the email was created and before the email was actually seat.

An example might be that the background queue is delayed, due to the mail
server’s spam protection kicking in, allowing the admin to unconfirm email
addresses of email addresses that are no longer valid to prevent more bounces
from being generated.

Remove comments column from ACP's ArticleListPage

This column was effectively broken since the introduction of pluggable
discussion providers and is completely broken (always showing zero) since the
`comments` column was moved the the article to the article content in
75c21dfd1231389b2e3f527fc202dfec8f5c808b.

Merge pull request #5098 from Krymonota/patch-23

Fix typos in German language items

Fix package update error message typos (`de.xml`)

Fix capitalization and add missing hyphen (`de.xml`)

Add missing commas preceding "um" (`de.xml`)

Update outdated links within phrases (#5096)

see #5095
see https://www.woltlab.com/community/thread/297811-neuer-twitter-link-registrierung/

Update Twitter Developer link (#5095)

See https://www.woltlab.com/community/thread/297811-neuer-twitter-link-registrierung/

Fix typo in PHPDoc in `ValueFormFieldDependency` (#5089)

Fix CryptoUtil API misuse in SessionHandler

The previous version was not incorrect (we never sign a falsy string in
SessionHandler). However the return value should always be explicitly compared
to `null` to make verification of correctness easier.

Update `guzzlehttp/psr7` composer dependency

Update `guzzlehttp/psr7` composer dependency

Change autoIncrement default type to false if autoIncrement is not set

Merge pull request #5065 from HanashiDev/fix/timedatabasetablecolumn

Fix double implements in TimeDatabaseTableColumn

Fix double "implements" in TimeDatabaseTableColumn

Add missing module comments to `.d.ts` files

Merge pull request #5057 from WoltLab/smtp-email-fails-to-stringify

Improve the behavior of the SmtpEmailTransport if generating the DATA value fails

Improve the behavior of the SmtpEmailTransport if generating the DATA value fails

Release 5.5.6

Merge branch '5.4' into 5.5

Release 5.4.24

Merge branch '5.3' into 5.4

Release 5.3.25

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Merge branch 'js-relocate-xss' into 5.3

Add missing return value to cleanup() in AclFormField/WysiwygFormField

see #5032

Fix XSS vulnerability within the JavaScript relocator

If the relocation placeholder appeared multiple times within the source code,
it would also be replaced multiple times. This might allow an attacker to blow
up the HTML structure by including the placeholder within UGC.

Fix this issue by only ever replacing the last placeholder, which should be the
“real” one from footer.tpl. In the future this should be protected further by
including a random nonce to prevent this attack entirely.

Release 5.5.6 dev 1

Updating minified JavaScript files

Force the editor to be focused after inserting quotes

Fix a race condition when inserting into the editor from a dialog context

Fix the reference to the current class name

Incorrect detection of numerical lists using `[list=1]`

Skip unrelated stylesheets when compiling the frontend style

See https://www.woltlab.com/community/thread/296365-scss-dateien-unter-acp-style-werden-f%C3%BCr-das-frontend-kompiliert/

Preserve the category parameter after submitting the settings

See https://www.woltlab.com/community/thread/297264-zur%C3%BCcksetzen-der-verlinkung-in-den-einstellungen/

Skip attempts to install the same package twice

See https://www.woltlab.com/community/thread/297390-fehler-bei-installation-mit-storecode/

Skip incomplete articles that contain no content

See https://www.woltlab.com/community/thread/297481-trying-to-get-property-comments-of-non-object/

Fix the check for enabled wysiwyg form tabs

See https://www.woltlab.com/community/thread/297588-settingstab-wird-nur-angezeigt-wenn-module-smiley-aktiviert-ist/

Merge pull request #5029 from WoltLab/template-pip-validation

Improve validation of the file list within the template PIP archives

Check WCF::AVAILABLE_UPGRADE_VERSION in PackageEnableUpgradeOverrideForm

Improve validation of the file list within the template PIP archives

It previously was possible to deploy all kinds of garbage that was impossible
to clean up later, because the uninstallation assumes that the templates have a
`.tpl` file extension.

Resolves #4698

Merge pull request #5025 from WoltLab/user-storage-parameter-types

Add proper parameter types to UserStorageHandler

Add proper parameter types to UserStorageHandler

Previously attempting to store an array in the user storage without manually
serializing it appeared to succeed, as the `->update()` call accepted it and
stored it in the `->log`. However the actually persisting would fail during
shutdown, as the array would be implicitly converted to a string, thus causing
a PHP Warning to be emitted.

Add parameter types to ensure the error is immediately detected when calling
`->update()`, as errors during shutdown are very hard to debug.

Return types are not added, as UserStorageHandler is not final and thus
technically child classes could exist. Adding types would break these child
classes.

see https://www.woltlab.com/community/thread/297525-array-to-string-conversion-fehlermeldung/

Show the email address in the English version of the Email MFA method's description

Fixes #5018

Properly handle enterprise mode permissions when showing systemIdMismatch in acp/index.tpl

see b7a2fba6ba6498a1c3a618addf6084ddfc7aa5c9

Release 5.5.5

Merge branch '5.4' into 5.5

Release 5.4.23

Remove redundant TTestableCategorizedUserNotificationEvent from ArticleComment*UserNotificationEvent

This is already provided by TTestableArticleCommentUserNotificationEvent.

see #4777

Update for PHP CS Fixer 3.11.0

Release 5.5.5 dev 2

Merge remote-tracking branch 'origin/5.5' into 5.5

Merge branch '5.4' into 5.5

Merge pull request #5009 from WoltLab/attachment-csp

Add security headers to AttachmentPage

Prevent MIME sniffing for attachments

Configure a restrictive content-security-policy for attachments

Pass int to FileUtil::checkMemoryLimit() in ImageAdapter::checkMemoryLimit()

see 74accff0500ef3d635605b21dd838e8ac673be04

Fix wrong indentation in box menus in left sidebar

For active menu items the padding was overwritten causing the indentation to be lost.

Merge pull request #5000 from WoltLab/purifier

Update htmlpurifier to 4.16.0

Update htmlpurifier to 4.16.0

Merge remote-tracking branch 'origin/5.5' into 5.5

Guard against throwing unserialize handlers when unserializing template metadata

Fix editing of active paid subscription that has a permanent length

Add `type="button"` to jsButtonAttachmentInsertThumbnail

This was missed in 35c59174f69bbaca5bc72c3a49beb886594ada11.

Release 5.5.5 dev 1

Updating minified JavaScript files

Unescape the apostrophe in notification messages

See https://www.woltlab.com/community/thread/296665-unicode-dezimal-bei-desktop-benachrichtigungen/

Append a cache buster to upload images

See https://www.woltlab.com/community/thread/296221-cache-in-template-uploadfieldcomponent/

Fix the scroll behavior on iOS when opening the comment editor

See https://www.woltlab.com/community/thread/297023-kommentare-unter-ios-antworten-springt-mobil-zum-seitenende/

Mark the attachment controls as plain buttons

See https://www.woltlab.com/community/thread/297057-absenden-des-formulars-via-eingabetaste-m%C3%B6chte-dateianhang-l%C3%B6schen/

Merge branch '5.4' into 5.5

DatabaseObjectList::seekTo() lead to an error if the list was empty

Merge pull request #4995 from WoltLab/package-fix-installation

Fix handling of multi-step upgrades that need to happen in lock-step

Add assertion to PackageInstallationNodeBuilder::buildPluginNodes()

Fix handling of multi-step upgrades that need to happen in lock-step

Consider the following situation:

- Package com.example.foo is installed in version 1.0.0.
- Version 1.0.1 can be upgraded from 1.0.0.
- Version 1.0.2 can be upgraded from 1.0.1 and adds a dependency on
  com.woltlab.bar which is not yet installed.
- Version 1.0.3 can be upgraded from 1.0.2.

Now the PackageinstallationScheduler will build the following plan when it's
desired to upgrade com.woltlab.foo from 1.0.0 to 1.0.2:

- Upgrade com.woltlab.foo to 1.0.1
- Install com.woltlab.bar to satisfy the dependencies for 1.0.2
- Upgrade com.woltlab.foo to 1.0.2
- Upgrade com.woltlab.foo to 1.0.3

Now when build the nodes for this plan, the upgrade instructions for 1.0.2 will
not be detected, as the "previous package" logic used for iterative upgrades
will set the previous package of com.woltlab.foo in 1.0.2 to com.woltlab.bar.
Thus when upgrading to 1.0.2 the node builder will believe that com.woltlab.foo
is installed in 1.0.0 when it actually is already upgraded to 1.0.1.

Fix this by leveraging the $pendingPackages list which is already kept up to
date for dependency resolution.

Add safety check to PackageInstallationNodeBuilder to detect corrupted installation plans

Merge pull request #4994 from WoltLab/insert-real-database-table-names-performance

Bypass all logic in ApplicationHandler::insertRealDatabaseTableNames() for `WCF_N == 1`

Bypass all logic in ApplicationHandler::insertRealDatabaseTableNames() for `WCF_N == 1`

see 1cfa4eb41df8014d1c001f612c46d788e9434144

Add identifiers to the mobile menu items

See https://www.woltlab.com/community/thread/297081-eindeutige-bezeichner-f%C3%BCr-mobile-men%C3%BCpunkte-fehlen/

Add the missing language switch to the mobile guest menu

Fixes #4990

Update `@types/google.maps` npm dependency

Update npm dependencies

Fix FeedArticle::getComments()

Closes #4983

Remove Article::$comments from PHPDoc

Closes #4982

Merge pull request #4989 from WoltLab/search-message-abstract-fix

Fix consideration of the search query when generating message excerpts in search results

Fix consideration of the search query when generating message excerpts in search results

Since the rebuild of the search function the results are loaded via AJAX. This accidentally broke the generation of the message excerpt, since the highlight parameter is no longer present at this point.

Report the timing in error messages in SmtpEmailTransport::read()

Set `->lastWrite = '*connect*'` after connecting to the SMTP server

Consume the response to HELO

see b04881b5e13300a590c1292461e58116dcbb7d89

Merge branch '5.4' into 5.5

Fix possible SMTP desync if a timeout strikes

see 9ae8a0e5da751e2abfcb00a621056c3a15ed009f

Explicitly handle `fgets()` returning `false` in SmtpEmailTransport

(cherry picked from commit a6ed0b255968e9ef44c6e37f7eb71fa4ad5256ea)

Fix handling of unsupported EHLO in SmtpEmailTransport

This was broken, because the SMTP exceptions are no longer `SystemException`s
since 280b49db3ccccb2c3db34d429cf760221d56116f.

Explicitly handle `fgets()` returning `false` in SmtpEmailTransport