reginaldojunior / winners

lib/Cake/Test/Case/View/Helper/FormHelperTest.php

Doc Comments +4 added lines, -4 removed lines

@@ -310,7 +310,7 @@ 
 /**
  * beforeValidate method
  *
- * @return void
+ * @return boolean
  */
 	public function beforeValidate($options = array()) {
 		$this->invalidate('openid_not_registered');
@@ -359,7 +359,7 @@ 
 /**
  * beforeValidate method
  *
- * @return void
+ * @return boolean
  */
 	public function beforeValidate($options = array()) {
 		$this->invalidate('email');
@@ -417,7 +417,7 @@ 
 /**
  * beforeValidate method
  *
- * @return void
+ * @return boolean
  */
 	public function beforeValidate($options = array()) {
 		$this->invalidate('full_name');
@@ -467,7 +467,7 @@ 
 /**
  * beforeValidate method
  *
- * @return void
+ * @return boolean
  */
 	public function beforeValidate($options = array()) {
 		$this->invalidate('description');

Spacing +16 added lines, -16 removed lines

@@ -690,8 +690,8 @@ 
 		$this->Form->request['_Token'] = array('key' => $key);
 		$result = $this->Form->secure($fields);
 
-		$hash = Security::hash(serialize($fields) . Configure::read('Security.salt'));
-		$hash .= ':' . 'Model.valid';
+		$hash = Security::hash(serialize($fields).Configure::read('Security.salt'));
+		$hash .= ':'.'Model.valid';
 		$hash = urlencode($hash);
 
 		$expected = array(
@@ -708,7 +708,7 @@ 
 		);
 		$this->assertTags($result, $expected);
 
-		$path = CAKE . 'Test' . DS . 'test_app' . DS . 'Config' . DS;
+		$path = CAKE.'Test'.DS.'test_app'.DS.'Config'.DS;
 		$this->Form->Html->loadConfig('htmlhelper_tags', $path);
 		$result = $this->Form->secure($fields);
 		$expected = array(
@@ -2286,7 +2286,7 @@ 
 
 		$result = $this->Form->input('published', array('type' => 'time'));
 		$now = strtotime('now');
-		$this->assertContains('<option value="' . date('h', $now) . '" selected="selected">' . date('g', $now) . '</option>', $result);
+		$this->assertContains('<option value="'.date('h', $now).'" selected="selected">'.date('g', $now).'</option>', $result);
 
 		$now = strtotime('2013-03-09 00:42:21');
 		$result = $this->Form->input('published', array('type' => 'time', 'selected' => $now));
@@ -3236,7 +3236,7 @@ 
 	public function testInputsPluginModel() {
 		$this->loadFixtures('Post');
 		App::build(array(
-			'Plugin' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Plugin' . DS)
+			'Plugin' => array(CAKE.'Test'.DS.'test_app'.DS.'Plugin'.DS)
 		));
 		CakePlugin::load('TestPlugin');
 		$this->Form->request['models'] = array(
@@ -5387,7 +5387,7 @@ 
 
 		$result = $this->Form->secure($this->Form->fields);
 		$key = 'f7d573650a295b94e0938d32b323fde775e5f32b%3A';
-		$this->assertRegExp('/"' . $key . '"/', $result);
+		$this->assertRegExp('/"'.$key.'"/', $result);
 	}
 
 /**
@@ -6109,7 +6109,7 @@ 
 		$this->assertTags($result, $expected);
 		$this->assertNotRegExp('/<option[^<>]+value=""[^<>]+selected="selected"[^>]*>/', $result);
 
-		$this->Form->request->data['Model']['field'] = date('Y') . '-01-01 00:00:00';
+		$this->Form->request->data['Model']['field'] = date('Y').'-01-01 00:00:00';
 		$now = strtotime($this->Form->data['Model']['field']);
 		$result = $this->Form->dateTime('Model.field', 'DMY', '12', array('empty' => false));
 		$expected = array(
@@ -6929,7 +6929,7 @@ 
 		$result = $this->Form->hour('Model.field', true, array('value' => 'now'));
 		$thisHour = date('H');
 		$optValue = date('G');
-		$this->assertRegExp('/<option value="' . $thisHour . '" selected="selected">' . $optValue . '<\/option>/', $result);
+		$this->assertRegExp('/<option value="'.$thisHour.'" selected="selected">'.$optValue.'<\/option>/', $result);
 
 		$this->Form->request->data['Model']['field'] = '2050-10-10 01:12:32';
 		$result = $this->Form->hour('Model.field', true);
@@ -7227,7 +7227,7 @@ 
 					'maxYear' => 2008
 				)
 		);
-		$this->assertContains('value="' . date('m') . '" selected="selected"', $result);
+		$this->assertContains('value="'.date('m').'" selected="selected"', $result);
 		$this->assertNotContains('value="2008" selected="selected"', $result);
 
 		$result = $this->Form->input('just_year',
@@ -7239,7 +7239,7 @@ 
 				'maxYear' => date('Y', strtotime('+20 years'))
 			)
 		);
-		$this->assertNotContains('value="' . date('Y') . '" selected="selected"', $result);
+		$this->assertNotContains('value="'.date('Y').'" selected="selected"', $result);
 
 		$result = $this->Form->input('just_month',
 			array(
@@ -7249,7 +7249,7 @@ 
 				'empty' => false,
 			)
 		);
-		$this->assertNotContains('value="' . date('m') . '" selected="selected"', $result);
+		$this->assertNotContains('value="'.date('m').'" selected="selected"', $result);
 
 		$result = $this->Form->input('just_day',
 			array(
@@ -7259,7 +7259,7 @@ 
 				'empty' => false,
 			)
 		);
-		$this->assertNotContains('value="' . date('d') . '" selected="selected"', $result);
+		$this->assertNotContains('value="'.date('d').'" selected="selected"', $result);
 	}
 
 /**
@@ -7623,9 +7623,9 @@ 
  */
 	public function testPostLinkSecurityHash() {
 		$hash = Security::hash(
-			'/posts/delete/1' .
-			serialize(array()) .
-			'' .
+			'/posts/delete/1'.
+			serialize(array()).
+			''.
 			Configure::read('Security.salt')
 		);
 		$hash .= '%3A';
@@ -9636,7 +9636,7 @@ 
 	public function testIntrospectModelFromRequest() {
 		$this->loadFixtures('Post');
 		App::build(array(
-			'Plugin' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Plugin' . DS)
+			'Plugin' => array(CAKE.'Test'.DS.'test_app'.DS.'Plugin'.DS)
 		));
 		CakePlugin::load('TestPlugin');
 		$this->Form->request['models'] = array('TestPluginPost' => array('plugin' => 'TestPlugin', 'className' => 'TestPluginPost'));

Indentation +5 added lines, -5 removed lines

@@ -5408,11 +5408,11 @@
 		$this->assertEquals(array('Model.select'), $this->Form->fields);
 	}
 /**
- * When a select box has no options it should not be added to the fields list
- * as it always fail post validation.
- *
- * @return void
- */
+	 * When a select box has no options it should not be added to the fields list
+	 * as it always fail post validation.
+	 *
+	 * @return void
+	 */
 	public function testSelectNoSecureWithNoOptions() {
 		$this->Form->request['_Token'] = array('key' => 'testkey');
 		$this->assertEquals(array(), $this->Form->fields);

lib/Cake/Test/Case/View/Helper/HtmlHelperTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
  * @param string $exclude
  * @param string $insertBefore
  * @param string $insertAfter
- * @return void
+ * @return string
  */
 	public function parseAttributes($options, $exclude = null, $insertBefore = ' ', $insertAfter = null) {
 		return $this->_parseAttributes($options, $exclude, $insertBefore, $insertAfter);

Spacing +43 added lines, -43 removed lines

@@ -24,7 +24,7 @@ 
 App::uses('ClassRegistry', 'Utility');
 App::uses('Folder', 'Utility');
 
-if (!defined('FULL_BASE_URL')) {
+if ( ! defined('FULL_BASE_URL')) {
 	define('FULL_BASE_URL', 'http://cakephp.org');
 }
 
@@ -72,7 +72,7 @@ 
  * @return mixed
  */
 	public function getAttribute($attribute) {
-		if (!isset($this->{$attribute})) {
+		if ( ! isset($this->{$attribute})) {
 			return null;
 		}
 		return $this->{$attribute};
@@ -150,7 +150,7 @@ 
 		$this->Html->request->webroot = '';
 
 		App::build(array(
-			'Plugin' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Plugin' . DS)
+			'Plugin' => array(CAKE.'Test'.DS.'test_app'.DS.'Plugin'.DS)
 		));
 
 		Configure::write('Asset.timestamp', false);
@@ -208,7 +208,7 @@ 
 		Router::reload();
 
 		$result = $this->Html->link('Posts', array('controller' => 'posts', 'action' => 'index', 'full_base' => true));
-		$expected = array('a' => array('href' => Router::fullBaseUrl() . '/posts'), 'Posts', '/a');
+		$expected = array('a' => array('href' => Router::fullBaseUrl().'/posts'), 'Posts', '/a');
 		$this->assertTags($result, $expected);
 
 		$result = $this->Html->link('Home', '/home', array('confirm' => 'Are you sure you want to do this?'));
@@ -464,11 +464,11 @@ 
 	public function testImageWithFullBase() {
 		$result = $this->Html->image('test.gif', array('fullBase' => true));
 		$here = $this->Html->url('/', true);
-		$this->assertTags($result, array('img' => array('src' => $here . 'img/test.gif', 'alt' => '')));
+		$this->assertTags($result, array('img' => array('src' => $here.'img/test.gif', 'alt' => '')));
 
 		$result = $this->Html->image('sub/test.gif', array('fullBase' => true));
 		$here = $this->Html->url('/', true);
-		$this->assertTags($result, array('img' => array('src' => $here . 'img/sub/test.gif', 'alt' => '')));
+		$this->assertTags($result, array('img' => array('src' => $here.'img/sub/test.gif', 'alt' => '')));
 
 		$request = $this->Html->request;
 		$request->webroot = '/myproject/';
@@ -477,7 +477,7 @@ 
 
 		$result = $this->Html->image('sub/test.gif', array('fullBase' => true));
 		$here = $this->Html->url('/', true);
-		$this->assertTags($result, array('img' => array('src' => $here . 'img/sub/test.gif', 'alt' => '')));
+		$this->assertTags($result, array('img' => array('src' => $here.'img/sub/test.gif', 'alt' => '')));
 	}
 
 /**
@@ -512,16 +512,16 @@ 
  * @return void
  */
 	public function testImageTagWithTheme() {
-		$this->skipIf(!is_writable(WWW_ROOT), 'Cannot write to webroot.');
-		$themeExists = is_dir(WWW_ROOT . 'theme');
+		$this->skipIf( ! is_writable(WWW_ROOT), 'Cannot write to webroot.');
+		$themeExists = is_dir(WWW_ROOT.'theme');
 
 		App::uses('File', 'Utility');
 
-		$testfile = WWW_ROOT . 'theme' . DS . 'test_theme' . DS . 'img' . DS . '__cake_test_image.gif';
+		$testfile = WWW_ROOT.'theme'.DS.'test_theme'.DS.'img'.DS.'__cake_test_image.gif';
 		new File($testfile, true);
 
 		App::build(array(
-			'View' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS)
+			'View' => array(CAKE.'Test'.DS.'test_app'.DS.'View'.DS)
 		));
 		Configure::write('Asset.timestamp', true);
 		Configure::write('debug', 1);
@@ -544,10 +544,10 @@ 
 				'alt' => ''
 		)));
 
-		$dir = new Folder(WWW_ROOT . 'theme' . DS . 'test_theme');
+		$dir = new Folder(WWW_ROOT.'theme'.DS.'test_theme');
 		$dir->delete();
-		if (!$themeExists) {
-			$dir = new Folder(WWW_ROOT . 'theme');
+		if ( ! $themeExists) {
+			$dir = new Folder(WWW_ROOT.'theme');
 			$dir->delete();
 		}
 	}
@@ -559,10 +559,10 @@ 
  */
 	public function testThemeAssetsInMainWebrootPath() {
 		App::build(array(
-			'View' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS)
+			'View' => array(CAKE.'Test'.DS.'test_app'.DS.'View'.DS)
 		));
 		$webRoot = Configure::read('App.www_root');
-		Configure::write('App.www_root', CAKE . 'Test' . DS . 'test_app' . DS . 'webroot' . DS);
+		Configure::write('App.www_root', CAKE.'Test'.DS.'test_app'.DS.'webroot'.DS);
 
 		$this->Html->theme = 'test_theme';
 		$result = $this->Html->css('webroot_test');
@@ -741,7 +741,7 @@ 
 
 		$result = $this->Html->css('screen', null, array('fullBase' => true));
 		$expected = array(
-			'link' => array('rel' => 'stylesheet', 'type' => 'text/css', 'href' => $here . 'css/screen.css')
+			'link' => array('rel' => 'stylesheet', 'type' => 'text/css', 'href' => $here.'css/screen.css')
 		);
 		$this->assertTags($result, $expected);
 	}
@@ -879,22 +879,22 @@ 
  * @return void
  */
 	public function testScriptTimestamping() {
-		$this->skipIf(!is_writable(WWW_ROOT . 'js'), 'webroot/js is not Writable, timestamp testing has been skipped.');
+		$this->skipIf( ! is_writable(WWW_ROOT.'js'), 'webroot/js is not Writable, timestamp testing has been skipped.');
 
 		Configure::write('debug', 2);
 		Configure::write('Asset.timestamp', true);
 
-		touch(WWW_ROOT . 'js' . DS . '__cake_js_test.js');
+		touch(WWW_ROOT.'js'.DS.'__cake_js_test.js');
 		$timestamp = substr(strtotime('now'), 0, 8);
 
 		$result = $this->Html->script('__cake_js_test', array('inline' => true, 'once' => false));
-		$this->assertRegExp('/__cake_js_test.js\?' . $timestamp . '[0-9]{2}"/', $result, 'Timestamp value not found %s');
+		$this->assertRegExp('/__cake_js_test.js\?'.$timestamp.'[0-9]{2}"/', $result, 'Timestamp value not found %s');
 
 		Configure::write('debug', 0);
 		Configure::write('Asset.timestamp', 'force');
 		$result = $this->Html->script('__cake_js_test', array('inline' => true, 'once' => false));
-		$this->assertRegExp('/__cake_js_test.js\?' . $timestamp . '[0-9]{2}"/', $result, 'Timestamp value not found %s');
-		unlink(WWW_ROOT . 'js' . DS . '__cake_js_test.js');
+		$this->assertRegExp('/__cake_js_test.js\?'.$timestamp.'[0-9]{2}"/', $result, 'Timestamp value not found %s');
+		unlink(WWW_ROOT.'js'.DS.'__cake_js_test.js');
 		Configure::write('Asset.timestamp', false);
 	}
 
@@ -907,23 +907,23 @@ 
 		CakePlugin::load('TestPlugin');
 
 		$pluginPath = App::pluginPath('TestPlugin');
-		$pluginJsPath = $pluginPath . 'webroot/js';
-		$this->skipIf(!is_writable($pluginJsPath), $pluginJsPath . ' is not Writable, timestamp testing has been skipped.');
+		$pluginJsPath = $pluginPath.'webroot/js';
+		$this->skipIf( ! is_writable($pluginJsPath), $pluginJsPath.' is not Writable, timestamp testing has been skipped.');
 
 		Configure::write('debug', 2);
 		Configure::write('Asset.timestamp', true);
 
-		touch($pluginJsPath . DS . '__cake_js_test.js');
+		touch($pluginJsPath.DS.'__cake_js_test.js');
 		$timestamp = substr(strtotime('now'), 0, 8);
 
 		$result = $this->Html->script('TestPlugin.__cake_js_test', array('inline' => true, 'once' => false));
-		$this->assertRegExp('/test_plugin\/js\/__cake_js_test.js\?' . $timestamp . '[0-9]{2}"/', $result, 'Timestamp value not found %s');
+		$this->assertRegExp('/test_plugin\/js\/__cake_js_test.js\?'.$timestamp.'[0-9]{2}"/', $result, 'Timestamp value not found %s');
 
 		Configure::write('debug', 0);
 		Configure::write('Asset.timestamp', 'force');
 		$result = $this->Html->script('TestPlugin.__cake_js_test', array('inline' => true, 'once' => false));
-		$this->assertRegExp('/test_plugin\/js\/__cake_js_test.js\?' . $timestamp . '[0-9]{2}"/', $result, 'Timestamp value not found %s');
-		unlink($pluginJsPath . DS . '__cake_js_test.js');
+		$this->assertRegExp('/test_plugin\/js\/__cake_js_test.js\?'.$timestamp.'[0-9]{2}"/', $result, 'Timestamp value not found %s');
+		unlink($pluginJsPath.DS.'__cake_js_test.js');
 		Configure::write('Asset.timestamp', false);
 
 		CakePlugin::unload('TestPlugin');
@@ -1149,15 +1149,15 @@ 
 
 		$result = $this->Html->script('foo', array('fullBase' => true));
 		$expected = array(
-			'script' => array('type' => 'text/javascript', 'src' => $here . 'js/foo.js')
+			'script' => array('type' => 'text/javascript', 'src' => $here.'js/foo.js')
 		);
 		$this->assertTags($result, $expected);
 
 		$result = $this->Html->script(array('foobar', 'bar'), array('fullBase' => true));
 		$expected = array(
-			array('script' => array('type' => 'text/javascript', 'src' => $here . 'js/foobar.js')),
+			array('script' => array('type' => 'text/javascript', 'src' => $here.'js/foobar.js')),
 			'/script',
-			array('script' => array('type' => 'text/javascript', 'src' => $here . 'js/bar.js')),
+			array('script' => array('type' => 'text/javascript', 'src' => $here.'js/bar.js')),
 			'/script',
 		);
 		$this->assertTags($result, $expected);
@@ -1169,16 +1169,16 @@ 
  * @return void
  */
 	public function testScriptInTheme() {
-		$this->skipIf(!is_writable(WWW_ROOT), 'Cannot write to webroot.');
-		$themeExists = is_dir(WWW_ROOT . 'theme');
+		$this->skipIf( ! is_writable(WWW_ROOT), 'Cannot write to webroot.');
+		$themeExists = is_dir(WWW_ROOT.'theme');
 
 		App::uses('File', 'Utility');
 
-		$testfile = WWW_ROOT . 'theme' . DS . 'test_theme' . DS . 'js' . DS . '__test_js.js';
+		$testfile = WWW_ROOT.'theme'.DS.'test_theme'.DS.'js'.DS.'__test_js.js';
 		new File($testfile, true);
 
 		App::build(array(
-			'View' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS)
+			'View' => array(CAKE.'Test'.DS.'test_app'.DS.'View'.DS)
 		));
 
 		$this->Html->webroot = '/';
@@ -1189,11 +1189,11 @@ 
 		);
 		$this->assertTags($result, $expected);
 
-		$Folder = new Folder(WWW_ROOT . 'theme' . DS . 'test_theme');
+		$Folder = new Folder(WWW_ROOT.'theme'.DS.'test_theme');
 		$Folder->delete();
 
-		if (!$themeExists) {
-			$dir = new Folder(WWW_ROOT . 'theme');
+		if ( ! $themeExists) {
+			$dir = new Folder(WWW_ROOT.'theme');
 			$dir->delete();
 		}
 	}
@@ -1994,7 +1994,7 @@ 
 			array('pathPrefix' => 'videos/', 'poster' => 'poster.jpg', 'text' => 'Your browser does not support the HTML5 Video element.')
 		);
 		$expected = array(
-			'video' => array('poster' => Configure::read('App.imageBaseUrl') . 'poster.jpg'),
+			'video' => array('poster' => Configure::read('App.imageBaseUrl').'poster.jpg'),
 				array('source' => array('src' => 'videos/video.webm', 'type' => 'video/webm')),
 				array('source' => array('src' => 'videos/video.ogv', 'type' => 'video/ogg; codecs='theora, vorbis'')),
 				'Your browser does not support the HTML5 Video element.',
@@ -2181,7 +2181,7 @@ 
  */
 
 	public function testLoadConfig() {
-		$path = CAKE . 'Test' . DS . 'test_app' . DS . 'Config' . DS;
+		$path = CAKE.'Test'.DS.'test_app'.DS.'Config'.DS;
 
 		$result = $this->Html->loadConfig('htmlhelper_tags', $path);
 		$expected = array(
@@ -2222,7 +2222,7 @@ 
  * @expectedException ConfigureException
  */
 	public function testLoadConfigWrongReader() {
-		$path = CAKE . 'Test' . DS . 'test_app' . DS . 'Config' . DS;
+		$path = CAKE.'Test'.DS.'test_app'.DS.'Config'.DS;
 		$this->Html->loadConfig(array('htmlhelper_tags', 'wrong_reader'), $path);
 	}
 
@@ -2239,8 +2239,8 @@ 
 		foreach ($compact as $attribute) {
 			foreach (array('true', true, 1, '1', $attribute) as $value) {
 				$attrs = array($attribute => $value);
-				$expected = ' ' . $attribute . '="' . $attribute . '"';
-				$this->assertEquals($expected, $helper->parseAttributes($attrs), '%s Failed on ' . $value);
+				$expected = ' '.$attribute.'="'.$attribute.'"';
+				$this->assertEquals($expected, $helper->parseAttributes($attrs), '%s Failed on '.$value);
 			}
 		}
 		$this->assertEquals(' compact="compact"', $helper->parseAttributes(array('compact')));

lib/Cake/Test/Case/View/Helper/JsHelperTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -69,8 +69,8 @@
  * test method for option parsing
  *
  * @param $options
- * @param array $safe
- * @return void
+ * @param string[] $safe
+ * @return string
  */
 	public function testParseOptions($options, $safe = array()) {
 		return $this->_parseOptions($options, $safe);

Spacing +8 added lines, -8 removed lines

@@ -360,7 +360,7 @@ 
  * @return void
  */
 	public function testWriteScriptsInFile() {
-		$this->skipIf(!is_writable(WWW_ROOT . 'js'), 'webroot/js is not Writable, script caching test has been skipped.');
+		$this->skipIf( ! is_writable(WWW_ROOT.'js'), 'webroot/js is not Writable, script caching test has been skipped.');
 
 		Configure::write('Cache.disable', false);
 		$this->Js->request->webroot = '/';
@@ -373,11 +373,11 @@ 
 		);
 		$this->assertTags($result, $expected);
 		preg_match('/src="(.*\.js)"/', $result, $filename);
-		$this->assertTrue(file_exists(WWW_ROOT . $filename[1]));
-		$contents = file_get_contents(WWW_ROOT . $filename[1]);
+		$this->assertTrue(file_exists(WWW_ROOT.$filename[1]));
+		$contents = file_get_contents(WWW_ROOT.$filename[1]);
 		$this->assertRegExp('/one\s=\s1;\ntwo\s=\s2;/', $contents);
-		if (file_exists(WWW_ROOT . $filename[1])) {
-			unlink(WWW_ROOT . $filename[1]);
+		if (file_exists(WWW_ROOT.$filename[1])) {
+			unlink(WWW_ROOT.$filename[1]);
 		}
 
 		Configure::write('Cache.disable', true);
@@ -385,7 +385,7 @@ 
 		$this->Js->buffer('two = 2;');
 		$result = $this->Js->writeBuffer(array('onDomReady' => false, 'cache' => true));
 		$this->assertRegExp('/one\s=\s1;\ntwo\s=\s2;/', $result);
-		$this->assertFalse(file_exists(WWW_ROOT . $filename[1]));
+		$this->assertFalse(file_exists(WWW_ROOT.$filename[1]));
 	}
 
 /**
@@ -776,11 +776,11 @@ 
 		$expected = '';
 		$this->assertEquals($expected, $result);
 
-		$result = $this->JsEngine->escape('CakePHP' . "\n" . 'Rapid Development Framework');
+		$result = $this->JsEngine->escape('CakePHP'."\n".'Rapid Development Framework');
 		$expected = 'CakePHP\\nRapid Development Framework';
 		$this->assertEquals($expected, $result);
 
-		$result = $this->JsEngine->escape('CakePHP' . "\r\n" . 'Rapid Development Framework' . "\r" . 'For PHP');
+		$result = $this->JsEngine->escape('CakePHP'."\r\n".'Rapid Development Framework'."\r".'For PHP');
 		$expected = 'CakePHP\\r\\nRapid Development Framework\\rFor PHP';
 		$this->assertEquals($expected, $result);
 

lib/Cake/Test/Case/View/ScaffoldViewTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
  * testGetFilename method
  *
  * @param string $action
- * @return void
+ * @return string
  */
 	public function testGetFilename($action) {
 		return $this->_getViewFileName($action);

Spacing +20 added lines, -20 removed lines

@@ -21,7 +21,7 @@ 
 App::uses('ScaffoldView', 'View');
 App::uses('AppModel', 'Model');
 
-require_once dirname(dirname(__FILE__)) . DS . 'Model' . DS . 'models.php';
+require_once dirname(dirname(__FILE__)).DS.'Model'.DS.'models.php';
 
 /**
  * TestScaffoldView class
@@ -90,8 +90,8 @@ 
 		$this->Controller->response = $this->getMock('CakeResponse', array('_sendHeader'));
 
 		App::build(array(
-			'View' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS),
-			'Plugin' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Plugin' . DS)
+			'View' => array(CAKE.'Test'.DS.'test_app'.DS.'View'.DS),
+			'Plugin' => array(CAKE.'Test'.DS.'test_app'.DS.'Plugin'.DS)
 		));
 		CakePlugin::load('TestPlugin');
 	}
@@ -118,39 +118,39 @@ 
 		$this->Controller->request->params['action'] = 'index';
 		$ScaffoldView = new TestScaffoldView($this->Controller);
 		$result = $ScaffoldView->testGetFilename('index');
-		$expected = CAKE . 'View' . DS . 'Scaffolds' . DS . 'index.ctp';
+		$expected = CAKE.'View'.DS.'Scaffolds'.DS.'index.ctp';
 		$this->assertEquals($expected, $result);
 
 		$result = $ScaffoldView->testGetFilename('edit');
-		$expected = CAKE . 'View' . DS . 'Scaffolds' . DS . 'form.ctp';
+		$expected = CAKE.'View'.DS.'Scaffolds'.DS.'form.ctp';
 		$this->assertEquals($expected, $result);
 
 		$result = $ScaffoldView->testGetFilename('add');
-		$expected = CAKE . 'View' . DS . 'Scaffolds' . DS . 'form.ctp';
+		$expected = CAKE.'View'.DS.'Scaffolds'.DS.'form.ctp';
 		$this->assertEquals($expected, $result);
 
 		$result = $ScaffoldView->testGetFilename('view');
-		$expected = CAKE . 'View' . DS . 'Scaffolds' . DS . 'view.ctp';
+		$expected = CAKE.'View'.DS.'Scaffolds'.DS.'view.ctp';
 		$this->assertEquals($expected, $result);
 
 		$result = $ScaffoldView->testGetFilename('admin_index');
-		$expected = CAKE . 'View' . DS . 'Scaffolds' . DS . 'index.ctp';
+		$expected = CAKE.'View'.DS.'Scaffolds'.DS.'index.ctp';
 		$this->assertEquals($expected, $result);
 
 		$result = $ScaffoldView->testGetFilename('admin_view');
-		$expected = CAKE . 'View' . DS . 'Scaffolds' . DS . 'view.ctp';
+		$expected = CAKE.'View'.DS.'Scaffolds'.DS.'view.ctp';
 		$this->assertEquals($expected, $result);
 
 		$result = $ScaffoldView->testGetFilename('admin_edit');
-		$expected = CAKE . 'View' . DS . 'Scaffolds' . DS . 'form.ctp';
+		$expected = CAKE.'View'.DS.'Scaffolds'.DS.'form.ctp';
 		$this->assertEquals($expected, $result);
 
 		$result = $ScaffoldView->testGetFilename('admin_add');
-		$expected = CAKE . 'View' . DS . 'Scaffolds' . DS . 'form.ctp';
+		$expected = CAKE.'View'.DS.'Scaffolds'.DS.'form.ctp';
 		$this->assertEquals($expected, $result);
 
 		$result = $ScaffoldView->testGetFilename('error');
-		$expected = CAKE . 'View' . DS . 'Errors' . DS . 'scaffold_error.ctp';
+		$expected = CAKE.'View'.DS.'Errors'.DS.'scaffold_error.ctp';
 		$this->assertEquals($expected, $result);
 
 		$Controller = new ScaffoldViewMockController($this->request);
@@ -160,11 +160,11 @@ 
 
 		$ScaffoldView = new TestScaffoldView($Controller);
 		$result = $ScaffoldView->testGetFilename('admin_edit');
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS . 'Posts' . DS . 'scaffold.form.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'View'.DS.'Posts'.DS.'scaffold.form.ctp';
 		$this->assertEquals($expected, $result);
 
 		$result = $ScaffoldView->testGetFilename('edit');
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS . 'Posts' . DS . 'scaffold.form.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'View'.DS.'Posts'.DS.'scaffold.form.ctp';
 		$this->assertEquals($expected, $result);
 
 		$Controller = new ScaffoldViewMockController($this->request);
@@ -179,13 +179,13 @@ 
 
 		$ScaffoldView = new TestScaffoldView($Controller);
 		$result = $ScaffoldView->testGetFilename('admin_add');
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'Plugin' .
-			DS . 'TestPlugin' . DS . 'View' . DS . 'Tests' . DS . 'scaffold.form.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'Plugin'.
+			DS.'TestPlugin'.DS.'View'.DS.'Tests'.DS.'scaffold.form.ctp';
 		$this->assertEquals($expected, $result);
 
 		$result = $ScaffoldView->testGetFilename('add');
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'Plugin' .
-			DS . 'TestPlugin' . DS . 'View' . DS . 'Tests' . DS . 'scaffold.form.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'Plugin'.
+			DS.'TestPlugin'.DS.'View'.DS.'Tests'.DS.'scaffold.form.ctp';
 		$this->assertEquals($expected, $result);
 
 		Configure::write('Routing.prefixes', $_admin);
@@ -203,8 +203,8 @@ 
 		$ScaffoldView = new TestScaffoldView($this->Controller);
 
 		$result = $ScaffoldView->testGetFilename('index');
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS .
-			'Themed' . DS . 'TestTheme' . DS . 'Posts' . DS . 'scaffold.index.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'View'.DS.
+			'Themed'.DS.'TestTheme'.DS.'Posts'.DS.'scaffold.index.ctp';
 		$this->assertEquals($expected, $result);
 	}
 

lib/Cake/Test/Case/View/ThemeViewTest.php

Doc Comments +3 added lines, -3 removed lines

@@ -63,7 +63,7 @@ 
  *
  * @param string $name
  * @param array $params
- * @return void
+ * @return string
  */
 	public function renderElement($name, $params = array()) {
 		return $name;
@@ -73,7 +73,7 @@ 
  * getViewFileName method
  *
  * @param string $name
- * @return void
+ * @return string
  */
 	public function getViewFileName($name = null) {
 		return $this->_getViewFileName($name);
@@ -83,7 +83,7 @@ 
  * getLayoutFileName method
  *
  * @param string $name
- * @return void
+ * @return string
  */
 	public function getLayoutFileName($name = null) {
 		return $this->_getLayoutFileName($name);

Spacing +11 added lines, -11 removed lines

@@ -112,8 +112,8 @@ 
 		$this->PostsController->index();
 		$this->ThemeView = new ThemeView($this->PostsController);
 		App::build(array(
-			'Plugin' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Plugin' . DS),
-			'View' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS)
+			'Plugin' => array(CAKE.'Test'.DS.'test_app'.DS.'Plugin'.DS),
+			'View' => array(CAKE.'Test'.DS.'test_app'.DS.'View'.DS)
 		));
 		App::objects('plugins', null, false);
 		CakePlugin::load(array('TestPlugin'));
@@ -145,15 +145,15 @@ 
 		$this->Controller->theme = 'TestTheme';
 
 		$ThemeView = new TestTheme2View($this->Controller);
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS . 'Themed' . DS . 'TestTheme' . DS . 'Plugin' . DS . 'TestPlugin' . DS . 'Tests' . DS . 'index.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'View'.DS.'Themed'.DS.'TestTheme'.DS.'Plugin'.DS.'TestPlugin'.DS.'Tests'.DS.'index.ctp';
 		$result = $ThemeView->getViewFileName('index');
 		$this->assertEquals($expected, $result);
 
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS . 'Themed' . DS . 'TestTheme' . DS . 'Plugin' . DS . 'TestPlugin' . DS . 'Layouts' . DS . 'plugin_default.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'View'.DS.'Themed'.DS.'TestTheme'.DS.'Plugin'.DS.'TestPlugin'.DS.'Layouts'.DS.'plugin_default.ctp';
 		$result = $ThemeView->getLayoutFileName('plugin_default');
 		$this->assertEquals($expected, $result);
 
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS . 'Themed' . DS . 'TestTheme' . DS . 'Layouts' . DS . 'default.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'View'.DS.'Themed'.DS.'TestTheme'.DS.'Layouts'.DS.'default.ctp';
 		$result = $ThemeView->getLayoutFileName('default');
 		$this->assertEquals($expected, $result);
 	}
@@ -172,25 +172,25 @@ 
 
 		$ThemeView = new TestTheme2View($this->Controller);
 		$ThemeView->theme = 'TestTheme';
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS . 'Pages' . DS . 'home.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'View'.DS.'Pages'.DS.'home.ctp';
 		$result = $ThemeView->getViewFileName('home');
 		$this->assertEquals($expected, $result);
 
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS . 'Themed' . DS . 'TestTheme' . DS . 'Posts' . DS . 'index.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'View'.DS.'Themed'.DS.'TestTheme'.DS.'Posts'.DS.'index.ctp';
 		$result = $ThemeView->getViewFileName('/Posts/index');
 		$this->assertEquals($expected, $result);
 
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS . 'Themed' . DS . 'TestTheme' . DS . 'Layouts' . DS . 'default.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'View'.DS.'Themed'.DS.'TestTheme'.DS.'Layouts'.DS.'default.ctp';
 		$result = $ThemeView->getLayoutFileName();
 		$this->assertEquals($expected, $result);
 
 		$ThemeView->layoutPath = 'rss';
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS . 'Layouts' . DS . 'rss' . DS . 'default.ctp';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'View'.DS.'Layouts'.DS.'rss'.DS.'default.ctp';
 		$result = $ThemeView->getLayoutFileName();
 		$this->assertEquals($expected, $result);
 
-		$ThemeView->layoutPath = 'Emails' . DS . 'html';
-		$expected = CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS . 'Layouts' . DS . 'Emails' . DS . 'html' . DS . 'default.ctp';
+		$ThemeView->layoutPath = 'Emails'.DS.'html';
+		$expected = CAKE.'Test'.DS.'test_app'.DS.'View'.DS.'Layouts'.DS.'Emails'.DS.'html'.DS.'default.ctp';
 		$result = $ThemeView->getLayoutFileName();
 		$this->assertEquals($expected, $result);
 	}

lib/Cake/TestSuite/CakeTestLoader.php

Doc Comments +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
  *
  * @param string $filePath
  * @param string $params
- * @return void
+ * @return string
  */
 	protected function _resolveTestFile($filePath, $params) {
 		$basePath = $this->_basePath($params) . DS . $filePath;
@@ -55,7 +55,7 @@ 
 /**
  * Generates the base path to a set of tests based on the parameters.
  *
- * @param array $params
+ * @param string $params
  * @return string The base path.
  */
 	protected static function _basePath($params) {

Spacing +10 added lines, -10 removed lines

@@ -47,9 +47,9 @@ 
  * @return void
  */
 	protected function _resolveTestFile($filePath, $params) {
-		$basePath = $this->_basePath($params) . DS . $filePath;
+		$basePath = $this->_basePath($params).DS.$filePath;
 		$ending = 'Test.php';
-		return (strpos($basePath, $ending) === (strlen($basePath) - strlen($ending))) ? $basePath : $basePath . $ending;
+		return (strpos($basePath, $ending) === (strlen($basePath) - strlen($ending))) ? $basePath : $basePath.$ending;
 	}
 
 /**
@@ -60,19 +60,19 @@ 
  */
 	protected static function _basePath($params) {
 		$result = null;
-		if (!empty($params['core'])) {
+		if ( ! empty($params['core'])) {
 			$result = CORE_TEST_CASES;
-		} elseif (!empty($params['plugin'])) {
-			if (!CakePlugin::loaded($params['plugin'])) {
+		} elseif ( ! empty($params['plugin'])) {
+			if ( ! CakePlugin::loaded($params['plugin'])) {
 				try {
 					CakePlugin::load($params['plugin']);
-					$result = CakePlugin::path($params['plugin']) . 'Test' . DS . 'Case';
+					$result = CakePlugin::path($params['plugin']).'Test'.DS.'Case';
 				} catch (MissingPluginException $e) {
 				}
 			} else {
-				$result = CakePlugin::path($params['plugin']) . 'Test' . DS . 'Case';
+				$result = CakePlugin::path($params['plugin']).'Test'.DS.'Case';
 			}
-		} elseif (!empty($params['app'])) {
+		} elseif ( ! empty($params['app'])) {
 			$result = APP_TEST_CASES;
 		}
 		return $result;
@@ -90,7 +90,7 @@ 
 
 		$testCases = array();
 		foreach ($fileList as $testCaseFile) {
-			$case = str_replace($directory . DS, '', $testCaseFile);
+			$case = str_replace($directory.DS, '', $testCaseFile);
 			$case = str_replace('Test.php', '', $case);
 			$testCases[$testCaseFile] = $case;
 		}
@@ -107,7 +107,7 @@ 
  */
 	protected static function _getRecursiveFileList($directory = '.') {
 		$fileList = array();
-		if (!is_dir($directory)) {
+		if ( ! is_dir($directory)) {
 			return $fileList;
 		}
 

lib/Cake/TestSuite/CakeTestRunner.php

Doc Comments +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
  *
  * @param PHPUnit_Framework_Test $suite
  * @param array $arguments
- * @return void
+ * @return PHPUnit_Framework_TestResult
  */
 	public function doRun(PHPUnit_Framework_Test $suite, array $arguments = array()) {
 		if (isset($arguments['printer'])) {

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
  */
 	protected function createTestResult() {
 		$result = new PHPUnit_Framework_TestResult;
-		if (!empty($this->_params['codeCoverage'])) {
+		if ( ! empty($this->_params['codeCoverage'])) {
 			if (method_exists($result, 'collectCodeCoverageInformation')) {
 				$result->collectCodeCoverageInformation(true);
 			}

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()) {