[PATCH] notifications.el: fixed image-data and image-path hints passing

[Mike Kazantsev]

Good day,

Before the patch, hints were passed as "image_path" and "image_data",
which is incorrect, as Desktop Notifications Specification v1.2 clearly
states:

  The icon is defined by the "app_icon" parameter. The image can be
  defined by the "image-path", the "image-data" hint or the deprecated
  "icon_data" hint.

Most common (by far) implementation of spec - libnotify - seem to pass
them as described in spec as well, so current behavior looks like a bug.

Patch to pass "image-path" and "image-data" hints - as per spec - is
attached below.

Desktop Notifications Specification:

  http://developer.gnome.org/notification-spec/#icons-and-images
  
http://people.gnome.org/~mccann/docs/notification-spec/notification-spec-latest.html#icons-and-images



From 4acc11ec45623a695a5db508b54bd107beb9c8bd Mon Sep 17 00:00:00 2001
From: Mike Kazantsev <address@hidden>
Date: Mon, 7 Nov 2011 14:08:08 +0600
Subject: [PATCH] notifications.el: fixed "image-data" and "image-path"
 notification hints passing

---
 notifications.el |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/notifications.el b/notifications.el
index a4e7fdd..71b2a63 100644
--- a/notifications.el
+++ b/notifications.el
@@ -226,11 +226,11 @@ used to manipulate the notification item with
                             (:variant :string ,desktop-entry)) t))
     (when image-data
       (add-to-list 'hints `(:dict-entry
-                            "image_data"
+                            "image-data"
                             (:variant :struct ,image-data)) t))
     (when image-path
       (add-to-list 'hints `(:dict-entry
-                            "image_path"
+                            "image-path"
                             (:variant :string ,image-path)) t))
     (when sound-file
       (add-to-list 'hints `(:dict-entry
-- 
1.7.7


-- 
Mike Kazantsev // fraggod.net

signature.asc
Description: PGP signature