Bug 307582

Summary:	Having an option to enable now playing is pointless.
Product:	[Unmaintained] telepathy	Reporter:	Alex Fiestas <afiestas>
Component:	kded-module	Assignee:	Telepathy Bugs <kde-telepathy-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	kde
Priority:	NOR	Flags:	kde: Decision-Required+
Version:	git-latest
Target Milestone:	Future
Platform:	unspecified
OS:	Linux
Latest Commit:	http://commits.kde.org/telepathy-kded-module/e0e61ace680533b613d19e442aaa9d2e90ecf08e	Version Fixed In:	0.7
Sentry Crash Report:

Description Alex Fiestas 2012-09-29 17:44:07 UTC

In the kded KCM there is an option called: Now playing... presence: Enabled which is enabled by default.

Discussed with d_ed about this, he told me that Now Playing won't be enabled even if this option is enabled, that this option enables the option in the contact list.

So, either way:
-In case it enables the actual now playing, it should be off by default.
-In case it enables an option... why do we need that? and option that enables an option?

Reproducible: Always

Actual Results:  
 

Expected Results:

Comment 1 David Edmundson 2012-10-12 01:42:51 UTC

The option to in the config to enable now playing is pointless. It doesn't activate until you enable it from the combo box. If this isn't enable you get a dialog telling you to enable this.

Changed title. Set to unconfirmed, and flagged for discussion.

Comment 2 Alex Fiestas 2013-02-24 22:24:03 UTC

So, what should we do about this?

Comment 3 Weng Xuetian 2013-10-08 10:37:09 UTC

Git commit e0e61ace680533b613d19e442aaa9d2e90ecf08e by Weng Xuetian.
Committed on 08/10/2013 at 10:29.
Pushed by xuetianweng into branch 'kde-telepathy-0.7'.

Refactor now playing plugin

Enable option now behaves as "enable on login", and changes at contact
list will be temporary for this login. And use requested presence to
evaluate the new presence by this plugin.
Related: bug 313126
REVIEW: 113066
FIXED-IN: 0.7

M  +1    -4    config/telepathy-kded-config.cpp
M  +118  -83   telepathy-mpris.cpp
M  +8    -4    telepathy-mpris.h

http://commits.kde.org/telepathy-kded-module/e0e61ace680533b613d19e442aaa9d2e90ecf08e