Merge pull request #5190 from WoltLab/php-ddl-foreign-key-drop

Fix dropping of misnamed foreign keys in PHP DDL

Fix dropping of misnamed foreign keys in PHP DDL

Foreign keys are matched up by their `getDiffData()` which includes the column
list, referenced column list and referenced table, but does not include the name.

This effectively ensures that only a single foreign key exists for each
possible combination of source and target columns.

Dropping foreign keys however relies on the foreign key’s name being sent to
the database and this is currently broken when the foreign key name differs
from the expected name:

The misnamed key will be matched up, but the DROP query will send the expected
name, instead of the actual name.

Fix this by inserting the `$matchingExistingForeignKey` into the list of keys
to drop, which makes sense, because the existing key is what should be dropped
in the first place.

Merge pull request #5186 from WoltLab/fileDelete-validation

Fix validation in AbstractFileDeletePackageInstallationPlugin

Silently skip files belonging to other packages in AbstractFileDeletePackageInstallationPlugin

Instead of throwing, we just silently skip files belonging to other packages,
because this implies that the file no longer exists for the package in
question.

Fix validation in AbstractFileDeletePackageInstallationPlugin

The query for logged files must not include the `packageID` in its condition,
as the entire purpose is retrieving all the `packageID`s for installed files to
compare them against the package that is currently installed or updated. Thus
by selecting only files for the current packageID the purpose is defeated.

see 672cd6166b684767a3fc8ee1fd6a4516d2061285

Merge pull request #5184 from WoltLab/external-image-internal-hostname

Merge `INTERNAL_HOSTNAMES` into list of allowed image domains

Merge `INTERNAL_HOSTNAMES` into list of allowed image domains

Fixes #5146

Use BBCodeHandler::getImageExternalSourceWhitelist() in HtmlOutputNodeImg

Related to #5146

Fix baseclass inheritance check in DatabaseObjectList::__construct()

see https://www.woltlab.com/community/thread/298219-fehlerhafte-%C3%BCberpr%C3%BCfung-der-databaseobjectdecorator-baseclass/

Fix the insertion of tables into the editor

See https://www.woltlab.com/community/thread/298250-tabellen-einf%C3%BCgen-via-ipad-nicht-mehr-m%C3%B6glich/

Fix the logic to display the button for the participant list

See https://www.woltlab.com/community/thread/298009-teilnehmer-bei-umfragen-nicht-%C3%B6ffentlich-sichtbar/

The `$value` for `getFormElement()` is nullable

See https://www.woltlab.com/community/thread/298187-explode-passing-null-to-parameter-2-string-of-type-string-is-deprecated-php-8-1/

Stop using undefined properties in SearchHandler::saveSearch()

These are not intended to be stored on the class and not used elsewhere, the
`this->` is likely a copy and paste mistake in the initial implementation.

see 11b63e8aa4e1a71425e1bffd7eb28f9db4a4bf13

Merge pull request #5151 from SoftCreatR/bugfix/tpl

Make unfurlUrl template available in ACP

Made unfurlUrl template available in ACP

See https://www.woltlab.com/community/thread/298168-artikel-erstellen-im-acp-bringt-fehler-bei-vorschau/

Merge pull request #5142 from WoltLab/php-ddl-index-validate

Fix validation of changed indices in PHP DDL

Fix validation of changed indices in PHP DDL

The validation of the to-be-performed DDL operation currently identifies
indices by their data (i.e. column list and type), whereas the actual DDL
operation uses the `->diffIndices()` operation which also takes into account
the name of the index.

This mismatch allows a package to drop a foreign index, consider the following
situation:

Package A:

    DatabaseTable::create('wcf1_test')
        ->columns([
            NotNullInt10DatabaseTableColumn::create('a'),
            NotNullInt10DatabaseTableColumn::create('b'),
        ])
        ->indices([
            DatabaseTableIndex::create('testing')
                ->columns(['a']),
        ])

The package creates a table with two columns and a named index (“testing”) that
includes one of the columns.

Now Package B:

    DatabaseTable::create('wcf1_test')
        ->indices([
            DatabaseTableIndex::create('testing')
                ->columns(['a', 'b'])
                ->drop(),
        ])

This definition drops the named index (“testing”), but with a different column
definition. Thus the validation believes the indices to be different, allowing
the operation to proceed. The actual operation however identifies the index by
its name and thus drops the “testing” index that belongs to a different
package.

Release 5.5.7

The `$value` for `getSearchFormElement()` is nullable

See https://www.woltlab.com/community/thread/297958-fehler-bei-suche-mit-php-8-1/

Revert "`getSearchFormElement()` expects a string for the value parameter"

This reverts commit d2beab69c1aee0a33cfe8bdb7d967da2300b1691.

Merge pull request #5137 from WoltLab/store-code-refresh

Refresh the package database in Package/QuickInstallation.ts

Merge pull request #5133 from WoltLab/i18n-form-field-one-language

Fix TI18nFormField if only one language is active

Update to TypeScript 4.9

Update eslint npm dependency

Refresh the package database in Package/QuickInstallation.ts

Fixes #5135

Fix TI18nFormField if only one language is active

Fixes #5131

Update eslint npm dependency

Fix erroneous encoding of ampersands in HtmlInputProcessor::convertToHtml()

see f8deb9077dc485487bf53f9930ff956fbca1127b
see https://www.woltlab.com/community/thread/298042-werden-beim-import-falsch-ersetzt/

Release 5.5.7 dev 1

Updating minified JavaScript files

Construct the profile url using just the user id

The attempt to embed the username into the URL is flawed, because it provides none of the server side transformations performed when naturally generating those URLs.

The currently implementation causes a redirect for most usernames anyway. Besides those redirects take place early in the processing of the controller and thus are acceptable.

Removing the username from the artificially constructed URL will now always cause a redirect, but do not break web servers with less permissive rewrite rules.

See https://www.woltlab.com/community/thread/297758-url-memberlist-suche-%C3%ACst-falsch/

Schedule the restore of the scroll position for the next loop

The previous delay of 1ms causes issue by actions that also try to delay their execution to the next run of the event loop. These will get executed while the page offset has not been adjusted, causing the calculations to be incorrect.

This primarily affects the editor which makes use of `setTimeout()` to workaround some browser limitations.

See  https://www.woltlab.com/community/thread/297841-einf%C3%BCgen-von-links-in-den-editor/

Update `@types/google.maps` and `@types/facebook-js-sdk`

Update eslint

Update `tslib`

Add the CSS class `.formAttachmentListItem` to existing attachments on page load

See https://www.woltlab.com/community/thread/297604-dateianh%C3%A4nge-werden-bei-mehrsprachigkeit-nicht-zwischen-editoren-synchronisiert/

`getSearchFormElement()` expects a string for the value parameter

Validate that the searched option implements the required interface

Remove selection markers after discarding the link dialog

See https://www.woltlab.com/community/thread/297178-links-bearbeiten-und-kopieren/

Bind the event listeners for the RSS feed dialog everytime

The dialog is replaced with the provided HTML on every invocation. Since this is a named dialog, the instance is set up once, but rebuild with every invocation.

See https://www.woltlab.com/community/thread/297896-kopieren-schaltfl%C3%A4che-nach-erneutem-%C3%B6ffnen-des-dialogs-nicht-mehr-funktionsf%C3%A4hig/

Fix the scroll offset when the first message is being targeted

See https://www.woltlab.com/community/thread/297814-mobil-zus%C3%A4tzlicher-container-sichtbar-bei-direktlink-auf-den-ersten-beitrag-eine/

Merge pull request #5119 from WoltLab/acp-not-authorized-ux

Improve user experience when accessing the ACP with an unauthorized user

Typo

Prevent wrapping of text on narrow resolutions

There is plenty of whitespace around each column to allow for some small overlaps.

https://www.woltlab.com/community/thread/297666-blog-unsch%C3%B6ne-darstellung-der-reaktionen/

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