Project

General

Profile

バグ #671

Webで登録した「保存した検索」がmikutter起動時に反映されない

Added by Satoshi Okuno over 5 years ago. Updated over 5 years ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
saved_search
ブランチ:
クラッシュする:

Description

saved_search.rbで起動時に「保存した検索」を同期しようとしますが、refresh(true)の引数がアレで動作していない様です。
(refresh()の引数は:keepまたは:getが値域に見えます。)

デフォルトの:keepを使う様にして正常動作しました。

パッチ添付しますのでご確認ください。


Files

0001-Saved-Search.patch (1.17 KB) 0001-Saved-Search.patch Satoshi Okuno, 2014-05-11 08:56

Associated revisions

Revision 709b789b (diff)
Added by toshi_a 初音 over 5 years ago

saved search: マルチアカウントに対応 refs #671, #680

History

#1

Updated by toshi_a 初音 over 5 years ago

仕様

mikutterでは、起動時はできるだけキャッシュを読み、APIリクエストをしないようになっています。
例えば以下の手順を踏むと

  1. mikutterを起動
  2. mikutterを終了
  3. 保存した検索をWebから削除
  4. mikutterを起動

起動直後は消したはずのタブが出現したままだが、「設定>基本設定>保存した検索」で設定した時間待つと、保存した検索タブが同期され、タブが削除されるというのが期待する動作です。

現在の不具合

しかし、ざっと見たところ、起動時一回しか refresh を呼び出していないようで、結果指摘頂いたとおり、常に最古のデータを取得してしまい、それが更新されることはないということが起こっています。
とすると、今回の問題は、定期更新時に保存した検索の一覧も再取得するという修正が適切かと思うのですが、それで問題ないでしょうか。もしそうならこちらで修正してしまいます。

#2

Updated by toshi_a 初音 over 5 years ago

  • Status changed from パッチ適用待ち to 新規
#3

Updated by toshi_a 初音 over 5 years ago

  • Status changed from 新規 to 実装待ち
#4

Updated by Satoshi Okuno over 5 years ago

なるほど。仕様了解です。

デフォルトの更新が60分なので結構長いタイムラグにはなるんですよねーとか色々思い巡らせてみたのですが、よくよく考えたらかなりレアなケースですね。

初回の更新時の反映で問題ないと思います。
よろしくお願いします。

#5

Updated by toshi_a 初音 over 5 years ago

  • Assignee set to toshi_a 初音
#6

Updated by toshi_a 初音 over 5 years ago

  • Status changed from 実装待ち to 終了

対応しました

Also available in: Atom PDF