changes in the skin are not communicated to the menubar options

[Max Linke]

To reproduce

1. activate samplers in the skin
2. check that the 'View'->show Samplers still thinks they are hidden

You'll also need to hit CTRL+1 two times to make them dissapear again

[Raf Van De Meirssche (rafvdm)]

Judging the date, this is about version 1.11.0

It's not only so for the Samplers, it's also for the Vinyl option.

I notice the same problem, somewhat different in version 1.12.0.
I have set up a new bug report for 1.12.0 (https://bugs.launchpad.net/mixxx/+bug/1392454), refering to this one.

[alex (alex-jercaianu)]

I would like to be assigned to this bug as my first patch.

[jus (jus)]

To assign a bug to yourself, click on the ``Unassigned`` link at the top.
The ``Assign`` dialog opens, there click on on the ``Assign Me`` link.

The bug is now assigned for you to work on it.
If there are any questions let us know, also see http://mixxx.org/wiki/doku.php/bugfix_workflow

Looking forward to your first contribution.

[Max Linke (max-linke)]

You can assign yourself in the ui. Just click the word Unassigned

On 03/10/2015 01:00 AM, alex wrote:
> I would like to be assigned to this bug as my first patch.
>

[alex (alex-jercaianu)]

I have looked into this issue and the problem seems to occur because the "SAMPLER" button from the skin xml must be somehow tied to "m_pViewShowSamplers" or "slotViewShotSamplers" in MixxxMainWindow. The same thing happens for the other widgets. I am not sure how i should approach this, since i think that it may imply to either change the skin parser, or the MixxxMainWindow class. Any hints?

[Daniel Schürmann (daschuer)]

The "SAMPLER" button just changes the value of a ControlObject. This is a global double value, that can be accessed form everywhere by a Key/Item string pair.
In your case I think it is required that you hock on the value change event of the ControlObject.

This can be done by instantiating a ControlObjectSlave and register a callback function (slot).
Please make sure that you instantiate the Cos only fafter the skin has introduced the show sampler CO.
Be aware that instantiating the COS only works for skins that have instantiated the show sampler CO.

By the way: Is your build environment fully on duty? Do you have a IDE with a powerful indexer and are you able to single step though the code? My favorite IDE is Eclipse http://www.mixxx.org/wiki/doku.php/eclipse
This link is also helful: http://www.mixxx.org/wiki/doku.php/bugfix_workflow

[alex (alex-jercaianu)]

Thank you for your response! I submitted a pull request in which I solved the problem for the SAMPLER button and i want to see if I should work in the same manner for the other buttons. This solution uses a workaround in which I need to disconnect m_pViewShowSamplers from slotViewShowSamplers because checking the QAction while it is connected to the function that toggles the ControlObject, would trigger infinite recursion. For the moment I can't find an alternative solution. Any ideas?

Link to the pull request: https://github.com/mixxxdj/mixxx/pull/521

[Daniel Schürmann (daschuer)]

Disconnect and connect are not fast functions.
You may consider to use a boolean flag to detect the recursion.

[Nicu Badescu (badescunicu)]

Fix commited: https://github.com/mixxxdj/mixxx/pull/526

[Raf Van De Meirssche (rafvdm)]

Looks OK for those that I have checked.

Thank you.

[Swiftb0y (swiftb0y)]

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/7078