reginaldojunior / winners

lib/Cake/TestSuite/CakeTestSuiteCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 /**
  * Construct method
  *
- * @param mixed $loader
+ * @param string $loader
  * @param array $params list of options to be used for this run
  * @throws MissingTestLoaderException When a loader class could not be found.
  */

Spacing +6 added lines, -6 removed lines

@@ -40,7 +40,7 @@ 
  * @throws MissingTestLoaderException When a loader class could not be found.
  */
 	public function __construct($loader, $params = array()) {
-		if ($loader && !class_exists($loader)) {
+		if ($loader && ! class_exists($loader)) {
 			throw new MissingTestLoaderException(array('class' => $loader));
 		}
 		$this->arguments['loader'] = $loader;
@@ -95,13 +95,13 @@ 
 		try {
 			$result = $runner->doRun($suite, $this->arguments);
 		} catch (PHPUnit_Framework_Exception $e) {
-			print $e->getMessage() . "\n";
+			print $e->getMessage()."\n";
 		}
 
 		if ($exit) {
 			if (isset($result) && $result->wasSuccessful()) {
 				exit(PHPUnit_TextUI_TestRunner::SUCCESS_EXIT);
-			} elseif (!isset($result) || $result->errorCount() > 0) {
+			} elseif ( ! isset($result) || $result->errorCount() > 0) {
 				exit(PHPUnit_TextUI_TestRunner::EXCEPTION_EXIT);
 			}
 			exit(PHPUnit_TextUI_TestRunner::FAILURE_EXIT);
@@ -138,13 +138,13 @@ 
 		$object = null;
 
 		$reporter = ucwords($reporter);
-		$coreClass = 'Cake' . $reporter . 'Reporter';
+		$coreClass = 'Cake'.$reporter.'Reporter';
 		App::uses($coreClass, 'TestSuite/Reporter');
 
-		$appClass = $reporter . 'Reporter';
+		$appClass = $reporter.'Reporter';
 		App::uses($appClass, 'TestSuite/Reporter');
 
-		if (!class_exists($appClass)) {
+		if ( ! class_exists($appClass)) {
 			$object = new $coreClass(null, $this->_params);
 		} else {
 			$object = new $appClass(null, $this->_params);

lib/Cake/TestSuite/Coverage/BaseCoverageReport.php

Doc Comments +2 added lines, -2 removed lines

@@ -95,7 +95,7 @@ 
 /**
  * Gets the base path that the files we are interested in live in.
  *
- * @return void
+ * @return string
  */
 	public function getPathFilter() {
 		$path = ROOT . DS;
@@ -136,7 +136,7 @@ 
  *
  * @param array $fileLines
  * @param array $coverageData
- * @return array Array of covered, total lines.
+ * @return integer[] Array of covered, total lines.
  */
 	protected function _calculateCoveredLines($fileLines, $coverageData) {
 		$covered = $total = 0;

Spacing +4 added lines, -4 removed lines

@@ -98,9 +98,9 @@ 
  * @return void
  */
 	public function getPathFilter() {
-		$path = ROOT . DS;
+		$path = ROOT.DS;
 		if ($this->appTest) {
-			$path .= APP_DIR . DS;
+			$path .= APP_DIR.DS;
 		} elseif ($this->pluginTest) {
 			$path = App::pluginPath($this->pluginTest);
 		} else {
@@ -146,10 +146,10 @@ 
 		unset($fileLines[0]);
 
 		foreach ($fileLines as $lineno => $line) {
-			if (!isset($coverageData[$lineno])) {
+			if ( ! isset($coverageData[$lineno])) {
 				continue;
 			}
-			if (is_array($coverageData[$lineno]) && !empty($coverageData[$lineno])) {
+			if (is_array($coverageData[$lineno]) && ! empty($coverageData[$lineno])) {
 				$covered++;
 				$total++;
 			} elseif ($coverageData[$lineno] === -1 || $coverageData[$lineno] === array()) {

lib/Cake/TestSuite/Coverage/HtmlCoverageReport.php

Doc Comments +4 added lines, -3 removed lines

@@ -108,6 +108,7 @@ 
  * Guess the class name the test was for based on the test case filename.
  *
  * @param ReflectionClass $testReflection.
+ * @param ReflectionClass $testReflection
  * @return string Possible test subject name.
  */
 	protected function _guessSubjectName($testReflection) {
@@ -127,7 +128,7 @@ 
  * @param integer $linenumber
  * @param string $class
  * @param array $coveringTests
- * @return void
+ * @return string
  */
 	protected function _paintLine($line, $linenumber, $class, $coveringTests) {
 		$coveredBy = '';
@@ -150,7 +151,7 @@ 
 /**
  * generate some javascript for the coverage report.
  *
- * @return void
+ * @return string
  */
 	public function coverageScript() {
 		return <<<HTML
@@ -199,7 +200,7 @@ 
 /**
  * Generate an HTML snippet for coverage footers
  *
- * @return void
+ * @return string
  */
 	public function coverageFooter() {
 		return "</pre></div></div>";

Spacing +4 added lines, -4 removed lines

@@ -77,7 +77,7 @@ 
 		foreach ($fileLines as $lineno => $line) {
 			$class = 'ignored';
 			$coveringTests = array();
-			if (!empty($coverageData[$lineno]) && is_array($coverageData[$lineno])) {
+			if ( ! empty($coverageData[$lineno]) && is_array($coverageData[$lineno])) {
 				$coveringTests = array();
 				foreach ($coverageData[$lineno] as $test) {
 					$class = (is_array($test) && isset($test['id'])) ? $test['id'] : $test;
@@ -113,7 +113,7 @@ 
 	protected function _guessSubjectName($testReflection) {
 		$basename = basename($testReflection->getFilename());
 		if (strpos($basename, '.test') !== false) {
-			list($subject, ) = explode('.', $basename, 2);
+			list($subject,) = explode('.', $basename, 2);
 			return $subject;
 		}
 		$subject = str_replace('Test.php', '', $basename);
@@ -131,10 +131,10 @@ 
  */
 	protected function _paintLine($line, $linenumber, $class, $coveringTests) {
 		$coveredBy = '';
-		if (!empty($coveringTests)) {
+		if ( ! empty($coveringTests)) {
 			$coveredBy = "Covered by:\n";
 			foreach ($coveringTests as $test) {
-				$coveredBy .= $test . "\n";
+				$coveredBy .= $test."\n";
 			}
 		}
 

lib/Cake/TestSuite/Fixture/CakeTestFixture.php

Doc Comments +1 added lines, -1 removed lines

@@ -264,7 +264,7 @@
  * of this fixture could be executed successfully.
  *
  * @param DboSource $db An instance of the database into which the records will be inserted
- * @return boolean on success or if there are no records to insert, or false on failure
+ * @return boolean|null on success or if there are no records to insert, or false on failure
  * @throws CakeException if counts of values and fields do not match.
  */
 	public function insert($db) {

Spacing +12 added lines, -12 removed lines

@@ -95,7 +95,7 @@ 
 			}
 		}
 		$connection = 'test';
-		if (!empty($this->useDbConfig)) {
+		if ( ! empty($this->useDbConfig)) {
 			$connection = $this->useDbConfig;
 			if (strpos($connection, 'test') !== 0) {
 				$message = __d(
@@ -127,8 +127,8 @@ 
 			$this->Schema->connection = $import['connection'];
 			if (isset($import['model'])) {
 				list($plugin, $modelClass) = pluginSplit($import['model'], true);
-				App::uses($modelClass, $plugin . 'Model');
-				if (!class_exists($modelClass)) {
+				App::uses($modelClass, $plugin.'Model');
+				if ( ! class_exists($modelClass)) {
 					throw new MissingModelException(array('class' => $modelClass));
 				}
 				$model = new $modelClass(null, null, $import['connection']);
@@ -155,7 +155,7 @@ 
 				ClassRegistry::flush();
 			}
 
-			if (!empty($db->config['prefix']) && strpos($this->table, $db->config['prefix']) === 0) {
+			if ( ! empty($db->config['prefix']) && strpos($this->table, $db->config['prefix']) === 0) {
 				$this->table = str_replace($db->config['prefix'], '', $this->table);
 			}
 
@@ -172,17 +172,17 @@ 
 				);
 				$records = $db->fetchAll($db->buildStatement($query, $model), false, $model->alias);
 
-				if ($records !== false && !empty($records)) {
-					$this->records = Hash::extract($records, '{n}.' . $model->alias);
+				if ($records !== false && ! empty($records)) {
+					$this->records = Hash::extract($records, '{n}.'.$model->alias);
 				}
 			}
 		}
 
-		if (!isset($this->table)) {
+		if ( ! isset($this->table)) {
 			$this->table = Inflector::underscore(Inflector::pluralize($this->name));
 		}
 
-		if (!isset($this->primaryKey) && isset($this->fields['id'])) {
+		if ( ! isset($this->primaryKey) && isset($this->fields['id'])) {
 			$this->primaryKey = 'id';
 		}
 	}
@@ -194,7 +194,7 @@ 
  * @return boolean True on success, false on failure
  */
 	public function create($db) {
-		if (!isset($this->fields) || empty($this->fields)) {
+		if ( ! isset($this->fields) || empty($this->fields)) {
 			return false;
 		}
 
@@ -204,7 +204,7 @@ 
 
 				if (is_string($args)) {
 					$type = $args;
-				} elseif (!empty($args['type'])) {
+				} elseif ( ! empty($args['type'])) {
 					$type = $args['type'];
 				} else {
 					continue;
@@ -268,9 +268,9 @@ 
  * @throws CakeException if counts of values and fields do not match.
  */
 	public function insert($db) {
-		if (!isset($this->_insert)) {
+		if ( ! isset($this->_insert)) {
 			$values = array();
-			if (isset($this->records) && !empty($this->records)) {
+			if (isset($this->records) && ! empty($this->records)) {
 				$fields = array();
 				foreach ($this->records as $record) {
 					$fields = array_merge($fields, array_keys(array_intersect_key($record, $this->fields)));

lib/Cake/TestSuite/Fixture/CakeTestModel.php

Doc Comments +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
  *
  * @param array $data
  * @param boolean|array $validate
- * @param array $fieldList
+ * @param string[] $fieldList
  * @return void
  */
 	public function save($data = null, $validate = true, $fieldList = array()) {

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
  */
 	public function __construct($id = false, $table = null, $ds = null) {
 		parent::__construct($id, $table, $ds);
-		$this->order = array($this->alias . '.' . $this->primaryKey => 'ASC');
+		$this->order = array($this->alias.'.'.$this->primaryKey => 'ASC');
 	}
 
 /**

lib/Cake/TestSuite/Reporter/CakeHtmlReporter.php

Doc Comments -1 removed lines

@@ -295,7 +295,6 @@
 /**
  * Paints a PHP exception.
  *
- * @param Exception $exception Exception to display.
  * @param mixed $test
  * @return void
  */

Spacing +42 added lines, -42 removed lines

@@ -46,8 +46,8 @@ 
  * @return void
  */
 	public function sendContentType() {
-		if (!headers_sent()) {
-			header('Content-Type: text/html; charset=' . Configure::read('App.encoding'));
+		if ( ! headers_sent()) {
+			header('Content-Type: text/html; charset='.Configure::read('App.encoding'));
 		}
 	}
 
@@ -59,7 +59,7 @@ 
 	public function paintDocumentStart() {
 		ob_start();
 		$baseDir = $this->params['baseDir'];
-		include CAKE . 'TestSuite' . DS . 'templates' . DS . 'header.php';
+		include CAKE.'TestSuite'.DS.'templates'.DS.'header.php';
 	}
 
 /**
@@ -69,10 +69,10 @@ 
  * @return void
  */
 	public function paintTestMenu() {
-		$cases = $this->baseUrl() . '?show=cases';
+		$cases = $this->baseUrl().'?show=cases';
 		$plugins = App::objects('plugin', null, false);
 		sort($plugins);
-		include CAKE . 'TestSuite' . DS . 'templates' . DS . 'menu.php';
+		include CAKE.'TestSuite'.DS.'templates'.DS.'menu.php';
 	}
 
 /**
@@ -91,8 +91,8 @@ 
 			$buffer = "<h3>Core Test Cases:</h3>\n<ul>";
 			$urlExtra = '&core=true';
 		} elseif ($plugin) {
-			$buffer = "<h3>" . Inflector::humanize($plugin) . " Test Cases:</h3>\n<ul>";
-			$urlExtra = '&plugin=' . $plugin;
+			$buffer = "<h3>".Inflector::humanize($plugin)." Test Cases:</h3>\n<ul>";
+			$urlExtra = '&plugin='.$plugin;
 		}
 
 		if (count($testCases) < 1) {
@@ -103,7 +103,7 @@ 
 			$title = explode(DS, str_replace('.test.php', '', $testCase));
 			$title[count($title) - 1] = Inflector::camelize($title[count($title) - 1]);
 			$title = implode(' / ', $title);
-				$buffer .= "<li><a href='" . $this->baseUrl() . "?case=" . urlencode($testCase) . $urlExtra . "'>" . $title . "</a></li>\n";
+				$buffer .= "<li><a href='".$this->baseUrl()."?case=".urlencode($testCase).$urlExtra."'>".$title."</a></li>\n";
 		}
 		$buffer .= "</ul>\n";
 		echo $buffer;
@@ -117,9 +117,9 @@ 
  * @return void
  */
 	public function sendNoCacheHeaders() {
-		if (!headers_sent()) {
+		if ( ! headers_sent()) {
 			header("Expires: Mon, 26 Jul 1997 05:00:00 GMT");
-			header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
+			header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
 			header("Cache-Control: no-store, no-cache, must-revalidate");
 			header("Cache-Control: post-check=0, pre-check=0", false);
 			header("Pragma: no-cache");
@@ -140,16 +140,16 @@ 
 		echo "<div style=\"";
 		echo "padding: 8px; margin: 1em 0; background-color: $colour; color: white;";
 		echo "\">";
-		echo ($result->count() - $result->skippedCount()) . "/" . $result->count();
+		echo ($result->count() - $result->skippedCount())."/".$result->count();
 		echo " test methods complete:\n";
-		echo "<strong>" . count($result->passed()) . "</strong> passes, ";
-		echo "<strong>" . $result->failureCount() . "</strong> fails, ";
-		echo "<strong>" . $this->numAssertions . "</strong> assertions and ";
-		echo "<strong>" . $result->errorCount() . "</strong> exceptions.";
+		echo "<strong>".count($result->passed())."</strong> passes, ";
+		echo "<strong>".$result->failureCount()."</strong> fails, ";
+		echo "<strong>".$this->numAssertions."</strong> assertions and ";
+		echo "<strong>".$result->errorCount()."</strong> exceptions.";
 		echo "</div>\n";
 		echo '<div style="padding:0 0 5px;">';
-		echo '<p><strong>Time:</strong> ' . $result->time() . ' seconds</p>';
-		echo '<p><strong>Peak memory:</strong> ' . number_format(memory_get_peak_usage()) . ' bytes</p>';
+		echo '<p><strong>Time:</strong> '.$result->time().' seconds</p>';
+		echo '<p><strong>Peak memory:</strong> '.number_format(memory_get_peak_usage()).' bytes</p>';
 		echo $this->_paintLinks();
 		echo '</div>';
 		if (isset($this->params['codeCoverage']) && $this->params['codeCoverage']) {
@@ -186,25 +186,25 @@ 
  */
 	protected function _paintLinks() {
 		$show = $query = array();
-		if (!empty($this->params['case'])) {
+		if ( ! empty($this->params['case'])) {
 			$show['show'] = 'cases';
 		}
 
-		if (!empty($this->params['core'])) {
+		if ( ! empty($this->params['core'])) {
 			$show['core'] = $query['core'] = 'true';
 		}
-		if (!empty($this->params['plugin'])) {
+		if ( ! empty($this->params['plugin'])) {
 			$show['plugin'] = $query['plugin'] = $this->params['plugin'];
 		}
-		if (!empty($this->params['case'])) {
+		if ( ! empty($this->params['case'])) {
 			$query['case'] = $this->params['case'];
 		}
 		$show = $this->_queryString($show);
 		$query = $this->_queryString($query);
 
-		echo "<p><a href='" . $this->baseUrl() . $show . "'>Run more tests</a> | <a href='" . $this->baseUrl() . $query . "&amp;show_passes=1'>Show Passes</a> | \n";
-		echo "<a href='" . $this->baseUrl() . $query . "&amp;debug=1'>Enable Debug Output</a> | \n";
-		echo "<a href='" . $this->baseUrl() . $query . "&amp;code_coverage=true'>Analyze Code Coverage</a></p>\n";
+		echo "<p><a href='".$this->baseUrl().$show."'>Run more tests</a> | <a href='".$this->baseUrl().$query."&amp;show_passes=1'>Show Passes</a> | \n";
+		echo "<a href='".$this->baseUrl().$query."&amp;debug=1'>Enable Debug Output</a> | \n";
+		echo "<a href='".$this->baseUrl().$query."&amp;code_coverage=true'>Analyze Code Coverage</a></p>\n";
 	}
 
 /**
@@ -230,7 +230,7 @@ 
  */
 	public function paintDocumentEnd() {
 		$baseDir = $this->params['baseDir'];
-		include CAKE . 'TestSuite' . DS . 'templates' . DS . 'footer.php';
+		include CAKE.'TestSuite'.DS.'templates'.DS.'footer.php';
 		if (ob_get_length()) {
 			ob_end_flush();
 		}
@@ -248,7 +248,7 @@ 
  */
 	public function paintFail($message, $test) {
 		$trace = $this->_getStackTrace($message);
-		$testName = get_class($test) . '(' . $test->getName() . ')';
+		$testName = get_class($test).'('.$test->getName().')';
 
 		$actualMsg = $expectedMsg = null;
 		if (method_exists($message, 'getComparisonFailure')) {
@@ -261,15 +261,15 @@ 
 
 		echo "<li class='fail'>\n";
 		echo "<span>Failed</span>";
-		echo "<div class='msg'><pre>" . $this->_htmlEntities($message->toString());
+		echo "<div class='msg'><pre>".$this->_htmlEntities($message->toString());
 
 		if ((is_string($actualMsg) && is_string($expectedMsg)) || (is_array($actualMsg) && is_array($expectedMsg))) {
-			echo "<br />" . PHPUnit_Util_Diff::diff($expectedMsg, $actualMsg);
+			echo "<br />".PHPUnit_Util_Diff::diff($expectedMsg, $actualMsg);
 		}
 
 		echo "</pre></div>\n";
-		echo "<div class='msg'>" . __d('cake_dev', 'Test case: %s', $testName) . "</div>\n";
-		echo "<div class='msg'>" . __d('cake_dev', 'Stack trace:') . '<br />' . $trace . "</div>\n";
+		echo "<div class='msg'>".__d('cake_dev', 'Test case: %s', $testName)."</div>\n";
+		echo "<div class='msg'>".__d('cake_dev', 'Stack trace:').'<br />'.$trace."</div>\n";
 		echo "</li>\n";
 	}
 
@@ -287,7 +287,7 @@ 
 			echo "<li class='pass'>\n";
 			echo "<span>Passed</span> ";
 
-			echo "<br />" . $this->_htmlEntities($test->getName()) . " ($time seconds)\n";
+			echo "<br />".$this->_htmlEntities($test->getName())." ($time seconds)\n";
 			echo "</li>\n";
 		}
 	}
@@ -301,14 +301,14 @@ 
  */
 	public function paintException($message, $test) {
 		$trace = $this->_getStackTrace($message);
-		$testName = get_class($test) . '(' . $test->getName() . ')';
+		$testName = get_class($test).'('.$test->getName().')';
 
 		echo "<li class='fail'>\n";
-		echo "<span>" . get_class($message) . "</span>";
+		echo "<span>".get_class($message)."</span>";
 
-		echo "<div class='msg'>" . $this->_htmlEntities($message->getMessage()) . "</div>\n";
-		echo "<div class='msg'>" . __d('cake_dev', 'Test case: %s', $testName) . "</div>\n";
-		echo "<div class='msg'>" . __d('cake_dev', 'Stack trace:') . '<br />' . $trace . "</div>\n";
+		echo "<div class='msg'>".$this->_htmlEntities($message->getMessage())."</div>\n";
+		echo "<div class='msg'>".__d('cake_dev', 'Test case: %s', $testName)."</div>\n";
+		echo "<div class='msg'>".__d('cake_dev', 'Stack trace:').'<br />'.$trace."</div>\n";
 		echo "</li>\n";
 	}
 
@@ -322,7 +322,7 @@ 
 	public function paintSkip($message, $test) {
 		echo "<li class='skipped'>\n";
 		echo "<span>Skipped</span> ";
-		echo $test->getName() . ': ' . $this->_htmlEntities($message->getMessage());
+		echo $test->getName().': '.$this->_htmlEntities($message->getMessage());
 		echo "</li>\n";
 	}
 
@@ -333,7 +333,7 @@ 
  * @return void
  */
 	public function paintFormattedMessage($message) {
-		echo '<pre>' . $this->_htmlEntities($message) . '</pre>';
+		echo '<pre>'.$this->_htmlEntities($message).'</pre>';
 	}
 
 /**
@@ -357,9 +357,9 @@ 
 		$out = array();
 		foreach ($trace as $frame) {
 			if (isset($frame['file']) && isset($frame['line'])) {
-				$out[] = $frame['file'] . ' : ' . $frame['line'];
+				$out[] = $frame['file'].' : '.$frame['line'];
 			} elseif (isset($frame['class']) && isset($frame['function'])) {
-				$out[] = $frame['class'] . '::' . $frame['function'];
+				$out[] = $frame['class'].'::'.$frame['function'];
 			} else {
 				$out[] = '[internal]';
 			}
@@ -374,10 +374,10 @@ 
  * @return void
  */
 	public function startTestSuite(PHPUnit_Framework_TestSuite $suite) {
-		if (!$this->_headerSent) {
+		if ( ! $this->_headerSent) {
 			echo $this->paintHeader();
 		}
-		echo '<h2>' . __d('cake_dev', 'Running  %s', $suite->getName()) . '</h2>';
+		echo '<h2>'.__d('cake_dev', 'Running  %s', $suite->getName()).'</h2>';
 	}
 
 }

lib/Cake/Utility/CakeNumber.php

Doc Comments +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
  *
  * @param float $value A floating point number.
  * @param integer $precision The precision of the returned number.
- * @return float Formatted float.
+ * @return string Formatted float.
  * @link http://book.cakephp.org/2.0/en/core-libraries/helpers/number.html#NumberHelper::precision
  */
 	public static function precision($value, $precision = 3) {

Indentation +17 added lines, -17 removed lines

@@ -1,22 +1,22 @@
 <?php
 /**
- * CakeNumber Utility.
- *
- * Methods to make numbers more readable.
- *
- * CakePHP(tm) : Rapid Development Framework (http://cakephp.org)
- * Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
- *
- * Licensed under The MIT License
- * For full copyright and license information, please see the LICENSE.txt
- * 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
- * @package       Cake.Utility
- * @since         CakePHP(tm) v 0.10.0.1076
- * @license       http://www.opensource.org/licenses/mit-license.php MIT License
- */
+		 * CakeNumber Utility.
+		 *
+		 * Methods to make numbers more readable.
+		 *
+		 * CakePHP(tm) : Rapid Development Framework (http://cakephp.org)
+		 * Copyright (c) Cake Software Foundation, Inc. (http://cakefoundation.org)
+		 *
+		 * Licensed under The MIT License
+		 * For full copyright and license information, please see the LICENSE.txt
+		 * 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
+		 * @package       Cake.Utility
+		 * @since         CakePHP(tm) v 0.10.0.1076
+		 * @license       http://www.opensource.org/licenses/mit-license.php MIT License
+		 */
 
 /**
  * Number helper library.

Spacing +16 added lines, -16 removed lines

@@ -137,7 +137,7 @@ 
  */
 	public static function fromReadableSize($size, $default = false) {
 		if (ctype_digit($size)) {
-			return (int)$size;
+			return (int) $size;
 		}
 		$size = strtoupper($size);
 
@@ -154,7 +154,7 @@ 
 
 		if (substr($size, -1) === 'B' && ctype_digit(substr($size, 0, -1))) {
 			$size = substr($size, 0, -1);
-			return (int)$size;
+			return (int) $size;
 		}
 
 		if ($default !== false) {
@@ -181,7 +181,7 @@ 
 		if ($options['multiply']) {
 			$value *= 100;
 		}
-		return self::precision($value, $precision) . '%';
+		return self::precision($value, $precision).'%';
 	}
 
 /**
@@ -202,15 +202,15 @@ 
 		$separators = array(',', '.', '-', ':');
 
 		$before = $after = null;
-		if (is_string($options) && !in_array($options, $separators)) {
+		if (is_string($options) && ! in_array($options, $separators)) {
 			$before = $options;
 		}
 		$thousands = ',';
-		if (!is_array($options) && in_array($options, $separators)) {
+		if ( ! is_array($options) && in_array($options, $separators)) {
 			$thousands = $options;
 		}
 		$decimals = '.';
-		if (!is_array($options) && in_array($options, $separators)) {
+		if ( ! is_array($options) && in_array($options, $separators)) {
 			$decimals = $options;
 		}
 
@@ -222,7 +222,7 @@ 
 		}
 
 		$value = self::_numberFormat($value, $places, '.', '');
-		$out = $before . self::_numberFormat($value, $places, $decimals, $thousands) . $after;
+		$out = $before.self::_numberFormat($value, $places, $decimals, $thousands).$after;
 
 		if ($escape) {
 			return h($out);
@@ -251,7 +251,7 @@ 
 		$places = isset($options['places']) ? $options['places'] : 0;
 		$value = self::_numberFormat($value, $places, '.', '');
 		$sign = $value > 0 ? '+' : '';
-		$options['before'] = isset($options['before']) ? $options['before'] . $sign : $sign;
+		$options['before'] = isset($options['before']) ? $options['before'].$sign : $sign;
 		return self::format($value, $options);
 	}
 
@@ -265,7 +265,7 @@ 
  * @return string
  */
 	protected static function _numberFormat($value, $places = 0, $decimals = '.', $thousands = ',') {
-		if (!isset(self::$_numberFormatSupport)) {
+		if ( ! isset(self::$_numberFormatSupport)) {
 			self::$_numberFormatSupport = version_compare(PHP_VERSION, '5.4.0', '>=');
 		}
 		if (self::$_numberFormatSupport) {
@@ -339,15 +339,15 @@ 
 		if (isset($options['before']) && $options['before'] !== '') {
 			$options['wholeSymbol'] = $options['before'];
 		}
-		if (isset($options['after']) && !$options['after'] !== '') {
+		if (isset($options['after']) && ! $options['after'] !== '') {
 			$options['fractionSymbol'] = $options['after'];
 		}
 
 		$result = $options['before'] = $options['after'] = null;
 
 		$symbolKey = 'whole';
-		$value = (float)$value;
-		if (!$value) {
+		$value = (float) $value;
+		if ( ! $value) {
 			if ($options['zero'] !== 0) {
 				return $options['zero'];
 			}
@@ -360,17 +360,17 @@ 
 			}
 		}
 
-		$position = $options[$symbolKey . 'Position'] !== 'after' ? 'before' : 'after';
-		$options[$position] = $options[$symbolKey . 'Symbol'];
+		$position = $options[$symbolKey.'Position'] !== 'after' ? 'before' : 'after';
+		$options[$position] = $options[$symbolKey.'Symbol'];
 
 		$abs = abs($value);
 		$result = self::format($abs, $options);
 
 		if ($value < 0) {
 			if ($options['negative'] === '()') {
-				$result = '(' . $result . ')';
+				$result = '('.$result.')';
 			} else {
-				$result = $options['negative'] . $result;
+				$result = $options['negative'].$result;
 			}
 		}
 		return $result;

lib/Cake/Utility/CakeTime.php

Doc Comments +5 added lines, -5 removed lines

@@ -295,7 +295,7 @@ 
 /**
  * Returns server's offset from GMT in seconds.
  *
- * @return integer Offset
+ * @return string Offset
  * @link http://book.cakephp.org/2.0/en/core-libraries/helpers/time.html#TimeHelper::serverOffset
  */
 	public static function serverOffset() {
@@ -623,7 +623,7 @@ 
  * @param integer|string|DateTime $dateString UNIX timestamp, strtotime() valid string or DateTime object
  * @param string|DateTimeZone $timezone Timezone string or DateTimeZone object
  * @param string $format date format string
- * @return mixed Formatted date
+ * @return false|string Formatted date
  * @link http://book.cakephp.org/2.0/en/core-libraries/helpers/time.html#TimeHelper::toServer
  */
 	public static function toServer($dateString, $timezone = null, $format = 'Y-m-d H:i:s') {
@@ -939,7 +939,7 @@ 
 /**
  * Returns true if specified datetime was within the interval specified, else false.
  *
- * @param string|integer $timeInterval the numeric value with space then time type.
+ * @param string $timeInterval the numeric value with space then time type.
  *    Example of valid types: 6 hours, 2 days, 1 minute.
  * @param integer|string|DateTime $dateString UNIX timestamp, strtotime() valid string or DateTime object
  * @param string|DateTimeZone $timezone Timezone string or DateTimeZone object
@@ -962,7 +962,7 @@ 
 /**
  * Returns true if specified datetime is within the interval specified, else false.
  *
- * @param string|integer $timeInterval the numeric value with space then time type.
+ * @param string $timeInterval the numeric value with space then time type.
  *    Example of valid types: 6 hours, 2 days, 1 minute.
  * @param integer|string|DateTime $dateString UNIX timestamp, strtotime() valid string or DateTime object
  * @param string|DateTimeZone $timezone Timezone string or DateTimeZone object
@@ -1019,7 +1019,7 @@ 
  *   CakeTime::format('2012-02-15 23:01:01', '%c', 'N/A', 'America/New_York'); // converts passed date to timezone
  * }}}
  *
- * @param integer|string|DateTime $date UNIX timestamp, strtotime() valid string or DateTime object (or a date format string)
+ * @param string $date UNIX timestamp, strtotime() valid string or DateTime object (or a date format string)
  * @param integer|string|DateTime $format date format string (or UNIX timestamp, strtotime() valid string or DateTime object)
  * @param boolean|string $default if an invalid date is passed it will output supplied default value. Pass false if you want raw conversion value
  * @param string|DateTimeZone $timezone Timezone string or DateTimeZone object

Spacing +48 added lines, -48 removed lines

@@ -132,7 +132,7 @@ 
  * @link http://book.cakephp.org/2.0/en/core-libraries/helpers/time.html#TimeHelper::convertSpecifiers
  */
 	public static function convertSpecifiers($format, $time = null) {
-		if (!$time) {
+		if ( ! $time) {
 			$time = time();
 		}
 		self::$_time = $time;
@@ -176,7 +176,7 @@ 
 				}
 				$day = date('j', self::$_time);
 				if ($day < 10) {
-					$day = ' ' . $day;
+					$day = ' '.$day;
 				}
 				return $day;
 			case 'eS' :
@@ -249,16 +249,16 @@ 
 		if ($serverTimezone === null || (date_default_timezone_get() !== $serverTimezone->getName())) {
 			$serverTimezone = new DateTimeZone(date_default_timezone_get());
 		}
-		$serverOffset = $serverTimezone->getOffset(new DateTime('@' . $serverTime));
+		$serverOffset = $serverTimezone->getOffset(new DateTime('@'.$serverTime));
 		$gmtTime = $serverTime - $serverOffset;
 		if (is_numeric($timezone)) {
 			$userOffset = $timezone * (60 * 60);
 		} else {
 			$timezone = self::timezone($timezone);
-			$userOffset = $timezone->getOffset(new DateTime('@' . $gmtTime));
+			$userOffset = $timezone->getOffset(new DateTime('@'.$gmtTime));
 		}
 		$userTime = $gmtTime + $userOffset;
-		return (int)$userTime;
+		return (int) $userTime;
 	}
 
 /**
@@ -328,9 +328,9 @@ 
 		) {
 			$clone = clone $dateString;
 			$clone->setTimezone(new DateTimeZone(date_default_timezone_get()));
-			$date = (int)$clone->format('U') + $clone->getOffset();
+			$date = (int) $clone->format('U') + $clone->getOffset();
 		} elseif ($dateString instanceof DateTime) {
-			$date = (int)$dateString->format('U');
+			$date = (int) $dateString->format('U');
 		} else {
 			$date = strtotime($dateString);
 		}
@@ -362,12 +362,12 @@ 
  * @link http://book.cakephp.org/2.0/en/core-libraries/helpers/time.html#TimeHelper::nice
  */
 	public static function nice($dateString = null, $timezone = null, $format = null) {
-		if (!$dateString) {
+		if ( ! $dateString) {
 			$dateString = time();
 		}
 		$date = self::fromString($dateString, $timezone);
 
-		if (!$format) {
+		if ( ! $format) {
 			$format = self::$niceFormat;
 		}
 		return self::_strftime(self::convertSpecifiers($format, $date), $date);
@@ -389,7 +389,7 @@ 
  * @link http://book.cakephp.org/2.0/en/core-libraries/helpers/time.html#TimeHelper::niceShort
  */
 	public static function niceShort($dateString = null, $timezone = null) {
-		if (!$dateString) {
+		if ( ! $dateString) {
 			$dateString = time();
 		}
 		$date = self::fromString($dateString, $timezone);
@@ -422,7 +422,7 @@ 
 		}
 
 		$y = '';
-		if (!self::isThisYear($date)) {
+		if ( ! self::isThisYear($date)) {
 			$y = ' %Y';
 		}
 		return self::_strftime(self::convertSpecifiers("%b %eS{$y}, %H:%M", $date), $date);
@@ -441,8 +441,8 @@ 
 	public static function daysAsSql($begin, $end, $fieldName, $timezone = null) {
 		$begin = self::fromString($begin, $timezone);
 		$end = self::fromString($end, $timezone);
-		$begin = date('Y-m-d', $begin) . ' 00:00:00';
-		$end = date('Y-m-d', $end) . ' 23:59:59';
+		$begin = date('Y-m-d', $begin).' 00:00:00';
+		$end = date('Y-m-d', $end).' 23:59:59';
 
 		return "($fieldName >= '$begin') AND ($fieldName <= '$end')";
 	}
@@ -589,13 +589,13 @@ 
 		$year = date('Y', $time);
 		switch ($date) {
 			case 1:
-				return array($year . '-01-01', $year . '-03-31');
+				return array($year.'-01-01', $year.'-03-31');
 			case 2:
-				return array($year . '-04-01', $year . '-06-30');
+				return array($year.'-04-01', $year.'-06-30');
 			case 3:
-				return array($year . '-07-01', $year . '-09-30');
+				return array($year.'-07-01', $year.'-09-30');
 			case 4:
-				return array($year . '-10-01', $year . '-12-31');
+				return array($year.'-10-01', $year.'-12-31');
 		}
 	}
 
@@ -631,16 +631,16 @@ 
 			$timezone = new DateTimeZone('UTC');
 		} elseif (is_string($timezone)) {
 			$timezone = new DateTimeZone($timezone);
-		} elseif (!($timezone instanceof DateTimeZone)) {
+		} elseif ( ! ($timezone instanceof DateTimeZone)) {
 			return false;
 		}
 
 		if ($dateString instanceof DateTime) {
 			$date = $dateString;
 		} elseif (is_int($dateString) || is_numeric($dateString)) {
-			$dateString = (int)$dateString;
+			$dateString = (int) $dateString;
 
-			$date = new DateTime('@' . $dateString);
+			$date = new DateTime('@'.$dateString);
 			$date->setTimezone($timezone);
 		} else {
 			$date = new DateTime($dateString, $timezone);
@@ -678,22 +678,22 @@ 
 		}
 
 		$userOffset = $timezone;
-		if (!is_numeric($timezone)) {
-			if (!is_object($timezone)) {
+		if ( ! is_numeric($timezone)) {
+			if ( ! is_object($timezone)) {
 				$timezone = new DateTimeZone($timezone);
 			}
-			$currentDate = new DateTime('@' . $date);
+			$currentDate = new DateTime('@'.$date);
 			$currentDate->setTimezone($timezone);
 			$userOffset = $timezone->getOffset($currentDate) / 60 / 60;
 		}
 
 		$timezone = '+0000';
 		if ($userOffset != 0) {
-			$hours = (int)floor(abs($userOffset));
-			$minutes = (int)(fmod(abs($userOffset), $hours) * 60);
-			$timezone = ($userOffset < 0 ? '-' : '+') . str_pad($hours, 2, '0', STR_PAD_LEFT) . str_pad($minutes, 2, '0', STR_PAD_LEFT);
+			$hours = (int) floor(abs($userOffset));
+			$minutes = (int) (fmod(abs($userOffset), $hours) * 60);
+			$timezone = ($userOffset < 0 ? '-' : '+').str_pad($hours, 2, '0', STR_PAD_LEFT).str_pad($minutes, 2, '0', STR_PAD_LEFT);
 		}
-		return date('D, d M Y H:i:s', $date) . ' ' . $timezone;
+		return date('D, d M Y H:i:s', $date).' '.$timezone;
 	}
 
 /**
@@ -791,7 +791,7 @@ 
 		}
 		$diff = $futureTime - $pastTime;
 
-		if (!$diff) {
+		if ( ! $diff) {
 			return __d('cake', 'just now', 'just now');
 		}
 
@@ -823,7 +823,7 @@ 
 				$daysInPastMonth = date('t', $pastTime);
 				$daysInFutureMonth = date('t', mktime(0, 0, 0, $future['m'] - 1, 1, $future['Y']));
 
-				if (!$backwards) {
+				if ( ! $backwards) {
 					$days = ($daysInPastMonth - $past['d']) + $future['d'];
 				} else {
 					$days = ($daysInFutureMonth - $past['d']) + $future['d'];
@@ -834,7 +834,7 @@ 
 				}
 			}
 
-			if (!$months && $years >= 1 && $diff < ($years * 31536000)) {
+			if ( ! $months && $years >= 1 && $diff < ($years * 31536000)) {
 				$months = 11;
 				$years--;
 			}
@@ -881,32 +881,32 @@ 
 
 		$relativeDate = '';
 		if ($fNum >= 1 && $years > 0) {
-			$relativeDate .= ($relativeDate ? ', ' : '') . __dn('cake', '%d year', '%d years', $years, $years);
+			$relativeDate .= ($relativeDate ? ', ' : '').__dn('cake', '%d year', '%d years', $years, $years);
 		}
 		if ($fNum >= 2 && $months > 0) {
-			$relativeDate .= ($relativeDate ? ', ' : '') . __dn('cake', '%d month', '%d months', $months, $months);
+			$relativeDate .= ($relativeDate ? ', ' : '').__dn('cake', '%d month', '%d months', $months, $months);
 		}
 		if ($fNum >= 3 && $weeks > 0) {
-			$relativeDate .= ($relativeDate ? ', ' : '') . __dn('cake', '%d week', '%d weeks', $weeks, $weeks);
+			$relativeDate .= ($relativeDate ? ', ' : '').__dn('cake', '%d week', '%d weeks', $weeks, $weeks);
 		}
 		if ($fNum >= 4 && $days > 0) {
-			$relativeDate .= ($relativeDate ? ', ' : '') . __dn('cake', '%d day', '%d days', $days, $days);
+			$relativeDate .= ($relativeDate ? ', ' : '').__dn('cake', '%d day', '%d days', $days, $days);
 		}
 		if ($fNum >= 5 && $hours > 0) {
-			$relativeDate .= ($relativeDate ? ', ' : '') . __dn('cake', '%d hour', '%d hours', $hours, $hours);
+			$relativeDate .= ($relativeDate ? ', ' : '').__dn('cake', '%d hour', '%d hours', $hours, $hours);
 		}
 		if ($fNum >= 6 && $minutes > 0) {
-			$relativeDate .= ($relativeDate ? ', ' : '') . __dn('cake', '%d minute', '%d minutes', $minutes, $minutes);
+			$relativeDate .= ($relativeDate ? ', ' : '').__dn('cake', '%d minute', '%d minutes', $minutes, $minutes);
 		}
 		if ($fNum >= 7 && $seconds > 0) {
-			$relativeDate .= ($relativeDate ? ', ' : '') . __dn('cake', '%d second', '%d seconds', $seconds, $seconds);
+			$relativeDate .= ($relativeDate ? ', ' : '').__dn('cake', '%d second', '%d seconds', $seconds, $seconds);
 		}
 
 		// When time has passed
-		if (!$backwards && $relativeDate) {
+		if ( ! $backwards && $relativeDate) {
 			return sprintf($relativeString, $relativeDate);
 		}
-		if (!$backwards) {
+		if ( ! $backwards) {
 			$aboutAgo = array(
 				'second' => __d('cake', 'about a second ago'),
 				'minute' => __d('cake', 'about a minute ago'),
@@ -920,7 +920,7 @@ 
 		}
 
 		// When time is to come
-		if (!$relativeDate) {
+		if ( ! $relativeDate) {
 			$aboutIn = array(
 				'second' => __d('cake', 'in about a second'),
 				'minute' => __d('cake', 'in about a minute'),
@@ -949,11 +949,11 @@ 
 	public static function wasWithinLast($timeInterval, $dateString, $timezone = null) {
 		$tmp = str_replace(' ', '', $timeInterval);
 		if (is_numeric($tmp)) {
-			$timeInterval = $tmp . ' ' . __d('cake', 'days');
+			$timeInterval = $tmp.' '.__d('cake', 'days');
 		}
 
 		$date = self::fromString($dateString, $timezone);
-		$interval = self::fromString('-' . $timeInterval);
+		$interval = self::fromString('-'.$timeInterval);
 		$now = self::fromString('now', $timezone);
 
 		return $date >= $interval && $date <= $now;
@@ -971,11 +971,11 @@ 
 	public static function isWithinNext($timeInterval, $dateString, $timezone = null) {
 		$tmp = str_replace(' ', '', $timeInterval);
 		if (is_numeric($tmp)) {
-			$timeInterval = $tmp . ' ' . __d('cake', 'days');
+			$timeInterval = $tmp.' '.__d('cake', 'days');
 		}
 
 		$date = self::fromString($dateString, $timezone);
-		$interval = self::fromString('+' . $timeInterval);
+		$interval = self::fromString('+'.$timeInterval);
 		$now = self::fromString('now', $timezone);
 
 		return $date <= $interval && $date >= $now;
@@ -1091,7 +1091,7 @@ 
 
 		if ($regex) {
 			foreach ($identifiers as $key => $tz) {
-				if (!preg_match($regex, $tz)) {
+				if ( ! preg_match($regex, $tz)) {
 					unset($identifiers[$key]);
 				}
 			}
@@ -1125,13 +1125,13 @@ 
 		$format = strftime($format, $date);
 		$encoding = Configure::read('App.encoding');
 
-		if (!empty($encoding) && $encoding === 'UTF-8') {
+		if ( ! empty($encoding) && $encoding === 'UTF-8') {
 			if (function_exists('mb_check_encoding')) {
 				$valid = mb_check_encoding($format, $encoding);
 			} else {
-				$valid = !Multibyte::checkMultibyte($format);
+				$valid = ! Multibyte::checkMultibyte($format);
 			}
-			if (!$valid) {
+			if ( ! $valid) {
 				$format = utf8_encode($format);
 			}
 		}

lib/Cake/Utility/Debugger.php

Doc Comments +1 added lines, -1 removed lines

@@ -199,7 +199,7 @@
  * @param string $file File on which error occurred
  * @param integer $line Line that triggered the error
  * @param array $context Context
- * @return boolean true if error was handled
+ * @return null|boolean true if error was handled
  * @deprecated Will be removed in 3.0. This function is superseded by Debugger::outputError().
  */
 	public static function showError($code, $description, $file = null, $line = null, $context = null) {

Spacing +40 added lines, -40 removed lines

@@ -99,7 +99,7 @@ 
 		if (empty($docRef) && function_exists('ini_set')) {
 			ini_set('docref_root', 'http://php.net/');
 		}
-		if (!defined('E_RECOVERABLE_ERROR')) {
+		if ( ! defined('E_RECOVERABLE_ERROR')) {
 			define('E_RECOVERABLE_ERROR', 4096);
 		}
 
@@ -153,12 +153,12 @@ 
  */
 	public static function getInstance($class = null) {
 		static $instance = array();
-		if (!empty($class)) {
-			if (!$instance || strtolower($class) != strtolower(get_class($instance[0]))) {
+		if ( ! empty($class)) {
+			if ( ! $instance || strtolower($class) != strtolower(get_class($instance[0]))) {
 				$instance[0] = new $class();
 			}
 		}
-		if (!$instance) {
+		if ( ! $instance) {
 			$instance[0] = new Debugger();
 		}
 		return $instance[0];
@@ -187,8 +187,8 @@ 
  * @link http://book.cakephp.org/2.0/en/development/debugging.html#Debugger::log
  */
 	public static function log($var, $level = LOG_DEBUG) {
-		$source = self::trace(array('start' => 1)) . "\n";
-		CakeLog::write($level, "\n" . $source . self::exportVar($var));
+		$source = self::trace(array('start' => 1))."\n";
+		CakeLog::write($level, "\n".$source.self::exportVar($var));
 	}
 
 /**
@@ -213,7 +213,7 @@ 
 		}
 
 		$info = compact('code', 'description', 'file', 'line');
-		if (!in_array($info, $self->errors)) {
+		if ( ! in_array($info, $self->errors)) {
 			$self->errors[] = $info;
 		} else {
 			return;
@@ -307,9 +307,9 @@ 
 				$next = array_merge($_trace, $backtrace[$i + 1]);
 				$signature = $reference = $next['function'];
 
-				if (!empty($next['class'])) {
-					$signature = $next['class'] . '::' . $next['function'];
-					$reference = $signature . '(';
+				if ( ! empty($next['class'])) {
+					$signature = $next['class'].'::'.$next['function'];
+					$reference = $signature.'(';
 					if ($options['args'] && isset($next['args'])) {
 						$args = array();
 						foreach ($next['args'] as $arg) {
@@ -354,12 +354,12 @@ 
  * @return string Normalized path
  */
 	public static function trimPath($path) {
-		if (!defined('CAKE_CORE_INCLUDE_PATH') || !defined('APP')) {
+		if ( ! defined('CAKE_CORE_INCLUDE_PATH') || ! defined('APP')) {
 			return $path;
 		}
 
 		if (strpos($path, APP) === 0) {
-			return str_replace(APP, 'APP' . DS, $path);
+			return str_replace(APP, 'APP'.DS, $path);
 		} elseif (strpos($path, CAKE_CORE_INCLUDE_PATH) === 0) {
 			return str_replace(CAKE_CORE_INCLUDE_PATH, 'CORE', $path);
 		} elseif (strpos($path, ROOT) === 0) {
@@ -390,7 +390,7 @@ 
  */
 	public static function excerpt($file, $line, $context = 2) {
 		$lines = array();
-		if (!file_exists($file)) {
+		if ( ! file_exists($file)) {
 			return array();
 		}
 		$data = file_get_contents($file);
@@ -400,16 +400,16 @@ 
 		if (strpos($data, "\n") !== false) {
 			$data = explode("\n", $data);
 		}
-		if (!isset($data[$line])) {
+		if ( ! isset($data[$line])) {
 			return $lines;
 		}
 		for ($i = $line - ($context + 1); $i < $line + $context; $i++) {
-			if (!isset($data[$i])) {
+			if ( ! isset($data[$i])) {
 				continue;
 			}
 			$string = str_replace(array("\r\n", "\n"), "", self::_highlight($data[$i]));
 			if ($i == $line) {
-				$lines[] = '<span class="code-highlight">' . $string . '</span>';
+				$lines[] = '<span class="code-highlight">'.$string.'</span>';
 			} else {
 				$lines[] = $string;
 			}
@@ -431,7 +431,7 @@ 
 		$added = false;
 		if (strpos($str, '<?php') === false) {
 			$added = true;
-			$str = "<?php \n" . $str;
+			$str = "<?php \n".$str;
 		}
 		$highlight = highlight_string($str, true);
 		if ($added) {
@@ -483,14 +483,14 @@ 
 			case 'boolean':
 				return ($var) ? 'true' : 'false';
 			case 'integer':
-				return '(int) ' . $var;
+				return '(int) '.$var;
 			case 'float':
-				return '(float) ' . $var;
+				return '(float) '.$var;
 			case 'string':
 				if (trim($var) === '') {
 					return "''";
 				}
-				return "'" . $var . "'";
+				return "'".$var."'";
 			case 'array':
 				return self::_array($var, $depth - 1, $indent + 1);
 			case 'resource':
@@ -537,9 +537,9 @@ 
 
 		$out = "array(";
 		$break = $end = null;
-		if (!empty($var)) {
-			$break = "\n" . str_repeat("\t", $indent);
-			$end = "\n" . str_repeat("\t", $indent - 1);
+		if ( ! empty($var)) {
+			$break = "\n".str_repeat("\t", $indent);
+			$end = "\n".str_repeat("\t", $indent - 1);
 		}
 		$vars = array();
 
@@ -551,14 +551,14 @@ 
 				} elseif ($val !== $var) {
 					$val = self::_export($val, $depth, $indent);
 				}
-				$vars[] = $break . self::exportVar($key) .
-					' => ' .
+				$vars[] = $break.self::exportVar($key).
+					' => '.
 					$val;
 			}
 		} else {
-			$vars[] = $break . '[maximum depth reached]';
+			$vars[] = $break.'[maximum depth reached]';
 		}
-		return $out . implode(',', $vars) . $end . ')';
+		return $out.implode(',', $vars).$end.')';
 	}
 
 /**
@@ -575,15 +575,15 @@ 
 		$props = array();
 
 		$className = get_class($var);
-		$out .= 'object(' . $className . ') {';
+		$out .= 'object('.$className.') {';
 
 		if ($depth > 0) {
-			$end = "\n" . str_repeat("\t", $indent - 1);
-			$break = "\n" . str_repeat("\t", $indent);
+			$end = "\n".str_repeat("\t", $indent - 1);
+			$break = "\n".str_repeat("\t", $indent);
 			$objectVars = get_object_vars($var);
 			foreach ($objectVars as $key => $value) {
 				$value = self::_export($value, $depth - 1, $indent);
-				$props[] = "$key => " . $value;
+				$props[] = "$key => ".$value;
 			}
 
 			if (version_compare(PHP_VERSION, '5.3.0') >= 0) {
@@ -606,7 +606,7 @@ 
 				}
 			}
 
-			$out .= $break . implode($break, $props) . $end;
+			$out .= $break.implode($break, $props).$end;
 		}
 		$out .= '}';
 		return $out;
@@ -625,7 +625,7 @@ 
 		if ($format === null) {
 			return $self->_outputFormat;
 		}
-		if ($format !== false && !isset($self->_templates[$format])) {
+		if ($format !== false && ! isset($self->_templates[$format])) {
 			throw new CakeException(__d('cake_dev', 'Invalid Debugger output format.'));
 		}
 		$self->_outputFormat = $format;
@@ -706,11 +706,11 @@ 
 			return Debugger::outputAs();
 		}
 
-		if (!empty($strings)) {
+		if ( ! empty($strings)) {
 			return Debugger::addFormat($format, $strings);
 		}
 
-		if ($format === true && !empty($self->_data)) {
+		if ($format === true && ! empty($self->_data)) {
 			$data = $self->_data;
 			$self->_data = array();
 			$format = false;
@@ -755,8 +755,8 @@ 
 		$links = array();
 		$info = '';
 
-		foreach ((array)$data['context'] as $var => $value) {
-			$context[] = "\${$var} = " . $this->exportVar($value, 3);
+		foreach ((array) $data['context'] as $var => $value) {
+			$context[] = "\${$var} = ".$this->exportVar($value, 3);
 		}
 
 		switch ($this->_outputFormat) {
@@ -769,7 +769,7 @@ 
 		}
 
 		$data['trace'] = $trace;
-		$data['id'] = 'cakeErr' . uniqid();
+		$data['id'] = 'cakeErr'.uniqid();
 		$tpl = array_merge($this->_templates['base'], $this->_templates[$this->_outputFormat]);
 
 		if (isset($tpl['links'])) {
@@ -778,13 +778,13 @@ 
 			}
 		}
 
-		if (!empty($tpl['escapeContext'])) {
+		if ( ! empty($tpl['escapeContext'])) {
 			$context = h($context);
 		}
 
 		$infoData = compact('code', 'context', 'trace');
 		foreach ($infoData as $key => $value) {
-			if (empty($value) || !isset($tpl[$key])) {
+			if (empty($value) || ! isset($tpl[$key])) {
 				continue;
 			}
 			if (is_array($value)) {