evolution-cms / evolution

core/vendor/agelxnash/modx-evo-database/tests/Real/MySqliQueryTest.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -1,9 +1,9 @@
 <?php namespace AgelxNash\Modx\Evo\Database\Tests\Real;
 
+use AgelxNash\Modx\Evo\Database\Drivers\MySqliDriver;
 use AgelxNash\Modx\Evo\Database\Tests\RealQueryTest;
 use mysqli;
 use mysqli_result;
-use AgelxNash\Modx\Evo\Database\Drivers\MySqliDriver;
 
 /**
  * @property MySqliDriver $instance

Spacing +2 added lines, -2 removed lines

@@ -28,7 +28,7 @@ 
 
     public function testDataSeek()
     {
-        $query = 'SELECT `id`, `alias` FROM ' . $this->table . ' WHERE id = 1';
+        $query = 'SELECT `id`, `alias` FROM '.$this->table.' WHERE id = 1';
 
         $result = $this->instance->query($query);
         $this->assertInstanceOf(
@@ -65,7 +65,7 @@ 
         $this->instance->getDriver()->dataSeek($result, 0);
 
         $this->assertEquals(
-            (object)['id' => 1, 'alias' => 'index'],
+            (object) ['id' => 1, 'alias' => 'index'],
             $this->instance->getRow($result, 'object')
         );
         $this->instance->getDriver()->dataSeek($result, 0);

core/vendor/agelxnash/modx-evo-database/tests/Source/DatabaseTest.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php namespace AgelxNash\Modx\Evo\Database\Tests\Source;
 
-use PHPUnit\Framework\TestCase;
 use AgelxNash\Modx\Evo\Database;
+use PHPUnit\Framework\TestCase;
 
 class DatabaseTest extends TestCase
 {

core/vendor/agelxnash/modx-evo-database/tests/Source/Traits/SupportTraitTest.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php namespace AgelxNash\Modx\Evo\Database\Tests\Traits\Source;
 
-use PHPUnit\Framework\TestCase;
 use AgelxNash\Modx\Evo\Database;
+use PHPUnit\Framework\TestCase;
 use ReflectionMethod;
 
 class SupportTraitTest extends TestCase

Spacing +1 added lines, -1 removed lines

@@ -342,7 +342,7 @@
                 $this->assertThat(
                     $method->invoke($this->instance, $item),
                     $this->isType($type),
-                    '[ STEP ' . ++$step . '] Check ' . $item . ' as ' . $type
+                    '[ STEP '.++$step.'] Check '.$item.' as '.$type
                 );
             }
         }

core/vendor/composer/ClassLoader.php

Doc Comments +5 added lines, -1 removed lines

@@ -103,7 +103,7 @@ 
      * appending or prepending to the ones previously set for this prefix.
      *
      * @param string       $prefix  The prefix
-     * @param array|string $paths   The PSR-0 root directories
+     * @param string $paths   The PSR-0 root directories
      * @param bool         $prepend Whether to prepend the directories
      */
     public function add($prefix, $paths, $prepend = false)
@@ -367,6 +367,10 @@ 
         return $file;
     }
 
+    /**
+     * @param string $class
+     * @param string $ext
+     */
     private function findFileWithExtension($class, $ext)
     {
         // PSR-4 lookup

Spacing +8 added lines, -8 removed lines

@@ -370,18 +370,18 @@ 
     private function findFileWithExtension($class, $ext)
     {
         // PSR-4 lookup
-        $logicalPathPsr4 = strtr($class, '\\', DIRECTORY_SEPARATOR) . $ext;
+        $logicalPathPsr4 = strtr($class, '\\', DIRECTORY_SEPARATOR).$ext;
 
         $first = $class[0];
         if (isset($this->prefixLengthsPsr4[$first])) {
             $subPath = $class;
             while (false !== $lastPos = strrpos($subPath, '\\')) {
                 $subPath = substr($subPath, 0, $lastPos);
-                $search = $subPath . '\\';
+                $search = $subPath.'\\';
                 if (isset($this->prefixDirsPsr4[$search])) {
-                    $pathEnd = DIRECTORY_SEPARATOR . substr($logicalPathPsr4, $lastPos + 1);
+                    $pathEnd = DIRECTORY_SEPARATOR.substr($logicalPathPsr4, $lastPos + 1);
                     foreach ($this->prefixDirsPsr4[$search] as $dir) {
-                        if (file_exists($file = $dir . $pathEnd)) {
+                        if (file_exists($file = $dir.$pathEnd)) {
                             return $file;
                         }
                     }
@@ -391,7 +391,7 @@ 
 
         // PSR-4 fallback dirs
         foreach ($this->fallbackDirsPsr4 as $dir) {
-            if (file_exists($file = $dir . DIRECTORY_SEPARATOR . $logicalPathPsr4)) {
+            if (file_exists($file = $dir.DIRECTORY_SEPARATOR.$logicalPathPsr4)) {
                 return $file;
             }
         }
@@ -403,14 +403,14 @@ 
                 . strtr(substr($logicalPathPsr4, $pos + 1), '_', DIRECTORY_SEPARATOR);
         } else {
             // PEAR-like class name
-            $logicalPathPsr0 = strtr($class, '_', DIRECTORY_SEPARATOR) . $ext;
+            $logicalPathPsr0 = strtr($class, '_', DIRECTORY_SEPARATOR).$ext;
         }
 
         if (isset($this->prefixesPsr0[$first])) {
             foreach ($this->prefixesPsr0[$first] as $prefix => $dirs) {
                 if (0 === strpos($class, $prefix)) {
                     foreach ($dirs as $dir) {
-                        if (file_exists($file = $dir . DIRECTORY_SEPARATOR . $logicalPathPsr0)) {
+                        if (file_exists($file = $dir.DIRECTORY_SEPARATOR.$logicalPathPsr0)) {
                             return $file;
                         }
                     }
@@ -420,7 +420,7 @@ 
 
         // PSR-0 fallback dirs
         foreach ($this->fallbackDirsPsr0 as $dir) {
-            if (file_exists($file = $dir . DIRECTORY_SEPARATOR . $logicalPathPsr0)) {
+            if (file_exists($file = $dir.DIRECTORY_SEPARATOR.$logicalPathPsr0)) {
                 return $file;
             }
         }

core/vendor/doctrine/cache/lib/Doctrine/Common/Cache/CacheProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -229,7 +229,7 @@
      *
      * @param string $id The id of the cache entry to fetch.
      *
-     * @return mixed|false The cached data or FALSE, if no cache entry exists for the given id.
+     * @return string The cached data or FALSE, if no cache entry exists for the given id.
      */
     abstract protected function doFetch($id);
 

Spacing +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
         // no internal array function supports this sort of mapping: needs to be iterative
         // this filters and combines keys in one pass
         foreach ($namespacedKeys as $requestedKey => $namespacedKey) {
-            if (! isset($items[$namespacedKey]) && ! array_key_exists($namespacedKey, $items)) {
+            if (!isset($items[$namespacedKey]) && !array_key_exists($namespacedKey, $items)) {
                 continue;
             }
 
@@ -214,7 +214,7 @@ 
 
         foreach ($keys as $key) {
             $item = $this->doFetch($key);
-            if ($item === false && ! $this->doContains($key)) {
+            if ($item === false && !$this->doContains($key)) {
                 continue;
             }
 

core/vendor/doctrine/dbal/lib/Doctrine/DBAL/Cache/QueryCacheProfile.php

Doc Comments +2 added lines, -1 removed lines

@@ -49,6 +49,7 @@ 
     /**
      * @param int         $lifetime
      * @param string|null $cacheKey
+     * @param Cache $resultCache
      */
     public function __construct($lifetime = 0, $cacheKey = null, ?Cache $resultCache = null)
     {
@@ -95,7 +96,7 @@ 
      * @param array  $types
      * @param array  $connectionParams
      *
-     * @return array
+     * @return string[]
      */
     public function generateCacheKeys($query, $params, $types, array $connectionParams = [])
     {

Spacing +4 added lines, -4 removed lines

@@ -99,10 +99,10 @@
      */
     public function generateCacheKeys($query, $params, $types, array $connectionParams = [])
     {
-        $realCacheKey = 'query=' . $query .
-            '&params=' . serialize($params) .
-            '&types=' . serialize($types) .
-            '&connectionParams=' . hash('sha256', serialize($connectionParams));
+        $realCacheKey = 'query='.$query.
+            '&params='.serialize($params).
+            '&types='.serialize($types).
+            '&connectionParams='.hash('sha256', serialize($connectionParams));
 
         // should the key be automatically generated using the inputs or is the cache key set?
         if ($this->cacheKey === null) {

core/vendor/doctrine/dbal/lib/Doctrine/DBAL/Configuration.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -19,8 +19,8 @@
 
 namespace Doctrine\DBAL;
 
-use Doctrine\DBAL\Logging\SQLLogger;
 use Doctrine\Common\Cache\Cache;
+use Doctrine\DBAL\Logging\SQLLogger;
 
 /**
  * Configuration container for the Doctrine DBAL.

core/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php

Doc Comments +2 added lines, -2 removed lines

@@ -601,7 +601,7 @@ 
      * @param int    $column    The 0-indexed column number to retrieve.
      * @param array  $types     The query parameter types.
      *
-     * @return mixed|bool False is returned if no rows are found.
+     * @return string|boolean False is returned if no rows are found.
      *
      * @throws \Doctrine\DBAL\DBALException
      */
@@ -1251,7 +1251,7 @@ 
      * Returns the savepoint name to use for nested transactions are false if they are not supported
      * "savepointFormat" parameter is not set
      *
-     * @return mixed A string with the savepoint name or false.
+     * @return string A string with the savepoint name or false.
      */
     protected function _getNestedTransactionSavePointName()
     {

Unused Use Statements +9 added lines, -9 removed lines

@@ -19,20 +19,20 @@
 
 namespace Doctrine\DBAL;
 
-use Doctrine\DBAL\Driver\ResultStatement;
-use Doctrine\DBAL\Driver\ServerInfoAwareConnection;
-use Doctrine\DBAL\Exception\InvalidArgumentException;
 use Closure;
-use Exception;
-use Doctrine\DBAL\Types\Type;
-use Doctrine\DBAL\Driver\Connection as DriverConnection;
-use Doctrine\DBAL\Driver\Statement as DriverStatement;
 use Doctrine\Common\EventManager;
-use Doctrine\DBAL\Cache\ResultCacheStatement;
-use Doctrine\DBAL\Cache\QueryCacheProfile;
 use Doctrine\DBAL\Cache\ArrayStatement;
 use Doctrine\DBAL\Cache\CacheException;
+use Doctrine\DBAL\Cache\QueryCacheProfile;
+use Doctrine\DBAL\Cache\ResultCacheStatement;
+use Doctrine\DBAL\Driver\Connection as DriverConnection;
 use Doctrine\DBAL\Driver\PingableConnection;
+use Doctrine\DBAL\Driver\ResultStatement;
+use Doctrine\DBAL\Driver\ServerInfoAwareConnection;
+use Doctrine\DBAL\Driver\Statement as DriverStatement;
+use Doctrine\DBAL\Exception\InvalidArgumentException;
+use Doctrine\DBAL\Types\Type;
+use Exception;
 use Throwable;
 use function assert;
 use function array_key_exists;

Spacing +19 added lines, -19 removed lines

@@ -229,7 +229,7 @@ 
         }
 
         if (isset($params["platform"])) {
-            if ( ! $params['platform'] instanceof Platforms\AbstractPlatform) {
+            if (!$params['platform'] instanceof Platforms\AbstractPlatform) {
                 throw DBALException::invalidPlatformType($params['platform']);
             }
 
@@ -238,11 +238,11 @@ 
         }
 
         // Create default config and event manager if none given
-        if ( ! $config) {
+        if (!$config) {
             $config = new Configuration();
         }
 
-        if ( ! $eventManager) {
+        if (!$eventManager) {
             $eventManager = new EventManager();
         }
 
@@ -438,7 +438,7 @@ 
     private function getDatabasePlatformVersion()
     {
         // Driver does not support version specific platforms.
-        if ( ! $this->_driver instanceof VersionAwarePlatformDriver) {
+        if (!$this->_driver instanceof VersionAwarePlatformDriver) {
             return null;
         }
 
@@ -496,7 +496,7 @@ 
     {
         // Automatic platform version detection.
         if ($this->_conn instanceof ServerInfoAwareConnection &&
-            ! $this->_conn->requiresQueryForServerVersion()
+            !$this->_conn->requiresQueryForServerVersion()
         ) {
             return $this->_conn->getServerVersion();
         }
@@ -654,7 +654,7 @@ 
 
             $columns[] = $columnName;
             $values[] = $value;
-            $conditions[] = $columnName . ' = ?';
+            $conditions[] = $columnName.' = ?';
         }
 
         return [$columns, $values, $conditions];
@@ -683,7 +683,7 @@ 
         list($columns, $values, $conditions) = $this->gatherConditions($identifier);
 
         return $this->executeUpdate(
-            'DELETE FROM ' . $tableExpression . ' WHERE ' . implode(' AND ', $conditions),
+            'DELETE FROM '.$tableExpression.' WHERE '.implode(' AND ', $conditions),
             $values,
             is_string(key($types)) ? $this->extractTypeValues($columns, $types) : $types
         );
@@ -752,7 +752,7 @@ 
         foreach ($data as $columnName => $value) {
             $setColumns[] = $columnName;
             $setValues[] = $value;
-            $set[] = $columnName . ' = ?';
+            $set[] = $columnName.' = ?';
         }
 
         list($conditionColumns, $conditionValues, $conditions) = $this->gatherConditions($identifier);
@@ -763,8 +763,8 @@ 
             $types = $this->extractTypeValues($columns, $types);
         }
 
-        $sql  = 'UPDATE ' . $tableExpression . ' SET ' . implode(', ', $set)
-                . ' WHERE ' . implode(' AND ', $conditions);
+        $sql = 'UPDATE '.$tableExpression.' SET '.implode(', ', $set)
+                . ' WHERE '.implode(' AND ', $conditions);
 
         return $this->executeUpdate($sql, $values, $types);
     }
@@ -785,7 +785,7 @@ 
     public function insert($tableExpression, array $data, array $types = [])
     {
         if (empty($data)) {
-            return $this->executeUpdate('INSERT INTO ' . $tableExpression . ' ()' . ' VALUES ()');
+            return $this->executeUpdate('INSERT INTO '.$tableExpression.' ()'.' VALUES ()');
         }
 
         $columns = [];
@@ -799,8 +799,8 @@ 
         }
 
         return $this->executeUpdate(
-            'INSERT INTO ' . $tableExpression . ' (' . implode(', ', $columns) . ')' .
-            ' VALUES (' . implode(', ', $set) . ')',
+            'INSERT INTO '.$tableExpression.' ('.implode(', ', $columns).')'.
+            ' VALUES ('.implode(', ', $set).')',
             $values,
             is_string(key($types)) ? $this->extractTypeValues($columns, $types) : $types
         );
@@ -967,7 +967,7 @@ 
     public function executeCacheQuery($query, $params, $types, QueryCacheProfile $qcp)
     {
         $resultCache = $qcp->getResultCacheDriver() ?: $this->_config->getResultCacheImpl();
-        if ( ! $resultCache) {
+        if (!$resultCache) {
             throw CacheException::noResultDriverConfigured();
         }
 
@@ -1230,7 +1230,7 @@ 
             throw ConnectionException::mayNotAlterNestedTransactionWithSavepointsInTransaction();
         }
 
-        if ( ! $this->getDatabasePlatform()->supportsSavepoints()) {
+        if (!$this->getDatabasePlatform()->supportsSavepoints()) {
             throw ConnectionException::savepointsNotSupported();
         }
 
@@ -1409,7 +1409,7 @@ 
      */
     public function createSavepoint($savepoint)
     {
-        if ( ! $this->getDatabasePlatform()->supportsSavepoints()) {
+        if (!$this->getDatabasePlatform()->supportsSavepoints()) {
             throw ConnectionException::savepointsNotSupported();
         }
 
@@ -1427,7 +1427,7 @@ 
      */
     public function releaseSavepoint($savepoint)
     {
-        if ( ! $this->getDatabasePlatform()->supportsSavepoints()) {
+        if (!$this->getDatabasePlatform()->supportsSavepoints()) {
             throw ConnectionException::savepointsNotSupported();
         }
 
@@ -1447,7 +1447,7 @@ 
      */
     public function rollbackSavepoint($savepoint)
     {
-        if ( ! $this->getDatabasePlatform()->supportsSavepoints()) {
+        if (!$this->getDatabasePlatform()->supportsSavepoints()) {
             throw ConnectionException::savepointsNotSupported();
         }
 
@@ -1474,7 +1474,7 @@ 
      */
     public function getSchemaManager()
     {
-        if ( ! $this->_schemaManager) {
+        if (!$this->_schemaManager) {
             $this->_schemaManager = $this->_driver->getSchemaManager($this);
         }
 

core/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connections/MasterSlaveConnection.php

Doc Comments +1 added lines

@@ -142,6 +142,7 @@
 
     /**
      * {@inheritDoc}
+     * @param string $connectionName
      */
     public function connect($connectionName = null)
     {

Unused Use Statements +2 added lines, -2 removed lines

@@ -19,11 +19,11 @@
 
 namespace Doctrine\DBAL\Connections;
 
+use Doctrine\Common\EventManager;
+use Doctrine\DBAL\Configuration;
 use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\Driver;
 use Doctrine\DBAL\Driver\Connection as DriverConnection;
-use Doctrine\DBAL\Configuration;
-use Doctrine\Common\EventManager;
 use Doctrine\DBAL\Event\ConnectionEventArgs;
 use Doctrine\DBAL\Events;
 use function array_rand;

Spacing +4 added lines, -4 removed lines

@@ -113,7 +113,7 @@ 
      */
     public function __construct(array $params, Driver $driver, Configuration $config = null, EventManager $eventManager = null)
     {
-        if (! isset($params['slaves'], $params['master'])) {
+        if (!isset($params['slaves'], $params['master'])) {
             throw new \InvalidArgumentException('master or slaves configuration missing');
         }
         if (count($params['slaves']) == 0) {
@@ -169,7 +169,7 @@ 
         if (isset($this->connections[$connectionName]) && $this->connections[$connectionName]) {
             $this->_conn = $this->connections[$connectionName];
 
-            if ($forceMasterAsSlave && ! $this->keepSlave) {
+            if ($forceMasterAsSlave && !$this->keepSlave) {
                 $this->connections['slave'] = $this->_conn;
             }
 
@@ -180,7 +180,7 @@ 
             $this->connections['master'] = $this->_conn = $this->connectTo($connectionName);
 
             // Set slave connection to master to avoid invalid reads
-            if ( ! $this->keepSlave) {
+            if (!$this->keepSlave) {
                 $this->connections['slave'] = $this->connections['master'];
             }
         } else {
@@ -230,7 +230,7 @@ 
 
         $config = $params['slaves'][array_rand($params['slaves'])];
 
-        if ( ! isset($config['charset']) && isset($params['master']['charset'])) {
+        if (!isset($config['charset']) && isset($params['master']['charset'])) {
             $config['charset'] = $params['master']['charset'];
         }