Summary: | /media not showing artist or album info for Kaffeine | ||
---|---|---|---|
Product: | [Applications] konversation | Reporter: | Chris <chris.kcat> |
Component: | general | Assignee: | Konversation Developers <konversation-devel> |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | ||
Priority: | NOR | ||
Version: | 0.18 | ||
Target Milestone: | --- | ||
Platform: | Gentoo Packages | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
Chris
2006-01-18 04:49:27 UTC
We've got a new version written in Python in the repository. Since it's easy to swap out, I'd ask you to test it: http://websvn.kde.org/*checkout*/trunk/extragear/network/konversation/scripts/media?rev=497755 ... and tell us if the problem still persists. It does still have the problem. This should fix it (not knowing much of anything about Python either, but it works for me): --- old_media 2006-01-17 20:56:53.000000000 -0800 +++ media 2006-01-17 20:59:51.000000000 -0800 @@ -80,10 +80,12 @@ kaffeine = check_running('kaffeine') if kaffeine: - string = os.popen('dcop '+kaffeine+' KaffeineIface title').readline().rstrip('\n') + title = os.popen('dcop '+kaffeine+' KaffeineIface title').readline().rstrip('\n') + artist = os.popen('dcop '+kaffeine+' KaffeineIface artist').readline().rstrip('\n') + album = os.popen('dcop '+kaffeine+' KaffeineIface album').readline().rstrip('\n') - if string: - string=pretty_print(string)+' [Kaffeine]' + if title: + string=pretty_print(title,artist,album)+' [Kaffeine]' Popen(['dcop', port, 'Konversation', 'say', server, target, string]).communicate() sys.exit(0) else: SVN commit 499936 by hein: Improved Kaffeine support in the media script. Patch by Chris. Thanks! BUG:120362 M +1 -0 ChangeLog M +7 -5 scripts/media --- trunk/extragear/network/konversation/ChangeLog #499935:499936 @@ -70,6 +70,7 @@ - Status bar now showing context-relevant information (link URLs, menu item descriptions) - Changed hard-coded channel join taskbar notification into a configurable notification event - Yammi media player is now supported by media script +- Improved Kaffeine support in the media script - QUIT events for users are now shown in any queries you have open with them - Fixed nickname context menu actions not working in queries - Redesigned tab notifications allowing for colored LEDs or text labels, the color being configurable per event class --- trunk/extragear/network/konversation/scripts/media #499935:499936 @@ -79,11 +79,13 @@ bad_apps = ['JuK'] kaffeine = check_running('kaffeine') - if kaffeine: - string = os.popen('dcop '+kaffeine+' KaffeineIface title').readline().rstrip('\n') - - if string: - string=pretty_print(string)+' [Kaffeine]' + if kaffeine: + title = os.popen('dcop '+kaffeine+' KaffeineIface title').readline().rstrip('\n') + artist = os.popen('dcop '+kaffeine+' KaffeineIface artist').readline().rstrip('\n') + album = os.popen('dcop '+kaffeine+' KaffeineIface album').readline().rstrip('\n') + + if title: + string=pretty_print(title,artist,album)+' [Kaffeine]' Popen(['dcop', port, 'Konversation', 'say', server, target, string]).communicate() sys.exit(0) else: |