Permalink
Browse files

Merge pull request #959 from ceeram/phpcsw

Make phpcs pass
  • Loading branch information...
2 parents fb275c5 + 6d0e32e commit 4611c987db80830d236071afd3d9eb5a387ad99d @ceeram ceeram committed Nov 19, 2012
Showing with 117 additions and 74 deletions.
  1. +10 −0 lib/Cake/Console/Command/ConsoleShell.php
  2. +0 −1 lib/Cake/Console/Command/Task/TemplateTask.php
  3. +2 −0 lib/Cake/Console/Shell.php
  4. +1 −1 lib/Cake/Controller/Controller.php
  5. +8 −0 lib/Cake/Error/exceptions.php
  6. +0 −1 lib/Cake/I18n/Multibyte.php
  7. +0 −1 lib/Cake/Model/Behavior/TreeBehavior.php
  8. +0 −1 lib/Cake/Model/BehaviorCollection.php
  9. +0 −1 lib/Cake/Model/Datasource/DataSource.php
  10. +3 −1 lib/Cake/Model/Datasource/Database/Sqlite.php
  11. +0 −1 lib/Cake/Model/Permission.php
  12. +2 −0 lib/Cake/Network/CakeSocket.php
  13. +2 −0 lib/Cake/Network/Email/MailTransport.php
  14. +0 −10 lib/Cake/Network/Http/HttpResponse.php
  15. +0 −3 lib/Cake/Network/Http/HttpSocket.php
  16. +6 −2 lib/Cake/Test/Case/BasicsTest.php
  17. +2 −0 lib/Cake/Test/Case/Cache/Engine/MemcacheEngineTest.php
  18. +6 −2 lib/Cake/Test/Case/Core/ConfigureTest.php
  19. +10 −4 lib/Cake/Test/Case/Error/ErrorHandlerTest.php
  20. +1 −1 lib/Cake/Test/Case/Log/CakeLogTest.php
  21. +0 −20 lib/Cake/Test/Case/Model/BehaviorCollectionTest.php
  22. +0 −1 lib/Cake/Test/Case/Model/ModelReadTest.php
  23. +0 −1 lib/Cake/Test/Case/Network/Http/HttpSocketTest.php
  24. +0 −1 lib/Cake/Test/Case/Routing/RouterTest.php
  25. +5 −1 lib/Cake/Test/Case/View/Helper/JsHelperTest.php
  26. +3 −3 lib/Cake/Test/Case/View/ScaffoldViewTest.php
  27. +3 −1 lib/Cake/Test/Case/View/ViewTest.php
  28. +1 −1 lib/Cake/TestSuite/CakeTestCase.php
  29. +2 −0 lib/Cake/TestSuite/CakeTestSuiteCommand.php
  30. +4 −4 lib/Cake/TestSuite/ControllerTestCase.php
  31. +4 −1 lib/Cake/Utility/Debugger.php
  32. +12 −0 lib/Cake/Utility/Folder.php
  33. +1 −1 lib/Cake/Utility/ObjectCollection.php
  34. +0 −1 lib/Cake/Utility/Security.php
  35. +6 −3 lib/Cake/View/Helper.php
  36. +6 −0 lib/Cake/View/MediaView.php
  37. +3 −1 lib/Cake/View/View.php
  38. +14 −4 lib/Cake/basics.php
@@ -238,7 +238,9 @@ public function main($command = null) {
if ($this->_isValidModel($modelToCheck)) {
$findCommand = "\$data = \$this->$command;";
+ //@codingStandardsIgnoreStart
@eval($findCommand);
+ //@codingStandardsIgnoreEnd
if (is_array($data)) {
foreach ($data as $idx => $results) {
@@ -294,7 +296,9 @@ public function main($command = null) {
list($foo, $data) = explode("->save", $command);
$data = preg_replace('/^\(*(array)?\(*(.+?)\)*$/i', '\\2', $data);
$saveCommand = "\$this->{$modelToSave}->save(array('{$modelToSave}' => array({$data})));";
+ //@codingStandardsIgnoreStart
@eval($saveCommand);
+ //@codingStandardsIgnoreEnd
$this->out(__d('cake_console', 'Saved record for %s', $modelToSave));
}
break;
@@ -304,7 +308,9 @@ public function main($command = null) {
if ($this->_isValidModel($modelToCheck)) {
// Get the column info for this model
$fieldsCommand = "\$data = \$this->{$modelToCheck}->getColumnTypes();";
+ //@codingStandardsIgnoreStart
@eval($fieldsCommand);
+ //@codingStandardsIgnoreEnd
if (is_array($data)) {
foreach ($data as $field => $type) {
@@ -326,7 +332,9 @@ public function main($command = null) {
$this->out(print_r(Hash::combine(Router::$routes, '{n}.template', '{n}.defaults'), true));
break;
case (preg_match("/^route\s+(\(.*\))$/i", $command, $tmp) == true):
+ //@codingStandardsIgnoreStart
if ($url = eval('return array' . $tmp[1] . ';')) {
+ //@codingStandardsIgnoreEnd
$this->out(Router::url($url));
}
break;
@@ -362,7 +370,9 @@ protected function _loadRoutes() {
Router::reload();
extract(Router::getNamedExpressions());
+ //@codingStandardsIgnoreStart
if (!@include APP . 'Config' . DS . 'routes.php') {
+ //@codingStandardsIgnoreEnd
return false;
}
CakePlugin::routes();
@@ -78,7 +78,6 @@ protected function _findThemes() {
$paths[] = $core;
- // TEMPORARY TODO remove when all paths are DS terminated
foreach ($paths as $i => $path) {
$paths[$i] = rtrim($path, DS) . DS;
}
@@ -686,7 +686,9 @@ public function createFile($path, $contents) {
protected function _checkUnitTest() {
if (class_exists('PHPUnit_Framework_TestCase')) {
return true;
+ //@codingStandardsIgnoreStart
} elseif (@include 'PHPUnit' . DS . 'Autoload.php') {
+ //@codingStandardsIgnoreEnd
return true;
} elseif (App::import('Vendor', 'phpunit', array('file' => 'PHPUnit' . DS . 'Autoload.php'))) {
return true;
@@ -755,7 +755,7 @@ public function redirect($url, $status = null, $exit = true) {
extract($status, EXTR_OVERWRITE);
}
$event = new CakeEvent('Controller.beforeRedirect', $this, array($url, $status, $exit));
- //TODO: Remove the following line when the events are fully migrated to the CakeEventManager
+
list($event->break, $event->breakOn, $event->collectReturn) = array(true, false, true);
$this->getEventManager()->dispatch($event);
@@ -224,9 +224,11 @@ class MissingControllerException extends CakeException {
protected $_messageTemplate = 'Controller class %s could not be found.';
+//@codingStandardsIgnoreStart
public function __construct($message, $code = 404) {
parent::__construct($message, $code);
}
+//@codingStandardsIgnoreEnd
}
@@ -240,9 +242,11 @@ class MissingActionException extends CakeException {
protected $_messageTemplate = 'Action %s::%s() could not be found.';
+//@codingStandardsIgnoreStart
public function __construct($message, $code = 404) {
parent::__construct($message, $code);
}
+//@codingStandardsIgnoreEnd
}
@@ -256,9 +260,11 @@ class PrivateActionException extends CakeException {
protected $_messageTemplate = 'Private Action %s::%s() is not directly accessible.';
+//@codingStandardsIgnoreStart
public function __construct($message, $code = 404, Exception $previous = null) {
parent::__construct($message, $code, $previous);
}
+//@codingStandardsIgnoreEnd
}
@@ -572,8 +578,10 @@ class NotImplementedException extends CakeException {
protected $_messageTemplate = '%s is not implemented.';
+//@codingStandardsIgnoreStart
public function __construct($message, $code = 501) {
parent::__construct($message, $code);
}
+//@codingStandardsIgnoreEnd
}
@@ -987,7 +987,6 @@ public static function substr($string, $start, $length = null) {
* @param string $charset charset to use for encoding. defaults to UTF-8
* @param string $newline
* @return string
- * @TODO: add support for 'Q'('Quoted Printable') encoding
*/
public static function mimeEncode($string, $charset = null, $newline = "\r\n") {
if (!Multibyte::checkMultibyte($string) && strlen($string) < 75) {
@@ -588,7 +588,6 @@ public function moveUp(Model $Model, $id = null, $number = 1) {
* 'parent' the values of the parent_id field will be used to populate the left and right fields. The missingParentAction
* parameter only applies to "parent" mode and determines what to do if the parent field contains an id that is not present.
*
- * @todo Could be written to be faster, *maybe*. Ideally using a subquery and putting all the logic burden on the DB.
* @param Model $Model Model instance
* @param string $mode parent or tree
* @param string|integer $missingParentAction 'return' to do nothing and return, 'delete' to
@@ -55,7 +55,6 @@ class BehaviorCollection extends ObjectCollection implements CakeEventListener {
/**
* Attaches a model object and loads a list of behaviors
*
- * @todo Make this method a constructor instead..
* @param string $modelName
* @param array $behaviors
* @return void
@@ -322,7 +322,6 @@ protected function _cacheDescription($object, $data = null) {
* @param Model $linkModel Instance of model to replace $__cakeForeignKey__$
* @param array $stack
* @return string String of query data with placeholders replaced.
- * @todo Remove and refactor $assocData, ensure uses of the method have the param removed too.
*/
public function insertQueryData($query, $data, $association, $assocData, Model $model, Model $linkModel, $stack) {
$keys = array('{$__cakeID__$}', '{$__cakeForeignKey__$}');
@@ -249,7 +249,9 @@ public function column($real) {
$col = strtolower(str_replace(')', '', $real));
$limit = null;
- @list($col, $limit) = explode('(', $col);
+ if (strpos($col, '(') !== false) {
+ list($col, $limit) = explode('(', $col);
+ }
if (in_array($col, array('text', 'integer', 'float', 'boolean', 'timestamp', 'date', 'datetime', 'time'))) {
return $col;
@@ -146,7 +146,6 @@ public function check($aro, $aco, $action = "*") {
return false;
case 0:
continue;
- break;
case 1:
return true;
}
@@ -105,11 +105,13 @@ public function connect() {
$scheme = 'ssl://';
}
+ //@codingStandardsIgnoreStart
if ($this->config['persistent'] == true) {
$this->connection = @pfsockopen($scheme . $this->config['host'], $this->config['port'], $errNum, $errStr, $this->config['timeout']);
} else {
$this->connection = @fsockopen($scheme . $this->config['host'], $this->config['port'], $errNum, $errStr, $this->config['timeout']);
}
+ //@codingStandardsIgnoreEnd
if (!empty($errNum) || !empty($errStr)) {
$this->setLastError($errNum, $errStr);
@@ -63,9 +63,11 @@ public function send(CakeEmail $email) {
* @return void
*/
protected function _mail($to, $subject, $message, $headers, $params = null) {
+ //@codingStandardsIgnoreStart
if (!@mail($to, $subject, $message, $headers, $params)) {
throw new SocketException(__d('cake_dev', 'Could not send email.'));
}
+ //@codingStandardsIgnoreEnd
}
}
@@ -219,27 +219,20 @@ protected function _decodeChunkedBody($body) {
$chunkSize = 0;
$hexLength = 0;
- $chunkExtensionName = '';
$chunkExtensionValue = '';
if (isset($match[0])) {
$chunkSize = $match[0];
}
if (isset($match[1])) {
$hexLength = $match[1];
}
- if (isset($match[2])) {
- $chunkExtensionName = $match[2];
- }
if (isset($match[3])) {
$chunkExtensionValue = $match[3];
}
$body = substr($body, strlen($chunkSize));
$chunkLength = hexdec($hexLength);
$chunk = substr($body, 0, $chunkLength);
- if (!empty($chunkExtensionName)) {
- // @todo See if there are popular chunk extensions we should implement
- }
$decodedBody .= $chunk;
if ($chunkLength !== 0) {
$body = substr($body, $chunkLength + strlen("\r\n"));
@@ -291,7 +284,6 @@ protected function _parseHeader($header) {
*
* @param array $header Header array containing one ore more 'Set-Cookie' headers.
* @return mixed Either false on no cookies, or an array of cookies received.
- * @todo Make this 100% RFC 2965 confirm
*/
public function parseCookies($header) {
$cookieHeader = $this->getHeader('Set-Cookie', $header);
@@ -334,7 +326,6 @@ public function parseCookies($header) {
* @param string $token Token to unescape
* @param array $chars
* @return string Unescaped token
- * @todo Test $chars parameter
*/
protected function _unescapeToken($token, $chars = null) {
$regex = '/"([' . implode('', $this->_tokenEscapeChars(true, $chars)) . '])"/';
@@ -348,7 +339,6 @@ protected function _unescapeToken($token, $chars = null) {
* @param boolean $hex true to get them as HEX values, false otherwise
* @param array $chars
* @return array Escape chars
- * @todo Test $chars parameter
*/
protected function _tokenEscapeChars($hex = true, $chars = null) {
if (!empty($chars)) {
@@ -901,7 +901,6 @@ protected function _buildHeader($header, $mode = 'standard') {
*
* @param array $cookies Array of cookies to send with the request.
* @return string Cookie header string to be sent with the request.
- * @todo Refactor token escape mechanism to be configurable
*/
public function buildCookies($cookies) {
$header = array();
@@ -917,7 +916,6 @@ public function buildCookies($cookies) {
* @param string $token Token to escape
* @param array $chars
* @return string Escaped token
- * @todo Test $chars parameter
*/
protected function _escapeToken($token, $chars = null) {
$regex = '/([' . implode('', $this->_tokenEscapeChars(true, $chars)) . '])/';
@@ -931,7 +929,6 @@ protected function _escapeToken($token, $chars = null) {
* @param boolean $hex true to get them as HEX values, false otherwise
* @param array $chars
* @return array Escape chars
- * @todo Test $chars parameter
*/
protected function _tokenEscapeChars($hex = true, $chars = null) {
if (!empty($chars)) {
@@ -282,7 +282,9 @@ public function testCache() {
$result = cache('basics_test');
$this->assertEquals('simple cache write', $result);
- @unlink(CACHE . 'basics_test');
+ if (file_exists(CACHE . 'basics_test')) {
+ unlink(CACHE . 'basics_test');
+ }
cache('basics_test', 'expired', '+1 second');
sleep(2);
@@ -603,7 +605,9 @@ public function testTranslateDomainCategoryPlural() {
* @return void
*/
public function testLogError() {
- @unlink(LOGS . 'error.log');
+ if (file_exists(LOGS . 'error.log')) {
+ unlink(LOGS . 'error.log');
+ }
// disable stderr output for this test
if (CakeLog::stream('stderr')) {
@@ -111,9 +111,11 @@ public function testMultipleServers() {
foreach ($servers as $server) {
list($host, $port) = explode(':', $server);
+ //@codingStandardsIgnoreStart
if (!@$Memcache->connect($host, $port)) {
$available = false;
}
+ //@codingStandardsIgnoreEnd
}
$this->skipIf(!$available, 'Need memcache servers at ' . implode(', ', $servers) . ' to run this test.');
@@ -383,7 +383,9 @@ public function testDump() {
$result = file_get_contents(TMP . 'config_test.php');
$this->assertContains('<?php', $result);
$this->assertContains('$config = ', $result);
- @unlink(TMP . 'config_test.php');
+ if (file_exists(TMP . 'config_test.php')) {
+ unlink(TMP . 'config_test.php');
+ }
}
/**
@@ -402,7 +404,9 @@ public function testDumpPartial() {
$this->assertContains('Error', $result);
$this->assertNotContains('debug', $result);
- @unlink(TMP . 'config_test.php');
+ if (file_exists(TMP . 'config_test.php')) {
+ unlink(TMP . 'config_test.php');
+ }
}
}
@@ -124,7 +124,9 @@ public function testErrorSuppressed() {
$this->_restoreError = true;
ob_start();
+ //@codingStandardsIgnoreStart
@include 'invalid.file';
+ //@codingStandardsIgnoreEnd
$result = ob_get_clean();
$this->assertTrue(empty($result));
}
@@ -138,7 +140,7 @@ public function testHandleErrorDebugOff() {
Configure::write('debug', 0);
Configure::write('Error.trace', false);
if (file_exists(LOGS . 'debug.log')) {
- @unlink(LOGS . 'debug.log');
+ unlink(LOGS . 'debug.log');
}
set_error_handler('ErrorHandler::handleError');
@@ -152,7 +154,9 @@ public function testHandleErrorDebugOff() {
'/^\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} (Notice|Debug): Notice \(8\): Undefined variable:\s+out in \[.+ line \d+\]$/',
$result[0]
);
- @unlink(LOGS . 'debug.log');
+ if (file_exists(LOGS . 'debug.log')) {
+ unlink(LOGS . 'debug.log');
+ }
}
/**
@@ -164,7 +168,7 @@ public function testHandleErrorLoggingTrace() {
Configure::write('debug', 0);
Configure::write('Error.trace', true);
if (file_exists(LOGS . 'debug.log')) {
- @unlink(LOGS . 'debug.log');
+ unlink(LOGS . 'debug.log');
}
set_error_handler('ErrorHandler::handleError');
@@ -179,7 +183,9 @@ public function testHandleErrorLoggingTrace() {
);
$this->assertRegExp('/^Trace:/', $result[1]);
$this->assertRegExp('/^ErrorHandlerTest\:\:testHandleErrorLoggingTrace\(\)/', $result[2]);
- @unlink(LOGS . 'debug.log');
+ if (file_exists(LOGS . 'debug.log')) {
+ unlink(LOGS . 'debug.log');
+ }
}
/**
@@ -150,7 +150,7 @@ public function testConfig() {
$this->assertEquals(array('file'), $result);
if (file_exists(LOGS . 'error.log')) {
- @unlink(LOGS . 'error.log');
+ unlink(LOGS . 'error.log');
}
CakeLog::write(LOG_WARNING, 'Test warning');
$this->assertTrue(file_exists(LOGS . 'error.log'));
Oops, something went wrong.

0 comments on commit 4611c98

Please sign in to comment.