make script.deckFromGroup(group) also filter deck from effect groups like [EqualizerRack1_[ChannelN]_Effect1]

[Frank Breitling]

The Javascript helper function for controller mappings script.deckFromGroup(group) fails for [EqualizerRack1_[ChannelN]_Effect1].

This group is needed for example to adjust the equalizers:
https://github.com/mixxxdj/mixxx/wiki/Mixxxcontrols#eqs-and-filters

[ronso0 (ronso0)]

That function is made for deck strings only.
[EqualizerRack1_[ChannelN]_Effect1] is a effect group.

https://github.com/mixxxdj/mixxx/blob/master/res/controllers/common-controller-scripts.js#L152

[ronso0 (ronso0)]

What are you tryig to achieve?

[ronso0 (ronso0)]

btw you can use the 'parseInt' from script.deckFromGroup():
// isolate 'N' from [ChannelN]
sourceDeck = parseInt(group.substring(8,9));

[Frank Breitling (frank-breitling)]

I am extending my controller mapping to four decks. Therefore I need to know from which Channel a command was sent. It would be nice if I could use script.deckFromGroup() not only for deck strings. I think replacing the if statement by

    // Extract deck number from the group text
    deck = group.match("Channel.")[0].substring(7);

should fix this.
Would you like me to try it out and write a PR?

[ronso0 (ronso0)]

Why don't you use midi-components? it has a handler for deck switches.

That way you'd also have an up-to-date maaping that could be merged into Mixxx.

[Frank Breitling (frank-breitling)]

Because it didn't work when I tried it and because of its complexity I didn't know how to debug it.

Anyway, you can set this to won't fix if you don't like to fixing it.
But the bug is valid, because the function should do what its name "deckFromGroup()" suggests.
It also would be useful and the fix is easy.

[ronso0 (ronso0)]

Sure, if you want to extend script.deckFromGroup() to filter decks from effect control groups (and others) go ahead!

[ronso0 (ronso0)]

maybe you should check https://mixxx.zulipchat.com/#narrow/stream/113295-controller-mapping/topic/C.2B.2B.20controller.20system.20refactoring

[Frank Breitling (frank-breitling)]

Thank you, I have created https://github.com/mixxxdj/mixxx/pull/3178.

[Swiftb0y (swiftb0y)]

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