Transmission should check server caps before using notification actions

Hey Cody,

Your patch only shows the notification at all if actions are supported, is that
the intended behaviour?

Thanks,

James

Nice catch, thanks very much James!

Thanks, Cody. Can you please file that upstream as well and link the bug here, so that we are notified about upstream comments/changes?

I'll upload that together with the sponsoring of 1.50 (bug 329161).

http://trac.transmissionbt.com/ticket/1865

Hi Martin, sorry.. I forgot to free the list of caps in the previous patch. This patch is correct.

This bug was fixed in the package transmission - 1.50-1ubuntu1

---------------
transmission (1.50-1ubuntu1) jaunty; urgency=low

  [ Andrew Starr-Bochicchio ]
  * Merge from debian unstable (LP: #329161), remaining changes:
   - debian/control:
    + Added replaces & provides clutch (now included as part of transmission).
    + Build-depends on quilt.
   - debian/rules:
    + Uncommented "include /usr/share/quilt/quilt.make".
    + Added patch/unpatch targets for Quilt.
    + Create a PO template during package build.
   - 20_add_X-Ubuntu-Gettext-Domain.diff: Add X-Ubuntu-Gettext-Domain
     to .desktop file.

  [ Martin Pitt ]
  * Add 01_check_notification_actions.diff: Check if notification
    agent supports actions, and do not use actions if not. (LP: #334252)

transmission (1.50-1) unstable; urgency=low

  * new upstream release (closes: #510743)
  * debian/control,
    debian/transmission-cli.install,
    debian/transmission-daemon.install: move transmission-remote to
    transmission-cli package (closes: #511572)
  * debian/control:
      - strip the "XS" prefix and use the official DM field name
      - changed build-dep on libcurl to "libcurl4-dev | libcurl-dev"

 -- Andrew Starr-Bochicchio <email address hidden> Thu, 26 Feb 2009 11:55:50 +0100