Merge pull request #4889 from WoltLab/5.5-fix-delete-button-unknown-images

Fix delete button appears on failed uploads

Merge tag '5.5.0_RC_3' into 5.5

Fix shrinking of content navigation buttons in desktop version

https://github.com/WoltLab/WCF/commit/a3e4ab16d9a566149de88e2d4b649accc768f012 caused the text on the buttons to be severely shortened when the content title became too long.

Fix delete button appears on failed uploads

Release 5.5.0 RC 3

Internal release 5.5.0 RC 3

Updating minified JavaScript files

Merge pull request #4886 from WoltLab/user-menu-incorrect-counter-update

Update the Unread Counter Using the Server-side Counter

Update focus-trap and tabbable

Merge branch '5.4' into 5.5

Update typescript

see 41b5a7f4fcc27a2fd4434d2feedb10e942930d51. We specifically upgrade
typescript also in 5.4 to ease merges, because the output of const enum
changed.

Update gsactions/commit-message-checker

Update npm dependencies

Update the unread badge using the server-side data

Update the unread badge using the server-side data

The previous implementation relied on the number of unread items currently visible in the user menu. This caused an incorrect number shown for >10 unread items.

Notice: This is a last-minute API change to report the correct number from the server-side.

Fixes #4877

Preserve the content when removing selection markers

In some rare cases the typed-in content could be placed within the selection markers, causing those to be removed whenever markers are being stripped.

This is a follow up for dd829b643d403f403fae8748a8f00e2e448af490 which already mitigates some of these issues by more carefully handling selections.

Merge pull request #4883 from WoltLab/main-menu-data-binding

Partial Data Binding for the Main Menu

Use the `null` value to indicate a missing identifier

Sync the unread counters of the main menu and the mobile menu

Expose the identifier of a menu item

Close the mobile navigation drop-down on page scroll

All drop-down menus are expected to close whenever the page is being scrolled. This is implemented for regular drop-down menus, but this one bypasses the general API.

See https://www.woltlab.com/community/thread/296092-fenster-zitieren-inhalt-melden-%C3%BCberlagert-men%C3%BC-mobil/

Delay the auto focus of the submit button

Safari delays the execution of the `blur` event on the document which causes the focus shift to happen too early.

Delaying it by two event cycles are enough to provide a consistent experience and to avoid conflicts with the focus trap.

See https://www.woltlab.com/community/thread/296069-best%C3%A4tigung-bei-installation-von-paket-%C3%BCber-entwickler-werkzeuge-nicht-direkt-pe/

Preserve the minimum height of the user header without ranks

The `.userMenuItemSingleLine` class is shared with other elements that are not as tall as the user header.

The `min-height: 0` will cause the user header to collapse too much when the user rank module is disabled.

See https://www.woltlab.com/community/thread/296134-darstellungsfehler-im-kontrollzentrum-ohne-rangmodul/

Fix missing check if conditions are available when adding a box

Merge branch '5.5' of https://github.com/WoltLab/WCF into 5.5

Ensure that avatars are world-readable in UserProfileAction::setAvatar()

Fixes #4769

Merge branch '5.5' of https://github.com/WoltLab/WCF into 5.5

Fix missing twitter icon

Merge pull request #4882 from WoltLab/secure-package-download

Improve the Security of the Plugin-Store StoreCode

Clearly mark the StoreCode as sensitive

The Plugin-Store StoreCode can contain credentials that are considered to be sensitive and should not be shared with third parties.

This change adds a unique prefix that serves as a warning to prevent users from unintentionally sharing these credentials with others.

In addition, this unique prefix makes it possible to block this string from appearing in messages using the censorship feature.

Guarantee integrity of packages downloaded via a Plugin-Store StoreCode

The package system was unaware of the context of an installation request and permitted the download from unintended package servers.

This can cause the download to be initiated from a different server than the user expected, potentially causing the download of a modified version.

This commit fixes this issue by restricting the package sources to official servers only when the download via the Plugin-Store‘s StoreCode is requested.

Explicitly define the text color in the mobile menu

The elements previously implicitly inherited the text color from the surrounding HTML context rather than the element itself. This caused the colors from the content area to bleed into the mobile menu.

See https://www.woltlab.com/community/thread/295243-pro-und-kontra-neues-kontrollzentrum/?postID=1897430#post1897430 ff

Force restore saved selections after inserting links into the editor

Redactor‘s selection handling is very fragile and “forgetting” to restore the selection can cause some very strange side-effects.

The user will start typing inside a selection marker, which will not cause any visual effects, but the content will get lost once the editor removes the markers.

See https://www.woltlab.com/community/thread/296022-einf%C3%BCgen-von-links-im-editor-unter-ipados/
See https://www.woltlab.com/community/thread/296116-einf%C3%BCgen-von-links-im-editor-unter-ipados-text-verschwindet/
See https://www.woltlab.com/community/thread/294753-text-wird-beim-speichern-abgeschnitten/

Merge branch '5.5' of https://github.com/WoltLab/WCF into 5.5

Fix sorting of articles by number of comments

Sorting by comments resulted in an exception because the comments column is no longer present in the article table.

Strengthen check for non-instantiable classes in ControllerMap::getClassData()

Refresh the mobile unread indicator on DOM update

The unread indicator might become out of sync if the menu is closed, but an action causes the last remaining element(s) to be marked as read.

Using a `MutationObserver` removes the need of signaling such change directly by all actions, for example a “Mark All as Read” button.

See https://www.woltlab.com/community/thread/296063-alle-als-gelesen-markieren-entfernt-badge-nicht/

Detect active menu items that have no children

Links with children have the `active` class set on them, but links without have it placed on the parent instead.

See https://www.woltlab.com/community/thread/296045-mobiles-hauptmen%C3%BC-teilweise-nicht-vorausgew%C3%A4hlt-wenn-man-auf-einer-unterseite-is/

Preserve the scroll position in CodeMirror on navigation

The previous code stopped working after the update of CodeMirror.

See https://www.woltlab.com/community/thread/296010-nach-css-%C3%A4nderung-nicht-an-selber-stelle/

Merge branch '5.4' into 5.5

Bind event listeners on editor elements only once

The previous code caused the event listeners to be bound with every iteration. Every invocation touches the DOM by updating the title, which in return caused `observe.load()` to be triggered again. This causes the number of bound event listeners (and thus DOM updates!) to increase exponentionally.

See https://www.woltlab.com/community/thread/296068-dialog-container-vom-spoiler-tag-kann-das-forum-aufh%C3%A4ngen/

Fix appearance of content navigation buttons in mobile version

Buttons went beyond the edge of the screen instead of wrapping.

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

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Update Guzzle

see guzzle/guzzle@a52f0440530b54fa079ce76e8c5d196a42cad981

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Update guzzlehttp/psr7

This is a dependency for an updated Guzzle.

see guzzle/psr7@e98e3e6d4f86621a9b75f623996e6bbdeb4b9318
see guzzle/guzzle@a52f0440530b54fa079ce76e8c5d196a42cad981

Merge branch '5.4' into 5.5

Merge branch '5.3' into 5.4

Regenerate composer files

Release 5.5.0 RC 2

Updating minified JavaScript files

Add missing newline at EOF in acp/style/layout.scss

see 89763fcf8fea68d5cda6bda419a5bd8326d4db58

Skip the search close handler if it was not open

Executing the callback causes some unintentional changes to the page state.

See https://www.woltlab.com/community/thread/295991-dialoge-hintergrund-scrollt/

Align the mobile ACP user drop-down to the right

See https://www.woltlab.com/community/thread/295980-acp-men%C3%BC-mobil-ausrichtung-%C3%A4ndern/

Merge branch '5.5' of https://github.com/WoltLab/WCF into 5.5

Fix non-functioning template scripting in language variable

Merge branch '5.4' into 5.5

Match foreign keys independent of referenced column casing in PHP DDL

Early versions of MySQL 8 contained a bug that would return the column names of
the referenced table in lowercase whenever querying the FOREIGN KEY or when
showing the table's structure:

https://bugs.mysql.com/bug.php?id=88718

This issue also affects the any created database dumps and is not corrected
when importing the dump into a fixed version of MySQL:

https://bugs.mysql.com/bug.php?id=98976

Thus any instances that were ever dumped with an affected version of MySQL 8
will have foreign keys with a mismatching column case.

The FOREIGN KEY itself will be completely functional, MySQL does not care about
the casing of column names.

However the PHP DDL considers those FOREIGN KEYs to be different when
attempting to find an existing FOREIGN KEY to update, thus believing that the
FOREIGN KEY does not yet exist. As a result it attemtps to create it,
generating a name that is identical to the existing FOREIGN KEY's, leading to
conflicts when attempting to log the change into package_installation_sql_log.

Fix this issue by lowercasing the referenced column names when grabbing the
`->getDiffData()`. The PHP DDL will then find the existing FOREIGN KEY, but it
will realize that it is different when diffing the `->getData()`.

This will lead to the "broken" FOREIGN KEY being dropped and it being recreated
correctly, both avoiding the duplicate key error and also actually correcting
the column name.

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

Merge branch 'php8.2-fixes' into 5.5

Fix PHP 8.2 compatibility in TemplateEngine

> Creation of dynamic property wcf\system\template\TemplateEngine::$pluginObjects is deprecated
> Creation of dynamic property wcf\system\template\TemplateEngine::$tagStack is deprecated

Fix PHP 8.2 compatibility in HascontentPrefilterTemplatePlugin

> Use of "self" in callables is deprecated

Fix saving box for page assignments

The methods still contained code that became obsolete with the PR linked below.

see https://github.com/WoltLab/WCF/pull/4866

Removed obsolete update script for box condition conversion

see https://github.com/WoltLab/WCF/pull/4866

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

Update composer dependencies

Merge pull request #4866 from WoltLab/box-visibility-conditions

Hiding boxes did not work properly

Modernized the embedded JavaScript

Hiding boxes did not work properly

Add missing semicolon in Quote.ts

see cc8832d874e4da51c58b7e44503eb3701abbd7a7

Mark closure in KeywordHighlighter as static

see 406348deb7debaeaa908db6fc1a724a40e82e8a3

Fix typo in PHPDoc of CheckboxFormField

Skip the highlighting of words with a length < 3

See https://www.woltlab.com/community/thread/295850-highlighter-in-suchergebnissen-hebt-zu-kurze-begriffe-hervor/

Quote links now imply the `https` protocol if not set

Default to `https` for protocol-less links

Use the same icon for quotes in the editor

Right sidebar boxes did not span the entire width

See https://www.woltlab.com/community/thread/295902-rechte-sidebardarstellung-im-userprofil-mobil-fehlerhaft/

The CSS was accidentally removed in c00bdb0694b993be21d779ecef6badc8524e8967

Merge pull request #4863 from WoltLab/thumbnail-graceful

Gracefully ignore Thumbnail generation failures in DefaultUploadFileSaveStrategy

Gracefully ignore Thumbnail generation failures in DefaultUploadFileSaveStrategy

`->generateThumbnails()` might already not do anything of the memory limit is
likely going to be exceeded, causing image upload to be thumbnail-less. Extend
this to all cases of thumbnail generation failure by catching exceptions thrown
when loading the image or when performing the resizing operation.

Fix SCSS formatting

Update composer dependencies

Especially for guzzle/guzzle@e3ff079b22820c2029d4c2a87796b6a0b8716ad8.

Merge branch '5.4' into 5.5

Update Guzzle

see guzzle/guzzle@e3ff079b22820c2029d4c2a87796b6a0b8716ad8

Merge branch '5.3' into 5.4

Update Guzzle

see guzzle/guzzle@724562fa861e21a4071c652c8a159934e4f05592

Use ellipsis to replace quotes in simplified text

Optimized message list / message group list for medium screens

New grid list element to replace `.containerList`

See https://www.woltlab.com/community/thread/295789-%C3%BCberlappung-in-suchergebnissen-bei-langem-themen-titel/

Incorrect alignment of drop-down menus on mobile

See https://www.woltlab.com/community/thread/295903-abonniert-dropdown-fehlerhafte-darstellung/

Update composer dependencies

Suppress PHP 8.1 deprecation for IDN domains

The true/punycode library appears to be no longer maintained and is not PHP 8.1
deprecation clean: true/php-punycode#35

Suppress the deprecation warning for now. The library should be replaced in a
future version.

Merge branch '5.4' into 5.5

Release 5.4.19

Fix update instructions from 5.4

Release 5.5.0 RC 1

Updating minified JavaScript files

Clear `AVAILABLE_UPGRADE_VERSION` for 5.5

see 15933d22a95b67d57ae794a88471098326911aba