Merge pull request #4945 from WoltLab/deprecated-js-modules

Removal of unused legacy components

Remove the unused `WCF.Search.Message.js` file

Merge pull request #4946 from WoltLab/remove-package-cli

Remove PackageCLICommand

Remove PackageCLICommand

This is broken by design and completely unmaintained.

Use `->prepare()` in PackageInstallationDispatcher

Initialize LegacySession with `null`

Otherwise ACP will access an uninitialized property during shutdown.

see 0867fa82fd1faaf2e1e9192102cae97a9759df70

Remove unused legacy components

Removed the unused `flexibleArea.js` plugin

Remove unused legacy components

Remove an unused legacy component

Remove the unused smartphone detection in jQuery

This was a custom detection that was entirely flawed and hard-coded to always return `false` since 5.5.

Remove an unused and dangerous window extension

Remove a workaround for `window.console` in IE<11

Merge pull request #4942 from WoltLab/fileutil-downloadFileFromHttp

Remove FileUtil::downloadFileFromHttp()

Merge pull request #4941 from WoltLab/legacy-mail

Remove the legacy Mail class

Merge pull request #4940 from WoltLab/prism-deprecation

Remove deprecated Prism.wscSplitIntoLines

Remove FileUtil::downloadFileFromHttp()

This method is deprecated since roughly forever, has a super questionable API
and even itself relies on the deprecated HTTPRequest API. Users really should
migrate to Guzzle.

Resolves #4373

Remove the legacy Mail class

This one is long-deprecated and comes with quite a few limitations compared to
the modern Email API. It's time to remove it for good.

see 4fb6319f03fe22334e6e16ce15bf736ff87c0b67
see #2756

Remove deprecated cookie management methods from SessionHandler

These are for internal use, third party code should not have any reason to call
those.

Add proper types to SessionHandler

Merge pull request #4939 from WoltLab/wcfacp-initAuth-middleware

Move ACP authentication check into middleware

Remove deprecated Prism.wscSplitIntoLines

This method was primarily meant for internal use, hijacks the original Prism
object, is not TypeScript and has a clean replacement in Prism/Helper.ts.

Remove obsolete imports in WCFACP

Move ACP authentication check into middleware

This is similar to #4935 / 51154ba3f8f1d09b54560d5d1933f9053ef409cb.

Merge pull request #4935 from WoltLab/ban-middleware

Move ban checking from WCF boot to middleware

Merge pull request #4937 from WoltLab/5.6-to-6.0

Adjust references to 5.6 to 6.0

Drop obsolete update_com.woltlab.wcf_5.5_deleteDsStore.php

Adjust references to 5.6 to 6.0

Use `Helper::isAjaxRequest()`

Add `\wcf\http\Helper`

Move ban checking from WCF boot to middleware

The previous location in WCF made sense back when `initBlacklist()` also
checked the IP address, User-Agent, or hostname blocklist to prevent processing
the request as much as possible. As all these checks are gone now, the only
thing that remains is the inexpensive ban check.

Move it into a middleware, it does not really belong into WCF which should just
be responsible for booting the framework independently of the request in
question.

Merge pull request #4934 from WoltLab/remove-security-token-constants

Remove the `SECURITY_TOKEN` PHP constants

Merge pull request #4933 from WoltLab/searchengine-parseSearchQuery

Remove always-throwing methods from AbstractSearchEngine

Use typed properties in Session class

Use typed property in UserLoggedIn event

Remove the `SECURITY_TOKEN` PHP constants

These are trivially replaced by the `{csrfToken}` template tag which was
backported all the way to 5.2.

see #3612

Remove always-throwing methods from AbstractSearchEngine

These dangerous methods were not immediately removed to allow pointing the
developer to the appropriate documentation, instead of them just seeing a call
to an undefined method.

Since they now served their purpose and don't do anything useful, they can be
removed.

Merge pull request #4932 from WoltLab/scssphp-compatibility-include

Remove scss.inc.php compatiblity include

Remove obsolete PHPDoc type in SearchIndexManager

Apparently the file did not properly safe before commiting.

see bd0439ac21208b02c347f7075632e7433e8235d0

Fix signature in SearchIndexManager::getTableName()

see db4325a863919ee1eea073b8ab8ac1fab377584e

Fix CronjobPackageInstallationPlugin

This fixes:

> 6 arguments are required, 5 given

The `startMinute` got lost in b355b1c75179f724cb0353247ce9b0b0f4712975.

Remove scss.inc.php compatiblity include

This is longish-deprecated and the fix is trivial: Just remove the include, the
classes are implicitly available by composer's autoloader.

Merge pull request #4929 from WoltLab/remove-gravatar

Remove Gravatar support

Remove Gravatar support

Resolves #3894

Fix formatting in en.xml

see 74477cc30f352f86b6aabe0e21d6a2a589636574

Merge pull request #4928 from WoltLab/remove-memcached

Remove Memcached support

Merge pull request #4927 from WoltLab/template-wcfversion

Remove __wcfVersion template variable

Merge pull request #4926 from WoltLab/remove-php-highlighters

Remove PHP-based syntax highlighters

Merge pull request #4925 from WoltLab/cleanup-search-index-manager

Clean up SearchIndexManager

Merge pull request #4924 from WoltLab/cleanup-cryptoutil

Clean up CryptoUtil

Remove Memcached support

While Memcached is only deprecated since 5.4, it also is effectively
unmaintained and untested since longer than that.

The Disk cache works fine for the vast majority of users and Redis is available
for the others and even more featureful compared to Memcached.

Resolves #3642

Remove PHP-based syntax highlighters

These are long-deprecated. If someone desires to still use them over Prism,
they can grab them from the commit history.

Resolves #4296

Remove long-deprecated methods from PasswordUtil

PasswordUtil itself is wholly deprecated, but these two methods are trivially
replaced and developers should to so, if only to improve the clarity of the
code.

Remove __wcfVersion template variable

This one is long-deprecated, breaks very obviously and is trivially replaced.

Add types to SearchIndexManager where easily possible

Parameter types to delete() and reset() could not be added, because they
violate LSP unless the interface is modified as well / modified first.

Remove deprecated methods from SearchIndexManager

As these are long deprecated and thin wrappers only, they can be trivially
adjusted.

Add parameter types to CryptoUtil

Remove deprecated methods from CryptoUtil

These are trivially replaced by their native counterparts.

Merge branch '5.5'

Merge branch '5.4' into 5.5

Merge pull request #4923 from WoltLab/crontab

Offload cronjob calculations to a library

Deprecate CronjobUtil

Replace use of CronjobUtil by dragonmantank/cron-expression

Add dragonmantank/cron-expression composer dependency

Merge branch '5.5'

Fix PHPDoc type in IFormField

Merge pull request #4918 from WoltLab/diff

Use sebastian/diff for diff generation

Merge branch '5.5'

Merge pull request #4920 from WoltLab/row-format

Set an explicit `ROW_FORMAT=dynamic` in CREATE TABLE

Merge pull request #4921 from SoftCreatR/patch-2

Fix Facebook media provider

Fixed Facebook media provider

Usernames CAN contain periods, but they're currently not supported. There's also no need for a trailing slash.

Example: https://www.facebook.com/RaiPlay.it/videos/1059491774481091

Remove useless array-emptiness checks in WCFSetup::logFiles()

a) It is impossible for theses arrays to be empty, unless the installation is
seriously broken.
b) Even if they were empty, it would not cause issues, as the foreach loop
simply won't be entered.

Store SHA-256 hashes in wcf1_package_installation_file_log during WCFSetup

see 01ec69848a725e649b045f26cd16720581071076

Stop referencing WCF_N in WCFSetup

This is guaranteed to be 1 since e3fd38fd7ef7e3bb1dbb871fdd6786c7009aa36d.

Fix use of `->prepare()` in FilesFileHandler

see 01ec69848a725e649b045f26cd16720581071076

Use `->prepare()` in SQL queries with static table names in PIPs

Set an explicit `ROW_FORMAT=dynamic` in CREATE TABLE

Resolves #4910.

Migrate to sebastian/diff

Deprecate the majority of methods in \wcf\util\Diff

Add Diff::rawDiffFromSebastianDiff()

This eases the migration, because consumers can switch to sebastian/diff's
generation logic, while preserving the legacy output format.

Add sebastian/diff composer dependency

Add `sort-packages: true` to composer.json

Merge branch '5.5'

Detect incorrect system font stack inserted in Chrome

Merge pull request #4917 from WoltLab/attachment-tab-a11y

Add missing button role in attachment management tab

Merge branch '5.5'

Add missing button role in attachment management tab

Merge pull request #4916 from WoltLab/assert

Attempt to enable proper `assert()` handling

Attempt to enable `zend.assertions` if debug mode is enabled

Force enable `AssertionError` for assertion failures

Fix missing copying of conditions when copying boxes

Add `str_contains`, `str_starts_with`, `str_ends_with` to allowed template modifiers

Replace `mt_rand()` with `random_int()` in non-deprecated methods

Update composer dependencies

Merge branch '5.5'

Update composer dependencies

Merge branch '5.5'

Update composer dependencies

Scroll to the editor after inserting the quote

See https://www.woltlab.com/community/thread/296552-zitieren-von-kompletten-beitr%C3%A4gen-springt-nicht-in-den-editor/

Merge pull request #4914 from WoltLab/file-log-sha256

Store SHA-256 hashes in `wcf1_package_installation_file_log`