reginaldojunior / winners

Plugin/DebugKit/Test/Case/DebugkitGroupTestCase.php

Doc Comments +2 added lines, -2 removed lines

@@ -32,8 +32,8 @@
 /**
  * Get Test Files
  *
- * @param null $directory
- * @param null $excludes
+ * @param string|null $directory
+ * @param string $excludes
  * @return array
  */
 	public static function getTestFiles($directory = null, $excludes = null) {

Indentation +14 added lines, -14 removed lines

@@ -1,19 +1,19 @@
 <?php
 /**
- * DebugKit Group Test Case
- *
- * PHP 5
- *
- * CakePHP(tm) : Rapid Development Framework (http://cakephp.org)
- * Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
- *
- * Licensed under The MIT License
- * Redistributions of files must retain the above copyright notice.
- *
- * @copyright     Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
- * @link          http://cakephp.org CakePHP(tm) Project
- * @license       http://www.opensource.org/licenses/mit-license.php MIT License
- */
+	 * DebugKit Group Test Case
+	 *
+	 * PHP 5
+	 *
+	 * CakePHP(tm) : Rapid Development Framework (http://cakephp.org)
+	 * Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
+	 *
+	 * Licensed under The MIT License
+	 * Redistributions of files must retain the above copyright notice.
+	 *
+	 * @copyright     Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
+	 * @link          http://cakephp.org CakePHP(tm) Project
+	 * @license       http://www.opensource.org/licenses/mit-license.php MIT License
+	 */
 
 /**
  * Class DebugKitGroupTestCase

Spacing +6 added lines, -6 removed lines

@@ -46,11 +46,11 @@ 
 		}
 
 		if ($excludes !== null) {
-			$excludes = self::getTestFiles((array)$excludes);
+			$excludes = self::getTestFiles((array) $excludes);
 		}
 		if ($directory === null || $directory !== realpath($directory)) {
-			$basePath = App::pluginPath('DebugKit') . 'Test' . DS . 'Case' . DS;
-			$directory = str_replace(DS . DS, DS, $basePath . $directory);
+			$basePath = App::pluginPath('DebugKit').'Test'.DS.'Case'.DS;
+			$directory = str_replace(DS.DS, DS, $basePath.$directory);
 		}
 
 		$it = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($directory));
@@ -58,14 +58,14 @@ 
 		$files = array();
 		while ($it->valid()) {
 
-			if (!$it->isDot()) {
+			if ( ! $it->isDot()) {
 				$file = $it->key();
 
 				if (
 					preg_match('|Test\.php$|', $file) &&
 					$file !== __FILE__ &&
-					!preg_match('|^All.+?\.php$|', basename($file)) &&
-					($excludes === null || !in_array($file, $excludes))
+					! preg_match('|^All.+?\.php$|', basename($file)) &&
+					($excludes === null || ! in_array($file, $excludes))
 				) {
 					$files[] = $file;
 				}

Plugin/DebugKit/View/Helper/TidyHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
  *
  * @param mixed $cmd
  * @param mixed $out null
- * @return boolean True if successful
+ * @return string|boolean True if successful
  */
 	protected function _exec($cmd, &$out = null) {
 		if (DS === '/') {

Spacing +10 added lines, -10 removed lines

@@ -48,7 +48,7 @@ 
 	public function process($html = '', &$out = '') {
 		$errors = $this->tidyErrors($html, $out);
 
-		if (!$errors) {
+		if ( ! $errors) {
 			return array();
 		}
 		$result = array('Error' => array(), 'Warning' => array(), 'Misc' => array());
@@ -64,7 +64,7 @@ 
 				if (isset($markup[$line - 1])) {
 					$string .= h($markup[$line - 1]);
 				}
-				$string .= '<strong>' . h(@$markup[$line]) . '</strong>';
+				$string .= '<strong>'.h(@$markup[$line]).'</strong>';
 				if (isset($markup[$line + 1])) {
 					$string .= h($markup[$line + 1]);
 				}
@@ -95,8 +95,8 @@ 
 		} elseif ($this->results === null) {
 			$this->process($this->_View->output);
 		}
-		if (!$this->results) {
-			return '<p>' . __d('debug_kit', 'No markup errors found') . '</p>';
+		if ( ! $this->results) {
+			return '<p>'.__d('debug_kit', 'No markup errors found').'</p>';
 		}
 		foreach ($this->results as &$results) {
 			foreach ($results as $type => &$messages) {
@@ -123,19 +123,19 @@ 
 		if (function_exists('tidy_parse_string')) {
 			$tidy = tidy_parse_string($out, array(), 'UTF8');
 			$tidy->cleanRepair();
-			$errors = $tidy->errorBuffer . "\n";
+			$errors = $tidy->errorBuffer."\n";
 			return $errors;
 		}
 
 		// cli
-		$File = new File(rtrim(TMP, DS) . DS . rand() . '.html', true);
+		$File = new File(rtrim(TMP, DS).DS.rand().'.html', true);
 		$File->write($out);
 		$path = $File->pwd();
-		$errors = $path . '.err';
+		$errors = $path.'.err';
 		$this->_exec("tidy -eq -utf8 -f $errors $path");
 		$File->delete();
 
-		if (!file_exists($errors)) {
+		if ( ! file_exists($errors)) {
 			return '';
 		}
 		$Error = new File($errors);
@@ -153,13 +153,13 @@ 
  */
 	protected function _exec($cmd, &$out = null) {
 		if (DS === '/') {
-			$_out = exec($cmd . ' 2>&1', $out, $return);
+			$_out = exec($cmd.' 2>&1', $out, $return);
 		} else {
 			$_out = exec($cmd, $out, $return);
 		}
 
 		if (Configure::read('debug')) {
-			$source = Debugger::trace(array('depth' => 1, 'start' => 2)) . "\n";
+			$source = Debugger::trace(array('depth' => 1, 'start' => 2))."\n";
 			//CakeLog::write('system_calls_' . date('Y-m-d'), "\n" . $source . Debugger::exportVar(compact('cmd','out','return')));
 			//CakeLog::write('system_calls', "\n" . $source . Debugger::exportVar(compact('cmd','out','return')));
 		}

Plugin/DebugKit/View/Helper/ToolbarHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -133,7 +133,7 @@
  *
  * @param string $name Name of the panel you want cached data for
  * @param integer $index
- * @return mixed Boolean false on failure, array of data otherwise.
+ * @return boolean|string Boolean false on failure, array of data otherwise.
  */
 	public function readCache($name, $index = 0) {
 		if (!$this->_cacheEnabled) {

Indentation +11 added lines, -11 removed lines

@@ -1,16 +1,16 @@
 <?php
 /**
- * CakePHP(tm) : Rapid Development Framework (http://cakephp.org)
- * Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
- *
- * Licensed under The MIT License
- * Redistributions of files must retain the above copyright notice.
- *
- * @copyright     Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
- * @link          http://cakephp.org CakePHP(tm) Project
- * @since         DebugKit 0.1
- * @license       http://www.opensource.org/licenses/mit-license.php MIT License
- */
+	 * CakePHP(tm) : Rapid Development Framework (http://cakephp.org)
+	 * Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
+	 *
+	 * Licensed under The MIT License
+	 * Redistributions of files must retain the above copyright notice.
+	 *
+	 * @copyright     Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
+	 * @link          http://cakephp.org CakePHP(tm) Project
+	 * @since         DebugKit 0.1
+	 * @license       http://www.opensource.org/licenses/mit-license.php MIT License
+	 */
 
 App::uses('DebugKitDebugger', 'DebugKit.Lib');
 App::uses('AppHelper', 'View/Helper');

Spacing +12 added lines, -12 removed lines

@@ -54,7 +54,7 @@ 
 			return;
 		}
 
-		if (!isset($options['output'])) {
+		if ( ! isset($options['output'])) {
 			$options['output'] = 'DebugKit.HtmlToolbar';
 		}
 		$className = $options['output'];
@@ -79,7 +79,7 @@ 
  * @return void
  */
 	public function afterLayout($layoutFile) {
-		if (!$this->request->is('requested')) {
+		if ( ! $this->request->is('requested')) {
 			$this->send();
 		}
 	}
@@ -120,10 +120,10 @@ 
  * @return boolean Success of write.
  */
 	public function writeCache($name, $content) {
-		if (!$this->_cacheEnabled) {
+		if ( ! $this->_cacheEnabled) {
 			return false;
 		}
-		$existing = (array)Cache::read($this->_cacheKey, $this->_cacheConfig);
+		$existing = (array) Cache::read($this->_cacheKey, $this->_cacheConfig);
 		$existing[0][$name]['content'] = $content;
 		return Cache::write($this->_cacheKey, $existing, $this->_cacheConfig);
 	}
@@ -136,11 +136,11 @@ 
  * @return mixed Boolean false on failure, array of data otherwise.
  */
 	public function readCache($name, $index = 0) {
-		if (!$this->_cacheEnabled) {
+		if ( ! $this->_cacheEnabled) {
 			return false;
 		}
-		$existing = (array)Cache::read($this->_cacheKey, $this->_cacheConfig);
-		if (!isset($existing[$index][$name]['content'])) {
+		$existing = (array) Cache::read($this->_cacheKey, $this->_cacheConfig);
+		if ( ! isset($existing[$index][$name]['content'])) {
 			return false;
 		}
 		return $existing[$index][$name]['content'];
@@ -164,7 +164,7 @@ 
 		$options += array('explain' => false, 'cache' => true, 'threshold' => 20);
 		$db = ConnectionManager::getDataSource($connection);
 
-		if (!method_exists($db, 'getLog')) {
+		if ( ! method_exists($db, 'getLog')) {
 			return array();
 		}
 
@@ -197,19 +197,19 @@ 
 			}
 			if ($isHtml) {
 				$query['query'] = h($query['query']);
-				if (!empty($query['params']) && is_array($query['params'])) {
+				if ( ! empty($query['params']) && is_array($query['params'])) {
 					$bindParam = $bindType = null;
 					if (preg_match('/.+ :.+/', $query['query'])) {
 						$bindType = true;
 					}
 					foreach ($query['params'] as $bindKey => $bindVal) {
 						if ($bindType === true) {
-							$bindParam .= h($bindKey) . " => " . h($bindVal) . ", ";
+							$bindParam .= h($bindKey)." => ".h($bindVal).", ";
 						} else {
-							$bindParam .= h($bindVal) . ", ";
+							$bindParam .= h($bindVal).", ";
 						}
 					}
-					$query['query'] .= " [ " . rtrim($bindParam, ', ') . " ]";
+					$query['query'] .= " [ ".rtrim($bindParam, ', ')." ]";
 				}
 			}
 			unset($query['params']);

Plugin/DebugKit/Console/Command/BenchmarkShell.php

Spacing +3 added lines, -3 removed lines

@@ -132,7 +132,7 @@ 
 	public function getOptionParser() {
 		$parser = parent::getOptionParser();
 		$parser->description(__d('debug_kit',
-			'Allows you to obtain some rough benchmarking statistics' .
+			'Allows you to obtain some rough benchmarking statistics'.
 			'about a fully qualified URL.'
 		))
 		->addArgument('url', array(
@@ -145,12 +145,12 @@ 
 		))
 		->addOption('t', array(
 			'default' => 100,
-			'help' => __d('debug_kit', 'Maximum total time for all iterations, in seconds.' .
+			'help' => __d('debug_kit', 'Maximum total time for all iterations, in seconds.'.
 				'If a single iteration takes more than the timeout, only one request will be made'
 			)
 		))
 		->epilog(__d('debug_kit',
-			'Example Use: `cake benchmark --n 10 --t 100 http://localhost/testsite`. ' .
+			'Example Use: `cake benchmark --n 10 --t 100 http://localhost/testsite`. '.
 			'<info>Note:</info> this benchmark does not include browser render times.'
 		));
 		return $parser;

Plugin/DebugKit/Console/Command/WhitespaceShell.php

Spacing +9 added lines, -9 removed lines

@@ -30,22 +30,22 @@ 
  */
 	public function main() {
 		$path = APP;
-		if (!empty($this->params['path']) && strpos($this->params['path'], '/') === 0) {
+		if ( ! empty($this->params['path']) && strpos($this->params['path'], '/') === 0) {
 			$path = $this->params['path'];
-		} elseif (!empty($this->params['path'])) {
+		} elseif ( ! empty($this->params['path'])) {
 			$path .= $this->params['path'];
 		}
 		$folder = new Folder($path);
 
 		$r = $folder->findRecursive('.*\.php');
-		$this->out("Checking *.php in " . $path);
+		$this->out("Checking *.php in ".$path);
 		foreach ($r as $file) {
 			$c = file_get_contents($file);
 			if (preg_match('/^[\n\r|\n\r|\n|\r|\s]+\<\?php/', $c)) {
-				$this->out('!!!contains leading whitespaces: ' . $this->shortPath($file));
+				$this->out('!!!contains leading whitespaces: '.$this->shortPath($file));
 			}
 			if (preg_match('/\?\>[\n\r|\n\r|\n|\r|\s]+$/', $c)) {
-				$this->out('!!!contains trailing whitespaces: ' . $this->shortPath($file));
+				$this->out('!!!contains trailing whitespaces: '.$this->shortPath($file));
 			}
 		}
 	}
@@ -58,19 +58,19 @@ 
  */
 	public function trim() {
 		$path = APP;
-		if (!empty($this->params['path']) && strpos($this->params['path'], '/') === 0) {
+		if ( ! empty($this->params['path']) && strpos($this->params['path'], '/') === 0) {
 			$path = $this->params['path'];
-		} elseif (!empty($this->params['path'])) {
+		} elseif ( ! empty($this->params['path'])) {
 			$path .= $this->params['path'];
 		}
 		$folder = new Folder($path);
 
 		$r = $folder->findRecursive('.*\.php');
-		$this->out("Checking *.php in " . $path);
+		$this->out("Checking *.php in ".$path);
 		foreach ($r as $file) {
 			$c = file_get_contents($file);
 			if (preg_match('/^[\n\r|\n\r|\n|\r|\s]+\<\?php/', $c) || preg_match('/\?\>[\n\r|\n\r|\n|\r|\s]+$/', $c)) {
-				$this->out('trimming' . $this->shortPath($file));
+				$this->out('trimming'.$this->shortPath($file));
 				$c = preg_replace('/^[\n\r|\n\r|\n|\r|\s]+\<\?php/', '<?php', $c);
 				$c = preg_replace('/\?\>[\n\r|\n\r|\n|\r|\s]+$/', '?>', $c);
 				file_put_contents($file, $c);

Plugin/DebugKit/Controller/ToolbarAccessController.php

Spacing +2 added lines, -2 removed lines

@@ -102,7 +102,7 @@ 
  */
 	public function sql_explain() {
 		if (
-			!$this->request->is('post') ||
+			! $this->request->is('post') ||
 			empty($this->request->data['log']['sql']) ||
 			empty($this->request->data['log']['ds']) ||
 			empty($this->request->data['log']['hash']) ||
@@ -110,7 +110,7 @@ 
 		) {
 			throw new BadRequestException('Invalid parameters');
 		}
-		$hash = Security::hash($this->request->data['log']['sql'] . $this->request->data['log']['ds'], 'sha1', true);
+		$hash = Security::hash($this->request->data['log']['sql'].$this->request->data['log']['ds'], 'sha1', true);
 		if ($hash !== $this->request->data['log']['hash']) {
 			throw new BadRequestException('Invalid parameters');
 		}

Plugin/DebugKit/Lib/DebugKitDebugger.php

Spacing +1 added lines, -1 removed lines

@@ -208,7 +208,7 @@
  * @return void
  */
 	public static function fireError($data, $links) {
-		$name = $data['error'] . ' - ' . $data['description'];
+		$name = $data['error'].' - '.$data['description'];
 		$message = "{$data['error']} {$data['code']} {$data['description']} on line: {$data['line']} in file: {$data['file']}";
 		FireCake::group($name);
 		FireCake::error($message, $name);

Plugin/DebugKit/Lib/DebugMemory.php

Spacing +3 added lines, -3 removed lines

@@ -55,17 +55,17 @@
  */
 	public static function record($message = null) {
 		$memoryUse = self::getCurrent();
-		if (!$message) {
+		if ( ! $message) {
 			$named = false;
 			$trace = debug_backtrace();
-			$message = Debugger::trimpath($trace[0]['file']) . ' line ' . $trace[0]['line'];
+			$message = Debugger::trimpath($trace[0]['file']).' line '.$trace[0]['line'];
 		}
 		if (isset(self::$_points[$message])) {
 			$originalMessage = $message;
 			$i = 1;
 			while (isset(self::$_points[$message])) {
 				$i++;
-				$message = $originalMessage . ' #' . $i;
+				$message = $originalMessage.' #'.$i;
 			}
 		}
 		self::$_points[$message] = $memoryUse;

Plugin/DebugKit/Lib/DebugTimer.php

Indentation +11 added lines, -11 removed lines

@@ -1,16 +1,16 @@
 <?php
 /**
- * CakePHP(tm) : Rapid Development Framework (http://cakephp.org)
- * Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
- *
- * Licensed under The MIT License
- * Redistributions of files must retain the above copyright notice.
- *
- * @copyright     Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
- * @link          http://cakephp.org CakePHP(tm) Project
- * @since         DebugKit 0.1
- * @license       http://www.opensource.org/licenses/mit-license.php MIT License
- */
+	 * CakePHP(tm) : Rapid Development Framework (http://cakephp.org)
+	 * Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
+	 *
+	 * Licensed under The MIT License
+	 * Redistributions of files must retain the above copyright notice.
+	 *
+	 * @copyright     Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
+	 * @link          http://cakephp.org CakePHP(tm) Project
+	 * @since         DebugKit 0.1
+	 * @license       http://www.opensource.org/licenses/mit-license.php MIT License
+	 */
 
 App::uses('Debugger', 'Utility');
 

Spacing +12 added lines, -12 removed lines

@@ -37,15 +37,15 @@ 
 	public static function start($name = null, $message = null) {
 		$start = microtime(true);
 
-		if (!$name) {
+		if ( ! $name) {
 			$named = false;
 			$calledFrom = debug_backtrace();
-			$_name = $name = Debugger::trimpath($calledFrom[0]['file']) . ' line ' . $calledFrom[0]['line'];
+			$_name = $name = Debugger::trimpath($calledFrom[0]['file']).' line '.$calledFrom[0]['line'];
 		} else {
 			$named = true;
 		}
 
-		if (!$message) {
+		if ( ! $message) {
 			$message = $name;
 		}
 
@@ -53,11 +53,11 @@ 
 		$i = 1;
 		while (isset(self::$_timers[$name])) {
 			$i++;
-			$name = $_name . ' #' . $i;
+			$name = $_name.' #'.$i;
 		}
 
 		if ($i > 1) {
-			$message .= ' #' . $i;
+			$message .= ' #'.$i;
 		}
 
 		self::$_timers[$name] = array(
@@ -78,10 +78,10 @@ 
  */
 	public static function stop($name = null) {
 		$end = microtime(true);
-		if (!$name) {
+		if ( ! $name) {
 			$names = array_reverse(array_keys(self::$_timers));
 			foreach ($names as $name) {
-				if (!empty(self::$_timers[$name]['end'])) {
+				if ( ! empty(self::$_timers[$name]['end'])) {
 					continue;
 				}
 				if (empty(self::$_timers[$name]['named'])) {
@@ -96,10 +96,10 @@ 
 					break;
 				}
 				$i++;
-				$name = $_name . ' #' . $i;
+				$name = $_name.' #'.$i;
 			}
 		}
-		if (!isset(self::$_timers[$name])) {
+		if ( ! isset(self::$_timers[$name])) {
 			return false;
 		}
 		self::$_timers[$name]['end'] = $end;
@@ -118,7 +118,7 @@ 
 		$now = microtime(true);
 
 		$times = array();
-		if (!empty(self::$_timers)) {
+		if ( ! empty(self::$_timers)) {
 			$firstTimer = reset(self::$_timers);
 			$_end = $firstTimer['start'];
 		} else {
@@ -132,7 +132,7 @@ 
 			'named' => null
 		);
 		foreach (self::$_timers as $name => $timer) {
-			if (!isset($timer['end'])) {
+			if ( ! isset($timer['end'])) {
 				$timer['end'] = $now;
 			}
 			$times[$name] = array_merge($timer, array(
@@ -165,7 +165,7 @@ 
  * @return float number of seconds elapsed for timer name, 0 on missing key
  */
 	public static function elapsedTime($name = 'default', $precision = 5) {
-		if (!isset(self::$_timers[$name]['start']) || !isset(self::$_timers[$name]['end'])) {
+		if ( ! isset(self::$_timers[$name]['start']) || ! isset(self::$_timers[$name]['end'])) {
 			return 0;
 		}
 		return round(self::$_timers[$name]['end'] - self::$_timers[$name]['start'], $precision);