Actions
バグ #951
closed
詳細タブのツイート時刻をクリックするとmikutterが暴走する.
Description
ツイートを開いたときに表示される 詳細タブ 中の ツイート時刻 をクリックすると, mikutterが暴走します.
関連付けをすべて消した状態でもそうなります.
Files
Updated by 
Updated by
Updated by Akira Ouchi over 7 years ago
私の環境で確認した場合は以下のエラーとともにクラッシュしました。
/Users/akkie/mikutter_dev/core/mui/gtk_retriever_header_widget.rb:71:in `block in message_opener': undefined local variable or method `retriever' for #<Gtk::RetrieverHeaderWidget:0x7ffb1a310b88 ptr=0x7ffb14e06400>
from /Users/akkie/mikutter_dev/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
from /Users/akkie/mikutter_dev/core/plugin/gtk/mainloop.rb:10:in `main'
from /Users/akkie/mikutter_dev/core/plugin/gtk/mainloop.rb:10:in `mainloop'
from mikutter.rb:66:in `boot!'
from mikutter.rb:95:in `<main>'
notice: mikutter.rb:72:in `rescue in boot!': catch exception `SystemExit'
notice: mikutter.rb:74:in `rescue in boot!': => `SystemExit'
notice: mikutter.rb:99:in `rescue in <main>': catch SystemExit
notice: {MIKUTTER_DIR}/core/configloader.rb:32:in `block in <module:ConfigLoader>': configloader: wrote 1 keys (UserConfig::ui_tab_order)
release/3.5ブランチのcommit fd560042e1d698be888363e16fe3380550d844c4です。
Updated by コスモ 虹川 over 7 years ago
- File 0001-display_requirements-Add-guard-to-be-message-id-unde.patch 0001-display_requirements-Add-guard-to-be-message-id-unde.patch added
dd1ac9aff50d0f667387ee0af12d2e26d6657c43 では通常のツイートでは開く方法が提示されるダイアログがでることになっており、
落ちなくなっているようです。
ただし、システムメッセージではまだ落ちるようです。
/media/Data2/mikutter/core/plugin/display_requirements/display_requirements.rb:181:in `clicked': undefined method `id' for #<Mikutter::System::Message:0x0055fa2bfcc530>
from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:160:in `block in event_hooks'
from /media/Data2/mikutter/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `signal_emit'
from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `block in tree='
from /media/Data2/mikutter/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `main'
from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `mainloop'
from mikutter.rb:66:in `boot!'
from mikutter.rb:95:in `<main>'
/media/Data2/mikutter/vendor/bundle/ruby/2.3.0/gems/glib2-3.0.8/lib/glib2.rb:41:in `exit': exit
from /media/Data2/mikutter/vendor/bundle/ruby/2.3.0/gems/glib2-3.0.8/lib/glib2.rb:41:in `exit_application'
from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `signal_emit'
from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `block in tree='
from /media/Data2/mikutter/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `main'
from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `mainloop'
from mikutter.rb:66:in `boot!'
from mikutter.rb:95:in `<main>'
notice: mikutter.rb:72:in `rescue in boot!': catch exception `SystemExit'
notice: mikutter.rb:74:in `rescue in boot!': => `SystemExit'
notice: mikutter.rb:99:in `rescue in <main>': catch SystemExit
notice: {MIKUTTER_DIR}/core/configloader.rb:32:in `block in <module:ConfigLoader>': configloader: wrote 1 keys (UserConfig::ui_tab_order)
修正がアドホックですがこれで一応システムメッセージの場合も落ちなくなります。
Updated by toshi_a 初音 over 7 years ago
パッチ見ましたが、コスモさんが言ってるのはツイートの詳細じゃなくてMiraclePainterですかね。とすると別の問題？あとで検証します
Updated by toshi_a 初音 over 7 years ago
- Status changed from 新規 to レビュー待ち
- Assignee set to Shi rusu
- % Done changed from 0 to 100
報告されたバグも確かにあるんですが、ここの正しい挙動は他のIntentを開く、vanillaのmikutterでの動作でいうと単にブラウザでそのツイートを開くという挙動が正しいので、実装しました。release/3.5ブランチです。
一応そちらでも確認してもらえますか。
日も迫ってるので、 Akira Ouchi も確認してみてもらって、どちらかから報告貰えたらこれでOKとします。
Actions