Project

General

Profile

Actions

バグ #1001

closed

インテントが発動しない

Added by Satoshi Okuno almost 8 years ago. Updated almost 8 years ago.

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

Description

(1):openイベントにURLを渡した場合に「どの方法で開きますか?」ダイアログで「次回から、次の内容から始まる〜」チェックボックスをONすると、対象が(未定義)の関連付けが作成されます。
しかし、条件に合致するURLを開いても関連付けが発動せず、再び「どの方法で開きますか?」ダイアログが表示されてしまいます。

直接的な原因は、core/plugin/intent_selector/intent_selector.rbの123行目。
divide_intents()で対象がnil(未定義)のインテントを除外しているからと思います。

単純に(未定義)インテントを除外しないようにすれば良いと考えています。

(2)同127行目
intentsの中身はStringです(or Stringの場合がある?)が、intent_slugsはSymbolなのでマッチしません。

intentsの中身を.to_symして比較すれば良いと思っています。


Files

0001-.patch (1.65 KB) 0001-.patch Satoshi Okuno, 2017-03-26 12:46
Actions #1

Updated by toshi_a 初音 almost 8 years ago

  • Assignee set to Satoshi Okuno

手元では試していませんが、コード見る感じだと確かにどちらの修正も妥当な気がしますね。特に二番目についてはバグっぽいですが、具体的にどこかわからないので、実際に修正された時のパッチとかもらえませんか

Actions #2

Updated by Satoshi Okuno almost 8 years ago

見解ありがとうございます。

修正部分のみのパッチを添付しました。
確認頂ければとおもいます。

Actions #3

Updated by toshi_a 初音 almost 8 years ago

  • Status changed from 新規 to 終了

確認しました。hotfix/3.5にmergeしました。ありがとうございます。

Actions

Also available in: Atom PDF