プロジェクト

全般

プロフィール

バグ #1539

完了

gtk3: Twitter プラグインでの client側設定 muted_user がタイムラインに表示されてしまう?

Izumi Tsutsui さんが約3年前に追加. 約3年前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
gtk3
クラッシュする:
いいえ

説明

プラグイン側の問題か mikutter 本体側の問題か確認していないのですがとりあえず投げます。

Twitter Plugin の user_detail_view の「ユーザーについて」タブで「ミュートする」を選択したユーザーが
mikutter_gtk3 だとホームタイムライン、保存した検索等で表示されるようです。
gtk2 版だと意図通り(?) mute されていたと思います。

mute の処理がどこで行われているかがわかればそこから確認してみます。


ファイル

Izumi Tsutsui さんが約3年前に更新

mute の処理がどこで行われているかがわかればそこから確認してみます。

結論からいうと gtk3 plugin の timeline 表示で
Plugin.filtering(:show_filter, ) の処理が漏れているのが問題のようです。
0001-gtk3-timeline-show_filter.patch

--- a/plugin/gtk3/widget/timeline.rb
+++ b/plugin/gtk3/widget/timeline.rb
@@ -96,7 +96,8 @@ module Plugin::Gtk3

     def bulk_add(models)
       update_ordinal = false
-      models.each do |message_or_share|
+      filtered_models = Plugin.filtering(:show_filter, models).first
+      filtered_models.each do |message_or_share|
         message = message_or_share.retweet_source || message_or_share
         mp = find_miracle_painter_by_message(message)
         if mp

以下は記憶を失った未来の自分のためのメモ

  • twitter plugin で mute で検索すると user_detail_view が引っかかって filter_show_filter あたりに処理がある
  • いろいろ試して show_filter で検索すると Plugin.filtering(:show_filter, ) が大量に引っかかる
  • mikutter show_filter で雑検索すると mikutterの薄い本vol.1 が引っかかってそこに filter の記事がある
  • 更に調べると Writing mikutter plugin に キーワードミュート機能 のエントリがあって notify プラグインでも使われているという記述がある
  • mikutter 本体を :show_filter@ で検索すると source:core/mui/gtk_timeline_utils.rb にも Plugin.filtering(:show_filter, ) があるが source:plugin/gtk3/widget/timeline.rb@c0c9997a には無い
  • 適当に勘で Plugin.filtering(:show_filter, )#1522 と同じ timeline 表示っぽいところに入れて意図通りの mute になるのを確認

toshi_a 初音 さんが約3年前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • 担当者Izumi Tsutsui にセット
  • ブランチtopic/1539-show-filter にセット

配列でない場合が考慮されていなかったので、そこだけパッチしました。確認お願いします

Izumi Tsutsui さんが約3年前に更新

  • ステータスレビュー待ち から マージ待ち に変更

topic/1539-show-filter をマージして
ミュートの試験はすぐにできませんが TL表示はできているので問題ないと思います。

toshi_a 初音 さんが約3年前に更新

  • ステータスマージ待ち から 終了 に変更

他の形式にエクスポート: Atom PDF