致命的 #982
closeddisplay_requirementを入れていない状態でメッセージの投稿時間を押すと落ちる
Description
display_requirementを入れていない状態でメッセージの投稿時間を押すと落ちます。
messageモデルにidの定義を要求されているのが原因みたいです。
見つけた際のやりとりを一応載せます。
みつけた
バックトレース
/Users/ahiru/mikutter/source/core/plugin/display_requirements/display_requirements.rb:182:in `clicked': undefined method `id' for #<Plugin::Slack::Message:0x007fbd206d24c0>
from /Users/ahiru/mikutter/source/core/mui/cairo_cell_renderer_message.rb:160:in `block in event_hooks'
from /Users/ahiru/mikutter/source/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
from /Users/ahiru/mikutter/source/core/mui/cairo_cell_renderer_message.rb:97:in `signal_emit'
from /Users/ahiru/mikutter/source/core/mui/cairo_cell_renderer_message.rb:97:in `block in tree='
from /Users/ahiru/mikutter/source/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
from /Users/ahiru/mikutter/source/core/plugin/gtk/mainloop.rb:10:in `main'
from /Users/ahiru/mikutter/source/core/plugin/gtk/mainloop.rb:10:in `mainloop'
from /Users/ahiru/mikutter/source/mikutter.rb:66:in `boot!'
from /Users/ahiru/mikutter/source/mikutter.rb:95:in `<main>'
Updated by toshi_a 初音 about 8 years ago
- Tracker changed from バグ to 致命的
- Assignee set to toshi_a 初音
- バックトレース updated (diff)
Updated by toshi_a 初音 about 8 years ago
- Status changed from 新規 to レビュー待ち
- Assignee changed from toshi_a 初音 to あひる 家鴨
- プラグイン名 set to display_requirements
修正しました。Message Modelに対してIntentを発生させるので、次のような挙動を想定しています。
- TwitterのMessage … ツイート詳細画面が開く
- システムメッセージ … 何も起こらない(noticeレベルのメッセージがコンソールに出力される)
- その他 … 入ってるインテントプラグインによる
Updated by あひる 家鴨 about 8 years ago
- Status changed from レビュー待ち to 解決
hotfixの最新(7d84a887)で確認しました。
Twitterではツイート詳細が開き、システムメッセージはnoticeレベルのメッセージが出力され、slackプラグインで確認したところ正しくwebに飛ばされました。
ありがとうございます。