decibel-audio-player shouldn't use notifications with actions

[David Barth]

Binary package hint: decibel-audio-player

When a new track starts playing, a notification bubble appears giving the song details, and containing a “Skip track” button. The button should be made conditional on whether the notification server supports actions.

[Scott Kitterman (kitterman)]

Won't fixing the Ubuntu task as this package is in Universe and MOTU is not manned to support this kind of divergence from upstream.

[Emilio Pozuelo Monfort (pochu)]

This is a valid bug as decibel-audio-player should check whether the notification-daemon implementation supports actions.

[Emilio Pozuelo Monfort (pochu)]

Hi François,

According to the notification-daemon spec, the "Actions" feature may or may not be implemented by the daemon, and apps should check whether support exists before using it.

Actions: "This functionality may not be implemented by the notification server, conforming clients should check if it is available before using it"

http://www.galago-project.org/specs/notification/0.9/x81.html

Checking for 'actions' in pynotify.get_server_caps() should do the trick.

Cheers

[Scott Kitterman (kitterman)]

I wontfixed the Ubuntu task because I don't think this is something we
should diverge from upstream on. I did not mark it invalid.

I'm not putting it to wontfix again because we don't revert reversions. I
would ask you to reconsider if we should maintain a diff for this
indefinitely and put it back to wontfix.

[Emilio Pozuelo Monfort (pochu)]

Scott Kitterman wrote:
> I'm not putting it to wontfix again because we don't revert reversions. I
> would ask you to reconsider if we should maintain a diff for this
> indefinitely and put it back to wontfix.

It if was a wishlist or something else I'd consider it, but since decibel is not
following the spec it can be considered buggy and thus I'll leave it open and
patch it if necessary (either here or even in Debian), but hope François will
consider fixing this upstream making a local fix unnecessary.

[Anonym25712 (anonym25712)]

Ok, I'll fix that.

[Anonym25712 (anonym25712)]

Fixed on trunk. Now, showing the button is disabled by default, and the checkbox in the configuration dialog is enabled only if 'actions' can be found in pynotify.get_server_caps().

[Emilio Pozuelo Monfort (pochu)]

Awesome, thanks François.

[Anonym25712 (anonym25712)]

De nada.

[Anonym25712 (anonym25712)]

See v1.01