プロジェクト

全般

プロフィール

バグ #830

完了

User.new_ifnecessary がメインスレッド以外から呼ばれた時にクラッシュすることがある

toshi_a 初音 さんが8年以上前に追加. 8年以上前に更新.

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

説明

#781 より

問題

source:core/user.rb では、 Retriever.new_ifnecessary を User.new_ifnecessary がオーバライドしていて、その中で User.findbyidname を呼んでいる。findbyidname はメインスレッドで呼ぶことはできないので、User.new_ifnecessaryがメインスレッドでは呼べないようになっている。

解決方法

User.new_ifnecessaryをやめる

そもそもRetriever.new_ifnecessaryは十分な機能をもっていて、Messageではオーバライドせずに使えている。こういうことをしているのはidnameでも引いてこれるようにしようとしているからだが、現状それは正しく動いていないので、他のRetriever同様、idのみでマッチングを行えば良い。

Retrieverの@storage相当のものをUserにもたせる

idnameで一意にUserのインスタンスが決まるのはUserだけなので、Retrieverがやっているのと全く同じようにインスタンス変数に保持する。


関連するチケット

関連している バグ #781: Userのインスタンスが生成されていないユーザのプロフィールを開けない終了toshi_a 初音2015-09-27操作

toshi_a 初音 さんが8年以上前に更新

  • 関連している バグ #781: Userのインスタンスが生成されていないユーザのプロフィールを開けない を追加

toshi_a 初音 さんが8年以上前に更新

idnameだけでもUserを取得可能になっているのは、Twitter API v1の頃の検索APIが、userのscreen_nameだけを返し、idを返さない仕様になっていたため、screen_nameのみによるuserの推定が必要になったことが原因らしい。

現在では少なくとも標準プラグインではこの挙動を期待したコードは残っていないように見える。これを消したときのサードパーティプラグインの影響だけが気になるところ。

恐らく別のサービスに対応するプラグインくらいしかこのメソッドを利用しないと思うので、影響はないんじゃないかな(願望)

toshi_a 初音 さんが8年以上前に更新

  • ステータス新規 から 終了 に変更
  • 対象バージョン3.3 から 3.4 に変更

Twitter API 1.0なんてなかったんや

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