Mixxx

control object creation order broken in main branch: cos created by skin not available to controller scripts

Bug #1914969 reported by Owen Williams
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Confirmed
High
Unassigned
Mixxx 2.4.0

Bug Description

My traktor s3 script stopped loading correctly in main. I see this:

Warning [Controller] ControlDoublePrivate::getControl returning NULL for ( "[Channel1]" , "end_of_track" )
Warning [Controller] ControllerScriptHandlerBase: "Uncaught exception at line 1729 in file file:///home/owen/src/github/mixxx/res/controllers/Traktor-Kontrol-S3-hid-scripts.js.\n\nException:\n Error: script tried to connect to ControlObject ([Channel1], end_of_track) which is non-existent."

Other times the control it barfs on is Master / maximize_library.

Both of these control objects do exist, but are not created until the skin parsing code comes across them and then creates them on the fly. It looks like something happened in the startup ordering to break this. I am doing a git bisect to see if I can locate the change.

Revision history for this message
Be (be.ing) wrote :

Likely a regression from https://github.com/mixxxdj/mixxx/pull/3446

Revision history for this message
Owen Williams (ywwg) wrote :

I don't think that pull request is necessarily wrong. maximize_library is referenced in main code, but isn't ever created there. The main menubar could own it, for instance. There may be other GUI actions we'd like to make available and should have explicit COs, like show_mixer / show_vinylcontrol / show_coverart

Be (be.ing)
Changed in mixxx:
milestone: none → 2.4.0
ronso0 (ronso0)
Changed in mixxx:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
ronso0 (ronso0) wrote :

my Terminalmix mappings fail when trying to connect LEDs to ..,show_mixer, show_4decks and other essential skin COs

Revision history for this message
Jan Holthuis (holthuis-jan) wrote :

I'm setting this to high. We should make these COs either first-class Mixxx COs that always work, or remove them from all builtin mappings.

At the moment, all mappings that use one of these COs are unusable with skins that donvt support them and won't work in QML mode either.

Changed in mixxx:
importance: Medium → High
Revision history for this message
Swiftb0y (swiftb0y) wrote :

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

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.