Fixed problem with PHP < 5.6

Author: Stolz

phpcodesniffer

@@ -21,14 +21,14 @@ done
 cd vendor/squizlabs/php_codesniffer/CodeSniffer/Standards/ || exit_error "PHP_CodeSniffer not installed"
 if [ -d PHPCompatibility ]; then
 	cd PHPCompatibility
-	git pull
+	git pull -q
 else
-	git clone http://github.com/wimg/PHPCompatibility.git
+	git clone http://github.com/wimg/PHPCompatibility.git -q
 fi
 
 # Run PHP_CodeSniffer
 
 cd "$DIR"
-./vendor/bin/phpcs --report-width=100 --tab-width=4 --standard=phpcodesniffer.xml,PHPCompatibility src/ -as --runtime-set testVersion 5.4
+./vendor/bin/phpcs --report-width=100 --tab-width=4 --standard=phpcodesniffer.xml,PHPCompatibility src/ -as --runtime-set testVersion 5.3
 
 exit 0

phpcodesniffer.xml

@@ -10,10 +10,7 @@
 	<rule ref="Generic.Formatting.SpaceAfterCast"/>
 
 	<!--Function name in camelCase-->
-	<rule ref="Generic.NamingConventions.CamelCapsFunctionName">
-		<!--Allow helpers with snake_case-->
-		<exclude-pattern>helpers.php</exclude-pattern>
-	</rule>
+	<rule ref="Generic.NamingConventions.CamelCapsFunctionName"/>
 
 	<!--PHP5 constructor syntax "function __construct()"-->
 	<rule ref="Generic.NamingConventions.ConstructorName"/>
@@ -24,10 +21,7 @@
 	<!--Force tabs identation-->
 	<rule ref="Generic.WhiteSpace.DisallowSpaceIndent"/>
 
-	<!--Suppress warning for some files without PHP code-->
-	<rule ref="Internal.NoCodeFound">
-		<exclude-pattern>/views/</exclude-pattern>
-	</rule>
+	<rule ref="Internal.NoCodeFound"/>
 
 	<rule ref="PSR2">
 		<!--Allow one line control structures to have no curly braces-->
@@ -71,5 +65,3 @@
 	<rule ref="Squiz.WhiteSpace.SemicolonSpacing"/>
 
 </ruleset>
-
-

src/Command.php

@@ -43,10 +43,10 @@ protected function setParametersFromConfig()
 		{
 			$this->info("Using '$connection' connection");
 
-			$this->parameters['-host']	= $connections[$connection]['host'];
-			$this->parameters['-db']	= $connections[$connection]['database'];
-			$this->parameters['-u']		= $connections[$connection]['username'];
-			$this->parameters['-p']		= $connections[$connection]['password'];
+			$this->parameters['-host'] = $connections[$connection]['host'];
+			$this->parameters['-db']   = $connections[$connection]['database'];
+			$this->parameters['-u']    = $connections[$connection]['username'];
+			$this->parameters['-p']    = $connections[$connection]['password'];
 		}
 		else
 			$this->comment("Unknown connection '$connection'. Command will fail unless you provide DB credentials.");

src/ServiceProvider.php

@@ -4,13 +4,6 @@
 
 class ServiceProvider extends LaravelServiceProvider
 {
-	/**
-	 * Path to the default config file.
-	 *
-	 * @var string
-	 */
-	protected $configFile = __DIR__ . '/config.php';
-
 	/**
 	 * Register bindings in the container.
 	 *
@@ -19,7 +12,7 @@ class ServiceProvider extends LaravelServiceProvider
 	public function register()
 	{
 		// Merge user's configuration
-		$this->mergeConfigFrom($this->configFile, 'spy');
+		$this->mergeConfigFrom(__DIR__ . '/config.php', 'spy');
 
 		// Bind 'stolz.schemaspy.command.spy' component to the IoC container
 		$this->app->bind('stolz.schemaspy.command.spy', function ($app) {
@@ -36,12 +29,10 @@ public function boot()
 	{
 		// Register paths to be published by 'vendor:publish' artisan command
 		$this->publishes([
-			$this->configFile => config_path('spy.php'),
+			__DIR__ . '/config.php' => config_path('spy.php'),
 		]);
 
 		// Add artisan command
 		$this->commands('stolz.schemaspy.command.spy');
 	}
-
-
 }