バグ #1318
popup_notifyイベントの第二引数にPlugin::Twitter::Message以外のMessage Modelを渡すと通知が発生しない
プラグイン名:
libnotify
ブランチ:
topic/1318-popup-notify-suddenly-death
クラッシュする:
No
Description
popup_notify イベントでは2つ目の引数にDiva::Modelが渡されることがあるが、Messageクラスであるかどうかだけを判定している。
そのため、Kernel.#spawnの最後の引数としてDiva::Modelが渡ることがあり、名前付き引数として解釈されてしまった結果、引数が不足して通知が発生しないなどの予想できない動作に繋がる。
popup_notify イベントがテキストでもModelでも受け付けてしまうのはもう仕方がないので、Diva::Model一般を受け付ける。
再現手順
Plugin.call(:popup_notify, user_model, some_model) のようなコードを実行する
Related issues
Updated by