Update comments, rearrange file, fix case statement and handle empty parameter correctly.

Config/Nextcloud/cron.sh

@@ -19,32 +19,44 @@ if (( $RUNNING > 0 )); then
 	exit $RUNNING
 fi
 
+function usage() {
+	cat <<- EOF
+		Script to help with scheduling Nextcloud's cron requirements.
+		Usage: $PROG [-h|-v]
+			-h) Display the usage and help text.
+			-v) Pass a verbose request to cron.php.
+	EOF
+	exit $1
+}
+
 verbose=""
 case "$1" in
+	"") ;;
 	"-h")
-		cat <<- EOF
+		usage 0
-			Script to help with scheduling Nextcloud's cron requirements.
-			Usage: $PROG [-h|-v]
-				-h) Display the usage and help text.
-				-v) Pass a verbose request to cron.php.
-		EOF
-		exit 0
 		;;
 	"-v")
 		verbose="-v"
 		;;
 	*)
-		echo "ERROR: Unknown parameter '$1', exiting."
+		echo -e "ERROR: Unknown parameter '$1'. Exiting.\n"
-		$DIR/$PROG -h
+		usage 1
 		;;
 esac
 
-# 2023-08-25 From crontab.
+# Keep ownership correct and apps up to date. Also exists in fixes.sh.
+sh -c "docker exec -i nc-app chown -Rc www-data:www-data ."
+# No longer update apps in advance of NC updates, allow the upgrade process to do it.
+#sh -c "docker exec -itu www-data nc-app ./occ app:update --all"
+
+# Prepare the variables being passed to the execution command.
 if [[ -f $DIR/.env ]]; then
 	source $DIR/.env
 else
 	PHP_MEMORY_LIMIT=256M
 fi
+
+# Main part of what would go in the crontab.
 sh -c "
 	docker exec nc-app \
 	sudo -u www-data \
@@ -54,9 +66,4 @@ sh -c "
 	-f cron.php $verbose \
 "
 
-# 2023-08-25 From fixes.sh, keep ownership correct and apps up to date.
-sh -c "docker exec -i nc-app chown -Rc www-data:www-data ."
-# No longer update apps in advance of NC updates, allow the upgrade process to do it.
-#sh -c "docker exec -itu www-data nc-app ./occ app:update --all"
-
 exit 0