プロジェクト

全般

プロフィール

バグ #1001

インテントが発動しない

Satoshi Okuno3ヶ月前に追加. 3ヶ月前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
開始日:
2017-03-23
期日:
進捗率:

0%

プラグイン名:
intent_selector.rb
再現手順:

説明

(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して比較すれば良いと思っています。

0001-.patch 表示 (1.65 KB) Satoshi Okuno, 2017-03-26 12:46

関係しているリビジョン

リビジョン 1c7f492b (差分)
Satoshi Okuno3ヶ月前に追加

対象が(未定義)の関連付けが発動しない問題を修正した。

履歴

#1 toshi_a 初音3ヶ月前に更新

  • 担当者Satoshi Okuno にセット

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

#2 Satoshi Okuno3ヶ月前に更新

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

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

#3 toshi_a 初音3ヶ月前に更新

  • ステータス新規 から 終了 に変更

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

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