Skip to content
This repository has been archived by the owner on Jul 13, 2024. It is now read-only.

refactor: メッセージ処理のリファクタリングなど #166

Merged
merged 1 commit into from
May 22, 2022

Conversation

book000
Copy link
Member

@book000 book000 commented May 21, 2022

なんかちょこちょこいじったりしてたら色々混ざったプルリクになっちゃいました。申し訳ないです。

  • 「メッセージ」と一言に言っても「ユーザから送られたメッセージ」・「Botから送られたメッセージ」・「ユーザが送ったシステムメッセージ」など様々なメッセージ形態があります。
    • このようなさまざまな形態のメッセージに対応するため、「メッセージプロセッサ (MessageProcessor)」パッケージを生やしました。
    • 現時点で、「ユーザーが送信したデフォルトメッセージ」・「スタンプ (Sticker) メッセージ」・「添付ファイル (メッセージ)」・「スレッド開始通知メッセージ (メッセージ派生かどうかを問わない)」に対応しています。
  • TrackInfo.SpeakFromType に RECEIVED_STICKER, CREATED_THREAD を追加
  • 画像内のテキストがあまりにも多い場合に2000文字を超えてしまいメッセージが送信できない不具合を修正。本文は1500文字で切り上げるように
  • ignore 系の変数を LibValue から LibIgnore に移動
  • IntelliJのインスペクション適用
    • Lib/EmojiWrapper にて、 EmojiManager::getByUnicode が nullable なのにも関わらず適切な Null フィルタリングが行われていない問題の修正
    • 正規表現 [0-9]\d に置き換え
  • 依存パッケージアップデート
    • JDA: 5.0.0-alpha.115.0.0-alpha.12
    • org.json.json: 2021030720220320

close #163
ref #109

@book000 book000 requested a review from yuuahp May 21, 2022 21:37
@book000 book000 marked this pull request as ready for review May 21, 2022 21:54
@book000 book000 added the 💡強化 新機能のリクエスト label May 21, 2022
@yuuahp yuuahp merged commit 94eff1a into jaoafa:main May 22, 2022
@book000 book000 deleted the refactor/rework-message-processor branch May 22, 2022 00:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💡強化 新機能のリクエスト
Projects
None yet
Development

Successfully merging this pull request may close these issues.

スレッド作成通知
2 participants