Bug 334542 - Contact list doesn't change to the presence selected when disabling the now playing plugin
Summary: Contact list doesn't change to the presence selected when disabling the now p...
Status: RESOLVED FIXED
Alias: None
Product: telepathy
Classification: Frameworks and Libraries
Component: contactlist (show other bugs)
Version: 0.7.1
Platform: openSUSE Linux
: NOR major
Target Milestone: Future
Assignee: Telepathy Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-05-09 06:02 UTC by Jamie Smith
Modified: 2017-11-24 19:49 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 18.04


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jamie Smith 2014-05-09 06:02:19 UTC
When selecting a desired presence message in the contact list that also disables a status message plugin, the selected presence isn't applied, only the presence that the presence message plugin was engaged from ends up re-activating.

Reproducible: Always

Steps to Reproduce:
1.Engage now playing plugin from an empty or full custom presence.
2.Disengage the plugin by changing to another presence.
Actual Results:  
The presence returns from the selected presence to a different presence that was active prior to the plugin having been activated.

Expected Results:  
The presence should stay at the user-selected presence.
Comment 1 Jamie Smith 2017-11-22 01:04:17 UTC
Git commit 20f845ce0c2e828700a79efd455f9700deafb1d7 by James D. Smith.
Committed on 22/11/2017 at 01:03.
Pushed by smithjd into branch 'master'.

Independent account presences.
Activities support.
Make unit testing compile.
Implement status message parser with Now Playing.
Status message parser unit testing.
REVIEW: 130192
Related: bug 284517, bug 293443, bug 294940, bug 297924

M  +5    -3    CMakeLists.txt
A  +208  -0    account-status-helper.cpp     [License: LGPL (v2.1+)]
A  +115  -0    account-status-helper.h     [License: LGPL (v2.1+)]
M  +31   -41   autoaway.cpp
M  +4    -8    autoaway.h
D  +0    -62   autoconnect.cpp
D  +0    -99   autoconnect.h
M  +0    -2    config/CMakeLists.txt
D  +0    -80   config/nowplaying-lineedit.cpp
D  +0    -43   config/nowplaying-lineedit.h
D  +0    -107  config/nowplaying-listwidget.cpp
D  +0    -50   config/nowplaying-listwidget.h
M  +34   -122  config/telepathy-kded-config.cpp
M  +0    -4    config/telepathy-kded-config.h
M  +10   -73   config/telepathy-kded-config.ui
M  +18   -27   screensaveraway.cpp
M  +2    -5    screensaveraway.h
M  +182  -149  status-handler.cpp
M  +24   -32   status-handler.h
A  +303  -0    status-message-parser.cpp     [License: LGPL (v2.1+)]
A  +97   -0    status-message-parser.h     [License: LGPL (v2.1+)]
M  +17   -13   telepathy-kded-module-plugin.cpp
M  +34   -22   telepathy-kded-module-plugin.h
M  +4    -1    telepathy-module.cpp
M  +0    -1    telepathy-module.h
M  +192  -236  telepathy-mpris.cpp
M  +33   -28   telepathy-mpris.h
M  +33   -11   tests/CMakeLists.txt
M  +3    -3    tests/status-handling-main.cpp
A  +132  -0    tests/status-message-parser.cpp     [License: LGPL (v2.1+)]

https://commits.kde.org/ktp-kded-module/20f845ce0c2e828700a79efd455f9700deafb1d7