player.in: exit if SONG_URI not set

Author: mbhangui

README.md

@@ -49,7 +49,7 @@ now-playing    - Passed when a song starts playing
 end-song       - Passed when a song finishes playing
 ```
 
-The default installation makes a copy of `/usr/libexec/mpdev/player` and `/usr/libexec/mpdev/playpause` in `$HOME/.mpdev` for the user with uid `1000`. The player script inserts or updates songs in the stat.db, sticker.db sqlite database when a song gets played. For bulk inserts, updates, deletion, the **mpdev** package also comes with `mpdev_update(1)` and `mpdev_cleanup(1)` programs that help in maintaining `stats.db`, `sticker.db` sqlite databases. This two programs use SQL Transactions and are extremely fast and takes just a few seconds to create/update the stats.db, sticker.db sqlite databases. You can disable all database updates by the player script, by setting **NO_DB_UPDATE** environment variable. In such a case, the player script will just print information about the song being played and actions like pause, stop, play. The default installation also makes a copy of `/usr/libexec/mpdev/output` and `/usr/libexec/mpdev/mixer` in `$HOME/mpdev`. At the moment, these scripts just print information of the state of output devices and the volume level. You can edit them if you can think of something useful. If you are installing mpdev from the [Open Build Service repository](https://software.opensuse.org//download.html?project=home%3Ambhangui%3Araspi&package=mpdev), the mpdev sevice will automatically update these scripts if they change in the package. If you don't want auto-udpate you can create an override file. e.g. If you create a file $HOME/.mpdev/.player.nooverwrite, the script $HOME/.mpdev/player will never be replaced with a newer script from the binary package.
+The default installation makes a copy of `/usr/libexec/mpdev/player` and `/usr/libexec/mpdev/playpause` in `$HOME/.mpdev` for the user with uid `1000`. The player script inserts or updates songs in the stat.db, sticker.db sqlite database when a song gets played. For bulk inserts, updates, deletion, the **mpdev** package also comes with `mpdev_update(1)` and `mpdev_cleanup(1)` programs that help in maintaining `stats.db`, `sticker.db` sqlite databases. This two programs use SQL Transactions and are extremely fast and takes just a few seconds to create/update the stats.db, sticker.db sqlite databases. You can disable all database updates by the player script, by setting **NO_DB_UPDATE** environment variable. In such a case, the player script will just print information about the song being played and actions like pause, stop, play. The default installation also makes a copy of `/usr/libexec/mpdev/output` and `/usr/libexec/mpdev/mixer` in `$HOME/mpdev`. At the moment, these scripts just print information of the state of output devices and the volume level. You can edit them if you can think of something useful. If you are installing mpdev from the [Open Build Service repository](https://software.opensuse.org//download.html?project=home%3Ambhangui%3Araspi&package=mpdev), the mpdev sevice will automatically update these scripts if they change in the package. If you don't want auto-update you can create an override file. e.g. If you create a file $HOME/.mpdev/.player.nooverwrite, the script $HOME/.mpdev/player will never be replaced with a newer script from the binary package.
 
 Example 1. create stats.db in the current directory
 

doc/ChangeLog

@@ -59,6 +59,8 @@ Release 1.1 Start 30/06/2022 End --/--/----
 31. songi: fixed last_played when song has never been played
 - 11/02/2025
 32. player.in: return db operation 5 times on failure
+- 17/11/2025
+33. player.in: exit if SONG_URI not set
 
 * Thu Jun 30 2022 18:32:58 IST Manvendra Bhangui <mpdev@indimail.org> 1.0-1.1%{?dist}
 Release 1.0 Start 02/03/2020 End 30/06/2022

player.in

@@ -1,7 +1,7 @@
 #!/bin/bash
 #
 #
-# $Id: player.in,v 1.56 2025-10-24 23:23:11+05:30 Cprogrammer Exp mbhangui $
+# $Id: player.in,v 1.57 2025-11-17 18:44:37+05:30 Cprogrammer Exp mbhangui $
 #
 get_mpd_conf_value()
 {
@@ -650,6 +650,10 @@ case "$SYSTEM" in
 	gnu_date=0
 	;;
 esac
+if [ -z "$SONG_URI" ] ; then
+	echo "SONG_URI not set" 1>&2
+	exit 0
+fi
 # we don't handle stream uri currently
 echo $SONG_URI|grep -E "^http:|^https:" > /dev/null
 if [ $? -eq 0 ] ; then
@@ -699,6 +703,9 @@ fi
 exit 0
 
 # $Log: player.in,v $
+# Revision 1.57  2025-11-17 18:44:37+05:30  Cprogrammer
+# exit if SONG_URI not set
+#
 # Revision 1.56  2025-10-24 23:23:11+05:30  Cprogrammer
 # fix syntax errors
 #