IM Status plugin does not reset status message on end playback.

Thank you for your bug report. The issue is an upstream one and it would be nice if somebody having it could send the bug the to the people writting the software (https://wiki.ubuntu.com/Bugs/Upstream/GNOME)

I can confirm this behavior. will look upstream.

It was known upstream, you can track it here: https://bugzilla.gnome.org/show_bug.cgi?id=612714

The problem seems to be this:

file: rhythmbox/plugins/im-status/im-status/__init__.py
----------------------------------
def get_mc5_status (self):
...
    for acct in am.Get (MC5_AM_IFACE_NAME, "ValidAccounts"):
        ...
        if status[2] != "":
              return status[2]
    ....
    return None
----------------------------------

This algorithm is doing the following:

Loop through all empathy accounts.
If the status of the current account is not an empty string then return it and
save it so we can restore it later.
If we do not find an account with a status that is not an empty string then
simply return None.

the None value is implicitly used by this plugin to mean that we never saved
the status in the first place, so we never bother restoring it.

Here is the kicker: Empathy (MC5, rather) uses an empty string as the default
status message. Even though the user is seeing the status as "Available"...
status[2] is actually an empty string.

The proper fix seems to be to change the return statement to return an empty
string instead of None.

NOTE: This patch has also been submitted upstream.

Thank you for your work on this issue, lucid is current frozen for beta2 but let's wait to see what upstream says about the change

According to the latest (upstream) comment this is in git upstream.

the bug has been fixed upstream now, the change will not go in lucid which is frozen now but could be consider for a stable update

just for the record, the commit is at : http://git.gnome.org/browse/rhythmbox/commit/?id=d2ca8e6cf68ccae21e93dbe8c31f1b78e1698bd1 in case someone wants to try it.

I will backport this change to lucid

SRU team, the debdiff is on bug #568887

Accepted rhythmbox into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

I've tested the proposed package and it fixes the issue, now the IM Status is set to the previous one you were using after using the plugin, the status is set back to the previous by either the end of the playback or quitting rhythmbox. Marking this as verification done, thanks a lot.

This bug was fixed in the package rhythmbox - 0.12.8-0ubuntu4

---------------
rhythmbox (0.12.8-0ubuntu4) lucid-proposed; urgency=low

  * debian/patches/92_git_mc5_status.patch:
    - upstream change to correctly update the im status (lp: #548851)
  * debian/patches/93_git_no_empty_playlist_crasher.patch:
    - upstream change to not crash on empty playlist saving (lp: #567399)
  * debian/patches/94_git_escaping_crasher.patch:
    - upstream change to fix a crash on copies (lp: #565517)
  * debian/patches/95_git_correct_dbversion.patch:
    - upstream change to db version detection issues in some locates leading
      to trying to convert datas on every run (lp: #568887)
  * debian/patches/96_git_magnatune_genre.patch:
    - upstream change to set correct magnatune genre names (lp: #537958)
 -- Sebastien Bacher <email address hidden> Wed, 28 Apr 2010 16:27:51 +0200

Copied lucid-proposed to maverick.

This bug was fixed in the package rhythmbox - 0.12.8-0ubuntu4

---------------
rhythmbox (0.12.8-0ubuntu4) lucid-proposed; urgency=low

  * debian/patches/92_git_mc5_status.patch:
    - upstream change to correctly update the im status (lp: #548851)
  * debian/patches/93_git_no_empty_playlist_crasher.patch:
    - upstream change to not crash on empty playlist saving (lp: #567399)
  * debian/patches/94_git_escaping_crasher.patch:
    - upstream change to fix a crash on copies (lp: #565517)
  * debian/patches/95_git_correct_dbversion.patch:
    - upstream change to db version detection issues in some locates leading
      to trying to convert datas on every run (lp: #568887)
  * debian/patches/96_git_magnatune_genre.patch:
    - upstream change to set correct magnatune genre names (lp: #537958)
 -- Sebastien Bacher <email address hidden> Wed, 28 Apr 2010 16:27:51 +0200