Fix syntax-check errors

As the CI job was not correctly run for some time, a lot of small errors
were in the repository.

Author: Kevinjil

.github/jobs/syntax-check

@@ -25,7 +25,7 @@ if [ ! -x /usr/bin/shellcheck ]; then
 fi
 
 find . \( \
-       -path ./webapp/vendor        -prune \
+       -path ./webapp/vendor     -prune \
 	-o -path ./webapp/var        -prune \
 	-o -path ./output            -prune \
 	-o -path ./.git              -prune \
@@ -34,7 +34,8 @@ find . \( \
     -o -type f \) \
 	-a -type f |  \
 while read -r i ; do
-	if [[ "$i" == *\.php ]] || grep -q "^#\\!.*php" "$i"; then
+	if [[ "$i" == *\.php ]] || grep -q "^#\\!.*php" "$i" && \
+		echo "$i" | grep -qvE '(^\./(webapp/resources/adminer)\.php)'; then
 		$PHP -l "$i" | sed -e 's|No syntax errors detected in ./|check PHP syntax: |'
 		if [[ "$i" == *\.php ]] && [ "$i" != "./webapp/config/bundles.php" ] && ! head -1 "$i"|grep -q "strict_types"; then
 			echo "PHP file missing strict types declaration: $i"

misc-tools/dj_make_chroot.in

@@ -234,7 +234,7 @@ if [ "$DISTRO" = 'Ubuntu' ]; then
         # x86(_64) can use the main Ubuntu repo's, other
         # architectures need to use a mirror from ubuntu-ports.
         MAINSTREAM="amd64 i386"
-        if [ -z "${MAINSTREAM##*$ARCH*}" ]; then
+        if [ -z "${MAINSTREAM##*"$ARCH"*}" ]; then
             DEBMIRROR="http://us.archive.ubuntu.com/ubuntu/"
         else
             DEBMIRROR="http://ports.ubuntu.com/ubuntu-ports/"
@@ -321,6 +321,7 @@ fi
 
 # Add indication to interactive shell that the user is in the chroot
 for bashrc in "root/.bashrc" "etc/bash.bashrc"; do
+    # shellcheck disable=SC2016
 	echo 'PS1=(chroot)$PS1' >> "$CHROOTDIR/$bashrc"
 done
 

misc-tools/dj_run_chroot.in

@@ -10,6 +10,7 @@
 # Abort when a single command fails:
 set -e
 
+# shellcheck disable=SC2317
 cleanup() {
     # Unmount things on cleanup
     umount -f "$CHROOTDIR/proc" >/dev/null 2>&1  || /bin/true

sql/dj_setup_database.in

@@ -53,24 +53,22 @@ EOF
 
 mysql_options()
 {
-	local user pass
-
 	# shellcheck disable=SC2153
 	if [ -n "$DBUSER" ]; then
-		user="-u $DBUSER"
+		_user="-u $DBUSER"
 	else
-		user="${DBA_USER:+-u ${DBA_USER}}"
+		_user="${DBA_USER:+-u ${DBA_USER}}"
 	fi
 	# shellcheck disable=SC2153
 	if [ -n "$PASSWD" ]; then
-		pass="-p$PASSWD"
+		_pass="-p$PASSWD"
 	else
-		[ -n "$PROMPT_PASSWD" ] && pass="-p"
-		[ -n "$DBA_PASSWD" ]    && pass="-p$DBA_PASSWD"
+		[ -n "$PROMPT_PASSWD" ] && _pass="-p"
+		[ -n "$DBA_PASSWD" ]    && _pass="-p$DBA_PASSWD"
 	fi
 
 	[ -z "$USE_SOCKET" ] && port="-P$DBPORT"
-	echo $user ${pass:+"$pass"} -h "$DBHOST" ${port:+"$port"}
+	echo $_user ${_pass:+"$_pass"} -h "$DBHOST" ${port:+"$port"}
 }
 
 # Wrapper around mysql command to allow setting options, user, etc.
@@ -363,7 +361,7 @@ upgrade)
 	# Check if we need to upgrade the Doctrine migrations table
 	if ! echo "SHOW CREATE TABLE \`doctrine_migration_versions\`" | mysql "$DBNAME" >/dev/null 2>&1; then
 		symfony_console doctrine:migrations:sync-metadata-storage -n
-		# shellcheck disable=SC2016
+		# shellcheck disable=SC2016,SC2028
 		echo 'INSERT INTO `doctrine_migration_versions`
 			(version, executed_at, execution_time)
 			SELECT concat("DoctrineMigrations\\\\Version", version), executed_at, 1
@@ -386,7 +384,8 @@ dump)
 
 	if [ -f "${DATABASEDUMPDIR}/${DUMPNAME}.sql.gz" ]; then
 		while true; do
-			read -p "Overwrite existing database dump (y/N)? " yn
+			printf "Overwrite existing database dump (y/N)? "
+			read -r yn
 			case $yn in
 				[Yy]* ) break ;;
 				''|[Nn]* ) exit 0;;
@@ -407,19 +406,20 @@ load)
 			exit 1
 		fi
 		ind=1
-		for i in "$databases"; do
+		for i in $databases; do
 			echo "$ind) $i"
-			: $((ind+=1))
+			ind=$((ind+1))
 		done
 		while true; do
-			read -p "Which database should be loaded? " db
+			echo "Which database should be loaded? "
+			read -r db
 			ind=1
-			for i in "$databases"; do
+			for i in $databases; do
 				if [ "$ind" = "$db" ]; then
 					FILE="$i"
 					break
 				fi
-				: $((ind+=1))
+				ind=$((ind+1))
 			done
 			if [ -n "$FILE" ]; then
 				break

sql/files/defaultdata/r/run

@@ -30,10 +30,14 @@ fi
 #
 # Store intermediate files in the current dir (/compile) as its only
 # available during compilation step.
-export TMPDIR=`pwd`
-Rscript -e "parse('"$@"')"
-EXITCODE=$?
-[ "$EXITCODE" -ne 0 ] && exit $EXITCODE
+TMPDIR=`pwd`
+export TMPDIR
+for f in "$@"
+do
+    Rscript -e "parse('$f')"
+    EXITCODE=$?
+    [ "$EXITCODE" -ne 0 ] && exit $EXITCODE
+done
 
 # Write executing script:
 cat > "$DEST" <<EOF

sql/files/defaultdata/swift/run

@@ -23,7 +23,7 @@ while [ $# -gt 0 ]; do
     if [ "x${1%.swift}" != "x$1" ]; then
         SOURCE="$1"
         [ -z "$MAINSOURCE" ] && MAINSOURCE="$1"
-        if [ "x$SOURCE" = "x$MAINSOURCE" ] && [ "x$MAINSOURCE" != "xmain.swift" ]; then
+        if [ "$SOURCE" = "$MAINSOURCE" ] && [ "$MAINSOURCE" != "main.swift" ]; then
             if [ -f "main.swift" ]; then
                 echo "main.swift exists but is not the main source; not allowed by Swift compiler"
                 exit 1

webapp/config/preload.php

@@ -1,4 +1,4 @@
-<?php
+<?php declare(strict_types=1);
 
 if (file_exists(dirname(__DIR__) . '/var/cache/prod/App_KernelProdContainer.preload.php')) {
     require dirname(__DIR__) . '/var/cache/prod/App_KernelProdContainer.preload.php';

webapp/src/Utils/EventFeedFormat.php

@@ -1,4 +1,4 @@
-<?php
+<?php declare(strict_types=1);
 
 namespace App\Utils;
 

webapp/tests/bootstrap.php

@@ -1,4 +1,4 @@
-<?php
+<?php declare(strict_types=1);
 
 use Symfony\Component\Dotenv\Dotenv;