Release 5.4.33

Merge pull request #189 from WoltLab/conversation-modification-log-draft

Fix opening draft conversations

Fix opening draft conversations

This got broken in 40edd0891d0003c5441dfd2c067eeb5632fbc2ed, because drafts do
not have any participants, thus resulting in an empty list of visible
participants, breaking the database query for the modification log.

Fix this by adding the conversation author to the array if it's a draft
conversation. They are able to see it by definition.

Release 5.4.32

Merge pull request #184 from WoltLab/conversation-modification-log-invisible

Properly suppress modification log entries for invisible participants

Suppress the `removeParticipant` modification log message for invisible participants

These cannot efficiently be filtered after-the-fact, thus we suppress them
entirely.

For the same reason this fix will not affect existing conversations with the
issue.

Fix filtering the modification log from invisible participants

There is a previous commit relating to this issue in
28484add293eac5d0299d991ffaaa9f478c418e1, but it either never worked or no
longer works. Possibly this got broken when the conversations were refactored
to allow for removal of participants, while still allowing them to see the
contents up to their removal.

The ConversationParticipantList implicitly filters out invisible participants
internally, thus they don't will never be added to the invisibleParticipantIDs.

Fix this issue by inverting the logic: Instead of filtering out all invisible
participants, the query will now only include visible participants that are
guaranteed to be visible.

This fix will affect existing conversations.

see https://www.woltlab.com/community/thread/301779-konversation-mit-bcc-leakt-teilnehmer/

Release 5.4.21

Add event after quick reply is done

Update to setup-node@v3

Update npm dependencies

Remove the codestyle workflow from branches that are not master

Release 5.4.15

Fix ts dir

Delete old JS dir, before recompile TS

Release 5.4.15 dev 1

Updating minified JavaScript files

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

Update npm dependencies

Merge branch '5.3' into 5.4

Release 5.3.20

Release 5.4.14

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.

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