[TASK] Adjust Cs-Fixer 2.0 hits 28/51428/2
authorAnja Leichsenring <aleichsenring@ab-softlab.de>
Wed, 25 Jan 2017 17:37:09 +0000 (18:37 +0100)
committerTymoteusz Motylewski <t.motylewski@gmail.com>
Wed, 25 Jan 2017 19:15:01 +0000 (20:15 +0100)
With the update of cs-fixer to version 2.0, some checks
got minor changes in behaviour. The core follows now in
applying those suggested changes.

Resolves: #79469
Relates: #79361
Releases: master
Change-Id: I5d17b750231dcb386002b3e46b3a89c59eca2989
Reviewed-on: https://review.typo3.org/51428
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Jigal van Hemert <jigal.van.hemert@typo3.org>
Tested-by: Jigal van Hemert <jigal.van.hemert@typo3.org>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
23 files changed:
typo3/sysext/backend/Classes/Form/Container/FullRecordContainer.php
typo3/sysext/backend/Classes/Form/Container/SingleFieldContainer.php
typo3/sysext/backend/Classes/Form/Element/CheckboxElement.php
typo3/sysext/core/Classes/Cache/Backend/RedisBackend.php
typo3/sysext/core/Classes/Cache/Backend/Typo3DatabaseBackend.php
typo3/sysext/core/Classes/Configuration/TypoScript/ConditionMatching/AbstractConditionMatcher.php
typo3/sysext/core/Classes/DataHandling/DataHandler.php
typo3/sysext/core/Classes/Database/QueryGenerator.php
typo3/sysext/core/Classes/Package/PackageManager.php
typo3/sysext/core/Classes/Utility/ExtensionManagementUtility.php
typo3/sysext/core/Classes/Utility/MonitorUtility.php
typo3/sysext/core/Tests/Unit/Imaging/IconFactoryTest.php
typo3/sysext/core/Tests/Unit/Utility/GeneralUtilityTest.php
typo3/sysext/extbase/Classes/Mvc/Cli/Response.php
typo3/sysext/extbase/Classes/Property/PropertyMapper.php
typo3/sysext/extbase/Classes/Property/TypeConverter/DateTimeConverter.php
typo3/sysext/extbase/Classes/Utility/ArrayUtility.php
typo3/sysext/form/Classes/Controller/FormEditorController.php
typo3/sysext/frontend/Classes/ContentObject/ContentDataProcessor.php
typo3/sysext/frontend/Classes/DataProcessing/FilesProcessor.php
typo3/sysext/impexp/Classes/Controller/ImportExportController.php
typo3/sysext/rtehtmlarea/Classes/Controller/SpellCheckingController.php
typo3/sysext/workspaces/Tests/Functional/Service/WorkspaceServiceTest.php

index 6dadb47..9595852 100644 (file)
@@ -54,7 +54,7 @@ class FullRecordContainer extends AbstractContainer
             array_unshift($fieldsArray, '--div--;LLL:EXT:lang/Resources/Private/Language/locallang_core.xlf:labels.generalTab');
         }
         // If first tab has no label definition, add "general" label
-        $firstTabHasLabel = count(GeneralUtility::trimExplode(';',  $fieldsArray[0])) > 1;
+        $firstTabHasLabel = count(GeneralUtility::trimExplode(';', $fieldsArray[0])) > 1;
         if (!$firstTabHasLabel) {
             $fieldsArray[0] = '--div--;LLL:EXT:lang/Resources/Private/Language/locallang_core.xlf:labels.generalTab';
         }
index c2eb736..ed517c5 100644 (file)
@@ -602,7 +602,7 @@ class SingleFieldContainer extends AbstractContainer
      */
     protected function isAssociativeArray($object)
     {
-        return is_array($object) && !empty($object) && array_keys($object) !== range(0, sizeof($object) - 1);
+        return is_array($object) && !empty($object) && array_keys($object) !== range(0, count($object) - 1);
     }
 
     /**
index b3dbc81..62350bb 100644 (file)
@@ -78,7 +78,7 @@ class CheckboxElement extends AbstractFormElement
                 $label = $itemDefinition[0];
                 $html .=
                     '<div class="checkbox-column ' . $colClass . '">'
-                        . $this->renderSingleCheckboxElement($label, $itemKey,  $formElementValue, $numberOfItems, $this->data['parameterArray'], $disabled) .
+                        . $this->renderSingleCheckboxElement($label, $itemKey, $formElementValue, $numberOfItems, $this->data['parameterArray'], $disabled) .
                     '</div>';
                 $counter = $counter + 1;
                 if ($counter < $numberOfItems && !empty($colClear)) {
index 74fe92a..c035d7f 100644 (file)
@@ -217,7 +217,7 @@ class RedisBackend extends AbstractBackend implements TaggableBackendInterface
      */
     public function setDatabase($database)
     {
-        if (!is_integer($database)) {
+        if (!is_int($database)) {
             throw new \InvalidArgumentException('The specified database number is of type "' . gettype($database) . '" but an integer is expected.', 1279763057);
         }
         if ($database < 0) {
@@ -266,7 +266,7 @@ class RedisBackend extends AbstractBackend implements TaggableBackendInterface
      */
     public function setCompressionLevel($compressionLevel)
     {
-        if (!is_integer($compressionLevel)) {
+        if (!is_int($compressionLevel)) {
             throw new \InvalidArgumentException('The specified compression of type "' . gettype($compressionLevel) . '" but an integer is expected.', 1289679154);
         }
         if ($compressionLevel >= -1 && $compressionLevel <= 9) {
@@ -300,7 +300,7 @@ class RedisBackend extends AbstractBackend implements TaggableBackendInterface
             throw new \TYPO3\CMS\Core\Cache\Exception\InvalidDataException('The specified data is of type "' . gettype($data) . '" but a string is expected.', 1279469941);
         }
         $lifetime = $lifetime === null ? $this->defaultLifetime : $lifetime;
-        if (!is_integer($lifetime)) {
+        if (!is_int($lifetime)) {
             throw new \InvalidArgumentException('The specified lifetime is of type "' . gettype($lifetime) . '" but an integer or NULL is expected.', 1279488008);
         }
         if ($lifetime < 0) {
index 89fe79d..e079ba6 100644 (file)
@@ -231,7 +231,7 @@ class Typo3DatabaseBackend extends AbstractBackend implements TaggableBackendInt
             ->from($this->cacheTable)
             ->from($this->tagsTable)
             ->where(
-                $queryBuilder->expr()->eq($this->cacheTable . '.identifier',  $queryBuilder->quoteIdentifier($this->tagsTable . '.identifier')),
+                $queryBuilder->expr()->eq($this->cacheTable . '.identifier', $queryBuilder->quoteIdentifier($this->tagsTable . '.identifier')),
                 $queryBuilder->expr()->eq(
                     $this->tagsTable . '.tag',
                     $queryBuilder->createNamedParameter($tag, \PDO::PARAM_STR)
index bc860d9..e6207d7 100644 (file)
@@ -63,7 +63,7 @@ abstract class AbstractConditionMatcher
      */
     public function setPageId($pageId)
     {
-        if (is_integer($pageId) && $pageId > 0) {
+        if (is_int($pageId) && $pageId > 0) {
             $this->pageId = $pageId;
         }
     }
index 68da2db..73fd884 100644 (file)
@@ -6976,7 +6976,7 @@ class DataHandler
                 $errors = [];
                 foreach ($fieldArray as $key => $value) {
                     if (!$this->checkStoredRecords_loose || $value || $row[$key]) {
-                        if (is_double($row[$key])) {
+                        if (is_float($row[$key])) {
                             // if the database returns the value as double, compare it as double
                             if ((double)$value !== (double)$row[$key]) {
                                 $errors[] = $key;
index fecf77a..a14ce35 100644 (file)
@@ -436,13 +436,13 @@ class QueryGenerator
             // Initialize array to work on, save special parameters
             $ssArr = $this->getSubscript($POST['qG_del']);
             $workArr = &$this->queryConfig;
-            $ssArrSize = sizeof($ssArr) - 1;
+            $ssArrSize = count($ssArr) - 1;
             for ($i = 0; $i < $ssArrSize; $i++) {
                 $workArr = &$workArr[$ssArr[$i]];
             }
             // Delete the entry and move the other entries
             unset($workArr[$ssArr[$i]]);
-            $workArrSize = sizeof($workArr);
+            $workArrSize = count($workArr);
             for ($j = $ssArr[$i]; $j < $workArrSize; $j++) {
                 $workArr[$j] = $workArr[$j + 1];
                 unset($workArr[$j + 1]);
@@ -453,12 +453,12 @@ class QueryGenerator
             // Initialize array to work on, save special parameters
             $ssArr = $this->getSubscript($POST['qG_ins']);
             $workArr = &$this->queryConfig;
-            $ssArrSize = sizeof($ssArr) - 1;
+            $ssArrSize = count($ssArr) - 1;
             for ($i = 0; $i < $ssArrSize; $i++) {
                 $workArr = &$workArr[$ssArr[$i]];
             }
             // Move all entries above position where new entry is to be inserted
-            $workArrSize = sizeof($workArr);
+            $workArrSize = count($workArr);
             for ($j = $workArrSize; $j > $ssArr[$i]; $j--) {
                 $workArr[$j] = $workArr[$j - 1];
             }
@@ -471,7 +471,7 @@ class QueryGenerator
             // Initialize array to work on
             $ssArr = $this->getSubscript($POST['qG_up']);
             $workArr = &$this->queryConfig;
-            $ssArrSize = sizeof($ssArr) - 1;
+            $ssArrSize = count($ssArr) - 1;
             for ($i = 0; $i < $ssArrSize; $i++) {
                 $workArr = &$workArr[$ssArr[$i]];
             }
@@ -485,7 +485,7 @@ class QueryGenerator
             // Initialize array to work on
             $ssArr = $this->getSubscript($POST['qG_nl']);
             $workArr = &$this->queryConfig;
-            $ssArraySize = sizeof($ssArr) - 1;
+            $ssArraySize = count($ssArr) - 1;
             for ($i = 0; $i < $ssArraySize; $i++) {
                 $workArr = &$workArr[$ssArr[$i]];
             }
@@ -506,7 +506,7 @@ class QueryGenerator
             // Initialize array to work on
             $ssArr = $this->getSubscript($POST['qG_remnl']);
             $workArr = &$this->queryConfig;
-            $ssArrSize = sizeof($ssArr) - 1;
+            $ssArrSize = count($ssArr) - 1;
             for ($i = 0; $i < $ssArrSize; $i++) {
                 $workArr = &$workArr[$ssArr[$i]];
             }
index b6e7dad..c95c779 100644 (file)
@@ -288,7 +288,7 @@ class PackageManager implements \TYPO3\CMS\Core\SingletonInterface
     {
         // reset the active packages so they are rebuilt.
         $this->activePackages = [];
-        $this->packageStatesConfiguration['packages'][$package->getPackageKey()] = ['packagePath' => str_replace($this->packagesBasePath, '',  $package->getPackagePath())];
+        $this->packageStatesConfiguration['packages'][$package->getPackageKey()] = ['packagePath' => str_replace($this->packagesBasePath, '', $package->getPackagePath())];
     }
 
     /**
index a6d1d47..81b7910 100644 (file)
@@ -433,7 +433,7 @@ class ExtensionManagementUtility
                 }
                 if (
                     isset($fieldArrayWithOptions[$fieldNumber + 1])
-                    && strpos($fieldArrayWithOptions[$fieldNumber + 1],  '--palette--') === 0
+                    && strpos($fieldArrayWithOptions[$fieldNumber + 1], '--palette--') === 0
                 ) {
                     // Match for $field and next field is a palette - add fields to this one
                     $paletteName = GeneralUtility::trimExplode(';', $fieldArrayWithOptions[$fieldNumber + 1]);
@@ -1482,7 +1482,7 @@ tt_content.' . $key . $suffix . ' {
             throw new \InvalidArgumentException('No TCA definition for table "pages".', 1447789492);
         }
 
-        $value = str_replace(',',  '', 'EXT:' . $extKey . '/' . $filePath);
+        $value = str_replace(',', '', 'EXT:' . $extKey . '/' . $filePath);
         $itemArray = [trim($title . ' (' . $extKey . ')'), $value];
         $GLOBALS['TCA']['pages']['columns']['tsconfig_includes']['config']['items'][] = $itemArray;
     }
index 98ba580..2ee1b1e 100644 (file)
@@ -28,7 +28,7 @@ class MonitorUtility
     {
         $peakUsage = memory_get_peak_usage(true);
         $memoryLimit = GeneralUtility::getBytesFromSizeMeasurement(ini_get('memory_limit'));
-        if (is_double($memoryLimit) && $memoryLimit != 0) {
+        if (is_float($memoryLimit) && $memoryLimit != 0) {
             if ($peakUsage / $memoryLimit >= 0.9) {
                 /** @var $registry \TYPO3\CMS\Core\Registry */
                 $registry = GeneralUtility::makeInstance(\TYPO3\CMS\Core\Registry::class);
index 925f2ea..9af74ee 100644 (file)
@@ -385,7 +385,7 @@ class IconFactoryTest extends \TYPO3\CMS\Components\TestingFramework\Core\UnitTe
     public function getIconForResourceWithOpenFolderReturnsOpenFolderIcon()
     {
         $folderObject = $this->getTestSubjectFolderObject('/test');
-        $result = $this->subject->getIconForResource($folderObject, Icon::SIZE_DEFAULT, null,  ['folder-open' => true])->render();
+        $result = $this->subject->getIconForResource($folderObject, Icon::SIZE_DEFAULT, null, ['folder-open' => true])->render();
         $this->assertContains('<span class="t3js-icon icon icon-size-default icon-state-default icon-apps-filetree-folder-opened" data-identifier="apps-filetree-folder-opened">', $result);
     }
 
index cc5b546..9ccafca 100644 (file)
@@ -4586,7 +4586,7 @@ class GeneralUtilityTest extends \TYPO3\CMS\Components\TestingFramework\Core\Uni
         $inputData = ['called' => []];
         GeneralUtility::callUserFunction('&TYPO3\\CMS\\Core\\Tests\\Unit\\Utility\\GeneralUtilityTest->user_calledUserFunctionCountCallers', $inputData, $this);
         GeneralUtility::callUserFunction('&TYPO3\\CMS\\Core\\Tests\\Unit\\Utility\\GeneralUtilityTest->user_calledUserFunctionCountCallers', $inputData, $this);
-        $this->assertEquals(1, sizeof($inputData['called']));
+        $this->assertEquals(1, count($inputData['called']));
     }
 
     /**
index b2c5dbb..1fa1906 100644 (file)
@@ -36,7 +36,7 @@ class Response extends \TYPO3\CMS\Extbase\Mvc\Response
      */
     public function setExitCode($exitCode)
     {
-        if (!is_integer($exitCode)) {
+        if (!is_int($exitCode)) {
             throw new \InvalidArgumentException(sprintf('Tried to set invalid exit code. The value must be integer, %s given.', gettype($exitCode)), 1312222064);
         }
         $this->exitCode = $exitCode;
index 2c5f70f..712db66 100644 (file)
@@ -367,7 +367,7 @@ class PropertyMapper implements \TYPO3\CMS\Core\SingletonInterface
             return 'array';
         } elseif (is_float($source)) {
             return 'float';
-        } elseif (is_integer($source)) {
+        } elseif (is_int($source)) {
             return 'integer';
         } elseif (is_bool($source)) {
             return 'boolean';
index 0a0f155..0d92302 100644 (file)
@@ -105,7 +105,7 @@ class DateTimeConverter extends \TYPO3\CMS\Extbase\Property\TypeConverter\Abstra
         if (is_array($source)) {
             return true;
         }
-        if (is_integer($source)) {
+        if (is_int($source)) {
             return true;
         }
         return is_string($source);
@@ -126,12 +126,12 @@ class DateTimeConverter extends \TYPO3\CMS\Extbase\Property\TypeConverter\Abstra
         $dateFormat = $this->getDefaultDateFormat($configuration);
         if (is_string($source)) {
             $dateAsString = $source;
-        } elseif (is_integer($source)) {
+        } elseif (is_int($source)) {
             $dateAsString = strval($source);
         } else {
             if (isset($source['date']) && is_string($source['date'])) {
                 $dateAsString = $source['date'];
-            } elseif (isset($source['date']) && is_integer($source['date'])) {
+            } elseif (isset($source['date']) && is_int($source['date'])) {
                 $dateAsString = strval($source['date']);
             } elseif ($this->isDatePartKeysProvided($source)) {
                 if ($source['day'] < 1 || $source['month'] < 1 || $source['year'] < 1) {
index e39e862..6d49c36 100644 (file)
@@ -355,7 +355,7 @@ class ArrayUtility
         GeneralUtility::logDeprecatedFunction();
         $containsNumericalKeysOnly = true;
         array_walk($array, function ($value, $key) use (&$containsNumericalKeysOnly) {
-            if (!is_integer($key)) {
+            if (!is_int($key)) {
                 $containsNumericalKeysOnly = false;
                 return;
             }
index fd2df64..1392535 100644 (file)
@@ -352,7 +352,7 @@ class FormEditorController extends AbstractBackendController
     {
         $output = [];
         foreach ($input as $key => $value) {
-            if (is_integer($key) && is_array($value) && isset($value['_label']) && isset($value['_value'])) {
+            if (is_int($key) && is_array($value) && isset($value['_label']) && isset($value['_value'])) {
                 $key = $value['_value'];
                 $value = $value['_label'];
             }
index da3140a..40fa5d4 100644 (file)
@@ -43,7 +43,7 @@ class ContentDataProcessor
             foreach ($processorKeys as $key) {
                 $className = $processors[$key];
                 if (!class_exists($className)) {
-                    throw new \UnexpectedValueException('Processor class name "' . $className . '" does not exist!',  1427455378);
+                    throw new \UnexpectedValueException('Processor class name "' . $className . '" does not exist!', 1427455378);
                 }
 
                 if (!in_array(DataProcessorInterface::class, class_implements($className), true)) {
index 4413704..c66298b 100644 (file)
@@ -47,7 +47,7 @@ class FilesProcessor implements DataProcessorInterface
      * @param array $processedData Key/value store of processed data (e.g. to be passed to a Fluid View)
      * @return array the processed data as key/value store
      */
-    public function process(ContentObjectRenderer $cObj, array $contentObjectConfiguration, array $processorConfiguration,  array $processedData)
+    public function process(ContentObjectRenderer $cObj, array $contentObjectConfiguration, array $processorConfiguration, array $processedData)
     {
         if (isset($processorConfiguration['if.']) && !$cObj->checkIf($processorConfiguration['if.'])) {
             return $processedData;
index 3e9e2f3..185e3e5 100644 (file)
@@ -887,7 +887,7 @@ class ImportExportController extends BaseScriptClass
                         }
                     }
                     $import->display_import_pid_record = $this->pageinfo;
-                    $this->standaloneView->assign('contentOverview',  $import->displayContentOverview());
+                    $this->standaloneView->assign('contentOverview', $import->displayContentOverview());
                 }
                 // Compile messages which are inhibiting a proper import and add them to output.
                 if (!empty($importInhibitedMessages)) {
index 749bfc0..a177ea3 100644 (file)
@@ -345,7 +345,7 @@ var selectedDictionary = "' . $this->dictionary . '";
             // Calculating parsing and spell checkting time
             $time = number_format(microtime(true) - $time_start, 2, ',', ' ');
             // Insert spellcheck info
-            $this->result .= 'var spellcheckInfo = { "Total words":"' . $this->wordCount . '","Misspelled words":"' . sizeof($this->misspelled) . '","Total suggestions":"' . $this->suggestionCount . '","Total words suggested":"' . $this->suggestedWordCount . '","Spelling checked in":"' . $time . '" };
+            $this->result .= 'var spellcheckInfo = { "Total words":"' . $this->wordCount . '","Misspelled words":"' . count($this->misspelled) . '","Total suggestions":"' . $this->suggestionCount . '","Total words suggested":"' . $this->suggestedWordCount . '","Spelling checked in":"' . $time . '" };
 // -->
 /*]]>*/
 </script>
@@ -586,13 +586,13 @@ var selectedDictionary = "' . $this->dictionary . '";
                 if ($this->pspell_is_available && !$this->forceCommandMode) {
                     if (!pspell_check($this->pspell_link, $word)) {
                         if (!in_array($word, $this->misspelled)) {
-                            if (sizeof($this->misspelled) != 0) {
+                            if (count($this->misspelled) != 0) {
                                 $this->suggestedWords .= ',';
                             }
                             $suggest = pspell_suggest($this->pspell_link, $word);
-                            if (sizeof($suggest) != 0) {
+                            if (count($suggest) != 0) {
                                 $this->suggestionCount++;
-                                $this->suggestedWordCount += sizeof($suggest);
+                                $this->suggestedWordCount += count($suggest);
                             }
                             $this->suggestedWords .= '"' . $word . '":"' . implode(',', $suggest) . '"';
                             $this->misspelled[] = $word;
@@ -632,7 +632,7 @@ var selectedDictionary = "' . $this->dictionary . '";
                     GeneralUtility::unlink_tempfile($tmpFileName);
                     if ($AspellResultLines['1'][0] !== '*') {
                         if (!in_array($word, $this->misspelled)) {
-                            if (sizeof($this->misspelled) != 0) {
+                            if (count($this->misspelled) != 0) {
                                 $this->suggestedWords .= ',';
                             }
                             $suggest = [];
@@ -641,9 +641,9 @@ var selectedDictionary = "' . $this->dictionary . '";
                                 $suggestions = GeneralUtility::trimExplode(':', $AspellResultLines['1'], true);
                                 $suggest = GeneralUtility::trimExplode(',', $suggestions['1'], true);
                             }
-                            if (sizeof($suggest) != 0) {
+                            if (count($suggest) != 0) {
                                 $this->suggestionCount++;
-                                $this->suggestedWordCount += sizeof($suggest);
+                                $this->suggestedWordCount += count($suggest);
                             }
                             $this->suggestedWords .= '"' . $word . '":"' . implode(',', $suggest) . '"';
                             $this->misspelled[] = $word;
index 85d2322..9d9e55b 100644 (file)
@@ -54,7 +54,7 @@ class WorkspaceServiceTest extends \TYPO3\CMS\Components\TestingFramework\Core\F
         $service = new \TYPO3\CMS\Workspaces\Service\WorkspaceService();
         $result = $service->selectVersionsInWorkspace(91, 0, -99, 2);
         $this->assertTrue(is_array($result), 'The result from workspace 91 is supposed to be an array');
-        $this->assertEquals(1, sizeof($result['pages']), 'The result is supposed to contain one version for this page in workspace 91');
+        $this->assertEquals(1, count($result['pages']), 'The result is supposed to contain one version for this page in workspace 91');
         $this->assertEquals(102, $result['pages'][0]['uid'], 'Wrong workspace overlay record picked');
         $this->assertEquals(1, $result['pages'][0]['livepid'], 'Real pid wasn\'t resolved correctly');
     }
@@ -69,7 +69,7 @@ class WorkspaceServiceTest extends \TYPO3\CMS\Components\TestingFramework\Core\F
         $service = new \TYPO3\CMS\Workspaces\Service\WorkspaceService();
         $result = $service->selectVersionsInWorkspace(\TYPO3\CMS\Workspaces\Service\WorkspaceService::SELECT_ALL_WORKSPACES, 0, -99, 2);
         $this->assertTrue(is_array($result), 'The result from workspace 91 is supposed to be an array');
-        $this->assertEquals(2, sizeof($result['pages']), 'The result is supposed to contain one version for this page in workspace 91');
+        $this->assertEquals(2, count($result['pages']), 'The result is supposed to contain one version for this page in workspace 91');
     }
 
     /**
@@ -82,7 +82,7 @@ class WorkspaceServiceTest extends \TYPO3\CMS\Components\TestingFramework\Core\F
         $service = new \TYPO3\CMS\Workspaces\Service\WorkspaceService();
         $result = $service->selectVersionsInWorkspace(91, 0, -99, 1, 99);
         $this->assertTrue(is_array($result), 'The result from workspace 91 is supposed to be an array');
-        $this->assertEquals(4, sizeof($result['pages']), 'The result is supposed to contain four versions for this page in workspace 91');
+        $this->assertEquals(4, count($result['pages']), 'The result is supposed to contain four versions for this page in workspace 91');
     }
 
     /**
@@ -96,13 +96,13 @@ class WorkspaceServiceTest extends \TYPO3\CMS\Components\TestingFramework\Core\F
         // testing stage 1
         $result = $service->selectVersionsInWorkspace(91, 0, 1, 1, 99);
         $this->assertTrue(is_array($result), 'The result from workspace 91 is supposed to be an array');
-        $this->assertEquals(2, sizeof($result['pages']), 'The result is supposed to contain two versions for this page in workspace 91');
+        $this->assertEquals(2, count($result['pages']), 'The result is supposed to contain two versions for this page in workspace 91');
         $this->assertEquals(102, $result['pages'][0]['uid'], 'First records is supposed to have the uid 102');
         $this->assertEquals(105, $result['pages'][1]['uid'], 'First records is supposed to have the uid 105');
         // testing stage 2
         $result = $service->selectVersionsInWorkspace(91, 0, 2, 1, 99);
         $this->assertTrue(is_array($result), 'The result from workspace 91 is supposed to be an array');
-        $this->assertEquals(2, sizeof($result['pages']), 'The result is supposed to contain two versions for this page in workspace 91');
+        $this->assertEquals(2, count($result['pages']), 'The result is supposed to contain two versions for this page in workspace 91');
         $this->assertEquals(104, $result['pages'][0]['uid'], 'First records is supposed to have the uid 106');
         $this->assertEquals(106, $result['pages'][1]['uid'], 'First records is supposed to have the uid 106');
     }
@@ -118,13 +118,13 @@ class WorkspaceServiceTest extends \TYPO3\CMS\Components\TestingFramework\Core\F
         // testing all "draft" records
         $result = $service->selectVersionsInWorkspace(91, 1, -99, 1, 99);
         $this->assertTrue(is_array($result), 'The result from workspace 91 is supposed to be an array');
-        $this->assertEquals(2, sizeof($result['pages']), 'The result is supposed to contain three versions for this page in workspace 91');
+        $this->assertEquals(2, count($result['pages']), 'The result is supposed to contain three versions for this page in workspace 91');
         // testing all "archive" records
         $result = $service->selectVersionsInWorkspace(91, 2, -99, 1, 99);
-        $this->assertEquals(2, sizeof($result['pages']), 'The result is supposed to contain two versions for this page in workspace 91');
+        $this->assertEquals(2, count($result['pages']), 'The result is supposed to contain two versions for this page in workspace 91');
         // testing both types records
         $result = $service->selectVersionsInWorkspace(91, 0, -99, 1, 99);
-        $this->assertEquals(4, sizeof($result['pages']), 'The result is supposed to contain two versions for this page in workspace 91');
+        $this->assertEquals(4, count($result['pages']), 'The result is supposed to contain two versions for this page in workspace 91');
     }
 
     /**
@@ -140,8 +140,8 @@ class WorkspaceServiceTest extends \TYPO3\CMS\Components\TestingFramework\Core\F
         // Test if the placeholder can be found when we ask using recursion (same result)
         $service = new \TYPO3\CMS\Workspaces\Service\WorkspaceService();
         $result = $service->selectVersionsInWorkspace(91, 0, -99, 2, 99);
-        $this->assertEquals(0, sizeof($result['pages']), 'Changes should not show up in this branch of the tree within workspace 91');
-        $this->assertEquals(0, sizeof($result['tt_content']), 'Changes should not show up in this branch of the tree within workspace 91');
+        $this->assertEquals(0, count($result['pages']), 'Changes should not show up in this branch of the tree within workspace 91');
+        $this->assertEquals(0, count($result['tt_content']), 'Changes should not show up in this branch of the tree within workspace 91');
     }
 
     /**
@@ -153,11 +153,11 @@ class WorkspaceServiceTest extends \TYPO3\CMS\Components\TestingFramework\Core\F
         // Test if the placeholder can be found when we ask using recursion (same result)
         $service = new \TYPO3\CMS\Workspaces\Service\WorkspaceService();
         $result = $service->selectVersionsInWorkspace(91, 0, -99, 5, 99);
-        $this->assertEquals(1, sizeof($result['pages']), 'Wrong amount of page versions found within workspace 91');
+        $this->assertEquals(1, count($result['pages']), 'Wrong amount of page versions found within workspace 91');
         $this->assertEquals(103, $result['pages'][0]['uid'], 'Wrong move-to pointer found for page 3 in workspace 91');
         $this->assertEquals(5, $result['pages'][0]['wspid'], 'Wrong workspace-pointer found for page 3 in workspace 91');
         $this->assertEquals(2, $result['pages'][0]['livepid'], 'Wrong live-pointer found for page 3 in workspace 91');
-        $this->assertEquals(1, sizeof($result['tt_content']), 'Wrong amount of tt_content versions found within workspace 91');
+        $this->assertEquals(1, count($result['tt_content']), 'Wrong amount of tt_content versions found within workspace 91');
         $this->assertEquals(106, $result['tt_content'][0]['uid'], 'Wrong move-to pointer found for page 3 in workspace 91');
         $this->assertEquals(7, $result['tt_content'][0]['wspid'], 'Wrong workspace-pointer found for page 3 in workspace 91');
         $this->assertEquals(2, $result['tt_content'][0]['livepid'], 'Wrong live-pointer found for page 3 in workspace 91');
@@ -172,8 +172,8 @@ class WorkspaceServiceTest extends \TYPO3\CMS\Components\TestingFramework\Core\F
         // Test if the placeholder can be found when we ask using recursion (same result)
         $service = new \TYPO3\CMS\Workspaces\Service\WorkspaceService();
         $result = $service->selectVersionsInWorkspace(91, 0, -99, 3, 99);
-        $this->assertEquals(1, sizeof($result), 'Wrong amount of versions found within workspace 91');
-        $this->assertEquals(1, sizeof($result['pages']), 'Wrong amount of page versions found within workspace 91');
+        $this->assertEquals(1, count($result), 'Wrong amount of versions found within workspace 91');
+        $this->assertEquals(1, count($result['pages']), 'Wrong amount of page versions found within workspace 91');
         $this->assertEquals(103, $result['pages'][0]['uid'], 'Wrong move-to pointer found for page 3 in workspace 91');
     }
 }