Merge pull request #3637 from WoltLab/remove_meta_keywords

Remove meta keywords

Merge branch 'master' into remove_meta_keywords

Merge branch '5.3'

Hide empty table after deleting last missing language item on page

Remove obsolete references to `missingLanguageItems.txt`

Merge pull request #3627 from WoltLab/composer-deps

Update composer dependencies

Tighten up minimum version of scssphp/scssphp

Update to ezyang/htmlpurifier 4.13.*

Merge branch '5.3'

Fix possible "Invalid parameter number" error in UserStorageHandler

Remove `Promise` polyfill

Merge branch '5.3'

Merge branch '5.2' into 5.3

Merge pull request #3648 from WoltLab/actions

Refactor Workflows

Add javascript workflow

Rename php-syntax workflow to php

Adjust human readable name for CodeSniffer check

Merge pull request #3636 from WoltLab/remove-gzip

Remove in-application gzip compression

Remove in-application gzip compression

Resolves #3634

Merge branch '5.3'

Merge pull request #3646 from WoltLab/user-storage-refactor

Clean up user storage update logic

Clean up user storage update logic

This commit attempts to reduce the lock contention during user storage
shutdown. The user storage is a constant source of deadlocks and the update
logic grew increasingly ugly over the years in attempts to fix this.

While the previous situation was acceptable in that errors did not surface due
to the automated retries it still trashed any useful information within `SHOW
ENGINE InnoDB STATUS`, making debugging other deadlocks unnecessarily hard.

The new updating logic uses a single `log` array, indexed by user and field.
`null` values indicate a reset, non-`null` values are an update.

When shutting the user storage down the database will be updated with one
transaction per user, ensuring a consistent view for that user. In most
requests it is expected that only a single user (the current one) is being
updated anyway. Before actually performing the updates the row in wcf1_user for
that user is locked using a SELECT FOR UPDATE query. This should cause
concurrent requests to wait at the `SELECT`, which hopefully stops them from
obtaining any locks that the later DELETE / INSERT will need.

Just locking the `wcf1_user_storage` will not work as the lock cannot be
grabbed atomically. Instead MySQL locks all the matching rows one-by-one.
Locking the user is not ideal, but the update of the user storage should happen
pretty quickly which makes it acceptable.

Select correct identifier in FloodControl::countGuestContent()

see https://github.com/WoltLab/WCF/pull/3633/files#r511879902

Remove deletion of options that were removed in 5.3

Merge branch '5.3'

Merge branch '5.2' into 5.3

Merge pull request #3644 from Krymonota/patch-14

Add like support check to comment responses

Add like support check to comment responses

If the provided comment manager doesn't support likes, it's not possible to like comments, as can be seen here:

- https://github.com/WoltLab/WCF/blob/6747fe291c584d7c8295912f8d94297588d142b2/com.woltlab.wcf/templates/commentList.tpl#L59
- https://github.com/WoltLab/WCF/blob/6747fe291c584d7c8295912f8d94297588d142b2/com.woltlab.wcf/templates/commentList.tpl#L59

However, it's never checked for comment responses whether the comment manager supports likes. This results in the like button being displayed for comment responses, even though likes aren't supported.

Merge pull request #3633 from WoltLab/flood_control

Add flood control API

Unify flood control method descriptions

Remove foreign key in flood control database table

Release 5.3.0 RC 3

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

Add update script to remove `wcf1_page_content.metaKeywords`

Use PHPDoc to mark template variable as deprecated

Co-authored-by: Tim Düsterhus <duesterhus@woltlab.com>

Unify phrasing of FloodControl method documentation

Improve query performance in FloodControl::getLastTimeByIdentifier()

Update flood control method description

Use WCF_UUID and class name if flood control identifier prefix

Use half-open interval when counting flood control entries

Make FloodControl a final class

Change FloodControl::PRUNE_TIME visability to private

Co-authored-by: Tim Düsterhus <duesterhus@woltlab.com>

Cleanup old flood control entries in daily cronjob

Add flood control methods to get last content creation time

Add flood control methods to count contents in certain period of time

Add flood control methods to register new content

Add flood control database table

Add flood control content object type

Remove CMS page meta keywords

Remove meta keywords from page importer

Remove metaKeywords from page PIP

Remove meta keywords option

Merge pull request #3573 from WoltLab/session-meta

Refactor Session Handling: Integration branch

Merge branch '5.3'

Merge branch '5.2' into 5.3

Merge pull request #3612 from WoltLab/csrf-token-template-backport

Backport CsrfTokenFunctionTemplatePlugin

Merge pull request #3626 from WoltLab/php7.0-compatibility

Remove PHP 7.0 compatibility code for Reflection

Re-add removed template variables relating to legacy auto login in LoginForm

Fix formatting in session refactoring (round 2)

Fix formatting in session refactoring

Co-authored-by: Matthias Schmidt <gravatronics@live.com>

Merge pull request #3630 from WoltLab/database-table-util

Remove DatabaseTableUtil

Merge pull request #3624 from WoltLab/deprecate_fileutil_downloadfilefromhttp

Use Guzzle instead of FileUtil::downloadFileFromHttp

Use Guzzle to download spider list

Use Guzzle to download package archives

Use Guzzle to test the Gravatar URL

Merge branch '5.3'

Merge branch '5.2' into 5.3

Whitelist `get_class` in enterprise mode

Merge pull request #3631 from WoltLab/header-cleanup

Clean up response headers

Remove `Expires` response header

No RFC compliant user-agent will ever take this header into account:

> If a response includes a Cache-Control field with the max-age
> directive (Section 5.2.2.8), a recipient MUST ignore the Expires
> field.

Remove `Pragma: no-cache` response header

`Pragma: no-cache` is HTTP/1.0 specific. Anything accessing WoltLab Suite
should be able to at least speak HTTP/1.1. HTTP/1.0 does not even define the
`Host` header for hosting multiple sites on a single IP address!

Remove `X-UA-Compatible` response header

Without support for Internet Explorer we no longer need this header.

Remove DatabaseTableUtil

The only method in there was required for a 5.2.x bugfix update. It should be
possible to safely remove this for 5.4.

Merge pull request #3625 from WoltLab/guzzle-sink

Remove the explicit passing of a Guzzle sink

Prevent potential `null` argument to bin2hex in SessionHandler

Merge branch 'master' into session-meta

For the cleaned package.xml

Remove update files for 5.2 -> 5.3

Consistently use fully qualified function names

Remove PHP 7.0 compatibility code for Reflection

see #3617
see 26ead9f58b1554956e8fb46133e20a5f3fdf5d25

Remove the explicit passing of a Guzzle sink

Merge branch '5.3'

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Ignore top-level node_modules/

Updating minified JavaScript files

Merge branch 'master' into session-meta

For the updated CodeSniffer.

Limit branches for templates workflow

Merge branch '5.3' into master

Merge branch '5.2' into 5.3

Limit workflows to release branches

Features branches will be tested using a PR.

Merge pull request #3622 from WoltLab/workflow-style

Add codestyle workflow

Add codestyle workflow

Add .phpcs.xml

Merge pull request #3617 from WoltLab/formbuilder-php8

Fix PHP 8 support in form builder

Use stock OpeningFunctionBraceKernighanRitchie sniff

Update CodeSniffer ruleset

Merge branch '5.3' into master

Merge branch '5.2' into 5.3

Tighten up the code style a bit

Remove `final` from `UserEditor::getPasswordHash()`

In PHP 8:

> Private methods cannot be final as they are never overridden by other classes