Bug 291163

Summary: KMix shows as always muted in systray and mouse wheel up sets always 5% sound
Product: [Applications] kmix Reporter: Martin Klapetek <mklapetek>
Component: KMix Panel DockingAssignee: Christian Esken <esken>
Status: RESOLVED FIXED    
Severity: normal CC: amantia, jpsinthemix, lindsay.mathieson, simon
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Unlisted Binaries   
OS: Linux   
Latest Commit: Version Fixed In: 4.10
Sentry Crash Report:
Attachments: Fix volstr append error for capture channels
Fix saving erroneous capture volume values
Fix kmix popup erroneous volume values
Fix bad Qt connection

Description Martin Klapetek 2012-01-10 13:17:58 UTC
Version:           unspecified (using Devel) 
OS:                Linux

These are actually two bugs together, but I believe they are related.

I think it was after git master update sometime last week when the KMix shows its tray icon as always muted, even though the sound is not muted and the mouse tooltip shows correct volume level.

Muting/unmuting through KMix or even Alsamixer has no effect, the icon appears all the time as muted.

Furthermore the volume control using mouse wheel is also broken - it works on lowering the volume, but scrolling up always sets it right to 5%, no more (scroll down works).

It /might/ be my local issue, but I tried removing all kmix configs etc, did not help.

Reproducible: Didn't try

Steps to Reproduce:
1) Start KMix

Actual Results:  
1) Muted icon all the time
2) Volume change on mouse wheel up goes straight to 5%

Expected Results:  
1) Icon reflecting the real state
2) Volume changing by mouse wheel correctly

I can provide any additional debug/info you need, just ask :)
Comment 1 András Manţia 2012-01-31 08:42:36 UTC
I can see the same with git master from last week.
Comment 2 John Stanley 2012-02-01 12:27:57 UTC
Created attachment 68392 [details]
Fix volstr append error for capture channels 

Fix volstr append error for capture channels (this caused one pc which has
common playback and capture volumes to not have ant sound functionality at
all since a kmix writetoHW would write bad capture volumes over playback
volumes for DAC:0)
Comment 3 John Stanley 2012-02-01 12:28:38 UTC
Created attachment 68393 [details]
Fix saving erroneous capture volume values
Comment 4 John Stanley 2012-02-01 12:29:27 UTC
Created attachment 68394 [details]
Fix kmix popup erroneous volume values
Comment 5 John Stanley 2012-02-01 12:31:12 UTC
Created attachment 68395 [details]
Fix bad Qt connection

I'm always amazed that Qt doesn't detect this sort of thing @ compile time.
Comment 6 John Stanley 2012-02-02 22:11:53 UTC
(In reply to comment #0)
> Version:           unspecified (using Devel) 
> OS:                Linux
> 
> These are actually two bugs together, but I believe they are related.
> 
> I think it was after git master update sometime last week when the KMix shows
> its tray icon as always muted, even though the sound is not muted and the mouse
> tooltip shows correct volume level.
> 
> Muting/unmuting through KMix or even Alsamixer has no effect, the icon appears
> all the time as muted.
> 
> Furthermore the volume control using mouse wheel is also broken - it works on
> lowering the volume, but scrolling up always sets it right to 5%, no more
> (scroll down works).
> 
> It /might/ be my local issue, but I tried removing all kmix configs etc, did
> not help.
> 
> Reproducible: Didn't try
> 
> Steps to Reproduce:
> 1) Start KMix
> 
> Actual Results:  
> 1) Muted icon all the time
> 2) Volume change on mouse wheel up goes straight to 5%
> 
> Expected Results:  
> 1) Icon reflecting the real state
> 2) Volume changing by mouse wheel correctly
> 
> I can provide any additional debug/info you need, just ask :)


The patch in Comment #4 (https://bugs.kde.org/attachment.cgi?id=68394) should fix this problem.
Comment 7 simon 2012-02-05 22:14:09 UTC
this should find its way into git master
Comment 8 Christian Esken 2012-09-20 12:26:55 UTC
Git commit 4a103ad08f62237ff6c6803ebe2dedd9afb38e61 by Christian Esken.
Committed on 20/09/2012 at 14:25.
Pushed by esken into branch 'master'.

Fix: Not set capture volume in case of error

M  +1    -1    backends/mixer_alsa9.cpp

http://commits.kde.org/kmix/4a103ad08f62237ff6c6803ebe2dedd9afb38e61
Comment 9 Christian Esken 2012-09-20 12:37:46 UTC
01-kdemultimedia-4.8.0-volstr-fix.patch.gz  Already fixed in trunk
02-kdemultimedia-4.8.0-read-capture-volume-fix.patch  Just commited to trunk
04-kdemultimedia-4.8.0-aboutToShow-connection-fix.patch I did some changes. As far as I can see I also did the change (new line contains "contextMenuAboutToShow()")

03-kdemultimedia-4.8.0-kmixdockwidget-refreshVolumes-fix.patch I am unsure about this. When does that problem happen? I can change the volume within KMix or with alsamixer and see all volume changes and muting.
Comment 10 John Stanley 2012-09-24 23:24:06 UTC
Sorry, for the delay.. The issue patch 03-kdemultimedia-4.8.0-kmixdockwidget-refreshVolumes-fix.patch targeted was resolved sometime after v4.8.0 so it not needed anymore. Been quite some time so I can't remember the details..
thanks much,
John
Comment 11 Christian Esken 2012-09-25 07:04:08 UTC
Thanks for the feedback. I'll close the bug. By the way, I am currently rewriting KMix's communication infrastructure, and will hopefully not break anything that was just fixed. Currently it is not finished, but if someone is able to test a bit in about a week (trunk version) would be appreciated:

http://kmix5.wordpress.com/2012/09/24/more-consistent-gui-and-updates-to-the-soundmenu/
Comment 12 John Stanley 2012-09-25 08:25:55 UTC
(In reply to comment #11)
> Thanks for the feedback. I'll close the bug. By the way, I am currently
> rewriting KMix's communication infrastructure, and will hopefully not break
> anything that was just fixed. Currently it is not finished, but if someone
> is able to test a bit in about a week (trunk version) would be appreciated:
> 
> http://kmix5.wordpress.com/2012/09/24/more-consistent-gui-and-updates-to-the-
> soundmenu/

Assuming it'll build against kde release 4.9.1, I'd be happy to build it and verify that things are ok in my normal daily kde use. Just post a comment/email me when its ready.
Comment 13 Christian Esken 2012-11-14 21:30:29 UTC
Everything is done, and can be tested. It will also be part of the KDE4.10 beta.