xfjx / TonUINO

Die DIY Musikbox (nicht nur) für Kinder
https://www.voss.earth/tonuino/
GNU General Public License v3.0
449 stars 404 forks source link

handleNext() des aktiven modifiers kann bei einem Liedwechsel mehrfach aufgerufen werden #76

Open joachimeichborn opened 4 years ago

joachimeichborn commented 4 years ago

OnPlayFinished kann bei einem track-Ende mehrfach aufgerufen werden. Der Code probiert sich dagegen zu schützen, indem verglichen wird, ob der beendete track gleich dem zuletzt beendeten track ist. Falls ja wird das event ignoriert.

Der Vergleich wird aber erst ausgeführt, nachdem handleNext des aktiven modifiers aufgerufen wurde. Dies wird also bei zwei events zu einem track-Ende zwei mal ausgeführt was zu unerwartetem Verhalten führt.