Project

General

Profile

バグ #1216

user_detail_view noで tweet_mode の指定が必要?

Added by Izumi Tsutsui over 2 years ago. Updated over 2 years ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
user_detail_view
ブランチ:
topic/1216-yakitori
クラッシュする:
No

Description

user_detail_view.rb で Service.primary を使って user_timeline を取得しているところがある
という根本的な問題はさておき、そこでのツイート取得に tweet_mode は必要でしょうか。

--- core/plugin/user_detail_view/user_detail_view.rb.orig    2018-04-21 02:59:04.000000000 +0000
+++ core/plugin/user_detail_view/user_detail_view.rb
@@ -73,7 +73,7 @@ Plugin.create :user_detail_view do
       order do |message|
         retweet = message.retweeted_statuses.find{ |r| user_id == r.user.id }
         (retweet || message)[:created].to_i end end
-    Service.primary.user_timeline(user_id: user_id, include_rts: 1, count: [UserConfig[:profile_show_tweet_once], 200].min).next{ |tl|
+    Service.primary.user_timeline(user_id: user_id, include_rts: 1, count: [UserConfig[:profile_show_tweet_once], 200].min, tweet_mode: 'extended'.freeze).next{ |tl|
       i_timeline << tl
     }.terminate(_("@%{user} の最近のつぶやきが取得できませんでした。見るなってことですかね") % {user: model[:idname]})
     timeline_storage[i_timeline.slug] = model end

Files

#1

Updated by Izumi Tsutsui over 2 years ago

題名修正中の変換操作誤りの Enter でチケット投稿
の罠にハマりました……(適当に修正してください)

#2

Updated by toshi_a 初音 over 2 years ago

特に必要ないと考えています。
自分がデバッグ用に使っているプラグインなので説明も何もないですが、mikutterがどういった通信をしているかを見ることができるプラグインを昔作ったので、役に立つかもしれないので置いておきます。

https://github.com/toshia/network_inspector

それによると、現在のmasterブランチでは、ユーザを開いたときは以下のようなリクエストを送っています。

(実際にはGETリクエストなので、JSONではなくこれを?a=1&b=2…のようなクエリストリング形式でURLに付与しています。このあたりが自分のデバッグ用なので雑なところです)

#3

Updated by toshi_a 初音 over 2 years ago

  • Status changed from 分類待ち to 実装待ち
#4

Updated by toshi_a 初音 over 2 years ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee set to Izumi Tsutsui
  • ブランチ set to topic/1216-yakitori

焼き鳥食べながら書いてたパッチ適用しました

#5

Updated by Izumi Tsutsui over 2 years ago

  • Status changed from レビュー待ち to マージ待ち
  • Assignee changed from Izumi Tsutsui to toshi_a 初音

mikutter的にはておくれな状況ですが
パッチ当てた mikutter で 12時間ほど compat 表示出ていないので問題ないと思います

#6

Updated by toshi_a 初音 over 2 years ago

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

merged

Also available in: Atom PDF