Merge pull request #4776 from WoltLab/scroll-to-comments

Scrolling to a comment not visible by default did not work

Visual separator for specialized search filters

See https://www.woltlab.com/community/thread/295502-kategorie-auswahl-ist-verrutscht/

Merge branch '5.4'

Merge branch '5.3' into 5.4

Merge branch '5.2' into 5.3

Revert "Show always an no selection option in custom select options build with the OptionHandler"

This reverts commit 6fef8b82e15794eee5317e6b15bb0670f137315c.

Incorrect margin of `.formSubmit` in dialogs

This was caused by the negative margin introduced in 1d224f10cdd05e7f5d09f869ee1fbbbdff025749 which caused the calculation to be off by 10px, causing a visible gap.

See https://www.woltlab.com/community/thread/295497-fehlerhafte-mobile-darstellung-von-dialogen/

Notification type was not hidden when modules were disabled

Scrolling to a comment not visible by default did not work

Inline editors had been not initialized properly

Unified phrases

Merge pull request #4774 from WoltLab/focus-trap-tabbable

Update focus-trap and tabbable npm dependencies

Fix dependency sync check in javascript.yml

Add check that dependencies are in sync to javascript.yml

Update focus-trap and tabbable npm dependencies

Merge branch '5.4'

Update npm dependencies

Add PHP 8.1 to the php.yml workflow

Drop obsolete exclude in php.yml workflow

This should no longer be required since 123aedf297e0402e6bd3562fe08e2d74e24760d1.

Update version constraints in composer.json

Require the currently locked versions as the minimum versions.

Merge pull request #4772 from WoltLab/htmlpurifier-4.14

Update to ezyang/htmlpurifier 4.14.*

Labeling for buttons (trash and delete) was identical

Update to ezyang/htmlpurifier 4.14.*

Sort dependencies in composer.json

Add explicit composer dependency for guzzlehttp/psr7

Remove validation attributes from __sourceCodeFormField.tpl

These are not supported with CodeMirror, because it only syncs the entered data
upon `submit()` which notably runs *after* validation.

see codemirror/CodeMirror#5092
Fixes #4732

Fix typo in function name in AbstractFileDeletePackageInstallationPlugin

This must be `strtr` not `strstr`.

see #4714

Load the page logo using the `eager` policy

Using `loading="lazy"` for the page logo causes the image to flicker noticeably in Safari on both macOS and iOS.

See https://www.woltlab.com/community/thread/295403-logo-auf-der-startseite-beim-aktualisieren-ein-rechteck/

Merge branch '5.4'

Merge pull request #4768 from WoltLab/sitemap-page

Fix handling of CMS pages in sitemap

Fix handling of CMS pages in sitemap

Delegate the visibility control and access control to the appropriate methods
in \wcf\data\page\Page instead of reimplementing it from scratch. Most notably
the inversion of the page ACL was not implemented correctly within the sitemap.

see 92fba0538afc1d88f411db1a80553af2d17c09b4
Closes #4767

Co-authored-by: mutec <mysterycode@mysterycode.de>

Release 5.5.0 Alpha 6

Updating minified JavaScript files

Merge pull request #4764 from WoltLab/content-interaction-button-icons

Use icons in content interaction buttons

Adapt icon dynamically to the actual sort order

Removed a superfluous data attribute

This was a temporary part of the development process.

Fixes #4765

Merge pull request #4766 from mutec/appmanmudose

fix application management in multi domain setups

fix application management in multi domain setups

Changing the landing pages of apps was failing since the domain name for single-domain-setups was validated for any case, but is not set when using a multi-domain-setup.
This lead to an un-meaningful error-message saying something is incorrect.

Page change in search results scrolls to top automatically

Added parameter to configure the scroll behavior

Incorrect pronoun in article comment notifications

Reduced size of form submit buttons (on mobile)

Swapped the position of the user and page menu

See https://www.woltlab.com/community/thread/295422-position-des-hamburger-men%C3%BCs/

Use icons in content interaction buttons

Stop hiding icons in content interaction buttons

Merge pull request #4753 from WoltLab/article-timestamp

Ensures that the date of a published article is not in the future.

Show error message if article date is in the future

Release 5.5.0 Alpha 5

Updating minified JavaScript files

Merge pull request #4763 from WoltLab/user-profile-buttons

Overhauled user profile buttons

Ignore connection errors caused by page navigation

See WoltLab/com.woltlab.wbb#539

Delay menu interaction until the page is loaded

User menu providers are registered asynchronously and the UI does not know once they are all ready.

See https://www.woltlab.com/community/thread/295264-userpanel-l%C3%A4dt-unvollst%C3%A4ndig/

Simplified the code logic

Use icons to represent the moderation queue types

Fixes #4742

Added standalone button for "edit profile"

Removed duplicate link to avatar form

The avatar itself is already linked to the avatar form.

Use content interactions button in user profiles (management menu)

Use content interactions button in user profiles (search menu)

Made use of content interaction dropdown in user profiles

Merge pull request #4704 from WoltLab/string-validation

Use is_string to validate strings

Merge pull request #4762 from WoltLab/password-cleanup

Cleanup PHPASS password implementations

Remove the Google Plus profile field

Fixes #4741

Deprecate TPhpass::hash() and TPhpass::needsRehash()

Explicitly implement `Phpass::hash()` and `Phpass::needsRehash()`

They don't really belong into the `TPhpass` trait.

Implement Drupal8::needsRehash()

Move the Drupal8 costs into a class constant

Delegate to TPhpass in Drupal8

Use `\hash('md5', …)` instead of `\md5()` in TPhpass

This makes it easier to parameterize the implementation.

Delegate base64 encoding for Drupal8 to TPhpass

Add TPhpass::encode64()

Merge branch '5.4'

Add the Drupal8 hashing algorithm

Fix the TPhpass algorithm

The salt extraction used `mb_strpos` where `mb_substr` needs to be used.

see c586e46e62dd1d714b7c7db7911eb6cf5d96cc7e

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.

Match stop words case insensitively in MysqlSearchEngine

Replace MysqlSearchEngine::getStopWords() by ::isStopWord()

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