Project

General

Profile

Actions

バグ #1539

closed

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

Added by Izumi Tsutsui almost 3 years ago. Updated almost 3 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
gtk3
クラッシュする:
No

Description

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

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

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


Files

Actions #1

Updated by Izumi Tsutsui almost 3 years ago

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 になるのを確認
Actions #2

Updated by toshi_a 初音 almost 3 years ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to Izumi Tsutsui
  • ブランチ set to topic/1539-show-filter

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

Actions #3

Updated by Izumi Tsutsui almost 3 years ago

  • Status changed from レビュー待ち to マージ待ち

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

Actions #4

Updated by toshi_a 初音 almost 3 years ago

  • Status changed from マージ待ち to 終了
Actions

Also available in: Atom PDF