Merge branch '5.4'

Release 5.4.15 dev 1

Updating minified JavaScript files

Merge branch '5.4'

Merge branch '5.3' into 5.4

Merge branch '5.2' into 5.3

Upgrade to `actions/checkout@v3`

Merge branch '5.3' into 5.4

Merge branch '5.2' into 5.3

Remove codestyle workflow for non-PSR-12 branches

see WoltLab/WCF@2163aa902ace5c899fe7ae42220f91e5a3d71865

Recompile JavaScript

Merge branch '5.4'

Update npm dependencies

Updating minified JavaScript files

Marking all items as read did not reset the count

Merge branch '5.4'

Merge branch '5.3' into 5.4

Release 5.3.20

Release 5.4.14

Merge branch '5.4'

Release 5.4.14 dev 1

Merge pull request #171 from WoltLab/conversation-order-removed

Fix conversation order when being removed from conversation

Make CASE statements more readable in queries in UserConversationList

Fix conversation order when being removed from conversation

When one is removed from a conversation, UserConversationList will use the time
of removal as the conversation's `lastPostTime` when ordering, instead of the
actual time of the newest message to not leak information about new messages
after removal.

However this was not properly accounted for within `readObjectIDs()`. Now if a
conversation one was removed from receives new messages after a long-ish time -
with several conversations in between - the conversation will appear on the
*pageNo* where it would be if one would still be a participant. But within the
page it would be ordered based on the timestamp of removal (i.e. appear at the
very bottom of the page).

Fix this by also overriding `lastPostTime` in `readObjectIDs()`. This will move
the conversation to the proper `pageNo` based on the removal time.

This might still be slightly misleading, because the “last message” column in
the template shows the time of the actual last visible message, but this
confusion can be resolved by opening the conversation and seeing the removal
timestamp matching up, whereas without this fix the conversation can be all
over the place.

Use `->prepare()` instead if `->prepareStatement()` in ConversationEditor::removeParticipant()

Merge branch '5.4'

Merge branch '5.3' into 5.4

Merge pull request #170 from WoltLab/remove-participant-last-message-id

Fix calculation of `lastMessageID` when removing other conversation participants

Fix calculation of `lastMessageID` when removing other conversation participants

Without this `ORDER BY` usually the first visible message would've been set as
the `lastMessageID`, as that's the first row within the `conversation_message`
table.

Improved the styling of the user menu

Incorrect calculation of the "other participant" avatar

See #169

Fix update_com.woltlab.wcf.conversation_5.5_cleanup_orphaned_attachments.php again

see 3880927fdc5fc427e4b485639d1813c4cb3bee52

Fix update_com.woltlab.wcf.conversation_5.5_cleanup_orphaned_attachments.php

Updating minified JavaScript files

Merge pull request #168 from WoltLab/55-user-menu

Implementation of the new user menu for conversations

Validate the module and permissions, improved code clarity

Mark files/js/WoltLabSuite/**/*.js as linguist-generated

Implementation of the new user menu for conversations

See #167

Update commit for WoltLab/WCF in package.json

This is to get the types for perfect-scrollbar.

Update commit for WoltLab/WCF in package.json

Merge branch '5.4'

Merge branch 'can-reply-can-read' into 5.4

Remove obsolete check for `->canRead()` in ConversationMessageAction::validateContainer()

Verify that the conversation is readable in Conversation::canReply()

Merge pull request #166 from WoltLab/attachment-orphan

Clean up orphaned attachments when upgrading to 5.5

Clean up orphaned attachments when upgrading to 5.5

Resolves #158

Merge pull request #165 from WoltLab/new-search-form

Search form overhaul

Fixed extended link

Fixed code style

Search form overhaul

see https://github.com/WoltLab/WCF/pull/4605

Unified buttons

Merge branch '5.4'

Update npm dependencies

Merge branch '5.4'

Release 5.4.10

Release 5.4.10 dev 1

Merge branch '5.4'

Update prettier and typescript

Merge branch '5.4'

Drop obsolete update_com.woltlab.wcf.conversation_5.4_deleteLanguageItems.php

Update fileDelete.xml

Generated with:

    #!/bin/bash

    cat <<'EOF'
    <?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>
    EOF

    comm -23 <(git log 2bf40f755228eeae2902562471acf14177777eb6..HEAD --diff-filter=D --no-renames --summary |awk '/delete mode/{print $NF}' |grep files/ |sort -u) <(find files/ |sort -u) \
     |sed 's~^files/~~' \
     |sed 's~^~\t\t<file>~;s~$~</file>~'

    cat <<'EOF'
     </delete>
    </data>
    EOF

Add update instructions from 5.4.*

Count drafts for flood control during actual submission

Otherwise a spammer might be able to silently prepare a large number of drafts,
submitting them in bulk when they prepared enough.

Merge pull request #164 from WoltLab/close-phrase

Clarify that "closing" a conversation prevents new replies

Clarify that "closing" a conversation prevents new replies

Remove obsolete import

see 8fb48c34b510262be8c5e1fffc445d50d72463a2

Merge pull request #162 from WoltLab/jumpToExtended

Remove broken ConversationMessageAction::jumpToExtended()

Remove broken ConversationMessageAction::jumpToExtended()

The target controllers for the forwarding no longer exist since
a3258a9f150a8febeef0f2c7d1899799d953e1cc.

see WoltLab/WCF#4575

Stop checking the `enable_censorship` option

see WoltLab/WCF#4567

Merge pull request #161 from WoltLab/search-index-worker

Add ConversationMessageSearchIndexRebuildDataWorker

Add ConversationMessageSearchIndexRebuildDataWorker

This also fixes a bug within the indexing process: Previously each message
within a conversation would have been assigned the conversation's subject as
the subject, while the regular creation process does not.

Fix this by only setting the title for the first message within the
conversation.

Resolves #160

Update for PHP CS Fixer 3.2.1

Link the conversation's time to the conversation itself

Resolves #159

Merge pull request #156 from WoltLab/markAsRead-notification

Refactor logic in ConversationAction::markAsRead()

Ensure explicit database INDEX names

Checking all revisions of all `.sql` files returned that a INDEX was never
added to an existing table using `ALTER TABLE`. Thus this update script should
not result in changes in *any* installation out there, making this process
quite fast even in large communities. It is added and should be execute
nonetheless for full correctness.

The following fish script was used to check the revisions:

    for file in (g log 2bf40f755228eeae2902562471acf14177777eb6..HEAD --diff-filter=D --no-renames --summary |awk '/delete mode/{print $NF}' |grep '.sql$')
     for rev in (git rev-list --all -- $file)
     git show $rev:$file
     end
    end 2>&1 |grep -v '^fatal:' |grep '^ALTER' |sort -u

see WoltLab/WCF#4505

Replace install.sql by database PIP

The PHP definition already includes explicit INDEX names, because the two
indices in wcf1_conversation_to_user would conflict otherwise.

An update script will be added in a follow-up commit.

see WoltLab/WCF#4505

Merge branch '5.4'

Removed the upgrade instructions

Release 5.4.8

Merge branch '5.4'

Merge branch '5.4' of https://github.com/WoltLab/com.woltlab.wcf.conversation into 5.4

Release 5.4.8 dev 2

Run php-cs-fixer using PHP 8.0

Merge pull request #157 from WoltLab/attachment-orphan

Prevent method calls on `null` in ConversationMessageAttachmentObjectType::canDownload()

Prevent method calls on `null` in ConversationMessageAttachmentObjectType::canDownload()

This may happen for attachments where the corresponding conversation /
conversation message no longer exists.

Refactor logic in ConversationAction::markAsRead()

see WoltLab/com.woltlab.wbb#455

Release 5.4.8 dev 1

Updating minified JavaScript files

Merge branch '5.4'

Disable npm cache in javascript workflow

Update npm dependencies

Skip the JS for invisible participant if the user lacks the permissions

Incorrect check to display the signatur

This check was effectively redundant since 3.0, but worked until a legacy trait was removed in 5.4. This was overlooked and caused signatures to be hidden in 5.4.

Fixes 381191f4f1e26cdfff406e4ac6240c063d816068

Merge branch '5.4'

Merge branch '5.3' into 5.4

Merge branch '5.2' into 5.3

Release 5.2.15

Merge branch '3.1' into 5.2

Release 3.1.23

Merge branch '5.4'