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.

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.

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()

Update npm dependencies

Release 5.4.10

Release 5.4.10 dev 1

Update prettier and typescript

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.

Removed the upgrade instructions

Release 5.4.8

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.

Release 5.4.8 dev 1

Updating minified JavaScript files

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.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.3' into 5.4

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Updating minified JavaScript files

Release 5.4.3

Misleading remark about the comma to separate recipients

Release 5.4.0

Bad XML

Release 5.4.0 RC 1

Add modified PIPs to 5.3 -> 5.4 instructions

Revert "Removed the upgrade support from 5.3.*"

This reverts commit 0fe90b164697dbb4fc74a86b510a6c3de42406c2.

Update XSD paths in XML files

See WoltLab/WCF#4268

Release 5.4.0 Beta 2

Merge branch '5.3' into 5.4

Release 5.3.8

Updating minified JavaScript files

Force `LF` line endings for generated JavaScript (WoltLab/WCF#4285)

The `newLine` configuration option in tsconfig is platform specific. Explicitly set it to ensure consistent behavior.

[Tim: Rephrased the commit message]
[Tim: Adapted to the differing context in non-WoltLab/WCF repositories]

Add workflow for SCSS prettier (#154)

Add 5.4 branch to GitHub workflows

Release 5.4.0 Beta 1

Incorrect use of the `<void/>` instruction

Release 5.4.0 Beta 1

Removed the upgrade support from 5.3.*

Update npm dependencies

Release 5.4.0 Alpha 3

Update npm dependencies

Fix code style

Merge branch '5.3'

Delete conversations without messages in `ConversationRebuildDataWorker` (#153)

Close #152

Preserve names of method arguments during inheritance

See WoltLab/WCF#4211

Use/Add explicit `return null;` statement

Release 5.4.0 Alpha 2

Update npm dependencies

Fix typo in method documentation

Add `@method` comment for `DatabaseObjectList::getSingleObject()` in subclasses

Remove duplicate user ids before using local variable in `ConversationPage`

Release 5.4.0 Alpha 1

Pin `php-cs-fixer` version in codestyle workflow

Use null coalescing operator wherever possible

Update update instructions

Remove obsolete delete instruction for template listeners

Updating minified JavaScript files

Add explicit `return null;` statements

Merge branch '5.3'

Merge branch '5.2' into 5.3

Merge branch '3.1' into 5.2

Fix title of data section when editing conversation label

Close #151

Remove unused import

Improve IDE recognition of variable types

Correctly take user ignore types into account

see WoltLab/WCF#3981
see WoltLab/WCF#4064

Add language PIP to update instructions

Use `UserUtil::getIpAddress()` instead of deprecated `SessionHandler::$ipAddress`

See WoltLab/WCF#3593

Set-up php-cs-fixer using setup-php in codestyle workflow

This allows us to easily install extensions.

Enable new RSS feed dialog

See WoltLab/WCF#4123

Use new `lineBreakSeparatedText` option type

See WoltLab/WCF#4126
See WoltLab/WCF#4128

Updating minified JavaScript files

Merge branch '5.3'

Release 5.3.6

Use consistent title for flood control delay time

See WoltLab/WCF#3801

Consistently use "entered" in English phrases instead of "given"

See WoltLab/WCF#3801

Merge branch '5.3'

Redirect to GET request after using the participant filter

Update `Ui/Object/Action` implementation

See WoltLab/WCF@afc2ab81100a477e2fa1d92783837f6e064a8af6

Recompile TypeScript code

Use `Ui/Object/Action` instead of `WCF.Action.*`

See WoltLab/WCF#4080

Merge branch '5.3'

Do not check for `fixup!` commit in draft PRs

Merge pull request #150 from WoltLab/conversation-flood

Add flood control for conversation messages