Project

General

Profile

バグ #1318

popup_notifyイベントの第二引数にPlugin::Twitter::Message以外のMessage Modelを渡すと通知が発生しない

Added by toshi_a 初音 5 months ago. Updated 4 months ago.

Status:
終了
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
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

Related to バグ #1322: エラーメッセージを出さず突然正常終了することがある終了Actions

Associated revisions

Revision e7784a01 (diff)
Added by toshi_a 初音 5 months ago

[libnotify] Diva::Modelを受け取った時に通知が表示されない refs #1318

Revision d6656489 (diff)
Added by toshi_a 初音 4 months ago

libnotify gemでデスクトップ通知を表示する refs #1318

History

#1

Updated by toshi_a 初音 5 months ago

  • Subject changed from popup_notifyイベントの第二引数にPlugin::Twitter::Message以外のMessage Modelwo to popup_notifyイベントの第二引数にPlugin::Twitter::Message以外のMessage Modelを渡すと通知が発生しない
#2

Updated by toshi_a 初音 5 months ago

  • Status changed from 分類待ち to 実装待ち
#3

Updated by toshi_a 初音 5 months ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee deleted (toshi_a 初音)
  • ブランチ set to topic/1318-popup-notify-suddenly-death

ちゃんと見るといろいろとlibnotifyプラグインはいろいろと意味不明。

  • アイコンの表示・非表示をuserの有無で分岐してる
    • userは必須なので、ない場合の処理が要らない
    • しかも、通知タイトルの表示・非表示も同じ条件で分岐してる
  • アイコンの一時ファイルは、元画像の拡張子を踏襲する割には必ずpngになる
    • aaa.pngというjpegファイルー!
#4

Updated by toshi_a 初音 4 months ago

  • Related to バグ #1322: エラーメッセージを出さず突然正常終了することがある added
#5

Updated by toshi_a 初音 4 months ago

  • Status changed from レビュー待ち to マージ待ち
#6

Updated by toshi_a 初音 4 months ago

  • Status changed from マージ待ち to 終了

Also available in: Atom PDF