Sound menu has two Rhythmbox elements when using translations

I have to add that this installation dates back to 8.10 and was upgraded from Lucid to Maverick at Alpha 2.

Hi Jiri,

Thanks for the bug report.

In order make sure this is not an upgrade issue, do you think you could test this in a new installation? Using a virtual machine (e.g. virtualbox) or Testdrive (sudo apt-get install testdrive) should allow you to do that in a matter of minutes.

David, I was going to try RC (clean VB installation) yesterday before submitting this report but due to other work I didn't have enough time. I'll try RC/daily build today and report back. Still, this might be a bug somewhere in upgrade cycle and that's why I decided to report it before testing a clean installation.

Hi Jiri,

I have not seen this before or received a bug report with similar issues. One quick fix would be to delete the file at ~/.cache/indicators/sound/familiars-players-db.keyfile and reload the menu.

Conor

Hi Conor,

This is reproducible even with a clean Maverick RC install in VirtualBox. I've attached a screenshot. After installation indicator-sound doesn't have any player entries. After the first Rhythmbox launch the menu shows Rhythmbox and its controls. Closing Rhythmbox and relaunching it doesn't change the situation, only Rhythmbox entry appears.

After the first reboot (when I haven't launched Rhythmbox [for the second time] yet) the sound menu shows the translated text "Rytmilaatikko" without any controls. When I launch Rhythmbox for the first time after boot, "Rhythmbox" entry appears.

xyz@xyz:~$ cat .cache/indicators/sound/familiar-players-db.keyfile

[Seen Database]
DesktopFiles=/usr/share/applications/rhythmbox.desktop;

This is the only entry in the file, after the first launch of Rhythmbox, after reboots etc. There is definitely something wrong with indicator-sound or with (Finnish) translation(s).

I can confirm Jiri's findings.

It's probably some wrong assumption in indicator-sound that people wouldn't translate the actual name "Rhythmbox". Most languages do not, however it's wrong assumption as it's generally done on application-to-application and language-to-language basis. In Finnish, there was rather ok translation for the name as well, so it has been translated by the GNOME Translators.

Hmmm, I didn't think application names were translated... It's always our presumptions that let us down.

In order for me to get a fix in for this for the SRU, could someone please attach to this bug a /usr/share/applications/rhythmbox.desktop file which has the translated application name.

Conor

El dt 05 de 10 de 2010 a les 09:51 +0000, en/na Conor Curran va
escriure:
> Hmmm, I didn't think application names were translated... It's always
> our presumptions that let us down.
>

They are :) It is perhaps not as common for application names such as
Rhythmbox, but it is quite common for generic application names (e.g.
"Update Manager"). In addition to that, some languages with writing
systems different than Latin need to either adapt the name to the
particular writing system, or transcribe it.

I guess this hadn't come up in the Rhythmbox indicator because not many
languages translate "Rhythmbox".

> In order for me to get a fix in for this for the SRU, could someone
> please attach to this bug a /usr/share/applications/rhythmbox.desktop
> file which has the translated application name.
>

In Ubuntu generally .desktop files are empty wrt. translations. The
translations are only present in the .mo files from language packs,
where they are loaded from at runtime. More info on [1].

[1]
https://wiki.ubuntu.com/UbuntuDevelopment/Internationalisation/Packaging#DesktopEntries

Accepted indicator-sound into maverick-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!

The double entry (in English) is gone after the update. However, the controls don't work for me. Pressing the buttons doesn't affect Rhythmbox. Timo, coud you test this as well? Thanks.

Filename: pool/main/i/indicator-sound/indicator-sound_0.4.8-0ubuntu1_i386.deb

no I have not seen any regression atleast

I can confirm Jiri's findings: the translated text problem is gone, but now the controls do not work anymore.

Setting red flag here for now, for Conor to investigate further.

did you relogin after installing the update? works fine on two systems here.

On Mon, Oct 11, 2010 at 12:07 AM, Martin Pitt <email address hidden>wrote:

> Setting red flag here for now, for Conor to investigate further.
>
> ** Tags added: regression-proposed verification-failed
> ** Tags removed: verification-needed
>
> --
> Sound menu has two Rhythmbox elements when using translations
> https://bugs.launchpad.net/bugs/654140
> You received this bug notification because you are subscribed to
> indicator-sound in ubuntu.
>
> Status in Sound Menu: Fix Released
> Status in Ubuntu Translations: Triaged
> Status in “indicator-sound” package in Ubuntu: Fix Committed
> Status in “indicator-sound” source package in Maverick: Fix Committed
>
> Bug description:
> Sound menu shows duplicate Rhythmbox element with translations. I have set
> Finnish as the main language, yet indicator-sound shows Rhythmbox element as
> the main element. Due to the translation (or more likely a bug), there's
> another element called Rytmilaatikko (see picture) in Finnish. Translated
> element doesn't seem to do anything. I've deleted the whole panel already
> once and attached indicators but the problem persists.
>
>
>

Omer: Yes, I did and also reboot. Please note that if you didn't have the translated text problem, this new problem also probably doesn't affect you. It's now certainly something that used to use the application name "Rhythmbox" to send the signals, but now that the translated name is used the control signals no more go through (since they should probably still be sent with the original application name instead of the translated name).

hopefully 0.4.9 should fix this,. Thanks for all the feedback Timo

Rejecting current upload. Please reupload with -v to include previous SRU in source.changes

0.4.9 has been uploaded to proposed, Timo if you have a sec could you confirm that the problem has been fixed. After installing could you ensure you delete the cache file in /home/$user/.cache/indicators/sound/familiar-players.db and test again.
Thanks,
Conor

0.4.9-0ubuntu1 accepted into maverick-proposed, which should fix this regression. Since this only affected proposed, I'm closing this now.

Actually, let's leave it as "fix committed" until we get confirmation that it really works now.

It does work just fine for me. The element in English is gone and controls in the translated element work well now. Fix released?

Thanks for testing! Closing then.

Thanks Jiri for testing, I've been a bit busy and without my 10.10 laptop.