search

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

Fehlende disableStandbyTimer() aufrufe. #75

Open marco-117 opened 4 years ago

marco-117 commented 4 years ago

Hallo,

ich habe ein Fehler entdeckt. Es ist möglich, wenn der Tonuino pausiert, über die up und down Buttons einen Track spielen zu lassen. In Zeile 1024:

  if (!ignoreUpButton)
        if (!mySettings.invertVolumeButtons) {
          nextButton();
        }

und Zeile 1039:

else if (downButton.wasReleased()) {
if (!ignoreDownButton) {
        if (!mySettings.invertVolumeButtons) {
          previousButton();
        }

Es wird nicht abgefragt ob ein Track läuft und über nextButton() und previousButton() wird nextTrack() und previousTrack() ausgeführt. In beiden werden dann Tracks angespielt ohne das der Standby Timer deaktiviert wird.

Ich habe diese Schlussfolgerung hergeleitet und nicht getestet.

Grüße

jochenberger commented 3 years ago

Jepp, kann ich in der DEV reproduzieren.