Skip to content

Refactor ReactionHandler#6628

Merged
BurntimeX merged 20 commits into6.3from
63-refactor-reaction-handler
Mar 14, 2026
Merged

Refactor ReactionHandler#6628
BurntimeX merged 20 commits into6.3from
63-refactor-reaction-handler

Conversation

@BurntimeX
Copy link
Member

No description provided.

* Make `ReactionHandler` final
* Add typings / return types
* Remove obsolete phpdoc
This commit also improves the flow of actions to avoid race conditions.

In addition, `cachedReactions` is now stored as JSON in the database.
`getReactions()` returns an array keyed by `reactionTypeID`, so `count()` gives the number of distinct reaction types, not the total number of reactions. For example, if an object has 5 "thumbs up" + 3 "heart" reactions (8 total), this would subtract 2 instead of 8.
@BurntimeX BurntimeX merged commit 91a87bc into 6.3 Mar 14, 2026
7 of 8 checks passed
@BurntimeX BurntimeX deleted the 63-refactor-reaction-handler branch March 14, 2026 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants