[TASK] Fix some cgl violations 36/48836/2
authorChristian Kuhn <lolli@schwarzbu.ch>
Wed, 6 Jul 2016 22:12:00 +0000 (00:12 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Wed, 6 Jul 2016 22:19:33 +0000 (00:19 +0200)
To reduce worries about cgl violations from unrelated changes
in new patches a couple of existing violations are fixed with
the patch.

Change-Id: Iaf981fcce413bedbc3e7d42e87dfa375a424fd03
Resolves: #76967
Releases: master
Reviewed-on: https://review.typo3.org/48836
Reviewed-by: Oliver Klee <typo3-coding@oliverklee.de>
Tested-by: Bamboo TYPO3com <info@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/backend/Classes/Tree/View/AbstractTreeView.php
typo3/sysext/backend/Tests/Unit/Form/Wizard/SuggestWizardTest.php
typo3/sysext/extbase/Classes/Utility/DebuggerUtility.php
typo3/sysext/extbase/Tests/Unit/Mvc/RequestTest.php
typo3/sysext/fluid/Tests/Unit/ViewHelpers/ViewHelperBaseTestcase.php
typo3/sysext/form/Classes/ContentObject/FormContentObject.php
typo3/sysext/frontend/Tests/Unit/ContentObject/ContentObjectRendererTest.php
typo3/sysext/frontend/Tests/Unit/Controller/TypoScriptFrontendControllerTest.php
typo3/sysext/recordlist/Classes/LinkHandler/FileLinkHandler.php
typo3/sysext/version/Classes/Dependency/ElementEntity.php

index f660326..6367c7a 100644 (file)
@@ -936,7 +936,7 @@ abstract class AbstractTreeView
         if (is_array($this->data)) {
             return count($this->dataLookup[$res][$this->subLevelID]);
         } else {
-            return $res->rowCount();;
+            return $res->rowCount();
         }
     }
 
index 574b2cf..f60e766 100644 (file)
@@ -153,12 +153,14 @@ class SuggestWizardTest extends UnitTestCase
      * @test
      * @dataProvider isTableHiddenIsProperlyRetrievedDataProvider
      */
-    public function isTableHiddenIsProperlyRetrieved($expected, $array) {
+    public function isTableHiddenIsProperlyRetrieved($expected, $array)
+    {
         $subject = $this->getAccessibleMock(SuggestWizard::class, array('dummy'), array(), '', false);
         $this->assertEquals($expected, $subject->_call('isTableHidden', $array));
     }
 
-    public function isTableHiddenIsProperlyRetrievedDataProvider() {
+    public function isTableHiddenIsProperlyRetrievedDataProvider()
+    {
         return [
           'notSetValue' => [false, array('ctrl' => array('hideTable' => null))],
           'true' => [true, array('ctrl' => array('hideTable' => true))],
index d100e88..036b596 100644 (file)
@@ -391,7 +391,7 @@ class DebuggerUtility
                         } else {
                             $parameterDump .= '<span class="extbase-debug-string">'
                                 . htmlspecialchars(var_export($parameter->getDefaultValue(), true)) . '</span>';
-                         }
+                        }
                     }
                     $params[] = $parameterDump;
                 }
index 11598fe..b2f9621 100644 (file)
@@ -243,7 +243,7 @@ class RequestTest extends \TYPO3\CMS\Core\Tests\UnitTestCase
         $request = $this->getMockBuilder(\TYPO3\CMS\Extbase\Mvc\Request::class)
             ->setMethods(array('getControllerObjectName'))
             ->disableOriginalConstructor()
-            ->getMock(); 
+            ->getMock();
         $request->expects($this->once())->method('getControllerObjectName')->will($this->returnValue(''));
         $request->setControllerActionName('theAction');
         $this->assertEquals('theAction', $request->getControllerActionName());
index 5cd9eb3..fe06621 100644 (file)
@@ -15,8 +15,8 @@ namespace TYPO3\CMS\Fluid\Tests\Unit\ViewHelpers;
  */
 use TYPO3\CMS\Fluid\Core\Variables\CmsVariableProvider;
 use TYPO3Fluid\Fluid\Core\ViewHelper\AbstractTagBasedViewHelper;
-use TYPO3Fluid\Fluid\Core\ViewHelper\ViewHelperInterface;
 use TYPO3Fluid\Fluid\Core\ViewHelper\TagBuilder;
+use TYPO3Fluid\Fluid\Core\ViewHelper\ViewHelperInterface;
 use TYPO3Fluid\Fluid\Core\ViewHelper\ViewHelperVariableContainer;
 
 /**
index b3def56..0ebb8b5 100644 (file)
@@ -90,7 +90,7 @@ class FormContentObject extends AbstractContentObject
      */
     protected function prepareNonCacheableUserFunction($typoScript)
     {
-        $configuration = array (
+        $configuration = array(
             'userFunc' => 'TYPO3\\CMS\\Extbase\\Core\\Bootstrap->run',
             'pluginName' => 'Form',
             'extensionName' => 'Form',
index cf749c7..fc817f7 100644 (file)
@@ -537,7 +537,7 @@ class ContentObjectRendererTest extends UnitTestCase
         $subject
             ->expects($this->once())
             ->method('cropHTML')
-            ->with( $content, $conf['cropHTML'])
+            ->with($content, $conf['cropHTML'])
             ->willReturn($return);
         $this->assertSame($return,
             $subject->stdWrap_cropHTML($content, $conf));
@@ -561,8 +561,8 @@ class ContentObjectRendererTest extends UnitTestCase
             . 'original ' . 'version of the crop function.';
         $textWithLinebreaks = "Lorem ipsum dolor sit amet,\n"
             . "consetetur sadipscing elitr,\n"
-            . "sed diam nonumy eirmod tempor invidunt ut labore e"
-            . "t dolore magna aliquyam";
+            . 'sed diam nonumy eirmod tempor invidunt ut labore e'
+            . 't dolore magna aliquyam';
 
         return [
             'plain text; 11|...' => [
@@ -771,8 +771,8 @@ class ContentObjectRendererTest extends UnitTestCase
             ],
             'text with linebreaks' => [
                 "Lorem ipsum dolor sit amet,\nconsetetur sadipscing elitr,\ns"
-                . "ed diam nonumy eirmod tempor invidunt ut labore e"
-                . "t dolore magna",
+                . 'ed diam nonumy eirmod tempor invidunt ut labore e'
+                . 't dolore magna',
                 $textWithLinebreaks, '121',
             ],
         ];
@@ -836,7 +836,7 @@ class ContentObjectRendererTest extends UnitTestCase
      * @param string $expect Expected result.
      * @param string $input Given input string.
      * @param array $conf Property 'cache.'
-     * @param integer $times Times called mocked method.
+     * @param int $times Times called mocked method.
      * @param array $with Parameter passed to mocked method.
      * @param string|false $will Return value of mocked method.
      * @return void
@@ -1773,7 +1773,6 @@ class ContentObjectRendererTest extends UnitTestCase
         $this->assertSame($expected, $this->subject->stdWrap_stripHtml($content));
     }
 
-
     /**
      * Check if stdWrap_crop works properly.
      *
@@ -1800,13 +1799,12 @@ class ContentObjectRendererTest extends UnitTestCase
         $subject
             ->expects($this->once())
             ->method('crop')
-            ->with( $content, $conf['crop'])
+            ->with($content, $conf['crop'])
             ->willReturn($return);
         $this->assertSame($return,
             $subject->stdWrap_crop($content, $conf));
     }
 
-
     /**
      * Data provider for round
      *
@@ -2624,7 +2622,7 @@ class ContentObjectRendererTest extends UnitTestCase
         $subject
             ->expects($this->once())
             ->method('replacement')
-            ->with( $content, $conf['replacement.'])
+            ->with($content, $conf['replacement.'])
             ->willReturn($return);
         $this->assertSame($return,
             $subject->stdWrap_replacement($content, $conf));
@@ -3381,19 +3379,19 @@ class ContentObjectRendererTest extends UnitTestCase
     public function substringDataProvider()
     {
         return [
-            'sub -1'    => [ 'g', 'substring', '-1'],
-            'sub -1,0'  => [ 'g', 'substring', '-1,0'],
-            'sub -1,-1' => [ '', 'substring', '-1,-1'],
-            'sub -1,1'  => [ 'g', 'substring', '-1,1'],
-            'sub 0'     => [ 'substring', 'substring', '0'],
-            'sub 0,0'   => [ 'substring', 'substring', '0,0'],
-            'sub 0,-1'  => [ 'substrin', 'substring', '0,-1'],
-            'sub 0,1'   => [ 's', 'substring', '0,1'],
-            'sub 1'     => [ 'ubstring', 'substring', '1'],
-            'sub 1,0'   => [ 'ubstring', 'substring', '1,0'],
-            'sub 1,-1'  => [ 'ubstrin', 'substring', '1,-1'],
-            'sub 1,1'   => [ 'u', 'substring', '1,1'],
-            'sub'       => [ 'substring', 'substring', ''],
+            'sub -1'    => [ 'g', 'substring', '-1'],
+            'sub -1,0'  => [ 'g', 'substring', '-1,0'],
+            'sub -1,-1' => [ '', 'substring', '-1,-1'],
+            'sub -1,1'  => [ 'g', 'substring', '-1,1'],
+            'sub 0'     => [ 'substring', 'substring', '0'],
+            'sub 0,0'   => [ 'substring', 'substring', '0,0'],
+            'sub 0,-1'  => [ 'substrin', 'substring', '0,-1'],
+            'sub 0,1'   => [ 's', 'substring', '0,1'],
+            'sub 1'     => [ 'ubstring', 'substring', '1'],
+            'sub 1,0'   => [ 'ubstring', 'substring', '1,0'],
+            'sub 1,-1'  => [ 'ubstrin', 'substring', '1,-1'],
+            'sub 1,1'   => [ 'u', 'substring', '1,1'],
+            'sub'       => [ 'substring', 'substring', ''],
         ];
     }
 
@@ -3438,7 +3436,7 @@ class ContentObjectRendererTest extends UnitTestCase
         $subject
             ->expects($this->once())
             ->method('substring')
-            ->with( $content, $conf['substring'])
+            ->with($content, $conf['substring'])
             ->willReturn($return);
         $this->assertSame($return,
             $subject->stdWrap_substring($content, $conf));
@@ -7314,7 +7312,6 @@ class ContentObjectRendererTest extends UnitTestCase
         $this->assertSame(1, $pageRepo::$storeHashCallCount);
     }
 
-
     /**
      * Check if calculateCacheKey works properly.
      *
@@ -7375,7 +7372,7 @@ class ContentObjectRendererTest extends UnitTestCase
      * @dataProvider calculateCacheKeyDataProvider
      * @param string $expect Expected result.
      * @param array $conf Properties 'key', 'key.'
-     * @param integer $times Times called mocked method.
+     * @param int $times Times called mocked method.
      * @param array $with Parameter passed to mocked method.
      * @param string $will Return value of mocked method.
      * @return void
@@ -7423,7 +7420,7 @@ class ContentObjectRendererTest extends UnitTestCase
      * @param string $expect Expected result.
      * @param array $conf Configuration to pass to calculateCacheKey mock.
      * @param string $cacheKey Return from calculateCacheKey mock.
-     * @param integer $times Times the cache is expected to be called (0 or 1).
+     * @param int $times Times the cache is expected to be called (0 or 1).
      * @param string $cached Return from cacheFrontend mock.
      * @return void
      */
index 4335405..7fdec76 100644 (file)
@@ -302,5 +302,4 @@ class TypoScriptFrontendControllerTest extends \TYPO3\CMS\Core\Tests\UnitTestCas
         $this->subject->baseUrl = $baseUrl;
         $this->assertSame($expected, $this->subject->baseUrlWrap($url));
     }
-
 }
index 085a638..daf127a 100644 (file)
@@ -174,7 +174,6 @@ class FileLinkHandler extends AbstractLinkHandler implements LinkHandlerInterfac
         $this->view->assign('itemsInSelectedFolder', $itemsInSelectedFolder);
     }
 
-
     /**
      * @param Folder $folder
      * @param string $extensionList
@@ -269,7 +268,8 @@ class FileLinkHandler extends AbstractLinkHandler implements LinkHandlerInterfac
      * @param string $folderIdentifier
      * @return mixed the folder object or false if nothing was found
      */
-    protected function getSelectedFolder($folderIdentifier = '') {
+    protected function getSelectedFolder($folderIdentifier = '')
+    {
         $selectedFolder = false;
         if ($folderIdentifier) {
             try {
@@ -288,7 +288,6 @@ class FileLinkHandler extends AbstractLinkHandler implements LinkHandlerInterfac
             } catch (\Exception $e) {
                 // No path is selected
             }
-
         }
 
         // If no folder is selected, get the user's default upload folder
index 85799f7..603550d 100644 (file)
@@ -14,7 +14,6 @@ namespace TYPO3\CMS\Version\Dependency;
  * The TYPO3 project - inspiring people to share!
  */
 use TYPO3\CMS\Core\Database\ConnectionPool;
-use TYPO3\CMS\Core\Database\Query\Restriction\DeletedRestriction;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
 
 /**