make sure all files get checked and all errors are outputted

README.md

@@ -4,7 +4,7 @@
 
 ### php codesniffer
 
-http://pear.php.net/package/PHP_CodeSniffer/redirected
+[http://pear.php.net/package/PHP_CodeSniffer/redirected](http://pear.php.net/package/PHP_CodeSniffer/redirected)
 
 You can install php codesniffer with these commands:
 
@@ -13,12 +13,12 @@ You can install php codesniffer with these commands:
 
 ### Fork coding standards in ~/.phpcs/Fork
 
-This is based on this dotfiles setup: (https://github.com/WouterSioen/dotfiles).
+This is based on this dotfiles setup: [https://github.com/WouterSioen/dotfiles](https://github.com/WouterSioen/dotfiles).
 If you want to put it in another place, just change the --standard parameter in the phpcs command.
 
 ### esvalidate
 
-https://github.com/duereg/esvalidate
+[https://github.com/duereg/esvalidate](https://github.com/duereg/esvalidate)
 
 Esvalidate is easy to install with the node package manager:
 

pre-commit

@@ -10,27 +10,30 @@
 # @version	1.0.0
 # @author	Wouter Sioen <[email protected]>
 
+# create empty errors array
+declare -a errors
+
 # fetch all changed php files and validate them
 files=$(git diff-index --name-only HEAD | grep -P '\.php$')
 for file in $files; do
 
 	# first check if they are valid php files
-	output=`php -l $file | grep 'PHP Parse error'`
+	output=`php -l $file | grep 'Errors parsing'`
 
 	# if it did contain errors, we have output
 	if [ -n "$output" ]; then
 		echo "$file contains php syntax errors"
-		exit 1
+		errors=("${errors[@]}" "$output")
 	fi
 
 	# checks if the phpcs output contains '(0 errors, 0 warnings)'
-	output=`phpcs -v --standard="$HOME/.phpcs/Fork" --report=full $file | grep '| ERROR |'`
+	output=`phpcs --standard="$HOME/.phpcs/Fork" --report=full $file | grep '| ERROR |'`
 
 	# if it did contain errors, we have output
 	if [ -n "$output" ]; then
 		echo "$file fails coding standards"
-		echo $output
-		exit 1
+		phpcs --standard="$HOME/.phpcs/Fork" --report=full $file
+		errors=("${errors[@]}" "$output")
 	fi
 done
 
@@ -43,6 +46,11 @@ for file in $files; do
 	if [ -n "$output" ]; then
 		echo "$file contains javascript syntax errors"
 		echo $output
-		exit 1
+		errors=("${errors[@]}" "$output")
 	fi
-done
+done
+
+# if we have errors, exit with 1
+if [ -n "errors" ]; then
+	exit 1
+fi