diff options
| author | okayurisotto <47853651+okayurisotto@users.noreply.github.com> | 2024-02-21 14:31:50 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-02-21 14:31:50 +0900 |
| commit | 750d2626041b355459265a4a4148d08f6ac517fd (patch) | |
| tree | 728df33322fc43aa4358f1a6a764a9bb64f13cac /packages/backend/src/queue | |
| parent | refactor(frontend): 不必要なconsole.logを除去・抑制 (#13400) (diff) | |
| download | sharkey-750d2626041b355459265a4a4148d08f6ac517fd.tar.gz sharkey-750d2626041b355459265a4a4148d08f6ac517fd.tar.bz2 sharkey-750d2626041b355459265a4a4148d08f6ac517fd.zip | |
refactor(backend): `ReactionService.prototype.convertLegacyReactions` (#13375)
* add unit tests
* cleanup unnecessary type assertions
* `convertedReaction`変数の定義と変換表に対する存在確認処理の整理
* `count`変数の定義とループ処理での`Object.entries()`の活用
* 条件式の整理
* `Array.prototype.reduce`を使うように
* `Array.prototype.reduce`を使うように
* 配列操作を1つのメソッドチェーンに整理
これまでの実装では、`decodeReaction`の返り値が同一になる異なる入力値が同時に複数個存在した場合、後ろのもので上書きされてしまっていたはず。
これからの実装では、後ろのものは前のものに加算される。
(実際にこの挙動の変更が問題になるシチュエーションはまずないはず。)
* add unit test
* ドキュメントコメントの追加と型定義の調整
Diffstat (limited to 'packages/backend/src/queue')
0 files changed, 0 insertions, 0 deletions