Bug 261258

Summary: kmix with german ln do not show anything, and is broken
Product: [Applications] kmix Reporter: Fabian B. <apfelmausmail>
Component: Backend: PulseaudioAssignee: Christian Esken <esken>
Status: CLOSED FIXED    
Severity: normal CC: colin, esken, jisaitua, kde, mail, oldium.pro, schiwed
Priority: NOR    
Version: 1   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: with german ln
and the english one
Error message when adding a new tab with german locale
kmixrc with broken content (czech locale)

Description Fabian B. 2010-12-25 23:32:05 UTC
Created attachment 55243 [details]
with german ln

Version:           0.9 (using KDE 4.5.90) 
OS:                Linux

with the german language pack, kmix is clear. and i am unable to add a channel. i want to report the bug with english screenshots but after the change to te english language, kmix works fine. back to german = broken again. 
look at the screenshot

Reproducible: Always

Steps to Reproduce:
open kmix with german language

Actual Results:  
do not show a channel

Expected Results:  
should show a channel ^^

OS: Linux (i686) release 2.6.36-02063602-generic
Compiler: cc
Pulseaudio is active an works (using veromix plasmoid)
Comment 1 Fabian B. 2010-12-25 23:35:03 UTC
Created attachment 55244 [details]
and the english one
Comment 2 Christian (Fuchs) 2011-01-02 19:13:46 UTC
Confirmed here, Gentoo Linux, KDE 4.5.93  (RC1). 

Works with english, german it loses some tabs and it complains about an invalid profile when trying to add new ones. 

Fuchs
Comment 3 Christian Esken 2011-01-02 22:15:25 UTC
Please post a message or screenshot about what is shown when KMix "complains about an invalid profile".

BTW: I am on german locale, and it works for me.
Comment 4 Christian (Fuchs) 2011-01-02 22:20:20 UTC
Created attachment 55489 [details]
Error message when adding a new tab with german locale

File with the error message attached, no messages (despite kdeui (KIconLoader): Trying to remove an entry which is already invalid. This  cache is likely corrupt  which happens with other applications as well and is probably not related) are shown on stderr / stdout. 

Kind regards
Comment 5 Oldřich Jedlička 2011-01-06 23:23:39 UTC
Same here (Gentoo, KDE 4.6 RC2), czech locale. When I delete the .kde/share/config/kmixrc, the kmix works as expected. Second run it is ruined - empty GUI.

It looks like the configuration file is making problems. First run - OK; second run - unusable.
Comment 6 Oldřich Jedlička 2011-01-06 23:30:52 UTC
Created attachment 55659 [details]
kmixrc with broken content (czech locale)
Comment 7 Oldřich Jedlička 2011-01-06 23:43:03 UTC
It doesn't look like a good idea to have configuration file with locale-specific group names and keys.
Comment 8 Elias Probst 2011-01-08 16:02:43 UTC
I can confirm this issue.
German locale
Gentoo
PulseAudio 0.9.22
KDE SC 4.5.95 (4.6 RC2)

When using the German locale, only 2 tabs are shown:
- Streams wiedergeben (Playback Streams)
- Streams aufnehmen (Capture Streams)

Furthermore, the stream views aren't updated when a new sound application is there.
Comment 9 cIdde 2011-03-06 15:57:19 UTC
I can confirm the bug for Kubuntu 11.04 Alpha3, KDE 4.6.1, KMix version 3.8.
In english language I see the channel controls, in german setting not.
Comment 10 Juan Ignacio Saitua 2011-03-11 15:04:56 UTC
Same issue here (Kubuntu 10.10, KDE 4.6.1, Spanish locale using pulseaudio).

From what I see, the problem appears when a profile name has non standard characters as filename (spanish accents in my case). For example
"PulseAudio.Dispositivos_de_reproducción.1.default.xml" has problems, but
"PulseAudio.Dispositivos_de_reproduccion.1.default.xml" hasn't.

As a workaround: remove non standard characters on profile filenames and change the references in kmixctrlrc and kmixrc accordingly. I'm running this script (it only removes accents...so it has to be changed for other locales) before running kmix:

sed 'y/áéíóúÁÉÍÓÚ/aeiouAEIOU/' -i "$(kde4-config --localprefix)/share/config/kmixctrlrc"
sed 'y/áéíóúÁÉÍÓÚ/aeiouAEIOU/' -i "$(kde4-config --localprefix)/share/config/kmixrc"
rename 'y/áéíóúÁÉÍÓÚ/aeiouAEIOU/' "$(kde4-config --localprefix)/share/apps/kmix/profiles/"*
 
Regards.
Comment 11 Oldřich Jedlička 2011-03-11 21:31:03 UTC
It should use locale-independent (english) identifiers in the first place I think.
Comment 12 Juan Ignacio Saitua 2011-03-11 22:07:49 UTC
(In reply to comment #11)
> It should use locale-independent (english) identifiers in the first place I
> think.
+1
Comment 13 Christian Esken 2011-04-03 20:07:13 UTC
Yes, I fully agree that using locale-independent (english) identifiers should be used.  I guess that that the localization was put in because the Tabs can't be renamed.
This is a issue specific to Pulseaudio. => Changing Component
Comment 14 Colin Guthrie 2011-04-04 11:55:29 UTC
GUI Profiles for PA are no longer saved. It's just too complex to try and manipulate things when the elements therein are dynamic anyway.

As PA is already a simplification of the underlying ALSA hardware, I don't think there is really a compelling reason to add even more simplification in the GUI (perhaps there is but IMO the structure of the GUI Profiles code is not really suited to this job without a bit of further tweaks).

So I suspect that this problem is now "fixed" (aka avoided) in the trunk and 4.6 branches after the commits I did yesterday.

Please test if you can and let me know, otherwise I'll close this bug in a littel while.
Comment 15 Christian (Fuchs) 2011-04-04 23:44:26 UTC
These commits are not in 4.6.2 then, because this was after the freeze. A short test showed that it is still broken there. 

Could you link to the commits in the 4.6 branch, so that I can try to apply the patches manually?  Otherwise I will have to wait with my tests for 4.6.3, because I don't want to go to trunk right now. 

Thanks in advance, kind regards
Comment 16 Colin Guthrie 2011-04-05 00:56:17 UTC
Should be quite easy to create a patch from the 4.6.2 tag and the 4.6 branch via subversion, but here are the relevant patches (the first two just being prerequisites to avoid merge conflicts).

http://websvn.kde.org/?revision=1226956&view=revision
http://websvn.kde.org/?revision=1226957&view=revision
http://websvn.kde.org/?revision=1226958&view=revision
Comment 17 Christian (Fuchs) 2011-05-07 10:44:59 UTC
Fixed, and fix officially released in 4.6.3, thanks for the good work. 

Kind regards, 

Christian 

(Sorry for the late reply)
Comment 18 Colin Guthrie 2011-05-07 14:32:02 UTC
Thanks for the feedback.