mirror of
https://github.com/got-feedBack/feedBack.git
synced 2026-07-22 12:52:29 +00:00
e14ef64224
Some checks are pending
ship-ci / ci (push) Waiting to run
Tester: "Passports does not advance in the song queue." The play queue tells playSong "don't clear the queue I'm driving" by passing options.fromQueue. But window.playSong is wrapped by a CHAIN of plugins — nam_tone, midi_amp, fretboard, invert_highway, tabview — and each wrapper forwards only (filename, arrangement), silently dropping the options object. So fromQueue never reached playSong: it cleared the queue the instant its first song started, and a gig/album/playlist never advanced. Reproduced on the real build via a queue.start + a hooked clear(): the queue went inactive with 0 remaining immediately after start, and the clear stack ran through nam_tone -> midi_amp -> invert_highway -> fretboard -> session.js. Fixing six plugin wrappers is whack-a-mole and the next plugin re-breaks it. Fix it at the source instead: the queue raises an out-of-band flag (_consumeInternalPlay, one-shot) beside the wrapper chain, not through it, and playSong's clear-guard honours it. options.fromQueue stays as the in-band path. The flag is consumed on read so a later MANUAL play still abandons the queue. Verified on the real build: the gig queue stays active after start and advances on song:ended (Iron Maiden -> Blind Guardian), and a manual play still clears. Tests drive the real clear-guard against the queue for: a dropped-options wrapper (the bug), the one-shot manual-play-still-clears invariant, and the in-band fromQueue path on its own. All 3 fail on the pre-fix source. JS 1211/1211. |
||
|---|---|---|
| .. | ||
| audio-el.js | ||
| blob-io.js | ||
| count-in.js | ||
| diagnostics-export.js | ||
| dom.js | ||
| edit-modal.js | ||
| format.js | ||
| highway-colors.js | ||
| highway-constants.js | ||
| highway-draw.js | ||
| highway-geometry.js | ||
| highway-state-primitives.js | ||
| host.js | ||
| juce-audio.js | ||
| library-state.js | ||
| library.js | ||
| loops.js | ||
| player-controls.js | ||
| player-state.js | ||
| plugin-loader.js | ||
| resume-session.js | ||
| section-practice.js | ||
| session.js | ||
| settings-io.js | ||
| settings.js | ||
| shortcuts.js | ||
| transport.js | ||
| tuning-display.js | ||
| viz.js | ||