fixed formatting warnings (#872)

* save used git commit hash to version file (see #828)

* added first idea

* Get rid of debconf: falling back to frontend: Readline messages while installing

* Adapt for stretch dockerfile

* Added functions for logging

* Enable Logging during the installation

* Added config file information into logging

* Updated comments

* fixed flake8 warnings

* read current commit from Github

* Consolidate install scripts (#871)

* Remove duplicate chown/chmod

* Align stretch install scripts

* Remove stretch-install-spotify.sh script

* Add info about pkg installation (#879)

* Add info about pkg installation 

Fix for #818

* Update README.md

* added missing sudo

* Update Detect rfidoff function in gpio-buttons.py (#869)

* Update Detect rfidoff function in gpio-buttons.py

Adapted the RFID-Off function to the new gpio-buttons code structure. Works fine with the hardware hack for the USB-rfid reader described here:
#62

The rfidoff event is triggered when the used gpio is pulled down.

* Added Missing "Channel" to rfidoff Action

* add empty header context to spotify requests, fixes title and cover download (#884)

Co-authored-by: fhaertig <[email protected]>

* Add info how to post logging info (#882)

* fixed flake8 warnings

Co-authored-by: Kiri <[email protected]>
Co-authored-by: Groovylein <[email protected]>
Co-authored-by: Micz Flor <[email protected]>
Co-authored-by: Fred G <[email protected]>
Co-authored-by: MalteHST <[email protected]>
Co-authored-by: lovethisshit <[email protected]>
Co-authored-by: fhaertig <[email protected]>

misc/sampleconfigs/gpio-buttons.py.sample

@@ -31,6 +31,7 @@ from time import sleep
 # I have not yet had the time to test is, so I placed it in the misc folder.
 # If anybody has ideas or tests or experience regarding this solution, please create pull requests or contact me.
 
+
 # This function takes a holding time (fractional seconds), a channel, a GPIO state and an action reference (function).
 # It checks if the GPIO is in the state since the function was called. If the state
 # changes it return False. If the time is over the function returns True.
@@ -41,39 +42,59 @@ def checkGpioStaysInState(holdingTime, gpioChannel, gpioHoldingState):
     while(holdingTime >= (time.perf_counter() - startTime)):
         # Return if state does not match holding state
         if(gpioHoldingState != GPIO.input(gpioChannel)):
-           return False
+            return False
         # Else: Wait
     return True
 
+
 # Actions that call other processes of the phoniebox (Channel Parameter needed by RPi.GPIO)
 def shutdown_action(channel):
     check_call("./scripts/playout_controls.sh -c=shutdown", shell=True)
+
+
 def vol0_action(channel):
     check_call("./scripts/playout_controls.sh -c=mute", shell=True)
+
+
 def volD_action(channel):
     check_call("./scripts/playout_controls.sh -c=volumedown", shell=True)
+
+
 def volU_action(channel):
     check_call("./scripts/playout_controls.sh -c=volumeup", shell=True)
+
+
 def next_action(channel):
     check_call("./scripts/playout_controls.sh -c=playernext", shell=True)
+
+
 def prev_action(channel):
     check_call("./scripts/playout_controls.sh -c=playerprev", shell=True)
+
+
 def halt_action(channel):
     check_call("./scripts/playout_controls.sh -c=playerpause", shell=True)
+
+
 def recordstart_action(channel):
     check_call("./scripts/playout_controls.sh -c=recordstart", shell=True)
+
+
 def recordstop_action(channel):
     check_call("./scripts/playout_controls.sh -c=recordstop", shell=True)
+
+
 def recordplaylatest_action(channel):
     check_call("./scripts/playout_controls.sh -c=recordplaylatest", shell=True)
 def rfidoff_action(channel):
     check_call("./scripts/playout_controls.sh -c=playerpauseforce", shell=True)
 
+
 # Handlers that handle special behavior of the push of a button
-# When the shutdown button was held for more than 2 seconds LED flashed and afterwards 
+# When the shutdown button was held for more than 2 seconds LED flashed and afterwards
 def shutdown_handler(channel):
     # Detect holding of button
-    if True == checkGpioStaysInState(shutdownHoldTime, btn_shut, GPIO.LOW):
+    if checkGpioStaysInState(shutdownHoldTime, btn_shut, GPIO.LOW) is True:
         # Blink LED
         for x in range(0, 10):
             GPIO.output(led_power, x & 1)
@@ -82,7 +103,8 @@ def shutdown_handler(channel):
         GPIO.output(led_power, GPIO.HIGH)
         # Shutdown afterwards
         shutdown_action(channel)
-
+
+
 # When the Volume Down button was held for more than 0.3 seconds every 0.3 seconds he will lower t$
 def volU_handler(channel):
     # Rise volume as requested
@@ -91,6 +113,7 @@ def volU_handler(channel):
     while checkGpioStaysInState(volumeHoldTime, btn_volU, GPIO.LOW):
         volU_action(channel)
 
+
 # When the Volume Up button was held for more than 0.3 seconds every 0.3 seconds he will call a ra$
 def volD_handler(channel):
     # Rise volume as requested
@@ -99,6 +122,7 @@ def volD_handler(channel):
     while checkGpioStaysInState(volumeHoldTime, btn_volD, GPIO.LOW):
         volD_action(channel)
 
+
 # Define the used pins of the raspberry board
 btn_shut = 3
 btn_vol0 = 13
@@ -109,27 +133,27 @@ btn_prev = 20
 btn_halt = 21
 led_power = 12
 # btn_rfidoff = 
-#reco =
-#play =
+# reco =
+# play =
 
 # Set GPIO numbering to BCM instead of board numbering
 GPIO.setmode(GPIO.BCM)
 # Bounce tolerance time for buttons
 bouncetime = 500
-volumeHoldTime = 0.3 # Seconds
-shutdownHoldTime = 2 # Seconds
+volumeHoldTime = 0.3  # Seconds
+shutdownHoldTime = 2  # Seconds
 PledBlinkTime = 0.3  # Seconds
 
 # Set up GPIO pins and the power led
-GPIO.setup(btn_shut       , GPIO.IN, pull_up_down=GPIO.PUD_UP)
-GPIO.setup(btn_vol0       , GPIO.IN, pull_up_down=GPIO.PUD_UP)
-GPIO.setup(btn_volU       , GPIO.IN, pull_up_down=GPIO.PUD_UP)
-GPIO.setup(btn_volD       , GPIO.IN, pull_up_down=GPIO.PUD_UP)
-GPIO.setup(btn_next       , GPIO.IN, pull_up_down=GPIO.PUD_UP)
-GPIO.setup(btn_prev       , GPIO.IN, pull_up_down=GPIO.PUD_UP)
-GPIO.setup(btn_halt       , GPIO.IN, pull_up_down=GPIO.PUD_UP)
-#GPIO.setup(btn_rfidoff    , GPIO.IN, pull_up_down=GPIO.PUD_DOWN)
-GPIO.setup(led_power      , GPIO.OUT)
+GPIO.setup(btn_shut, GPIO.IN, pull_up_down=GPIO.PUD_UP)
+GPIO.setup(btn_vol0, GPIO.IN, pull_up_down=GPIO.PUD_UP)
+GPIO.setup(btn_volU, GPIO.IN, pull_up_down=GPIO.PUD_UP)
+GPIO.setup(btn_volD, GPIO.IN, pull_up_down=GPIO.PUD_UP)
+GPIO.setup(btn_next, GPIO.IN, pull_up_down=GPIO.PUD_UP)
+GPIO.setup(btn_prev, GPIO.IN, pull_up_down=GPIO.PUD_UP)
+GPIO.setup(btn_halt, GPIO.IN, pull_up_down=GPIO.PUD_UP)
+# GPIO.setup(btn_rfidoff, GPIO.IN, pull_up_down=GPIO.PUD_DOWN)
+GPIO.setup(led_power, GPIO.OUT)
 
 
 # Set standard events for the buttons. Callback functions define the actions of the events (THey are defined above)