Merge pull request #4758 from WoltLab/mailbox-filterAddress

Unify the email validation of UserUtil::isValidEmail() and Mailbox::__construct() - Take 2

Merge pull request #4760 from WoltLab/unified-mark-as-read-language-variables

Unified language variables for "mark as read" buttons

Unified language variables for "mark as read" buttons

Merge pull request #4759 from WoltLab/content-interaction-margins

Avoid empty content interaction elements to avoid obsolete margin

Avoid empty content interaction elements to avoid obsolete margin

Delegate to Mailbox::filterAddress() in UserUtil::isValidEmail()

see 1723b426d25ecaad5034a6f74e3b29db25dc1b29

Add Mailbox::filterAddress()

see 1723b426d25ecaad5034a6f74e3b29db25dc1b29

Add explicit support for `*` in excludedPackages in `PackageUpdateAction`

Fixes #4736

Revert "Unify the email validation of UserUtil::isValidEmail() and Mailbox::__construct()"

This change broke WCFSetup, because Mailboxes implicitly carry a language which
is not readily available in WCFSetup.

The validation of the email address should likely be moved into a dedicated
method in Mailbox and then UserUtil::isValidEmail() deprecated.

This reverts commit 395f9a6affffdd5381134d3f9d39b70de453b668.

Added standalone button for "mark as read"

Removed useless RSS feed dialog for notifications

The option for an anonymous RSS feed does not make sense at this point, as guests do not have access to notifications.

Release 5.5.0 Alpha 4

Updating minified JavaScript files

Merge pull request #4757 from WoltLab/visit-tracker

Improve VisitTracker behavior

Remove support for object specific lifetimes in VisitTracker

Fix cleaning of tracked visits in DailyCleanUpCronjob

This part of the cronjob never was functional, because stored visitTime was
compared to the lifetime which could only ever match near the start of the unix
epoch.

Fix this by correctly subtracting the lifetime from the current timestamp. I've
opted to use 120 days as the cleanup lifetime, as this allows one to adjust the
DEFAULT_LIFETIME gracefully.

Simplify VisitTracker::getVisitTime()

Remove the intransparent doubling of the visit tracker lifetime if existing data exists

In case of com.woltlab.wbb this only triggers if and only if a user *ever*
marked all boards as read. With the increase of the default lifetime in the
previous commit it becomes obsolete.

Increase VisitTracker::DEFAULT_LIFETIME to 31 days

see 0fb222af73eab6621c62c5bd4403bdf7330bfad8

Use constant expression for VisitTracker::DEFAULT_LIFETIME

Enforce equal heights of all buttons

See https://www.woltlab.com/community/thread/295373-buttons-unterschiedlich-gro%C3%9F-wenn-pagination-aktiv/

Rephrased the button to set up notifications

See https://www.woltlab.com/community/thread/295368-desktop-benachrichtigungen-aktivieren-pr%C3%A4ferenzen-festlegen-falsch/

Merge pull request #4756 from WoltLab/user-util-is-valid-email

Unify the email validation of UserUtil::isValidEmail() and Mailbox::__construct()

Unify the email validation of UserUtil::isValidEmail() and Mailbox::__construct()

Pasting code into a code box caused duplicates

Superfluous CSS for dialog alignment

This was supposed to be removed by 8fb8e05c07732280ff5fea3b926b71f35db9b0ff

Disable the focus trap before hiding the dialog

See https://www.woltlab.com/community/thread/295337-mobil-thema-bearbeiten-bug/

Disable hiding of the user menu on Redactor focus

This change was missing from 106207823daf1b1105afc918fa66718ae70707e2

Disable hiding of the user menu on Redactor focus

The behavior was intended to increase the available screen space when the editor is active. However, the height of smartphones has constantly increased, reducing the impact of a couple extra pixels.

The major downside was the focus behavior which is somewhat unintuitive on mobile and has repeatedly caused confusion among users, because the menu was seemingly “lost”.

Keep the scrollbar visible while scrolling

See https://www.woltlab.com/community/thread/295349-scrollbar-ohne-hover-nicht-sichtbar-selbst-beim-scrollen/

Synchronize the unread indicator on menu close

See https://www.woltlab.com/community/thread/295340-mobiler-hinweis-verschwindet-nicht-direkt-nach-benachrichtigungen-als-gelesen-ma/

Use the DOM attribute as notification counter

User-friendly messages for network errors

See https://www.woltlab.com/community/thread/295356-load-failed-wenn-keine-netzwerkverbindung-beim-klick-auf-die-benachrichtigungen/

Merge branch '5.4'

Set explicit `referrerpolicy` for Vimeo embeds

see 936c0f6d5d3f5a811110f7add4fa688a3e9a8377
see https://www.woltlab.com/community/thread/295351/?postID=1891462#post1891462
Resolves #4755

Set explicit `referrerpolicy` for YouTube embeds

YouTube requires the `Referer` header to be set for (some) embeds to work. By
using the attribute any `referrer-policy` header set on the top level document
is overridden (e.g. `same-origin`).

Co-authored-by: Sascha Greuel <github@1-2.dev>

Fix assertion in TMultiRecipientModerationQueueCommentUserNotificationObjectType

Release 5.5.0 Alpha 3

Updating minified JavaScript files

Added standalone button for "mark as read"

Show notification after "mark as read" is done

Skip the notification tab when in maintenance mode

See https://www.woltlab.com/community/thread/295237-leeres-benutzermen%C3%BC-im-wartungsmodus/

Improves visuals when marking user menu items read

See https://www.woltlab.com/community/thread/295256-mobiles-als-gelesen-markieren-von-benachrichtigungen-schlecht-erkenntlich-und-nu/

Incorrect selector target

Moved the page menu item counter to the text

Additional indicator for the active page menu

Fixed code style

Unused import

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

Wrapping of the pagination prevented

Fixed margins when pagination top was hidden

Unified code of content header / footer navigation

Do not focus the search suggestions automatically

This interferes with the ability to enter a search term that is similar to a suggestion. Pressing enter would cause the suggestion to be filed in instead of submitting the value as-is.

Selecting a search suggestion discarded the search

See https://www.woltlab.com/community/thread/295268-suchfunktion-teilweise-umst%C3%A4ndlich/

Superfluous whitespace

Restore the user menu DOM on close

See https://www.woltlab.com/community/thread/295304-kontrollzentrum-nicht-mehr-anklickbar-nach-wechsel-der-aufl%C3%B6sung/

Exclude the macOS emoji keyboard from keypresses

See https://www.woltlab.com/community/thread/295207-hotkey-f%C3%BCr-emoji-men%C3%BC-funktioniert-im-editor-nicht/

Incorrect selection when submitting editor dialogs

Fixes #4747

Update fileDelete.xml to remove files where only the case changed

It is not guaranteed that new file case was successfully applied on case
insensitive file system, so these files might be deleted even with the
fileDelete PIP verifying the actual casing since
49c488354f3fd5b02a851a48d912d6aad3601161.

Remove those files from fileDelete.xml to be on the safe side. This is no
longer easily doable with bash only. The following PHP script was used to
generate fileDelete.xml:

    <?php
    \exec(<<<'EOT'
    git log --root --diff-filter=D --no-renames --summary |awk '/delete mode/{print $NF}' |grep wcfsetup/install/files/ |sort -u
    EOT, $delete);
    \exec(<<<'EOT'
    find wcfsetup/install/files/ |sort -u
    EOT, $preserve);
    $preserve = \array_map('strtolower', $preserve);

    $delete = \array_filter($delete, static function ($file) use ($preserve) {
     return !\in_array(\strtolower($file), $preserve);
    });

    echo <<<'EOT'
    <?xml version="1.0" encoding="UTF-8"?>
    <data xmlns="http://www.woltlab.com" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.woltlab.com http://www.woltlab.com/XSD/2019/fileDelete.xsd">
     <delete>
     <!-- Special files -->
     <file>acp/masterPassword.inc.php</file>
     <!-- /Special files -->
    EOT, "\n";

    echo \implode("\n", \array_map(static function ($file) {
     $file = \preg_replace('/^wcfsetup\\/install\\/files\\//', '', $file);

     return "\t\t<file>{$file}</file>";
    }, $delete))."\n";

    echo <<<'EOT'
     </delete>
    </data>
    EOT, "\n";

Merge branch '5.4'

Remove reference to CONTRIBUTING.md from README.md

see 5d7b7d602e0f6efb077bffda4b3ecceed8534810

Added global language variable for "mark as read" buttons

Button label optimized

Merge branch '5.4'

Detect italic/underline text pasted from Google Docs

Pagination of search results was not accessible for guests

Closes #4749

Individual sorting of search results had no effect

Release 5.5.0 Alpha 2

Updating minified JavaScript files

Incorrect focus/scroll management on iOS

See https://www.woltlab.com/community/thread/295245-antworten-button-springt-zum-footer/

Incorrect DOM behavior of `dboAction`

The invocation of the `DomChangeListener` does not make any sense any more, because the API is designed to be asynchronous. The caller must take care of this themselves.

Merge pull request #4746 from WoltLab/55-user-menu-settings

Cleaned up the user menu, extra profile link

Inconsistent layout of multiline text

Cleaned up the user menu, extra profile link

Merge pull request #4744 from WoltLab/deprecate-fetchRemoteAvatar

Deprecate UserAvatarAction::fetchRemoteAvatar()

Deprecate UserAvatarAction::fetchRemoteAvatar()

Add `@since` to UserProfileAction::setAvatar()

see 7448a453886e504eaa1d70e8a39bbb1fad14dd42
see #4583

Fetching a remote avatar didn't work if user doesn't have to permission to upload avatars

Properly check permissions for the `systemIdMismatch` message in acp/index.tpl

Merge pull request #4739 from SoftCreatR/patch-4

Remove cursive font in quotes

Updating minified JavaScript files

Merge branch '5.4'

Fix fetching userID for multilanguage articles

Merge remote-tracking branch 'origin/master'

Merge branch '5.4'

Use spaces instead of tabs

Prevent firing notifications for awarding own trophies

Removed cursive font in quotes

See https://www.woltlab.com/community/thread/295216-kursivschrift-in-zitaten-entfernen

Release 5.4.16

Merge branch '5.3' into 5.4

Release 5.3.22

Merge branch '5.4'

Removed the outdated `CONTRIBUTING.md`

The file is heavily outdated and badly presented on GitHub. With WoltLab Suite 5.4 we switched to PSR-12 and Prettier anyway, so there is no need to explain any special rules.

Closes #4600

Merge branch '5.4'

Attached images with mismatched proportions got mangled

Merge pull request #4737 from WoltLab/exclusion-check-install

Always check exclusions in PackageUpdateAction::createQueue()

Always check exclusions in PackageUpdateAction::createQueue()

Ever since the method checked for package exclusions in
1d7f1d205ee64fd2723877c1f296c592518e516d it only applied the check to updates,
not to fresh installs. It's not clear why it did so, I assume this to be an
oversight.

Change this to always run the exclusion check, otherwise exclusions might be
circumvented using PackageUpdateAction::prepareInstallation() with a
package+version combination that would regularly be excluded.

Improve types in Acp/Ui/Package/QuickInstallation.ts

Fix error handling in Acp/Ui/Package/QuickInstallation.ts

Merge branch '5.4'