Add update instructions for index updates of `wcf1_tag_to_object`

Make wcf1_tag_to_object's PRIMARY KEY a UNIQUE KEY

see #3803

Remove redundant column from wcf1_tag_to_object.tagID key

This key was identical to the `(objectTypeID, tagID)` key. We don't need the
objectTypeID here.

Remove the languageID column from keys in wcf1_tag_to_object

This column is functionally dependent on tagID. Since the previous commits this
column is no longer used and only filled for backwards compatibility.

See #3803

Stop accessing wcf1_tag_to_object.languageID in TagEngine::getObjectsTags()

This stops accessing the redundant `languageID` column that is functionally
dependent on the tagID (see #3803).

This change will make the query a little bit slower, but this will be
remediated by adjusting the indices on the wcf1_tag_to_object table after which
the performance will be identical:

    MariaDB [*snip*]> EXPLAIN
        -> SELECT tag.*,
        ->        tag_to_object.objectid
        -> FROM   wcf1_tag_to_object tag_to_object
        ->        LEFT JOIN wcf1_tag tag
        ->               ON ( tag.tagid = tag_to_object.tagid )
        -> WHERE  tag_to_object.objecttypeid = 92
        ->        AND tag_to_object.objectid IN ( 3553, 7990 )
        ->        AND tag_to_object.languageid IN ( 1 );
    +------+-------------+---------------+--------+----------------------------------------------------------------------+-------------------------------------+---------+-------------------------------+------+--------------------------+
    | id   | select_type | table         | type   | possible_keys                                                        | key                                 | key_len | ref                           | rows | Extra                    |
    +------+-------------+---------------+--------+----------------------------------------------------------------------+-------------------------------------+---------+-------------------------------+------+--------------------------+
    |    1 | SIMPLE      | tag_to_object | range  | objectTypeID,objectTypeID_2,cbbba36334575d806c002a8756c8a107_fk,test | cbbba36334575d806c002a8756c8a107_fk | 12      | NULL                          |    8 | Using where; Using index |
    |    1 | SIMPLE      | tag           | eq_ref | PRIMARY                                                              | PRIMARY                             | 4       | *snip*.tag_to_object.tagID    |    1 |                          |
    +------+-------------+---------------+--------+----------------------------------------------------------------------+-------------------------------------+---------+-------------------------------+------+--------------------------+
    2 rows in set (0.00 sec)

    MariaDB [*snip*]>
    MariaDB [*snip*]> EXPLAIN
        -> SELECT tag.*,
        ->        tag_to_object.objectid
        -> FROM   wcf1_tag_to_object tag_to_object
        ->        LEFT JOIN wcf1_tag tag
        ->               ON ( tag.tagid = tag_to_object.tagid )
        -> WHERE  tag_to_object.objecttypeid = 92
        ->        AND tag_to_object.objectid IN ( 3553, 7990 )
        ->        AND tag.languageid IN ( 1 );
    +------+-------------+---------------+--------+----------------------------------------+---------+---------+-------------------------------+------+--------------------------+
    | id   | select_type | table         | type   | possible_keys                          | key     | key_len | ref                           | rows | Extra                    |
    +------+-------------+---------------+--------+----------------------------------------+---------+---------+-------------------------------+------+--------------------------+
    |    1 | SIMPLE      | tag_to_object | range  | objectTypeID,objectTypeID_2,tagID,test | test    | 8       | NULL                          |    8 | Using where; Using index |
    |    1 | SIMPLE      | tag           | eq_ref | PRIMARY,languageID                     | PRIMARY | 4       | *snip*.tag_to_object.tagID    |    1 | Using where              |
    +------+-------------+---------------+--------+----------------------------------------+---------+---------+-------------------------------+------+--------------------------+
    2 rows in set (0.00 sec)

Stop accessing wcf1_tag_to_object.languageID during DELETE in TagEngine

This stops accessing the redundant `languageID` column that is functionally
dependent on the tagID (see #3803).

Stop accessing wcf1_tag_to_object.languageID in TagCloudCacheBuilder::getTags()

This change comes with one primary benefit:

This stops accessing the redundant `languageID` column that is functionally
dependent on the tagID (see #3803).

On MariaDB 10.1 at a first glance this results in a *much* better query plan
(note the lower row count for the first query):

    MariaDB [*snip*]> EXPLAIN
        -> SELECT tag.tagid,
        ->        Count(object.objectid) AS counter
        -> FROM   wcf1_tag_to_object object
        ->        INNER JOIN wcf1_tag tag
        ->                ON tag.tagid = object.tagid
        -> WHERE  object.objecttypeid IN ( 92 )
        ->        AND tag.languageid IN ( 1 )
        -> GROUP  BY tag.tagid
        -> ORDER  BY counter DESC,
        ->           tag.tagid DESC
        -> LIMIT  500;
    +------+-------------+--------+------+-----------------------------------+------------+---------+---------------------------+------+-----------------------------------------------------------+
    | id   | select_type | table  | type | possible_keys                     | key        | key_len | ref                       | rows | Extra                                                     |
    +------+-------------+--------+------+-----------------------------------+------------+---------+---------------------------+------+-----------------------------------------------------------+
    |    1 | SIMPLE      | tag    | ref  | PRIMARY,languageID                | languageID | 4       | const                     | 5299 | Using where; Using index; Using temporary; Using filesort |
    |    1 | SIMPLE      | object | ref  | objectTypeID,objectTypeID_2,tagID | tagID      | 8       | *snip*.tag.tagID,const    |    3 | Using index                                               |
    +------+-------------+--------+------+-----------------------------------+------------+---------+---------------------------+------+-----------------------------------------------------------+
    2 rows in set (0.00 sec)

    MariaDB [*snip*]>
    MariaDB [*snip*]> EXPLAIN
        -> SELECT object.tagid,
        ->        Count(*) AS counter
        -> FROM   wcf1_tag_to_object object
        -> WHERE  object.objecttypeid IN ( 92 )
        ->        AND object.languageid IN ( 1 )
        -> GROUP  BY object.tagid
        -> ORDER  BY counter DESC,
        ->           object.tagid DESC
        -> LIMIT  500;
    +------+-------------+--------+------+-----------------------------------------------------------------+--------------+---------+-------------+-------+-----------------------------------------------------------+
    | id   | select_type | table  | type | possible_keys                                                   | key          | key_len | ref         | rows  | Extra                                                     |
    +------+-------------+--------+------+-----------------------------------------------------------------+--------------+---------+-------------+-------+-----------------------------------------------------------+
    |    1 | SIMPLE      | object | ref  | objectTypeID,objectTypeID_2,cbbba36334575d806c002a8756c8a107_fk | objectTypeID | 8       | const,const | 56293 | Using where; Using index; Using temporary; Using filesort |
    +------+-------------+--------+------+-----------------------------------------------------------------+--------------+---------+-------------+-------+-----------------------------------------------------------+
    1 row in set (0.00 sec)

When running the query this unfortunately is a bit slower 0.05 (new) vs 0.03
(old) seconds. This can also be confirmed with a (larger) MySQL 8 installation
and using EXPLAIN ANALYZE:

    > EXPLAIN ANALYZE
        -> SELECT tag.tagid,
        ->        Count(object.objectid) AS counter
        -> FROM   wcf1_tag_to_object object
        ->        INNER JOIN wcf1_tag tag
        ->                ON tag.tagid = object.tagid
        -> WHERE  object.objecttypeid IN ( 203, 337 )
        ->        AND tag.languageid IN ( 1 )
        -> GROUP  BY tag.tagid
        -> ORDER  BY counter DESC
        -> LIMIT  500 \G
    *************************** 1. row ***************************
    EXPLAIN: -> Limit: 500 row(s)  (actual time=166.506..166.540 rows=500 loops=1)
        -> Sort: counter DESC, limit input to 500 row(s) per chunk  (actual time=166.505..166.520 rows=500 loops=1)
            -> Table scan on <temporary>  (actual time=0.001..0.509 rows=16716 loops=1)
                -> Aggregate using temporary table  (actual time=163.863..165.054 rows=16716 loops=1)
                    -> Nested loop inner join  (cost=14797.71 rows=87401) (actual time=0.056..112.533 rows=201296 loops=1)
                        -> Index lookup on tag using languageID (languageID=1)  (cost=1132.88 rows=9229) (actual time=0.037..3.809 rows=18651 loops=1)
                        -> Filter: (object.objectTypeID in (203,337))  (cost=0.27 rows=9) (actual time=0.002..0.005 rows=11 loops=18651)
                            -> Index lookup on object using tagID (tagID=tag.tagID)  (cost=0.27 rows=12) (actual time=0.002..0.004 rows=11 loops=18651)

    1 row in set (0.16 sec)

    > EXPLAIN ANALYZE
        -> SELECT object.tagid,
        ->        Count(*) AS counter
        -> FROM   wcf1_tag_to_object object
        -> WHERE  object.objecttypeid IN ( 203, 337 )
        ->        AND object.languageid IN ( 1 )
        -> GROUP  BY object.tagid
        -> ORDER  BY counter DESC
        -> LIMIT  500 \G
    *************************** 1. row ***************************
    EXPLAIN: -> Limit: 500 row(s)  (actual time=135.420..135.461 rows=500 loops=1)
        -> Sort: counter DESC, limit input to 500 row(s) per chunk  (actual time=135.419..135.437 rows=500 loops=1)
            -> Table scan on <temporary>  (actual time=0.000..0.560 rows=16716 loops=1)
                -> Aggregate using temporary table  (actual time=132.599..133.936 rows=16716 loops=1)
                    -> Filter: ((object.languageID = 1) and (object.objectTypeID in (203,337)))  (cost=31642.06 rows=157965) (actual time=0.080..72.889 rows=201296 loops=1)
                        -> Index range scan on object using objectTypeID  (cost=31642.06 rows=157965) (actual time=0.076..53.904 rows=201296 loops=1)

    1 row in set (0.14 sec)

Nonetheless this appears to be worth it. Especially if we can remove the
`languageID` column in the future.

I also attempted to get rid of the second query, by simply putting a `tag.*`
into the column list of the first query. Unfortunately MariaDB 10.1 (which is
our minimum requirement) is too dumb to determine that all the columns in the
`tag` table are functionally dependent on `tag.tagID`. MySQL 8 is able to handle
that correctly.

I have verified that both queries result in the same results (except for the
undefined ordering when the counter is identical).

Merge branch '5.3'

Merge branch '5.2' into 5.3

Fix which index object is used when dropping indices with PHP API

Only `$matchingExistingIndex` is guaranteed to have the correct index name.

Merge branch '5.3'

Merge branch '5.2' into 5.3

Fix checked property when adding indices to `DatabaseTable`

Merge branch '5.2' into 5.3

Release 5.2.11

Merge branch '3.1' into 5.2

Release 3.1.19

Updating minified JavaScript files

Merge pull request #3911 from WoltLab/master-password

Deprecate Master Password

Merge branch '5.3'

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Fix converting float value to integer

Add reformatting of constants.php to .git-blame-ignore-revs

Add backslash before define() in constants.php

Incorrect comparison for the height of an image

Missing update of the version number in `wcf\system\WCF`

Use stronger wording in master password deprecation notice

Merge pull request #3912 from WoltLab/blocklist-deprecation

Deprecate the client blocklists

Bump version to 5.4.0 Alpha 1

The current development state diverged quite a lot from 5.3. Adjust the version
to prevent accidents with the developer tools.

Fix .gitattributes for Template.grammar.js

The `ts/` folder now resides in the root of the repository.

Add ILoggingAwareException

Add types for functions in core.functions.php

These cannot be inherited from, thus we can add the types without breaking
compatibility.

Merge pull request #3915 from WoltLab/composer

Update composer dependencies

Ignore symfony/polyfill-mbstring/bootstrap80.php during syntax check

Update composer dependencies

Remove bogus extra newline in WCF.class.php

Deprecate blacklist_ip_addresses and blacklist_user_agents

Resolves #3909

Remove the `blacklist_hostnames` option

The hostname blocklist requires a PTR lookup for every single request. This is
slow and unreliable.

see #3909

Show deprecation message on master password authentication

Resolves #3698

Mark the master password as deprecated in the option description

see #3698

Fix formatting in LogoutAction

Apparently the editor on GitHub.com defaults to tabs, even if the whole file consists of spaces only.

Clear the master password on ACP logout

Release 5.3.3

Add previous commit to .git-blame-ignore-revs

Fix bad merge

see 521f18fb9505d68091945d6ee484277dac89645c
see fa8aae2ad5f9299fdbce5feed23fe03a4fb335d4

Merge branch '5.3'

Object edit link led to the admin panel

Merge branch '5.3'

Make update_com.woltlab.wcf_5.3_packageServer.php compatible with WCF_N != 1

Merge branch '5.3'

Merge branch '5.2' into 5.3

Merge pull request #3908 from WoltLab/pr_build_package

Add wcfsetup workflow

Merge pull request #3901 from WoltLab/require-multifactor

Implement multi-factor requirement

Merge pull request #3906 from WoltLab/item-list-input-event

Use the `input` event to detect the comma on Chromium for Android

Fix informal phrase in de.xml

Add wcfsetup workflow
This workflow generates the WCFSetup and stores it as an artifact.

Merge pull request #3907 from WoltLab/mfa-update-multiple-requests

Split update_com.woltlab.wcf_5.4_migrate_multifactor across multiple requests

Split update_com.woltlab.wcf_5.4_migrate_multifactor across multiple requests

Resolves #3796

Use the `input` event to detect the comma on Chromium for Android

Merge pull request #3905 from WoltLab/benchmark-parameters

Replace placeholders by actual values in Benchmark

Replace placeholders by actual values in Benchmark

In most cases this allows one to simply copy the query to easily edit it within
a MySQL shell. The code (intentionally) does not handle single quotes
correctly. It also truncates the parameter after 100 characters and handles at
most 30 parameters.

Merge branch '5.3'

Improved message for rejected credentials on paid packages (#3903)

Throw NotImplementedException exception for unsupported method call

Add dev tools description for multi-factor object type definition (#3897)

see #3892

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

Change default username in dev installation to 'dev'
Closes #3866

Use FQN for internal PHP functions

Fix codestyle

Update phpBB3 password hash
Closes #3885

Reformat SQL queries in install and update scripts

missing id attribute

Enforce the multi-factor requirement in ACP

Add TMultifactorRequirementEnforcer

Check for multi-factor requirement in MultifactorDisableForm

Add RejectEverythingFormField

Reduce duplication in MultifactorDisableForm

Add User::requiresMultifactor()

Add requireMultifactor property to user groups

Merge remote-tracking branch 'origin/master'

Merge branch '5.3'

Merge branch '5.2' into 5.3

Merge pull request #3900 from WoltLab/hasOwnerAccess

Fix User::hasOwnerAccess()

Check owner access after checking controller blacklist in RequestHandler

This ensures that the check only happens when absolute required.

Remove caching from User::hasOwnerAccess()

The current caching logic is buggy as reported in issue #3899. This patch
removes this caching, as this method already is quite fast and it also is
seldomly called. Within the frontend it is only called when the user is banned.

Fixes #3899

Show content from all content languages for guests
Closes #3713

Remove useless WCFSetup workaround in SessionHandler::needsReauthentication()

Apparently I did not conduct my testing properly yesterday and piled on
non-effective workarounds for the WCFSetup issue. This one is particularly bad,
because I inverted the condition, disabling reauthentication everywhere, except
in WCFSetup.

Thus this patch removes this buggy workaround again.

see ff5d8cec55f0a953a353165b2d996f84a56838f6
see 4b5f3b084ef062b48eaba18b3f497ba89743ddcd

The compression quality was not applied in Imagick

Fix the cookie refresh after WCFSetup

We need to set user_session, as acp_session is gone.

Move the call to registerReauthentication() to WCFSetup

This was forgotten in the previous commit.

Unbreak WCFSetup

Dynamic WebP avatars (#3889)

Replace usage of `setObjectTitles()` with `replaceLinks()`

See #3881

Use `UserProfileRuntimeCache` instead of `UserProfile::getUserProfiles()`

See #3880

New UI design for the list of attachments (#3890)

* New UI design for the list of attachments

* Exchange the icon on focus (a11y)

* Improved a11y for attachments

* Inconsistent indentation

* Consistent use of whitespaces

* Fix indentation in en.xml

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

Replace usage of `LikeHandler` with `ReactionHandler`

… whereever possible.

Stop using `TLegacyUserPropertyAccess`

See #3880

Add dev tools description for flood control object type definition

See #3892

Add button to delete missing phrases logs for phrases existing now (#3896)

Replaces #3716

Merge pull request #3893 from WoltLab/deprecate-gravatar

Deprecate Gravatar support