Browse Source

Used PHP 7 list syntax where appropriate

tags/3.0.0-alpha1
Angel Guevara 7 years ago committed by Alexander Makarov
parent
commit
0117459444
  1. 3
      .travis.yml
  2. 6
      build/controllers/PhpDocController.php
  3. 2
      build/controllers/ReleaseController.php
  4. 4
      composer.json
  5. 4
      framework/base/Module.php
  6. 4
      framework/base/Security.php
  7. 2
      framework/captcha/CaptchaValidator.php
  8. 2
      framework/console/Application.php
  9. 4
      framework/console/UnknownCommandException.php
  10. 2
      framework/console/controllers/BaseMigrateController.php
  11. 14
      framework/console/controllers/HelpController.php
  12. 2
      framework/console/controllers/ServeController.php
  13. 17
      framework/db/ActiveQuery.php
  14. 2
      framework/db/ActiveRelationTrait.php
  15. 6
      framework/db/BaseActiveRecord.php
  16. 2
      framework/db/Query.php
  17. 28
      framework/db/QueryBuilder.php
  18. 4
      framework/db/mysql/QueryBuilder.php
  19. 4
      framework/db/oci/QueryBuilder.php
  20. 2
      framework/db/sqlite/QueryBuilder.php
  21. 2
      framework/di/Container.php
  22. 4
      framework/filters/RateLimiter.php
  23. 3
      framework/helpers/BaseArrayHelper.php
  24. 2
      framework/helpers/BaseConsole.php
  25. 6
      framework/i18n/Formatter.php
  26. 2
      framework/log/DbTarget.php
  27. 6
      framework/log/Logger.php
  28. 2
      framework/log/SyslogTarget.php
  29. 2
      framework/log/Target.php
  30. 4
      framework/mutex/PgsqlMutex.php
  31. 2
      framework/rbac/DbManager.php
  32. 4
      framework/rest/Serializer.php
  33. 2
      framework/validators/ImageValidator.php
  34. 6
      framework/validators/IpValidator.php
  35. 2
      framework/validators/Validator.php
  36. 2
      framework/web/Application.php
  37. 2
      framework/web/AssetBundle.php
  38. 2
      framework/web/AssetConverter.php
  39. 6
      framework/web/MultipartFormDataParser.php
  40. 4
      framework/web/Request.php
  41. 6
      framework/web/Response.php
  42. 2
      framework/web/User.php
  43. 2
      framework/widgets/LinkPager.php
  44. 6
      tests/framework/console/ControllerTest.php
  45. 2
      tests/framework/console/RequestTest.php
  46. 4
      tests/framework/db/DatabaseTestCase.php
  47. 57
      tests/framework/db/QueryBuilderTest.php
  48. 6
      tests/framework/db/mssql/QueryBuilderTest.php
  49. 4
      tests/framework/db/mysql/connection/DeadLockTest.php
  50. 4
      tests/framework/db/sqlite/ConnectionTest.php
  51. 2
      tests/framework/db/sqlite/QueryBuilderTest.php
  52. 2
      tests/framework/di/ContainerTest.php
  53. 4
      tests/framework/log/SyslogTargetTest.php
  54. 20
      tests/framework/rest/UrlRuleTest.php
  55. 4
      tests/framework/web/ControllerTest.php
  56. 13
      tests/framework/web/GroupUrlRuleTest.php
  57. 44
      tests/framework/web/UrlRuleTest.php

3
.travis.yml

@ -24,9 +24,6 @@ sudo: false
language: php
php:
- 5.5
- 5.6
- 7.0
- 7.1
- nightly

6
build/controllers/PhpDocController.php

@ -52,7 +52,7 @@ class PhpDocController extends Controller
foreach ($files as $file) {
$result = $this->generateClassPropertyDocs($file);
if ($result !== false) {
list($className, $phpdoc) = $result;
[$className, $phpdoc] = $result;
if ($this->updateFiles) {
if ($this->updateClassPropertyDocs($file, $className, $phpdoc)) {
$nFilesUpdated++;
@ -177,7 +177,7 @@ class PhpDocController extends Controller
$extensionPath = dirname(rtrim($root, '\\/'));
$this->setUpExtensionAliases($extensionPath);
list(, $extension) = $matches;
$extension = $matches[1];
Yii::setAlias("@yii/$extension", "$root");
if (is_file($autoloadFile = Yii::getAlias("@yii/$extension/vendor/autoload.php"))) {
include($autoloadFile);
@ -201,7 +201,7 @@ class PhpDocController extends Controller
$extensionPath = dirname(dirname(rtrim($root, '\\/'))) . '/extensions';
$this->setUpExtensionAliases($extensionPath);
list(, $appName) = $matches;
$appName = $matches[1];
Yii::setAlias("@app-$appName", "$root");
if (is_file($autoloadFile = Yii::getAlias("@app-$appName/vendor/autoload.php"))) {
include($autoloadFile);

2
build/controllers/ReleaseController.php

@ -841,7 +841,7 @@ class ReleaseController extends Controller
{
foreach ($this->getChangelogs($what) as $file) {
// split the file into relevant parts
list($start, $changelog, $end) = $this->splitChangelog($file, $version);
[$start, $changelog, $end] = $this->splitChangelog($file, $version);
$changelog = $this->resortChangelog($changelog);
file_put_contents($file, implode("\n", array_merge($start, $changelog, $end)));
}

4
composer.json

@ -68,7 +68,7 @@
"yiisoft/yii2": "self.version"
},
"require": {
"php": ">=5.5.0",
"php": ">=7.1.0",
"ext-mbstring": "*",
"ext-ctype": "*",
"lib-pcre": "*",
@ -99,7 +99,7 @@
}
},
"config": {
"platform": {"php": "5.5"}
"platform": {"php": "7.1"}
},
"bin": [
"framework/yii"

4
framework/base/Module.php

@ -517,7 +517,7 @@ class Module extends ServiceLocator
$parts = $this->createController($route);
if (is_array($parts)) {
/* @var $controller Controller */
list($controller, $actionID) = $parts;
[$controller, $actionID] = $parts;
$oldController = Yii::$app->controller;
Yii::$app->controller = $controller;
$result = $controller->runAction($actionID, $params);
@ -567,7 +567,7 @@ class Module extends ServiceLocator
}
if (strpos($route, '/') !== false) {
list($id, $route) = explode('/', $route, 2);
[$id, $route] = explode('/', $route, 2);
} else {
$id = $route;
$route = '';

4
framework/base/Security.php

@ -180,7 +180,7 @@ class Security extends Component
throw new InvalidConfigException($this->cipher . ' is not an allowed cipher');
}
list($blockSize, $keySize) = $this->allowedCiphers[$this->cipher];
[$blockSize, $keySize] = $this->allowedCiphers[$this->cipher];
$keySalt = $this->generateRandomKey($keySize);
if ($passwordBased) {
@ -230,7 +230,7 @@ class Security extends Component
throw new InvalidConfigException($this->cipher . ' is not an allowed cipher');
}
list($blockSize, $keySize) = $this->allowedCiphers[$this->cipher];
[$blockSize, $keySize] = $this->allowedCiphers[$this->cipher];
$keySalt = StringHelper::byteSubstr($data, 0, $keySize);
if ($passwordBased) {

2
framework/captcha/CaptchaValidator.php

@ -72,7 +72,7 @@ class CaptchaValidator extends Validator
$ca = Yii::$app->createController($this->captchaAction);
if ($ca !== false) {
/* @var $controller \yii\base\Controller */
list($controller, $actionID) = $ca;
[$controller, $actionID] = $ca;
$action = $controller->createAction($actionID);
if ($action !== null) {
return $action;

2
framework/console/Application.php

@ -142,7 +142,7 @@ class Application extends \yii\base\Application
*/
public function handleRequest($request)
{
list($route, $params) = $request->resolve();
[$route, $params] = $request->resolve();
$this->requestedRoute = $route;
$result = $this->runAction($route, $params);
if ($result instanceof Response) {

4
framework/console/UnknownCommandException.php

@ -71,7 +71,7 @@ class UnknownCommandException extends Exception
return [];
}
/** @var $helpController HelpController */
list($helpController, $actionID) = $help;
[$helpController, $actionID] = $help;
$availableActions = [];
$commands = $helpController->getCommands();
@ -85,7 +85,7 @@ class UnknownCommandException extends Exception
// add all actions of this controller
/** @var $controller Controller */
list($controller, $actionID) = $result;
[$controller, $actionID] = $result;
$actions = $helpController->getActions($controller);
if (!empty($actions)) {
$prefix = $controller->getUniqueId();

2
framework/console/controllers/BaseMigrateController.php

@ -578,7 +578,7 @@ abstract class BaseMigrateController extends Controller
throw new Exception('The migration name should contain letters, digits, underscore and/or backslash characters only.');
}
list($namespace, $className) = $this->generateClassName($name);
[$namespace, $className] = $this->generateClassName($name);
$migrationPath = $this->findMigrationPath($namespace);
$file = $migrationPath . DIRECTORY_SEPARATOR . $className . '.php';

14
framework/console/controllers/HelpController.php

@ -55,7 +55,7 @@ class HelpController extends Controller
throw new Exception("No help for unknown command \"$name\".");
}
list($controller, $actionID) = $result;
[$controller, $actionID] = $result;
$actions = $this->getActions($controller);
if ($actionID !== '' || count($actions) === 1 && $actions[0] === $controller->defaultAction) {
@ -82,7 +82,7 @@ class HelpController extends Controller
continue;
}
/** @var $controller Controller */
list($controller, $actionID) = $result;
[$controller, $actionID] = $result;
$actions = $this->getActions($controller);
if (!empty($actions)) {
$prefix = $controller->getUniqueId();
@ -110,7 +110,7 @@ class HelpController extends Controller
}
/** @var Controller $controller */
list($controller, $actionID) = $result;
[$controller, $actionID] = $result;
$action = $controller->createAction($actionID);
if ($action === null) {
return;
@ -145,7 +145,7 @@ class HelpController extends Controller
}
/** @var Controller $controller */
list($controller, $actionID) = $result;
[$controller, $actionID] = $result;
$action = $controller->createAction($actionID);
if ($action === null) {
return;
@ -195,7 +195,7 @@ class HelpController extends Controller
$result = Yii::$app->createController($command);
if ($result !== false && $result[0] instanceof Controller) {
list($controller, $actionID) = $result;
[$controller, $actionID] = $result;
/** @var Controller $controller */
$description = $controller->getHelpSummary();
}
@ -294,7 +294,7 @@ class HelpController extends Controller
$result = Yii::$app->createController($command);
if ($result !== false && $result[0] instanceof Controller) {
/** @var $controller Controller */
list($controller, $actionID) = $result;
[$controller, $actionID] = $result;
$actions = $this->getActions($controller);
if (!empty($actions)) {
$prefix = $controller->getUniqueId();
@ -320,7 +320,7 @@ class HelpController extends Controller
$result = Yii::$app->createController($command);
if ($result !== false && $result[0] instanceof Controller) {
list($controller, $actionID) = $result;
[$controller, $actionID] = $result;
$actions = $this->getActions($controller);
if (!empty($actions)) {
$prefix = $controller->getUniqueId();

2
framework/console/controllers/ServeController.php

@ -113,7 +113,7 @@ class ServeController extends Controller
*/
protected function isAddressTaken($address)
{
list($hostname, $port) = explode(':', $address);
[$hostname, $port] = explode(':', $address);
$fp = @fsockopen($hostname, $port, $errno, $errstr, 3);
if ($fp === false) {
return false;

17
framework/db/ActiveQuery.php

@ -154,7 +154,7 @@ class ActiveQuery extends Query implements ActiveQueryInterface
}
if (empty($this->select) && !empty($this->join)) {
list(, $alias) = $this->getTableNameAndAlias();
$alias = $this->getTableNameAndAlias()[1];
$this->select = ["$alias.*"];
}
@ -172,7 +172,7 @@ class ActiveQuery extends Query implements ActiveQueryInterface
} elseif (is_array($this->via)) {
// via relation
/* @var $viaQuery ActiveQuery */
list($viaName, $viaQuery) = $this->via;
[$viaName, $viaQuery] = $this->via;
if ($viaQuery->multiple) {
$viaModels = $viaQuery->all();
$this->primaryModel->populateRelation($viaName, $viaModels);
@ -315,7 +315,7 @@ class ActiveQuery extends Query implements ActiveQueryInterface
}
if ($this->sql === null) {
list ($sql, $params) = $db->getQueryBuilder()->build($this);
[$sql, $params] = $db->getQueryBuilder()->build($this);
} else {
$sql = $this->sql;
$params = $this->params;
@ -409,7 +409,7 @@ class ActiveQuery extends Query implements ActiveQueryInterface
if (preg_match('/^(.*?)(?:\s+AS\s+|\s+)(\w+)$/i', $name, $matches)) {
// relation is defined with an alias, adjust callback to apply alias
list(, $relation, $alias) = $matches;
[, $relation, $alias] = $matches;
$name = $relation;
$callback = function ($query) use ($callback, $alias) {
/** @var $query ActiveQuery */
@ -436,8 +436,7 @@ class ActiveQuery extends Query implements ActiveQueryInterface
$this->join = [];
$model = new $this->modelClass;
foreach ($this->joinWith as $config) {
list ($with, $eagerLoading, $joinType) = $config;
foreach ($this->joinWith as [$with, $eagerLoading, $joinType]) {
$this->joinWithRelations($model, $with, $joinType);
if (is_array($eagerLoading)) {
@ -602,8 +601,8 @@ class ActiveQuery extends Query implements ActiveQueryInterface
return;
}
list ($parentTable, $parentAlias) = $parent->getTableNameAndAlias();
list ($childTable, $childAlias) = $child->getTableNameAndAlias();
[$parentTable, $parentAlias] = $parent->getTableNameAndAlias();
[$childTable, $childAlias] = $child->getTableNameAndAlias();
if (!empty($child->link)) {
@ -777,7 +776,7 @@ class ActiveQuery extends Query implements ActiveQueryInterface
public function alias($alias)
{
if (empty($this->from) || count($this->from) < 2) {
list($tableName, ) = $this->getTableNameAndAlias();
$tableName = $this->getTableNameAndAlias()[0];
$this->from = [$alias => $tableName];
} else {
$tableName = $this->getPrimaryTableName();

2
framework/db/ActiveRelationTrait.php

@ -207,7 +207,7 @@ trait ActiveRelationTrait
} elseif (is_array($this->via)) {
// via relation
/* @var $viaQuery ActiveRelationTrait|ActiveQueryTrait */
list($viaName, $viaQuery) = $this->via;
[$viaName, $viaQuery] = $this->via;
if ($viaQuery->asArray === null) {
// inherit asArray from primary query
$viaQuery->asArray($this->asArray);

6
framework/db/BaseActiveRecord.php

@ -1251,7 +1251,7 @@ abstract class BaseActiveRecord extends Model implements ActiveRecordInterface
}
if (is_array($relation->via)) {
/* @var $viaRelation ActiveQuery */
list($viaName, $viaRelation) = $relation->via;
[$viaName, $viaRelation] = $relation->via;
$viaClass = $viaRelation->modelClass;
// unset $viaName so that it can be reloaded to reflect the change
unset($this->_related[$viaName]);
@ -1341,7 +1341,7 @@ abstract class BaseActiveRecord extends Model implements ActiveRecordInterface
if ($relation->via !== null) {
if (is_array($relation->via)) {
/* @var $viaRelation ActiveQuery */
list($viaName, $viaRelation) = $relation->via;
[$viaName, $viaRelation] = $relation->via;
$viaClass = $viaRelation->modelClass;
unset($this->_related[$viaName]);
} else {
@ -1436,7 +1436,7 @@ abstract class BaseActiveRecord extends Model implements ActiveRecordInterface
if ($relation->via !== null) {
if (is_array($relation->via)) {
/* @var $viaRelation ActiveQuery */
list($viaName, $viaRelation) = $relation->via;
[$viaName, $viaRelation] = $relation->via;
$viaClass = $viaRelation->modelClass;
unset($this->_related[$viaName]);
} else {

2
framework/db/Query.php

@ -123,7 +123,7 @@ class Query extends Component implements QueryInterface
if ($db === null) {
$db = Yii::$app->getDb();
}
list ($sql, $params) = $db->getQueryBuilder()->build($this);
[$sql, $params] = $db->getQueryBuilder()->build($this);
return $db->createCommand($sql, $params);
}

28
framework/db/QueryBuilder.php

@ -162,7 +162,7 @@ class QueryBuilder extends \yii\base\Object
$placeholders = [];
$values = ' DEFAULT VALUES';
if ($columns instanceof \yii\db\Query) {
list($names, $values, $params) = $this->prepareInsertSelectSubQuery($columns, $schema);
[$names, $values, $params] = $this->prepareInsertSelectSubQuery($columns, $schema);
} else {
foreach ($columns as $name => $value) {
$names[] = $schema->quoteColumnName($name);
@ -172,7 +172,7 @@ class QueryBuilder extends \yii\base\Object
$params[$n] = $v;
}
} elseif ($value instanceof \yii\db\Query) {
list($sql, $params) = $this->build($value, $params);
[$sql, $params] = $this->build($value, $params);
$placeholders[] = "($sql)";
} else {
$phName = self::PARAM_PREFIX . count($params);
@ -204,7 +204,7 @@ class QueryBuilder extends \yii\base\Object
throw new InvalidArgumentException('Expected select query object with enumerated (named) parameters');
}
list($values, $params) = $this->build($columns, $params);
[$values, $params] = $this->build($columns, $params);
$names = [];
$values = ' ' . $values;
foreach ($columns->select as $title => $field) {
@ -762,7 +762,7 @@ class QueryBuilder extends \yii\base\Object
}
$params = array_merge($params, $column->params);
} elseif ($column instanceof Query) {
list($sql, $params) = $this->build($column, $params);
[$sql, $params] = $this->build($column, $params);
$columns[$i] = "($sql) AS " . $this->db->quoteColumnName($i);
} elseif (is_string($i)) {
if (strpos($column, '(') === false) {
@ -814,7 +814,7 @@ class QueryBuilder extends \yii\base\Object
throw new Exception('A join clause must be specified as an array of join type, join table, and optionally join condition.');
}
// 0:join type, 1:join table, 2:on-condition (optional)
list ($joinType, $table) = $join;
[$joinType, $table] = $join;
$tables = $this->quoteTableNames((array) $table, $params);
$table = reset($tables);
$joins[$i] = "$joinType $table";
@ -840,7 +840,7 @@ class QueryBuilder extends \yii\base\Object
{
foreach ($tables as $i => $table) {
if ($table instanceof Query) {
list($sql, $params) = $this->build($table, $params);
[$sql, $params] = $this->build($table, $params);
$tables[$i] = "($sql) " . $this->db->quoteTableName($i);
} elseif (is_string($i)) {
if (strpos($table, '(') === false) {
@ -1002,7 +1002,7 @@ class QueryBuilder extends \yii\base\Object
foreach ($unions as $i => $union) {
$query = $union['query'];
if ($query instanceof Query) {
list($unions[$i]['query'], $params) = $this->build($query, $params);
[$unions[$i]['query'], $params] = $this->build($query, $params);
}
$result .= 'UNION ' . ($union['all'] ? 'ALL ' : '') . '( ' . $unions[$i]['query'] . ' ) ';
@ -1176,7 +1176,7 @@ class QueryBuilder extends \yii\base\Object
throw new InvalidArgumentException("Operator '$operator' requires three operands.");
}
list($column, $value1, $value2) = $operands;
[$column, $value1, $value2] = $operands;
if (strpos($column, '(') === false) {
$column = $this->db->quoteColumnName($column);
@ -1221,7 +1221,7 @@ class QueryBuilder extends \yii\base\Object
throw new Exception("Operator '$operator' requires two operands.");
}
list($column, $values) = $operands;
[$column, $values] = $operands;
if ($column === []) {
// no columns to test against
@ -1288,7 +1288,7 @@ class QueryBuilder extends \yii\base\Object
*/
protected function buildSubqueryInCondition($operator, $columns, $values, &$params)
{
list($sql, $params) = $this->build($values, $params);
[$sql, $params] = $this->build($values, $params);
if (is_array($columns)) {
foreach ($columns as $i => $col) {
if (strpos($col, '(') === false) {
@ -1378,7 +1378,7 @@ class QueryBuilder extends \yii\base\Object
$not = !empty($matches[3]);
$operator = $matches[2];
list($column, $values) = $operands;
[$column, $values] = $operands;
if (!is_array($values)) {
$values = [$values];
@ -1424,7 +1424,7 @@ class QueryBuilder extends \yii\base\Object
public function buildExistsCondition($operator, $operands, &$params)
{
if ($operands[0] instanceof Query) {
list($sql, $params) = $this->build($operands[0], $params);
[$sql, $params] = $this->build($operands[0], $params);
return "$operator ($sql)";
} else {
throw new InvalidArgumentException('Subquery for EXISTS operator must be a Query object.');
@ -1445,7 +1445,7 @@ class QueryBuilder extends \yii\base\Object
throw new InvalidArgumentException("Operator '$operator' requires two operands.");
}
list($column, $value) = $operands;
[$column, $value] = $operands;
if (strpos($column, '(') === false) {
$column = $this->db->quoteColumnName($column);
@ -1459,7 +1459,7 @@ class QueryBuilder extends \yii\base\Object
}
return "$column $operator {$value->expression}";
} elseif ($value instanceof Query) {
list($sql, $params) = $this->build($value, $params);
[$sql, $params] = $this->build($value, $params);
return "$column $operator ($sql)";
} else {
$phName = self::PARAM_PREFIX . count($params);

4
framework/db/mysql/QueryBuilder.php

@ -216,7 +216,7 @@ class QueryBuilder extends \yii\db\QueryBuilder
$placeholders = [];
$values = ' DEFAULT VALUES';
if ($columns instanceof \yii\db\Query) {
list($names, $values, $params) = $this->prepareInsertSelectSubQuery($columns, $schema, $params);
[$names, $values, $params] = $this->prepareInsertSelectSubQuery($columns, $schema, $params);
} else {
foreach ($columns as $name => $value) {
$names[] = $schema->quoteColumnName($name);
@ -226,7 +226,7 @@ class QueryBuilder extends \yii\db\QueryBuilder
$params[$n] = $v;
}
} elseif ($value instanceof \yii\db\Query) {
list($sql, $params) = $this->build($value, $params);
[$sql, $params] = $this->build($value, $params);
$placeholders[] = "($sql)";
} else {
$phName = self::PARAM_PREFIX . count($params);

4
framework/db/oci/QueryBuilder.php

@ -195,7 +195,7 @@ EOD;
$placeholders = [];
$values = ' DEFAULT VALUES';
if ($columns instanceof \yii\db\Query) {
list($names, $values, $params) = $this->prepareInsertSelectSubQuery($columns, $schema, $params);
[$names, $values, $params] = $this->prepareInsertSelectSubQuery($columns, $schema, $params);
} else {
foreach ($columns as $name => $value) {
$names[] = $schema->quoteColumnName($name);
@ -205,7 +205,7 @@ EOD;
$params[$n] = $v;
}
} elseif ($value instanceof \yii\db\Query) {
list($sql, $params) = $this->build($value, $params);
[$sql, $params] = $this->build($value, $params);
$placeholders[] = "($sql)";
} else {
$phName = self::PARAM_PREFIX . count($params);

2
framework/db/sqlite/QueryBuilder.php

@ -448,7 +448,7 @@ class QueryBuilder extends \yii\db\QueryBuilder
foreach ($unions as $i => $union) {
$query = $union['query'];
if ($query instanceof Query) {
list($unions[$i]['query'], $params) = $this->build($query, $params);
[$unions[$i]['query'], $params] = $this->build($query, $params);
}
$result .= ' UNION ' . ($union['all'] ? 'ALL ' : '') . ' ' . $unions[$i]['query'];

2
framework/di/Container.php

@ -362,7 +362,7 @@ class Container extends Component
protected function build($class, $params, $config)
{
/* @var $reflection ReflectionClass */
list($reflection, $dependencies) = $this->getDependencies($class);
[$reflection, $dependencies] = $this->getDependencies($class);
foreach ($params as $index => $param) {
$dependencies[$index] = $param;

4
framework/filters/RateLimiter.php

@ -108,8 +108,8 @@ class RateLimiter extends ActionFilter
{
$current = time();
list ($limit, $window) = $user->getRateLimit($request, $action);
list ($allowance, $timestamp) = $user->loadAllowance($request, $action);
[$limit, $window] = $user->getRateLimit($request, $action);
[$allowance, $timestamp] = $user->loadAllowance($request, $action);
$allowance += (int) (($current - $timestamp) * $limit / $window);
if ($allowance > $limit) {

3
framework/helpers/BaseArrayHelper.php

@ -873,8 +873,7 @@ class BaseArrayHelper
$result[$globalKey][$localKey] = $array[$globalKey][$localKey];
}
foreach ($forbiddenVars as $var) {
list($globalKey, $localKey) = $var;
foreach ($forbiddenVars as [$globalKey, $localKey]) {
if (array_key_exists($globalKey, $result)) {
unset($result[$globalKey][$localKey]);
}

2
framework/helpers/BaseConsole.php

@ -596,7 +596,7 @@ class BaseConsole
}
/**
* Usage: list($width, $height) = ConsoleHelper::getScreenSize();
* Usage: [$width, $height] = ConsoleHelper::getScreenSize();
*
* @param bool $refresh whether to force checking and not re-use cached size value.
* This is useful to detect changing window size while the application is running but may

6
framework/i18n/Formatter.php

@ -602,7 +602,7 @@ class Formatter extends Component
$timeZone = $this->timeZone;
// avoid time zone conversion for date-only and time-only values
if ($type === 'date' || $type === 'time') {
list($timestamp, $hasTimeInfo, $hasDateInfo) = $this->normalizeDatetimeValue($value, true);
[$timestamp, $hasTimeInfo, $hasDateInfo] = $this->normalizeDatetimeValue($value, true);
if ($type === 'date' && !$hasTimeInfo || $type === 'time' && !$hasDateInfo) {
$timeZone = $this->defaultTimeZone;
}
@ -1188,7 +1188,7 @@ class Formatter extends Component
return $this->nullDisplay;
}
list($params, $position) = $this->formatSizeNumber($value, $decimals, $options, $textOptions);
[$params, $position] = $this->formatSizeNumber($value, $decimals, $options, $textOptions);
if ($this->sizeFormatBase == 1024) {
switch ($position) {
@ -1244,7 +1244,7 @@ class Formatter extends Component
return $this->nullDisplay;
}
list($params, $position) = $this->formatSizeNumber($value, $decimals, $options, $textOptions);
[$params, $position] = $this->formatSizeNumber($value, $decimals, $options, $textOptions);
if ($this->sizeFormatBase == 1024) {
switch ($position) {

2
framework/log/DbTarget.php

@ -71,7 +71,7 @@ class DbTarget extends Target
VALUES (:level, :category, :log_time, :prefix, :message)";
$command = $this->db->createCommand($sql);
foreach ($this->messages as $message) {
list($text, $level, $category, $timestamp) = $message;
[$text, $level, $category, $timestamp] = $message;
if (!is_string($text)) {
// exceptions may not be serializable if in the call stack somewhere is a Closure
if ($text instanceof \Throwable || $text instanceof \Exception) {

6
framework/log/Logger.php

@ -278,8 +278,8 @@ class Logger extends Component
$stack = [];
foreach ($messages as $i => $log) {
list($token, $level, $category, $timestamp, $traces) = $log;
$memory = isset($log[5]) ? $log[5] : 0;
[$token, $level, $category, $timestamp, $traces] = $log;
$memory = $log[5] ?? 0;
$log[6] = $i;
$hash = md5(serialize($token));
if ($level == Logger::LEVEL_PROFILE_BEGIN) {
@ -294,7 +294,7 @@ class Logger extends Component
'level' => count($stack) - 1,
'duration' => $timestamp - $stack[$hash][3],
'memory' => $memory,
'memoryDiff' => $memory - (isset($stack[$hash][5]) ? $stack[$hash][5] : 0),
'memoryDiff' => $memory - ($stack[$hash][5] ?? 0),
];
unset($stack[$hash]);
}

2
framework/log/SyslogTarget.php

@ -76,7 +76,7 @@ class SyslogTarget extends Target
*/
public function formatMessage($message)
{
list($text, $level, $category, $timestamp) = $message;
[$text, $level, $category, $timestamp] = $message;
$level = Logger::getLevelName($level);
if (!is_string($text)) {
// exceptions may not be serializable if in the call stack somewhere is a Closure

2
framework/log/Target.php

@ -252,7 +252,7 @@ abstract class Target extends Component
*/
public function formatMessage($message)
{
list($text, $level, $category, $timestamp) = $message;
[$text, $level, $category, $timestamp] = $message;
$level = Logger::getLevelName($level);
if (!is_string($text)) {
// exceptions may not be serializable if in the call stack somewhere is a Closure

4
framework/mutex/PgsqlMutex.php

@ -70,7 +70,7 @@ class PgsqlMutex extends DbMutex
if ($timeout !== 0) {
throw new InvalidArgumentException('PgsqlMutex does not support timeout.');
}
list($key1, $key2) = $this->getKeysFromName($name);
[$key1, $key2] = $this->getKeysFromName($name);
return (bool) $this->db
->createCommand('SELECT pg_try_advisory_lock(:key1, :key2)', [':key1' => $key1, ':key2' => $key2])
->queryScalar();
@ -84,7 +84,7 @@ class PgsqlMutex extends DbMutex
*/
protected function releaseLock($name)
{
list($key1, $key2) = $this->getKeysFromName($name);
[$key1, $key2] = $this->getKeysFromName($name);
return (bool) $this->db
->createCommand('SELECT pg_advisory_unlock(:key1, :key2)', [':key1' => $key1, ':key2' => $key2])
->queryScalar();

2
framework/rbac/DbManager.php

@ -974,7 +974,7 @@ class DbManager extends BaseManager
$data = $this->cache->get($this->cacheKey);
if (is_array($data) && isset($data[0], $data[1], $data[2])) {
list ($this->items, $this->rules, $this->parents) = $data;
[$this->items, $this->rules, $this->parents] = $data;
return;
}

4
framework/rest/Serializer.php

@ -255,7 +255,7 @@ class Serializer extends Component
if ($this->request->getIsHead()) {
return null;
} else {
list ($fields, $expand) = $this->getRequestedFields();
[$fields, $expand] = $this->getRequestedFields();
return $model->toArray($fields, $expand);
}
}
@ -286,7 +286,7 @@ class Serializer extends Component
*/
protected function serializeModels(array $models)
{
list ($fields, $expand) = $this->getRequestedFields();
[$fields, $expand] = $this->getRequestedFields();
foreach ($models as $i => $model) {
if ($model instanceof Arrayable) {
$models[$i] = $model->toArray($fields, $expand);

2
framework/validators/ImageValidator.php

@ -134,7 +134,7 @@ class ImageValidator extends FileValidator
return [$this->notImage, ['file' => $image->name]];
}
list($width, $height) = $imageInfo;
[$width, $height] = $imageInfo;
if ($width == 0 || $height == 0) {
return [$this->notImage, ['file' => $image->name]];

6
framework/validators/IpValidator.php

@ -432,7 +432,7 @@ class IpValidator extends Validator
}
foreach ($this->ranges as $string) {
list($isNegated, $range) = $this->parseNegatedRange($string);
[$isNegated, $range] = $this->parseNegatedRange($string);
if ($this->inRange($ip, $cidr, $range)) {
return !$isNegated;
}
@ -469,11 +469,11 @@ class IpValidator extends Validator
{
$result = [];
foreach ($ranges as $string) {
list($isRangeNegated, $range) = $this->parseNegatedRange($string);
[$isRangeNegated, $range] = $this->parseNegatedRange($string);
if (isset($this->networks[$range])) {
$replacements = $this->prepareRanges($this->networks[$range]);
foreach ($replacements as &$replacement) {
list($isReplacementNegated, $replacement) = $this->parseNegatedRange($replacement);
[$isReplacementNegated, $replacement] = $this->parseNegatedRange($replacement);
$result[] = ($isRangeNegated && !$isReplacementNegated ? static::NEGATION_CHAR : '') . $replacement;
}
} else {

2
framework/validators/Validator.php

@ -298,7 +298,7 @@ class Validator extends Component
return true;
}
list($message, $params) = $result;
[$message, $params] = $result;
$params['attribute'] = Yii::t('yii', 'the input value');
if (is_array($value)) {
$params['value'] = 'array()';

2
framework/web/Application.php

@ -79,7 +79,7 @@ class Application extends \yii\base\Application
{
if (empty($this->catchAll)) {
try {
list ($route, $params) = $request->resolve();
[$route, $params] = $request->resolve();
} catch (UrlNormalizerRedirectException $e) {
$url = $e->url;
if (is_array($url)) {

2
framework/web/AssetBundle.php

@ -182,7 +182,7 @@ class AssetBundle extends Object
public function publish($am)
{
if ($this->sourcePath !== null && !isset($this->basePath, $this->baseUrl)) {
list($this->basePath, $this->baseUrl) = $am->publish($this->sourcePath, $this->publishOptions);
[$this->basePath, $this->baseUrl] = $am->publish($this->sourcePath, $this->publishOptions);
}
if (isset($this->basePath, $this->baseUrl) && ($converter = $am->getConverter()) !== null) {

2
framework/web/AssetConverter.php

@ -63,7 +63,7 @@ class AssetConverter extends Component implements AssetConverterInterface
if ($pos !== false) {
$ext = substr($asset, $pos + 1);
if (isset($this->commands[$ext])) {
list ($ext, $command) = $this->commands[$ext];
[$ext, $command] = $this->commands[$ext];
$result = substr($asset, 0, $pos + 1) . $ext;
if ($this->forceConvert || @filemtime("$basePath/$result") < @filemtime("$basePath/$asset")) {
$this->runCommand($command, $basePath, $asset, $result);

6
framework/web/MultipartFormDataParser.php

@ -141,7 +141,7 @@ class MultipartFormDataParser extends Object implements RequestParserInterface
if (empty($bodyPart)) {
continue;
}
list($headers, $value) = preg_split("/\\R\\R/", $bodyPart, 2);
[$headers, $value] = preg_split("/\\R\\R/", $bodyPart, 2);
$headers = $this->parseHeaders($headers);
if (!isset($headers['content-disposition']['name'])) {
@ -208,7 +208,7 @@ class MultipartFormDataParser extends Object implements RequestParserInterface
continue;
}
list($headerName, $headerValue) = explode(':', $headerPart, 2);
[$headerName, $headerValue] = explode(':', $headerPart, 2);
$headerName = strtolower(trim($headerName));
$headerValue = trim($headerValue);
@ -221,7 +221,7 @@ class MultipartFormDataParser extends Object implements RequestParserInterface
if (strpos($part, '=') === false) {
$headers[$headerName][] = $part;
} else {
list($name, $value) = explode('=', $part, 2);
[$name, $value] = explode('=', $part, 2);
$name = strtolower(trim($name));
$value = trim(trim($value), '"');
$headers[$headerName][$name] = $value;

4
framework/web/Request.php

@ -183,7 +183,7 @@ class Request extends \yii\base\Request
{
$result = Yii::$app->getUrlManager()->parseRequest($this);
if ($result !== false) {
list($route, $params) = $result;
[$route, $params] = $result;
if ($this->_queryParams === null) {
$_GET = $params + $_GET; // preserve numeric keys
} else {
@ -1153,7 +1153,7 @@ class Request extends \yii\base\Request
];
foreach ($params as $param) {
if (strpos($param, '=') !== false) {
list($key, $value) = explode('=', $param, 2);
[$key, $value] = explode('=', $param, 2);
if ($key === 'q') {
$values['q'][2] = (double) $value;
} else {

6
framework/web/Response.php

@ -418,7 +418,7 @@ class Response extends \yii\base\Response
$chunkSize = 8 * 1024 * 1024; // 8MB per chunk
if (is_array($this->stream)) {
list ($handle, $begin, $end) = $this->stream;
[$handle, $begin, $end] = $this->stream;
fseek($handle, $begin);
while (!feof($handle) && ($pos = ftell($handle)) <= $end) {
if ($pos + $chunkSize > $end) {
@ -516,7 +516,7 @@ class Response extends \yii\base\Response
throw new RangeNotSatisfiableHttpException();
}
list($begin, $end) = $range;
[$begin, $end] = $range;
if ($begin != 0 || $end != $contentLength - 1) {
$this->setStatusCode(206);
$headers->set('Content-Range', "bytes $begin-$end/$contentLength");
@ -571,7 +571,7 @@ class Response extends \yii\base\Response
throw new RangeNotSatisfiableHttpException();
}
list($begin, $end) = $range;
[$begin, $end] = $range;
if ($begin != 0 || $end != $fileSize - 1) {
$this->setStatusCode(206);
$headers->set('Content-Range', "bytes $begin-$end/$fileSize");

2
framework/web/User.php

@ -558,7 +558,7 @@ class User extends Component
}
$data = json_decode($value, true);
if (count($data) == 3) {
list ($id, $authKey, $duration) = $data;
[$id, $authKey, $duration] = $data;
/* @var $class IdentityInterface */
$class = $this->identityClass;
$identity = $class::findIdentity($id);

2
framework/widgets/LinkPager.php

@ -194,7 +194,7 @@ class LinkPager extends Widget
}
// internal pages
list($beginPage, $endPage) = $this->getPageRange();
[$beginPage, $endPage] = $this->getPageRange();
for ($i = $beginPage; $i <= $endPage; ++$i) {
$buttons[] = $this->renderPageButton($i + 1, $i, null, $this->disableCurrentPageButton && $i == $currentPage, $i == $currentPage);
}

6
tests/framework/console/ControllerTest.php

@ -32,12 +32,12 @@ class ControllerTest extends TestCase
$controller = new FakeController('fake', Yii::$app);
$params = ['from params'];
list($fromParam, $other) = $controller->run('aksi1', $params);
[$fromParam, $other] = $controller->run('aksi1', $params);
$this->assertEquals('from params', $fromParam);
$this->assertEquals('default', $other);
$params = ['from params', 'notdefault'];
list($fromParam, $other) = $controller->run('aksi1', $params);
[$fromParam, $other] = $controller->run('aksi1', $params);
$this->assertEquals('from params', $fromParam);
$this->assertEquals('notdefault', $other);
@ -54,7 +54,7 @@ class ControllerTest extends TestCase
$this->assertEquals('testAlias', $result);
$params = ['_aliases' => ['ta' => 'from params,notdefault']];
list($fromParam, $other) = $controller->runAction('aksi6', $params);
[$fromParam, $other] = $controller->runAction('aksi6', $params);
$this->assertEquals('from params', $fromParam);
$this->assertEquals('notdefault', $other);

2
tests/framework/console/RequestTest.php

@ -147,7 +147,7 @@ class RequestTest extends TestCase
$request = new Request();
$request->setParams($params);
list($route, $params) = $request->resolve();
[$route, $params] = $request->resolve();
$this->assertEquals($expected['route'], $route);
$this->assertEquals($expected['params'], $params);
}

4
tests/framework/db/DatabaseTestCase.php

@ -84,8 +84,8 @@ abstract class DatabaseTestCase extends TestCase
$db->open();
if ($fixture !== null) {
if ($this->driverName === 'oci') {
list($drops, $creates) = explode('/* STATEMENTS */', file_get_contents($fixture), 2);
list($statements, $triggers, $data) = explode('/* TRIGGERS */', $creates, 3);
[$drops, $creates] = explode('/* STATEMENTS */', file_get_contents($fixture), 2);
[$statements, $triggers, $data] = explode('/* TRIGGERS */', $creates, 3);
$lines = array_merge(explode('--', $drops), explode(';', $statements), explode('/', $triggers), explode(';', $data));
} else {
$lines = explode(';', file_get_contents($fixture));

57
tests/framework/db/QueryBuilderTest.php

@ -1007,8 +1007,8 @@ abstract class QueryBuilderTest extends DatabaseTestCase
$qb = $this->getQueryBuilder();
foreach ($this->columnTypes() as $item) {
list ($column, $builder, $expected) = $item;
$expectedColumnSchemaBuilder = isset($item[3]) ? $item[3] : $column;
[$column, $builder, $expected] = $item;
$expectedColumnSchemaBuilder = $item[3] ?? $column;
$this->assertEquals($expected, $qb->getColumnType($column));
$this->assertEquals($expected, $qb->getColumnType($builder));
@ -1024,8 +1024,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
}
$columns = [];
$i = 0;
foreach ($this->columnTypes() as $item) {
list ($column, $builder, $expected) = $item;
foreach ($this->columnTypes() as [$column, $builder, $expected]) {
if (!(strncmp($column, Schema::TYPE_PK, 2) === 0 ||
strncmp($column, Schema::TYPE_UPK, 3) === 0 ||
strncmp($column, Schema::TYPE_BIGPK, 5) === 0 ||
@ -1206,7 +1205,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
public function testBuildCondition($condition, $expected, $expectedParams)
{
$query = (new Query())->where($condition);
list($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$this->assertEquals('SELECT *' . (empty($expected) ? '' : ' WHERE ' . $this->replaceQuotes($expected)), $sql);
$this->assertEquals($expectedParams, $params);
}
@ -1217,7 +1216,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
public function testBuildFilterCondition($condition, $expected, $expectedParams)
{
$query = (new Query())->filterWhere($condition);
list($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$this->assertEquals('SELECT *' . (empty($expected) ? '' : ' WHERE ' . $this->replaceQuotes($expected)), $sql);
$this->assertEquals($expectedParams, $params);
}
@ -1276,7 +1275,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->from('TotalExample t')
->where([$cond, $subQuery]);
list($actualQuerySql, $actualQueryParams) = $this->getQueryBuilder()->build($query);
[$actualQuerySql, $actualQueryParams] = $this->getQueryBuilder()->build($query);
$this->assertEquals($expectedQuerySql, $actualQuerySql);
$this->assertEquals($expectedQueryParams, $actualQueryParams);
}
@ -1301,7 +1300,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->where(['exists', $subQuery])
->andWhere('t.some_column = :some_value', [':some_value' => "asd"]);
list($actualQuerySql, $queryParams) = $this->getQueryBuilder()->build($query);
[$actualQuerySql, $queryParams] = $this->getQueryBuilder()->build($query);
$this->assertEquals($expectedQuerySql, $actualQuerySql);
$this->assertEquals($expectedQueryParams, $queryParams);
}
@ -1325,7 +1324,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->where(['exists', $subQuery])
->andWhere(['t.some_column' => "asd"]);
list($actualQuerySql, $queryParams) = $this->getQueryBuilder()->build($query);
[$actualQuerySql, $queryParams] = $this->getQueryBuilder()->build($query);
$this->assertEquals($expectedQuerySql, $actualQuerySql);
$this->assertEquals($expectedQueryParams, $queryParams);
}
@ -1353,7 +1352,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->where(['and', 'w > 0', 'x < 2'])
->union($secondQuery)
->union($thirdQuery, TRUE);
list($actualQuerySql, $queryParams) = $this->getQueryBuilder()->build($query);
[$actualQuerySql, $queryParams] = $this->getQueryBuilder()->build($query);
$this->assertEquals($expectedQuerySql, $actualQuerySql);
$this->assertEquals([], $queryParams);
}
@ -1368,7 +1367,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->select('*')
->from('accounts')
->addSelect(['operations_count' => $subquery]);
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT *, (SELECT COUNT(*) FROM [[operations]] WHERE account_id = accounts.id) AS [[operations_count]] FROM [[accounts]]');
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1386,7 +1385,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
new Expression($this->replaceQuotes("case t.Status_Id when 1 then 'Acknowledge' when 2 then 'No Action' else 'Unknown Action' END as [[Next Action]]")),
])
->from('tablename');
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes(
'SELECT [[t]].[[id]] AS [[ID]], [[gsm]].[[username]] AS [[GSM]], [[part]].[[Part]], [[t]].[[Part_Cost]] AS [[Part Cost]], st_x(location::geometry) as lon,'
. ' case t.Status_Id when 1 then \'Acknowledge\' when 2 then \'No Action\' else \'Unknown Action\' END as [[Next Action]] FROM [[tablename]]');
@ -1399,7 +1398,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
$query = (new Query())
->select(new Expression("1 AS ab"))
->from('tablename');
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes("SELECT 1 AS ab FROM [[tablename]]");
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1409,7 +1408,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->addSelect(new Expression("2 AS cd"))
->addSelect(['ef' => new Expression("3")])
->from('tablename');
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes("SELECT 1 AS ab, 2 AS cd, 3 AS [[ef]] FROM [[tablename]]");
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1417,7 +1416,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
$query = (new Query())
->select(new Expression("SUBSTR(name, 0, :len)", [':len' => 4]))
->from('tablename');
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes("SELECT SUBSTR(name, 0, :len) FROM [[tablename]]");
$this->assertEquals($expected, $sql);
$this->assertEquals([':len' => 4], $params);
@ -1430,7 +1429,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
public function testFromIndexHint()
{
$query = (new Query)->from([new Expression('{{%user}} USE INDEX (primary)')]);
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM {{%user}} USE INDEX (primary)');
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1438,7 +1437,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
$query = (new Query)
->from([new Expression('{{user}} {{t}} FORCE INDEX (primary) IGNORE INDEX FOR ORDER BY (i1)')])
->leftJoin(['p' => 'profile'], 'user.id = profile.user_id USE INDEX (i2)');
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM {{user}} {{t}} FORCE INDEX (primary) IGNORE INDEX FOR ORDER BY (i1) LEFT JOIN [[profile]] [[p]] ON user.id = profile.user_id USE INDEX (i2)');
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1450,7 +1449,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
$subquery = (new Query)->from('user')->where('account_id = accounts.id');
$query = (new Query)->from(['activeusers' => $subquery]);
// SELECT * FROM (SELECT * FROM [[user]] WHERE [[active]] = 1) [[activeusers]];
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM (SELECT * FROM [[user]] WHERE account_id = accounts.id) [[activeusers]]');
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1459,7 +1458,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
$subquery = (new Query)->from('user')->where('account_id = :id', ['id' => 1]);
$query = (new Query)->from(['activeusers' => $subquery])->where('abc = :abc', ['abc' => 'abc']);
// SELECT * FROM (SELECT * FROM [[user]] WHERE [[active]] = 1) [[activeusers]];
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM (SELECT * FROM [[user]] WHERE account_id = :id) [[activeusers]] WHERE abc = :abc');
$this->assertEquals($expected, $sql);
$this->assertEquals([
@ -1471,7 +1470,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
$subquery = "(SELECT * FROM user WHERE account_id = accounts.id)";
$query = (new Query)->from(['activeusers' => $subquery]);
// SELECT * FROM (SELECT * FROM [[user]] WHERE [[active]] = 1) [[activeusers]];
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM (SELECT * FROM user WHERE account_id = accounts.id) [[activeusers]]');
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1484,7 +1483,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->select('*')
->from('operations')
->orderBy('name ASC, date DESC');
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM [[operations]] ORDER BY [[name]], [[date]] DESC');
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1494,7 +1493,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->select('*')
->from('operations')
->orderBy(['name' => SORT_ASC, 'date' => SORT_DESC]);
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM [[operations]] ORDER BY [[name]], [[date]] DESC');
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1505,7 +1504,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->from('operations')
->where('account_id = accounts.id')
->orderBy(new Expression('SUBSTR(name, 3, 4) DESC, x ASC'));
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM [[operations]] WHERE account_id = accounts.id ORDER BY SUBSTR(name, 3, 4) DESC, x ASC');
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1515,7 +1514,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->select('*')
->from('operations')
->orderBy(new Expression('SUBSTR(name, 3, :to) DESC, x ASC', [':to' => 4]));
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM [[operations]] ORDER BY SUBSTR(name, 3, :to) DESC, x ASC');
$this->assertEquals($expected, $sql);
$this->assertEquals([':to' => 4], $params);
@ -1528,7 +1527,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->select('*')
->from('operations')
->groupBy('name, date');
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM [[operations]] GROUP BY [[name]], [[date]]');
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1538,7 +1537,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->select('*')
->from('operations')
->groupBy(['name', 'date']);
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM [[operations]] GROUP BY [[name]], [[date]]');
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1549,7 +1548,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->from('operations')
->where('account_id = accounts.id')
->groupBy(new Expression('SUBSTR(name, 0, 1), x'));
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM [[operations]] WHERE account_id = accounts.id GROUP BY SUBSTR(name, 0, 1), x');
$this->assertEquals($expected, $sql);
$this->assertEmpty($params);
@ -1559,7 +1558,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
->select('*')
->from('operations')
->groupBy(new Expression('SUBSTR(name, 0, :to), x', [':to' => 4]));
list ($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$expected = $this->replaceQuotes('SELECT * FROM [[operations]] GROUP BY SUBSTR(name, 0, :to), x');
$this->assertEquals($expected, $sql);
$this->assertEquals([':to' => 4], $params);
@ -1719,7 +1718,7 @@ abstract class QueryBuilderTest extends DatabaseTestCase
public function testBuildLikeCondition($condition, $expected, $expectedParams)
{
$query = (new Query())->where($condition);
list($sql, $params) = $this->getQueryBuilder()->build($query);
[$sql, $params] = $this->getQueryBuilder()->build($query);
$this->assertEquals('SELECT *' . (empty($expected) ? '' : ' WHERE ' . $this->replaceQuotes($expected)), $sql);
$this->assertEquals($expectedParams, $params);
}

6
tests/framework/db/mssql/QueryBuilderTest.php

@ -28,7 +28,7 @@ class QueryBuilderTest extends \yiiunit\framework\db\QueryBuilderTest
$query = new Query();
$query->select('id')->from('example')->limit(10)->offset(5);
list($actualQuerySql, $actualQueryParams) = $this->getQueryBuilder()->build($query);
[$actualQuerySql, $actualQueryParams] = $this->getQueryBuilder()->build($query);
$this->assertEquals($expectedQuerySql, $actualQuerySql);
$this->assertEquals($expectedQueryParams, $actualQueryParams);
@ -42,7 +42,7 @@ class QueryBuilderTest extends \yiiunit\framework\db\QueryBuilderTest
$query = new Query();
$query->select('id')->from('example')->limit(10);
list($actualQuerySql, $actualQueryParams) = $this->getQueryBuilder()->build($query);
[$actualQuerySql, $actualQueryParams] = $this->getQueryBuilder()->build($query);
$this->assertEquals($expectedQuerySql, $actualQuerySql);
$this->assertEquals($expectedQueryParams, $actualQueryParams);
@ -56,7 +56,7 @@ class QueryBuilderTest extends \yiiunit\framework\db\QueryBuilderTest
$query = new Query();
$query->select('id')->from('example')->offset(10);
list($actualQuerySql, $actualQueryParams) = $this->getQueryBuilder()->build($query);
[$actualQuerySql, $actualQueryParams] = $this->getQueryBuilder()->build($query);
$this->assertEquals($expectedQuerySql, $actualQuerySql);
$this->assertEquals($expectedQueryParams, $actualQueryParams);

4
tests/framework/db/mysql/connection/DeadLockTest.php

@ -185,7 +185,7 @@ class DeadLockTest extends \yiiunit\framework\db\mysql\ConnectionTest
});
}, Transaction::REPEATABLE_READ);
} catch (Exception $e) {
list ($sqlError, $driverError, $driverMessage) = $e->errorInfo;
[$sqlError, $driverError, $driverMessage] = $e->errorInfo;
// Deadlock found when trying to get lock; try restarting transaction
if ('40001' === $sqlError && 1213 === $driverError) {
return self::CHILD_EXIT_CODE_DEADLOCK;
@ -245,7 +245,7 @@ class DeadLockTest extends \yiiunit\framework\db\mysql\ConnectionTest
});
}, Transaction::REPEATABLE_READ);
} catch (Exception $e) {
list ($sqlError, $driverError, $driverMessage) = $e->errorInfo;
[$sqlError, $driverError, $driverMessage] = $e->errorInfo;
// Deadlock found when trying to get lock; try restarting transaction
if ('40001' === $sqlError && 1213 === $driverError) {
return self::CHILD_EXIT_CODE_DEADLOCK;

4
tests/framework/db/sqlite/ConnectionTest.php

@ -46,9 +46,7 @@ class ConnectionTest extends \yiiunit\framework\db\ConnectionTest
{
$counts = [[0, 2], [1, 2], [2, 2]];
foreach ($counts as $count) {
list($masterCount, $slaveCount) = $count;
foreach ($counts as [$masterCount, $slaveCount]) {
$db = $this->prepareMasterSlave($masterCount, $slaveCount);
$this->assertInstanceOf(Connection::class, $db->getSlave());

2
tests/framework/db/sqlite/QueryBuilderTest.php

@ -106,7 +106,7 @@ class QueryBuilderTest extends \yiiunit\framework\db\QueryBuilderTest
->where(['and', 'w > 0', 'x < 2'])
->union($secondQuery)
->union($thirdQuery, TRUE);
list($actualQuerySql, $queryParams) = $this->getQueryBuilder()->build($query);
[$actualQuerySql, $queryParams] = $this->getQueryBuilder()->build($query);
$this->assertEquals($expectedQuerySql, $actualQuerySql);
$this->assertEquals([], $queryParams);
}

2
tests/framework/di/ContainerTest.php

@ -175,7 +175,7 @@ class ContainerTest extends TestCase
$myFunc = function (\yii\console\Request $request, \yii\console\Response $response) {
return [$request, $response];
};
list($request, $response) = Yii::$container->invoke($myFunc);
[$request, $response] = Yii::$container->invoke($myFunc);
$this->assertEquals($request, Yii::$app->request);
$this->assertEquals($response, Yii::$app->response);
}

4
tests/framework/log/SyslogTargetTest.php

@ -125,13 +125,13 @@ namespace yiiunit\framework\log {
static::$functions['openlog'] = function ($arguments) use ($syslogTarget) {
$this->assertCount(3, $arguments);
list($identity, $option, $facility) = $arguments;
[$identity, $option, $facility] = $arguments;
$syslogTarget->openlog($identity, $option, $facility);
};
static::$functions['syslog'] = function ($arguments) use ($syslogTarget) {
$this->assertCount(2, $arguments);
list($priority, $message) = $arguments;
[$priority, $message] = $arguments;
$syslogTarget->syslog($priority, $message);
};

20
tests/framework/rest/UrlRuleTest.php

@ -24,10 +24,8 @@ class UrlRuleTest extends TestCase
public function testInitControllerNamePluralization()
{
$suites = $this->getTestsForControllerNamePluralization();
foreach ($suites as $i => $suite) {
list ($name, $tests) = $suite;
foreach ($tests as $j => $test) {
list ($config, $expected) = $test;
foreach ($suites as $i => [$name, $tests]) {
foreach ($tests as $j => [$config, $expected]) {
$rule = new UrlRule($config);
$this->assertEquals($expected, $rule->controller, "Test#$i-$j: $name");
}
@ -39,14 +37,12 @@ class UrlRuleTest extends TestCase
$manager = new UrlManager(['cache' => null]);
$request = new Request(['hostInfo' => 'http://en.example.com', 'methodParam' => '_METHOD',]);
$suites = $this->getTestsForParseRequest();
foreach ($suites as $i => $suite) {
list ($name, $config, $tests) = $suite;
foreach ($suites as $i => [$name, $config, $tests]) {
$rule = new UrlRule($config);
foreach ($tests as $j => $test) {
$request->pathInfo = $test[0];
$route = $test[1];
$params = isset($test[2]) ? $test[2] : [];
$_POST['_METHOD'] = isset($test[3]) ? $test[3] : 'GET';
[$request->pathInfo, $route] = $test;
$params = $test[2] ?? [];
$_POST['_METHOD'] = $test[3] ?? 'GET';
$result = $rule->parseRequest($manager, $request);
if ($route === false) {
$this->assertFalse($result, "Test#$i-$j: $name");
@ -344,9 +340,7 @@ class UrlRuleTest extends TestCase
*/
public function testCreateUrl($rule, $tests)
{
foreach ($tests as $test) {
list($params, $expected) = $test;
foreach ($tests as [$params, $expected]) {
$this->mockWebApplication();
Yii::$app->set('request', new Request(['hostInfo' => 'http://api.example.com', 'scriptUrl' => '/index.php']));
$route = array_shift($params);

4
tests/framework/web/ControllerTest.php

@ -24,12 +24,12 @@ class ControllerTest extends TestCase
$aksi1 = new InlineAction('aksi1', $this->controller, 'actionAksi1');
$params = ['fromGet' => 'from query params', 'q' => 'd426', 'validator' => 'avaliable'];
list($fromGet, $other) = $this->controller->bindActionParams($aksi1, $params);
[$fromGet, $other] = $this->controller->bindActionParams($aksi1, $params);
$this->assertEquals('from query params', $fromGet);
$this->assertEquals('default', $other);
$params = ['fromGet' => 'from query params', 'q' => 'd426', 'other' => 'avaliable'];
list($fromGet, $other) = $this->controller->bindActionParams($aksi1, $params);
[$fromGet, $other] = $this->controller->bindActionParams($aksi1, $params);
$this->assertEquals('from query params', $fromGet);
$this->assertEquals('avaliable', $other);

13
tests/framework/web/GroupUrlRuleTest.php

@ -23,11 +23,10 @@ class GroupUrlRuleTest extends TestCase
{
$manager = new UrlManager(['cache' => null]);
$suites = $this->getTestsForCreateUrl();
foreach ($suites as $i => $suite) {
list ($name, $config, $tests) = $suite;
foreach ($suites as $i => [$name, $config, $tests]) {
$rule = new GroupUrlRule($config);
foreach ($tests as $j => $test) {
list ($route, $params, $expected, $status) = $test;
[$route, $params, $expected, $status] = $test;
$url = $rule->createUrl($manager, $route, $params);
$this->assertEquals($expected, $url, "Test#$i-$j: $name");
$this->assertSame($status, $rule->getCreateUrlStatus(), "Test#$i-$j: $name");
@ -40,13 +39,11 @@ class GroupUrlRuleTest extends TestCase
$manager = new UrlManager(['cache' => null]);
$request = new Request(['hostInfo' => 'http://en.example.com']);
$suites = $this->getTestsForParseRequest();
foreach ($suites as $i => $suite) {
list ($name, $config, $tests) = $suite;
foreach ($suites as $i => [$name, $config, $tests]) {
$rule = new GroupUrlRule($config);
foreach ($tests as $j => $test) {
$request->pathInfo = $test[0];
$route = $test[1];
$params = isset($test[2]) ? $test[2] : [];
[$request->pathInfo, $route] = $test;
$params = $test[2] ?? [];
$result = $rule->parseRequest($manager, $request);
if ($route === false) {
$this->assertFalse($result, "Test#$i-$j: $name");

44
tests/framework/web/UrlRuleTest.php

@ -27,11 +27,9 @@ class UrlRuleTest extends TestCase
{
$manager = new UrlManager(['cache' => null]);
$suites = $this->getTestsForCreateUrl();
foreach ($suites as $i => $suite) {
list ($name, $config, $tests) = $suite;
foreach ($suites as $i => [$name, $config, $tests]) {
$rule = new UrlRule($config);
foreach ($tests as $j => $test) {
list ($route, $params, $expected) = $test;
foreach ($tests as $j => [$route, $params, $expected]) {
$url = $rule->createUrl($manager, $route, $params);
$this->assertSame($expected, $url, "Test#$i-$j: $name");
}
@ -46,12 +44,10 @@ class UrlRuleTest extends TestCase
]);
$request = new Request(['hostInfo' => 'http://en.example.com']);
$suites = $this->getTestsForParseRequest();
foreach ($suites as $i => $suite) {
list ($name, $config, $tests) = $suite;
foreach ($suites as $i => [$name, $config, $tests]) {
$rule = new UrlRule($config);
foreach ($tests as $j => $test) {
$request->pathInfo = $test[0];
$expected = $test[1];
[$request->pathInfo, $expected] = $test;
$result = $rule->parseRequest($manager, $request);
if ($expected === false) {
$this->assertFalse($result, "Test#$i-$j: $name");
@ -69,12 +65,11 @@ class UrlRuleTest extends TestCase
]);
$request = new Request(['hostInfo' => 'http://en.example.com']);
$suites = $this->getTestsForParseRequest();
foreach ($suites as $i => $suite) {
list ($name, $config, $tests) = $suite;
foreach ($suites as $i => [$name, $config, $tests]) {
$rule = new UrlRule($config);
foreach ($tests as $j => $test) {
$request->pathInfo = $test[0];
$expected = isset($test[2]) ? $test[2] : $test[1];
$expected = $test[2] ?? $test[1];
try {
$result = $rule->parseRequest($manager, $request);
if ($expected === false) {
@ -100,12 +95,11 @@ class UrlRuleTest extends TestCase
]);
$request = new Request(['hostInfo' => 'http://en.example.com']);
$suites = $this->getTestsForParseRequest();
foreach ($suites as $i => $suite) {
list ($name, $config, $tests) = $suite;
foreach ($suites as $i => [$name, $config, $tests]) {
$rule = new UrlRule($config);
foreach ($tests as $j => $test) {
$request->pathInfo = $test[0];
$expected = isset($test[2]) ? $test[2] : $test[1];
$expected = $test[2] ?? $test[1];
try {
$result = $rule->parseRequest($manager, $request);
if ($expected === false) {
@ -129,12 +123,11 @@ class UrlRuleTest extends TestCase
]);
$request = new Request(['hostInfo' => 'http://en.example.com']);
$suites = $this->getTestsForParseRequest();
foreach ($suites as $i => $suite) {
list ($name, $config, $tests) = $suite;
foreach ($suites as $i => [$name, $config, $tests]) {
$rule = new UrlRule($config);
foreach ($tests as $j => $test) {
$request->pathInfo = $test[0];
$expected = isset($test[2]) ? $test[2] : $test[1];
$expected = $test[2] ?? $test[1];
try {
$result = $rule->parseRequest($manager, $request);
if ($expected === false) {
@ -158,12 +151,10 @@ class UrlRuleTest extends TestCase
]);
$request = new Request(['hostInfo' => 'http://en.example.com']);
$suites = $this->getTestsForParseRequest();
foreach ($suites as $i => $suite) {
list ($name, $config, $tests) = $suite;
foreach ($suites as $i => [$name, $config, $tests]) {
$rule = new UrlRule($config);
foreach ($tests as $j => $test) {
$request->pathInfo = $test[0];
$expected = $test[1];
[$request->pathInfo, $expected] = $test;
try {
$result = $rule->parseRequest($manager, $request);
if ($expected === false) {
@ -186,12 +177,11 @@ class UrlRuleTest extends TestCase
]);
$request = new Request(['hostInfo' => 'http://en.example.com']);
$suites = $this->getTestsForParseRequest();
foreach ($suites as $i => $suite) {
list ($name, $config, $tests) = $suite;
foreach ($suites as $i => [$name, $config, $tests]) {
$rule = new UrlRule($config);
foreach ($tests as $j => $test) {
$request->pathInfo = $test[0];
$expected = isset($test[2]) ? $test[2] : $test[1];
$expected = $test[2] ?? $test[1];
$result = $rule->parseRequest($manager, $request);
if ($expected === false) {
$this->assertFalse($result, "Test#$i-$j: $name");
@ -215,8 +205,7 @@ class UrlRuleTest extends TestCase
]);
$request = new Request(['hostInfo' => 'http://en.example.com']);
$suites = $this->getTestsForParseRequest();
foreach ($suites as $i => $suite) {
list ($name, $config, $tests) = $suite;
foreach ($suites as $i => [$name, $config, $tests]) {
$rule = new UrlRule($config);
foreach ($tests as $j => $test) {
$request->pathInfo = $test[0];
@ -276,8 +265,7 @@ class UrlRuleTest extends TestCase
public function testToString()
{
$suites = $this->getTestsForToString();
foreach ($suites as $i => $suite) {
list ($name, $config, $test) = $suite;
foreach ($suites as $i => [$name, $config, $test]) {
$rule = new UrlRule($config);
$this->assertEquals($rule->__toString(), $test, "Test#$i: $name");
}

Loading…
Cancel
Save