aimeos / aimeos-typo3

ext_tables.php

Spacing +7 added lines, -7 removed lines

@@ -1,11 +1,11 @@ 
 <?php
 
-defined('TYPO3') or die();
+defined( 'TYPO3' ) or die();
 
 
-$aimeosExtPath = \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::extPath('aimeos');
+$aimeosExtPath = \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::extPath( 'aimeos' );
 
-if (file_exists($aimeosExtPath . '/Resources/Libraries/autoload.php') === true) {
+if( file_exists( $aimeosExtPath . '/Resources/Libraries/autoload.php' ) === true ) {
     require_once $aimeosExtPath . '/Resources/Libraries/autoload.php';
 }
 
@@ -14,7 +14,7 @@ 
  * Register Aimeos icon
  */
 
-$iconRegistry = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance(\TYPO3\CMS\Core\Imaging\IconRegistry::class);
+$iconRegistry = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance( \TYPO3\CMS\Core\Imaging\IconRegistry::class );
 $iconRegistry->registerIcon(
     'aimeos-shop',
     \TYPO3\CMS\Core\Imaging\IconProvider\SvgIconProvider::class,
@@ -32,12 +32,12 @@ 
     'labels' => 'LLL:EXT:aimeos/Resources/Private/Language/admin.xlf',
 ];
 
-if ((bool) \Aimeos\Aimeos\Base::getExtConfig('showPageTree', false) == false) {
+if( (bool) \Aimeos\Aimeos\Base::getExtConfig( 'showPageTree', false ) == false ) {
     $_aimeosConfiguration['navigationComponentId'] = null;
     $_aimeosConfiguration['inheritNavigationComponentFromMainModule'] = false;
 }
 
-$name = defined('TYPO3_version') && version_compare(constant('TYPO3_version'), '11.0.0', '<') ? 'Aimeos.' : '';
+$name = defined( 'TYPO3_version' ) && version_compare( constant( 'TYPO3_version' ), '11.0.0', '<' ) ? 'Aimeos.' : '';
 
 \TYPO3\CMS\Extbase\Utility\ExtensionUtility::registerModule(
     $name . 'aimeos',
@@ -54,7 +54,7 @@ 
 );
 
 
-$signalSlotDispatcher = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Extbase\\SignalSlot\\Dispatcher');
+$signalSlotDispatcher = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance( 'TYPO3\\CMS\\Extbase\\SignalSlot\\Dispatcher' );
 
 /**
  * Execute the setup tasks automatically to create the required tables

class.ext_update.php

Spacing +20 added lines, -20 removed lines

@@ -14,7 +14,7 @@ 
 use TYPO3\CMS\Extbase\Utility\LocalizationUtility;
 
 
-if (file_exists(__DIR__ . '/Resources/Libraries/autoload.php') === true) {
+if( file_exists( __DIR__ . '/Resources/Libraries/autoload.php' ) === true ) {
     require_once __DIR__ . '/Resources/Libraries/autoload.php';
 }
 
@@ -45,16 +45,16 @@ 
     public function main()
     {
         ob_start();
-        $exectimeStart = microtime(true);
+        $exectimeStart = microtime( true );
 
-        if (($result = $this->checkEnvironment()) !== null) {
+        if( ( $result = $this->checkEnvironment() ) !== null ) {
             return $result;
         }
 
         try {
             \Aimeos\Aimeos\Setup::execute();
             $output = ob_get_contents();
-        } catch(Exception $e) {
+        } catch( Exception $e ) {
             $output = ob_get_contents();
             $output .= PHP_EOL . $e->getMessage();
             $output .= PHP_EOL . $e->getTraceAsString();
@@ -64,7 +64,7 @@ 
 
 
         return '<pre>' . $output . '</pre>' . PHP_EOL .
-            sprintf("Setup process lasted %1\$f sec</br>\n", (microtime(true) - $exectimeStart));
+            sprintf( "Setup process lasted %1\$f sec</br>\n", ( microtime( true ) - $exectimeStart ) );
     }
 
 
@@ -77,53 +77,53 @@ 
      */
     protected function checkEnvironment()
     {
-        $objectManager = GeneralUtility::makeInstance(ObjectManager::class);
-        $connectionPool = $objectManager->get(TYPO3\CMS\Core\Database\ConnectionPool::class);
-        $connection = $connectionPool->getQueryBuilderForTable('fe_user')->getConnection();
+        $objectManager = GeneralUtility::makeInstance( ObjectManager::class );
+        $connectionPool = $objectManager->get( TYPO3\CMS\Core\Database\ConnectionPool::class );
+        $connection = $connectionPool->getQueryBuilderForTable( 'fe_user' )->getConnection();
         $params = $connection->getParams();
 
-        if (strpos($connection->getServerVersion(), 'MySQL') === false) {
+        if( strpos( $connection->getServerVersion(), 'MySQL' ) === false ) {
             return;
         }
 
         // MariaDB might get identified as a 'MySQL 5.5.5' for some reason
-        $result = $connection->prepare('SELECT version()');
+        $result = $connection->prepare( 'SELECT version()' );
         $result->execute();
         $rows = $result->fetchAll();
 
-        if (!isset($rows[0]['version()'])) {
+        if( !isset( $rows[0]['version()'] ) ) {
             return;
         }
 
         $version = $rows[0]['version()']; // Something like '10.1.29-MariaDB' or '5.6.33-0ubuntu0'
 
         // Only MySQL < 5.7 and MariaDB < 10.2 don't work
-        if ((strpos($version, 'MariaDB') !== false && version_compare($version, '10.2', '>='))
-            || version_compare($version, '5.7', '>=')
+        if( ( strpos( $version, 'MariaDB' ) !== false && version_compare( $version, '10.2', '>=' ) )
+            || version_compare( $version, '5.7', '>=' )
         ) {
             return;
         }
 
         // MySQL < 5.7 and utf8mb4 charset doesn't work due to missing long index support
-        if (isset($params['tableoptions']['charset']) && $params['tableoptions']['charset'] !== 'utf8mb4') {
+        if( isset( $params['tableoptions']['charset'] ) && $params['tableoptions']['charset'] !== 'utf8mb4' ) {
             return;
         }
 
         // Retrieve the name of the connection (which is not part of the connection class)
-        foreach($connectionPool->getConnectionNames() as $name) {
-            if ($connectionPool->getConnectionByName($name) === $connection) {
+        foreach( $connectionPool->getConnectionNames() as $name ) {
+            if( $connectionPool->getConnectionByName( $name ) === $connection ) {
                 break;
             }
         }
 
         $msg = $objectManager->get(
             FlashMessage::class,
-            LocalizationUtility::translate('LLL:EXT:aimeos/Resources/Private/Language/admin.xlf:t3_9x_config_error_text', 'aimeos', [$name]),
-            LocalizationUtility::translate('LLL:EXT:aimeos/Resources/Private/Language/admin.xlf:t3_9x_config_error_header'),
+            LocalizationUtility::translate( 'LLL:EXT:aimeos/Resources/Private/Language/admin.xlf:t3_9x_config_error_text', 'aimeos', [$name] ),
+            LocalizationUtility::translate( 'LLL:EXT:aimeos/Resources/Private/Language/admin.xlf:t3_9x_config_error_header' ),
             FlashMessage::ERROR
         );
 
-        return $objectManager->get(FlashMessageRendererResolver::class)->resolve()->render([$msg]) .
-            LocalizationUtility::translate('LLL:EXT:aimeos/Resources/Private/Language/admin.xlf:t3_9x_config_error_remedy');
+        return $objectManager->get( FlashMessageRendererResolver::class )->resolve()->render( [$msg] ) .
+            LocalizationUtility::translate( 'LLL:EXT:aimeos/Resources/Private/Language/admin.xlf:t3_9x_config_error_remedy' );
     }
 }

Braces +5 added lines, -2 removed lines

@@ -51,10 +51,13 @@
             return $result;
         }
 
-        try {
+        try
+        {
             \Aimeos\Aimeos\Setup::execute();
             $output = ob_get_contents();
-        } catch(Exception $e) {
+        }
+        catch(Exception $e)
+        {
             $output = ob_get_contents();
             $output .= PHP_EOL . $e->getMessage();
             $output .= PHP_EOL . $e->getTraceAsString();

Tests/Unit/Flexform/CatalogTest.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
 
     public function tearDown()
     {
-        unset($this->object);
+        unset( $this->object );
     }
 
 
@@ -27,8 +27,8 @@ 
      */
     public function getCategories()
     {
-        $result = $this->object->getCategories(array('items' => []));
+        $result = $this->object->getCategories( array( 'items' => [] ) );
 
-        $this->assertArrayHasKey('items', $result);
+        $this->assertArrayHasKey( 'items', $result );
     }
 }
\ No newline at end of file

Tests/Unit/Scheduler/Task/Email6Test.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
 
     public function tearDown()
     {
-        unset($this->object);
+        unset( $this->object );
     }
 
 
@@ -29,6 +29,6 @@ 
     {
         $result = $this->object->execute();
 
-        $this->assertTrue($result);
+        $this->assertTrue( $result );
     }
 }

Tests/Unit/Scheduler/Provider/Email6Test.php

Spacing +34 added lines, -34 removed lines

@@ -22,7 +22,7 @@ 
 
     public function tearDown()
     {
-        unset($this->object);
+        unset( $this->object );
     }
 
 
@@ -34,15 +34,15 @@ 
         $taskInfo = [];
         $module = new SchedulerModuleController();
 
-        $result = $this->object->getAdditionalFields($taskInfo, $this->object, $module);
+        $result = $this->object->getAdditionalFields( $taskInfo, $this->object, $module );
 
-        $this->assertInternalType('array', $result);
-        $this->assertArrayHasKey('aimeos_controller', $result);
-        $this->assertArrayHasKey('aimeos_sitecode', $result);
-        $this->assertArrayHasKey('aimeos_config', $result);
-        $this->assertArrayHasKey('aimeos_sender_from', $result);
-        $this->assertArrayHasKey('aimeos_sender_email', $result);
-        $this->assertArrayHasKey('aimeos_reply_email', $result);
+        $this->assertInternalType( 'array', $result );
+        $this->assertArrayHasKey( 'aimeos_controller', $result );
+        $this->assertArrayHasKey( 'aimeos_sitecode', $result );
+        $this->assertArrayHasKey( 'aimeos_config', $result );
+        $this->assertArrayHasKey( 'aimeos_sender_from', $result );
+        $this->assertArrayHasKey( 'aimeos_sender_email', $result );
+        $this->assertArrayHasKey( 'aimeos_reply_email', $result );
     }
 
 
@@ -53,17 +53,17 @@ 
     {
         $taskInfo = [];
         $module = new SchedulerModuleController();
-        $module->setCurrentAction(new Action('EDIT'));
+        $module->setCurrentAction( new Action( 'EDIT' ) );
 
-        $mock = $this->getMockBuilder('\Aimeos\Aimeos\Scheduler\Provider\Email6')
-            ->setMethods(['getFields'])->getMock();
+        $mock = $this->getMockBuilder( '\Aimeos\Aimeos\Scheduler\Provider\Email6' )
+            ->setMethods( ['getFields'] )->getMock();
 
-        $mock->expects($this->once())->method('getFields')
-            ->will($this->throwException(new \RuntimeException()));
+        $mock->expects( $this->once() )->method( 'getFields' )
+            ->will( $this->throwException( new \RuntimeException() ) );
 
-        $result = $mock->getAdditionalFields($taskInfo, $mock, $module);
+        $result = $mock->getAdditionalFields( $taskInfo, $mock, $module );
 
-        $this->assertEquals([], $result);
+        $this->assertEquals( [], $result );
     }
 
 
@@ -82,14 +82,14 @@ 
         ];
         $task = new Scheduler\Task\Typo6();
 
-        $this->object->saveAdditionalFields($data, $task);
+        $this->object->saveAdditionalFields( $data, $task );
 
-        $this->assertEquals('testsite', $task->aimeos_sitecode);
-        $this->assertEquals('testcntl', $task->aimeos_controller);
-        $this->assertEquals('testconf', $task->aimeos_config);
-        $this->assertEquals('test name', $task->aimeos_sender_from);
-        $this->assertEquals('sender@test', $task->aimeos_sender_email);
-        $this->assertEquals('reply@test', $task->aimeos_reply_email);
+        $this->assertEquals( 'testsite', $task->aimeos_sitecode );
+        $this->assertEquals( 'testcntl', $task->aimeos_controller );
+        $this->assertEquals( 'testconf', $task->aimeos_config );
+        $this->assertEquals( 'test name', $task->aimeos_sender_from );
+        $this->assertEquals( 'sender@test', $task->aimeos_sender_email );
+        $this->assertEquals( 'reply@test', $task->aimeos_reply_email );
     }
 
 
@@ -101,7 +101,7 @@ 
         $data = [];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -115,7 +115,7 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -132,7 +132,7 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -148,7 +148,7 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -165,7 +165,7 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -183,7 +183,7 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -200,7 +200,7 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -218,7 +218,7 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -235,7 +235,7 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -252,7 +252,7 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -273,6 +273,6 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertTrue($this->object->validateAdditionalFields($data, $module));
+        $this->assertTrue( $this->object->validateAdditionalFields( $data, $module ) );
     }
 }

Tests/Unit/Scheduler/Provider/Typo6Test.php

Spacing +22 added lines, -22 removed lines

@@ -21,7 +21,7 @@ 
 
     public function tearDown()
     {
-        unset($this->object);
+        unset( $this->object );
     }
 
 
@@ -32,14 +32,14 @@ 
     {
         $taskInfo = [];
         $module = new SchedulerModuleController();
-        $module->setCurrentAction(new Action('EDIT'));
+        $module->setCurrentAction( new Action( 'EDIT' ) );
 
-        $result = $this->object->getAdditionalFields($taskInfo, $this->object, $module);
+        $result = $this->object->getAdditionalFields( $taskInfo, $this->object, $module );
 
-        $this->assertInternalType('array', $result);
-        $this->assertArrayHasKey('aimeos_controller', $result);
-        $this->assertArrayHasKey('aimeos_sitecode', $result);
-        $this->assertArrayHasKey('aimeos_config', $result);
+        $this->assertInternalType( 'array', $result );
+        $this->assertArrayHasKey( 'aimeos_controller', $result );
+        $this->assertArrayHasKey( 'aimeos_sitecode', $result );
+        $this->assertArrayHasKey( 'aimeos_config', $result );
     }
 
 
@@ -50,17 +50,17 @@ 
     {
         $taskInfo = [];
         $module = new SchedulerModuleController();
-        $module->setCurrentAction(new Action('EDIT'));
+        $module->setCurrentAction( new Action( 'EDIT' ) );
 
-        $mock = $this->getMockBuilder('\Aimeos\Aimeos\Scheduler\Provider\Typo6')
-            ->setMethods(['getFields'])->getMock();
+        $mock = $this->getMockBuilder( '\Aimeos\Aimeos\Scheduler\Provider\Typo6' )
+            ->setMethods( ['getFields'] )->getMock();
 
-        $mock->expects($this->once())->method('getFields')
-            ->will($this->throwException(new \RuntimeException()));
+        $mock->expects( $this->once() )->method( 'getFields' )
+            ->will( $this->throwException( new \RuntimeException() ) );
 
-        $result = $mock->getAdditionalFields($taskInfo, $mock, $module);
+        $result = $mock->getAdditionalFields( $taskInfo, $mock, $module );
 
-        $this->assertEquals([], $result);
+        $this->assertEquals( [], $result );
     }
 
 
@@ -76,11 +76,11 @@ 
         ];
         $task = new \Aimeos\Aimeos\Scheduler\Task\Typo6();
 
-        $this->object->saveAdditionalFields($data, $task);
+        $this->object->saveAdditionalFields( $data, $task );
 
-        $this->assertEquals('testsite', $task->aimeos_sitecode);
-        $this->assertEquals('testcntl', $task->aimeos_controller);
-        $this->assertEquals('testconf', $task->aimeos_config);
+        $this->assertEquals( 'testsite', $task->aimeos_sitecode );
+        $this->assertEquals( 'testcntl', $task->aimeos_controller );
+        $this->assertEquals( 'testconf', $task->aimeos_config );
     }
 
 
@@ -92,7 +92,7 @@ 
         $data = [];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -106,7 +106,7 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -122,7 +122,7 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertFalse($this->object->validateAdditionalFields($data, $module));
+        $this->assertFalse( $this->object->validateAdditionalFields( $data, $module ) );
     }
 
 
@@ -137,6 +137,6 @@ 
         ];
         $module = new SchedulerModuleController();
 
-        $this->assertTrue($this->object->validateAdditionalFields($data, $module));
+        $this->assertTrue( $this->object->validateAdditionalFields( $data, $module ) );
     }
 }

Tests/Unit/Controller/JqadmControllerTest.php

Spacing +52 added lines, -52 removed lines

@@ -16,43 +16,43 @@ 
     {
         \Aimeos\Aimeos\Base::aimeos(); // initialize autoloader
 
-        $this->object = $this->getAccessibleMock('Aimeos\Aimeos\Controller\JqadmController', array('dummy'));
+        $this->object = $this->getAccessibleMock( 'Aimeos\Aimeos\Controller\JqadmController', array( 'dummy' ) );
 
-        $this->request = $this->getMockBuilder('TYPO3\CMS\Extbase\Mvc\Request')
-            ->setMethods(array('hasArgument', 'getArgument'))
+        $this->request = $this->getMockBuilder( 'TYPO3\CMS\Extbase\Mvc\Request' )
+            ->setMethods( array( 'hasArgument', 'getArgument' ) )
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->response = $this->getMockBuilder('TYPO3\CMS\Extbase\Mvc\Response')
-            ->setMethods(array('setHeader'))
+        $this->response = $this->getMockBuilder( 'TYPO3\CMS\Extbase\Mvc\Response' )
+            ->setMethods( array( 'setHeader' ) )
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->view = $this->getMockBuilder('TYPO3\CMS\Extbase\Mvc\View\EmptyView')
-            ->setMethods(array('assign'))
+        $this->view = $this->getMockBuilder( 'TYPO3\CMS\Extbase\Mvc\View\EmptyView' )
+            ->setMethods( array( 'assign' ) )
             ->disableOriginalConstructor()
             ->getMock();
 
-        $uriBuilder = $this->getMockBuilder('TYPO3\CMS\Extbase\Mvc\Web\Routing\UriBuilder')
-            ->setMethods(array('buildBackendUri'))
+        $uriBuilder = $this->getMockBuilder( 'TYPO3\CMS\Extbase\Mvc\Web\Routing\UriBuilder' )
+            ->setMethods( array( 'buildBackendUri' ) )
             ->disableOriginalConstructor()
             ->getMock();
 
 
-        $uriBuilder->expects($this->any())->method('buildBackendUri');
+        $uriBuilder->expects( $this->any() )->method( 'buildBackendUri' );
 
-        $this->object->_set('uriBuilder', $uriBuilder);
-        $this->object->_set('response', $this->response);
-        $this->object->_set('request', $this->request);
-        $this->object->_set('view', $this->view);
+        $this->object->_set( 'uriBuilder', $uriBuilder );
+        $this->object->_set( 'response', $this->response );
+        $this->object->_set( 'request', $this->request );
+        $this->object->_set( 'view', $this->view );
 
-        $this->object->_call('initializeAction');
+        $this->object->_call( 'initializeAction' );
     }
 
 
     public function tearDown()
     {
-        unset($this->object, $this->request, $this->view);
+        unset( $this->object, $this->request, $this->view );
     }
 
 
@@ -61,12 +61,12 @@ 
      */
     public function fileActionCss()
     {
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->returnValue('css'));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->returnValue( 'css' ) );
 
         $result = $this->object->fileAction();
 
-        $this->assertContains('.aimeos', $result);
+        $this->assertContains( '.aimeos', $result );
     }
 
 
@@ -75,12 +75,12 @@ 
      */
     public function fileActionJs()
     {
-        $this->request->expects($this->any())->method('getArgument')
-        ->will($this->returnValue('js'));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+        ->will( $this->returnValue( 'js' ) );
 
         $result = $this->object->fileAction();
 
-        $this->assertContains('Aimeos = {', $result);
+        $this->assertContains( 'Aimeos = {', $result );
     }
 
 
@@ -89,13 +89,13 @@ 
      */
     public function copyAction()
     {
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls('product', 'de', 'unittest'));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( 'product', 'de', 'unittest' ) );
 
-        $this->view->expects($this->exactly(2))->method('assign');
+        $this->view->expects( $this->exactly( 2 ) )->method( 'assign' );
 
         $this->object->copyAction();
     }
@@ -106,13 +106,13 @@ 
      */
     public function createAction()
     {
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls('product', 'de', 'unittest'));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( 'product', 'de', 'unittest' ) );
 
-        $this->view->expects($this->exactly(2))->method('assign');
+        $this->view->expects( $this->exactly( 2 ) )->method( 'assign' );
 
         $this->object->createAction();
     }
@@ -123,13 +123,13 @@ 
      */
     public function deleteAction()
     {
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls('product', 'de', 'unittest'));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( 'product', 'de', 'unittest' ) );
 
-        $this->view->expects($this->exactly(2))->method('assign');
+        $this->view->expects( $this->exactly( 2 ) )->method( 'assign' );
 
         $this->object->deleteAction();
     }
@@ -140,13 +140,13 @@ 
      */
     public function getAction()
     {
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls('product', 'de', 'unittest'));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( 'product', 'de', 'unittest' ) );
 
-        $this->view->expects($this->exactly(2))->method('assign');
+        $this->view->expects( $this->exactly( 2 ) )->method( 'assign' );
 
         $this->object->getAction();
     }
@@ -157,13 +157,13 @@ 
      */
     public function saveAction()
     {
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls('product', 'de', 'unittest'));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( 'product', 'de', 'unittest' ) );
 
-        $this->view->expects($this->exactly(2))->method('assign');
+        $this->view->expects( $this->exactly( 2 ) )->method( 'assign' );
 
         $this->object->saveAction();
     }
@@ -174,13 +174,13 @@ 
      */
     public function searchAction()
     {
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls('product', 'de', 'unittest'));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( 'product', 'de', 'unittest' ) );
 
-        $this->view->expects($this->exactly(2))->method('assign');
+        $this->view->expects( $this->exactly( 2 ) )->method( 'assign' );
 
         $this->object->searchAction();
     }

Tests/Unit/Controller/CatalogControllerTest.php

Spacing +60 added lines, -60 removed lines

@@ -14,31 +14,31 @@ 
     {
         \Aimeos\Aimeos\Base::aimeos(); // initialize autoloader
 
-        $this->object = $this->getAccessibleMock('Aimeos\\Aimeos\\Controller\\CatalogController', array('dummy'));
+        $this->object = $this->getAccessibleMock( 'Aimeos\\Aimeos\\Controller\\CatalogController', array( 'dummy' ) );
 
         $objManager = new \TYPO3\CMS\Extbase\Object\ObjectManager();
 
-        $uriBuilder = $objManager->get('TYPO3\\CMS\\Extbase\\Mvc\\Web\\Routing\\UriBuilder');
-        $response = $objManager->get('TYPO3\\CMS\\Extbase\\Mvc\\Web\\Response');
-        $request = $objManager->get('TYPO3\\CMS\\Extbase\\Mvc\\Web\\Request');
+        $uriBuilder = $objManager->get( 'TYPO3\\CMS\\Extbase\\Mvc\\Web\\Routing\\UriBuilder' );
+        $response = $objManager->get( 'TYPO3\\CMS\\Extbase\\Mvc\\Web\\Response' );
+        $request = $objManager->get( 'TYPO3\\CMS\\Extbase\\Mvc\\Web\\Request' );
 
-        $uriBuilder->setRequest($request);
+        $uriBuilder->setRequest( $request );
 
-        if (method_exists($response, 'setRequest')) {
-            $response->setRequest($request);
+        if( method_exists( $response, 'setRequest' ) ) {
+            $response->setRequest( $request );
         }
 
-        $this->object->_set('uriBuilder', $uriBuilder);
-        $this->object->_set('response', $response);
-        $this->object->_set('request', $request);
+        $this->object->_set( 'uriBuilder', $uriBuilder );
+        $this->object->_set( 'response', $response );
+        $this->object->_set( 'request', $request );
 
-        $this->object->_call('initializeAction');
+        $this->object->_call( 'initializeAction' );
     }
 
 
     public function tearDown()
     {
-        unset($this->object);
+        unset( $this->object );
     }
 
 
@@ -48,16 +48,16 @@ 
     public function countAction()
     {
         $name = '\\Aimeos\\Client\\Html\\Catalog\\Count\\Standard';
-        $client = $this->getMock($name, array('getBody', 'getHeader', 'process'), [], '', false);
+        $client = $this->getMock( $name, array( 'getBody', 'getHeader', 'process' ), [], '', false );
 
-        $client->expects($this->once())->method('getBody')->will($this->returnValue('body'));
-        $client->expects($this->once())->method('getHeader')->will($this->returnValue('header'));
+        $client->expects( $this->once() )->method( 'getBody' )->will( $this->returnValue( 'body' ) );
+        $client->expects( $this->once() )->method( 'getHeader' )->will( $this->returnValue( 'header' ) );
 
-        \Aimeos\Client\Html\Catalog\Count\Factory::injectClient($name, $client);
+        \Aimeos\Client\Html\Catalog\Count\Factory::injectClient( $name, $client );
         $output = $this->object->countAction();
-        \Aimeos\Client\Html\Catalog\Count\Factory::injectClient($name, null);
+        \Aimeos\Client\Html\Catalog\Count\Factory::injectClient( $name, null );
 
-        $this->assertEquals('body', $output);
+        $this->assertEquals( 'body', $output );
     }
 
 
@@ -67,16 +67,16 @@ 
     public function detailAction()
     {
         $name = '\\Aimeos\\Client\\Html\\Catalog\\Detail\\Standard';
-        $client = $this->getMock($name, array('getBody', 'getHeader', 'process'), [], '', false);
+        $client = $this->getMock( $name, array( 'getBody', 'getHeader', 'process' ), [], '', false );
 
-        $client->expects($this->once())->method('getBody')->will($this->returnValue('body'));
-        $client->expects($this->once())->method('getHeader')->will($this->returnValue('header'));
+        $client->expects( $this->once() )->method( 'getBody' )->will( $this->returnValue( 'body' ) );
+        $client->expects( $this->once() )->method( 'getHeader' )->will( $this->returnValue( 'header' ) );
 
-        \Aimeos\Client\Html\Catalog\Detail\Factory::injectClient($name, $client);
+        \Aimeos\Client\Html\Catalog\Detail\Factory::injectClient( $name, $client );
         $output = $this->object->detailAction();
-        \Aimeos\Client\Html\Catalog\Detail\Factory::injectClient($name, null);
+        \Aimeos\Client\Html\Catalog\Detail\Factory::injectClient( $name, null );
 
-        $this->assertEquals('body', $output);
+        $this->assertEquals( 'body', $output );
     }
 
 
@@ -86,16 +86,16 @@ 
     public function filterAction()
     {
         $name = '\\Aimeos\\Client\\Html\\Catalog\\Filter\\Standard';
-        $client = $this->getMock($name, array('getBody', 'getHeader', 'process'), [], '', false);
+        $client = $this->getMock( $name, array( 'getBody', 'getHeader', 'process' ), [], '', false );
 
-        $client->expects($this->once())->method('getBody')->will($this->returnValue('body'));
-        $client->expects($this->once())->method('getHeader')->will($this->returnValue('header'));
+        $client->expects( $this->once() )->method( 'getBody' )->will( $this->returnValue( 'body' ) );
+        $client->expects( $this->once() )->method( 'getHeader' )->will( $this->returnValue( 'header' ) );
 
-        \Aimeos\Client\Html\Catalog\Filter\Factory::injectClient($name, $client);
+        \Aimeos\Client\Html\Catalog\Filter\Factory::injectClient( $name, $client );
         $output = $this->object->filterAction();
-        \Aimeos\Client\Html\Catalog\Filter\Factory::injectClient($name, null);
+        \Aimeos\Client\Html\Catalog\Filter\Factory::injectClient( $name, null );
 
-        $this->assertEquals('body', $output);
+        $this->assertEquals( 'body', $output );
     }
 
 
@@ -105,16 +105,16 @@ 
     public function listAction()
     {
         $name = '\\Aimeos\\Client\\Html\\Catalog\\Lists\\Standard';
-        $client = $this->getMock($name, array('getBody', 'getHeader', 'process'), [], '', false);
+        $client = $this->getMock( $name, array( 'getBody', 'getHeader', 'process' ), [], '', false );
 
-        $client->expects($this->once())->method('getBody')->will($this->returnValue('body'));
-        $client->expects($this->once())->method('getHeader')->will($this->returnValue('header'));
+        $client->expects( $this->once() )->method( 'getBody' )->will( $this->returnValue( 'body' ) );
+        $client->expects( $this->once() )->method( 'getHeader' )->will( $this->returnValue( 'header' ) );
 
-        \Aimeos\Client\Html\Catalog\Lists\Factory::injectClient($name, $client);
+        \Aimeos\Client\Html\Catalog\Lists\Factory::injectClient( $name, $client );
         $output = $this->object->listAction();
-        \Aimeos\Client\Html\Catalog\Lists\Factory::injectClient($name, null);
+        \Aimeos\Client\Html\Catalog\Lists\Factory::injectClient( $name, null );
 
-        $this->assertEquals('body', $output);
+        $this->assertEquals( 'body', $output );
     }
 
 
@@ -124,16 +124,16 @@ 
     public function suggestAction()
     {
         $name = '\\Aimeos\\Client\\Html\\Catalog\\Suggest\\Standard';
-        $client = $this->getMock($name, array('getBody', 'getHeader', 'process'), [], '', false);
+        $client = $this->getMock( $name, array( 'getBody', 'getHeader', 'process' ), [], '', false );
 
-        $client->expects($this->once())->method('getBody')->will($this->returnValue('body'));
-        $client->expects($this->once())->method('getHeader')->will($this->returnValue('header'));
+        $client->expects( $this->once() )->method( 'getBody' )->will( $this->returnValue( 'body' ) );
+        $client->expects( $this->once() )->method( 'getHeader' )->will( $this->returnValue( 'header' ) );
 
-        \Aimeos\Client\Html\Catalog\Suggest\Factory::injectClient($name, $client);
+        \Aimeos\Client\Html\Catalog\Suggest\Factory::injectClient( $name, $client );
         $output = $this->object->suggestAction();
-        \Aimeos\Client\Html\Catalog\Suggest\Factory::injectClient($name, null);
+        \Aimeos\Client\Html\Catalog\Suggest\Factory::injectClient( $name, null );
 
-        $this->assertEquals('body', $output);
+        $this->assertEquals( 'body', $output );
     }
 
 
@@ -143,16 +143,16 @@ 
     public function sessionAction()
     {
         $name = '\\Aimeos\\Client\\Html\\Catalog\\Session\\Standard';
-        $client = $this->getMock($name, array('getBody', 'getHeader', 'process'), [], '', false);
+        $client = $this->getMock( $name, array( 'getBody', 'getHeader', 'process' ), [], '', false );
 
-        $client->expects($this->once())->method('getBody')->will($this->returnValue('body'));
-        $client->expects($this->once())->method('getHeader')->will($this->returnValue('header'));
+        $client->expects( $this->once() )->method( 'getBody' )->will( $this->returnValue( 'body' ) );
+        $client->expects( $this->once() )->method( 'getHeader' )->will( $this->returnValue( 'header' ) );
 
-        \Aimeos\Client\Html\Catalog\Session\Factory::injectClient($name, $client);
+        \Aimeos\Client\Html\Catalog\Session\Factory::injectClient( $name, $client );
         $output = $this->object->sessionAction();
-        \Aimeos\Client\Html\Catalog\Session\Factory::injectClient($name, null);
+        \Aimeos\Client\Html\Catalog\Session\Factory::injectClient( $name, null );
 
-        $this->assertEquals('body', $output);
+        $this->assertEquals( 'body', $output );
     }
 
 
@@ -162,16 +162,16 @@ 
     public function stageAction()
     {
         $name = '\\Aimeos\\Client\\Html\\Catalog\\Stage\\Standard';
-        $client = $this->getMock($name, array('getBody', 'getHeader', 'process'), [], '', false);
+        $client = $this->getMock( $name, array( 'getBody', 'getHeader', 'process' ), [], '', false );
 
-        $client->expects($this->once())->method('getBody')->will($this->returnValue('body'));
-        $client->expects($this->once())->method('getHeader')->will($this->returnValue('header'));
+        $client->expects( $this->once() )->method( 'getBody' )->will( $this->returnValue( 'body' ) );
+        $client->expects( $this->once() )->method( 'getHeader' )->will( $this->returnValue( 'header' ) );
 
-        \Aimeos\Client\Html\Catalog\Stage\Factory::injectClient($name, $client);
+        \Aimeos\Client\Html\Catalog\Stage\Factory::injectClient( $name, $client );
         $output = $this->object->stageAction();
-        \Aimeos\Client\Html\Catalog\Stage\Factory::injectClient($name, null);
+        \Aimeos\Client\Html\Catalog\Stage\Factory::injectClient( $name, null );
 
-        $this->assertEquals('body', $output);
+        $this->assertEquals( 'body', $output );
     }
 
 
@@ -181,15 +181,15 @@ 
     public function stockAction()
     {
         $name = '\\Aimeos\\Client\\Html\\Catalog\\Stock\\Standard';
-        $client = $this->getMock($name, array('getBody', 'getHeader', 'process'), [], '', false);
+        $client = $this->getMock( $name, array( 'getBody', 'getHeader', 'process' ), [], '', false );
 
-        $client->expects($this->once())->method('getBody')->will($this->returnValue('body'));
-        $client->expects($this->once())->method('getHeader')->will($this->returnValue('header'));
+        $client->expects( $this->once() )->method( 'getBody' )->will( $this->returnValue( 'body' ) );
+        $client->expects( $this->once() )->method( 'getHeader' )->will( $this->returnValue( 'header' ) );
 
-        \Aimeos\Client\Html\Catalog\Stock\Factory::injectClient($name, $client);
+        \Aimeos\Client\Html\Catalog\Stock\Factory::injectClient( $name, $client );
         $output = $this->object->stockAction();
-        \Aimeos\Client\Html\Catalog\Stock\Factory::injectClient($name, null);
+        \Aimeos\Client\Html\Catalog\Stock\Factory::injectClient( $name, null );
 
-        $this->assertEquals('body', $output);
+        $this->assertEquals( 'body', $output );
     }
 }
\ No newline at end of file

Tests/Unit/Controller/JsonadmControllerTest.php

Spacing +70 added lines, -70 removed lines

@@ -14,38 +14,38 @@ 
     {
         \Aimeos\Aimeos\Base::aimeos(); // initialize autoloader
 
-        $this->object = $this->getAccessibleMock('Aimeos\\Aimeos\\Controller\\JsonadmController', array('dummy'));
+        $this->object = $this->getAccessibleMock( 'Aimeos\\Aimeos\\Controller\\JsonadmController', array( 'dummy' ) );
 
-        $this->request = $this->getMockBuilder('TYPO3\\CMS\\Extbase\\Mvc\\Web\\Request')
-            ->setMethods(array('hasArgument', 'getArgument', 'getMethod'))
+        $this->request = $this->getMockBuilder( 'TYPO3\\CMS\\Extbase\\Mvc\\Web\\Request' )
+            ->setMethods( array( 'hasArgument', 'getArgument', 'getMethod' ) )
             ->disableOriginalConstructor()
             ->getMock();
 
-        $this->response = $this->getMockBuilder('TYPO3\\CMS\\Extbase\\Mvc\\Web\\Response')
-            ->setMethods(array('setStatus'))
+        $this->response = $this->getMockBuilder( 'TYPO3\\CMS\\Extbase\\Mvc\\Web\\Response' )
+            ->setMethods( array( 'setStatus' ) )
             ->disableOriginalConstructor()
             ->getMock();
 
         $objManager = new \TYPO3\CMS\Extbase\Object\ObjectManager();
-        $uriBuilder = $objManager->get('TYPO3\\CMS\\Extbase\\Mvc\\Web\\Routing\\UriBuilder');
+        $uriBuilder = $objManager->get( 'TYPO3\\CMS\\Extbase\\Mvc\\Web\\Routing\\UriBuilder' );
 
-        $uriBuilder->setRequest($this->request);
+        $uriBuilder->setRequest( $this->request );
 
-        if (method_exists($response, 'setRequest')) {
-            $response->setRequest($this->request);
+        if( method_exists( $response, 'setRequest' ) ) {
+            $response->setRequest( $this->request );
         }
 
-        $this->object->_set('uriBuilder', $uriBuilder);
-        $this->object->_set('response', $this->response);
-        $this->object->_set('request', $this->request);
+        $this->object->_set( 'uriBuilder', $uriBuilder );
+        $this->object->_set( 'response', $this->response );
+        $this->object->_set( 'request', $this->request );
 
-        $this->object->_call('initializeAction');
+        $this->object->_call( 'initializeAction' );
     }
 
 
     public function tearDown()
     {
-        unset($this->object, $this->request, $this->response);
+        unset( $this->object, $this->request, $this->response );
     }
 
 
@@ -54,22 +54,22 @@ 
      */
     public function getAction()
     {
-        $this->request->expects($this->exactly(1))->method('getMethod')
-            ->will($this->returnValue('GET'));
+        $this->request->expects( $this->exactly( 1 ) )->method( 'getMethod' )
+            ->will( $this->returnValue( 'GET' ) );
 
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls('stock/type', null, 'unittest', null, null, 'unittest', null));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( 'stock/type', null, 'unittest', null, null, 'unittest', null ) );
 
-        $this->response->expects($this->exactly(1))->method('setStatus')
-            ->with($this->equalTo(200));
+        $this->response->expects( $this->exactly( 1 ) )->method( 'setStatus' )
+            ->with( $this->equalTo( 200 ) );
 
         $result = $this->object->indexAction();
-        $json = json_decode($result, true);
+        $json = json_decode( $result, true );
 
-        $this->assertNotEquals(null, $json);
+        $this->assertNotEquals( null, $json );
     }
 
 
@@ -78,26 +78,26 @@ 
      */
     public function optionsAction()
     {
-        $this->request->expects($this->exactly(1))->method('getMethod')
-            ->will($this->returnValue('OPTIONS'));
+        $this->request->expects( $this->exactly( 1 ) )->method( 'getMethod' )
+            ->will( $this->returnValue( 'OPTIONS' ) );
 
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls(null, null, 'unittest', null, null, 'unittest', null));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( null, null, 'unittest', null, null, 'unittest', null ) );
 
-        $this->response->expects($this->exactly(1))->method('setStatus')
-            ->with($this->equalTo(200));
+        $this->response->expects( $this->exactly( 1 ) )->method( 'setStatus' )
+            ->with( $this->equalTo( 200 ) );
 
         $result = $this->object->indexAction();
-        $json = json_decode($result, true);
+        $json = json_decode( $result, true );
 
-        $this->assertNotEquals(null, $json);
-        $this->assertArrayHasKey('meta', $json);
-        $this->assertArrayHasKey('resources', $json['meta']);
-        $this->assertGreaterThan(1, count($json['meta']['resources']));
-        $this->assertArrayHasKey('stock/type', $json['meta']['resources']);
+        $this->assertNotEquals( null, $json );
+        $this->assertArrayHasKey( 'meta', $json );
+        $this->assertArrayHasKey( 'resources', $json['meta'] );
+        $this->assertGreaterThan( 1, count( $json['meta']['resources'] ) );
+        $this->assertArrayHasKey( 'stock/type', $json['meta']['resources'] );
     }
 
 
@@ -106,19 +106,19 @@ 
      */
     public function deleteAction()
     {
-        $this->request->expects($this->exactly(1))->method('getMethod')
-            ->will($this->returnValue('DELETE'));
+        $this->request->expects( $this->exactly( 1 ) )->method( 'getMethod' )
+            ->will( $this->returnValue( 'DELETE' ) );
 
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls('stock/type', null, 'unittest', null, null, 'unittest', null));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( 'stock/type', null, 'unittest', null, null, 'unittest', null ) );
 
         $result = $this->object->indexAction();
-        $json = json_decode($result, true);
+        $json = json_decode( $result, true );
 
-        $this->assertNotEquals(null, $json);
+        $this->assertNotEquals( null, $json );
     }
 
 
@@ -127,19 +127,19 @@ 
      */
     public function patchAction()
     {
-        $this->request->expects($this->exactly(1))->method('getMethod')
-            ->will($this->returnValue('PATCH'));
+        $this->request->expects( $this->exactly( 1 ) )->method( 'getMethod' )
+            ->will( $this->returnValue( 'PATCH' ) );
 
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls('stock/type', null, 'unittest', null, null, 'unittest', null));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( 'stock/type', null, 'unittest', null, null, 'unittest', null ) );
 
         $result = $this->object->indexAction();
-        $json = json_decode($result, true);
+        $json = json_decode( $result, true );
 
-        $this->assertNotEquals(null, $json);
+        $this->assertNotEquals( null, $json );
     }
 
 
@@ -148,19 +148,19 @@ 
      */
     public function postAction()
     {
-        $this->request->expects($this->exactly(1))->method('getMethod')
-            ->will($this->returnValue('POST'));
+        $this->request->expects( $this->exactly( 1 ) )->method( 'getMethod' )
+            ->will( $this->returnValue( 'POST' ) );
 
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls('stock/type', null, 'unittest', null, null, 'unittest', null));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( 'stock/type', null, 'unittest', null, null, 'unittest', null ) );
 
         $result = $this->object->indexAction();
-        $json = json_decode($result, true);
+        $json = json_decode( $result, true );
 
-        $this->assertNotEquals(null, $json);
+        $this->assertNotEquals( null, $json );
     }
 
 
@@ -169,18 +169,18 @@ 
      */
     public function putAction()
     {
-        $this->request->expects($this->exactly(1))->method('getMethod')
-            ->will($this->returnValue('PUT'));
+        $this->request->expects( $this->exactly( 1 ) )->method( 'getMethod' )
+            ->will( $this->returnValue( 'PUT' ) );
 
-        $this->request->expects($this->atLeastOnce())->method('hasArgument')
-            ->will($this->returnValue(true));
+        $this->request->expects( $this->atLeastOnce() )->method( 'hasArgument' )
+            ->will( $this->returnValue( true ) );
 
-        $this->request->expects($this->any())->method('getArgument')
-            ->will($this->onConsecutiveCalls('stock/type', null, 'unittest', null, null, 'unittest', null));
+        $this->request->expects( $this->any() )->method( 'getArgument' )
+            ->will( $this->onConsecutiveCalls( 'stock/type', null, 'unittest', null, null, 'unittest', null ) );
 
         $result = $this->object->indexAction();
-        $json = json_decode($result, true);
+        $json = json_decode( $result, true );
 
-        $this->assertNotEquals(null, $json);
+        $this->assertNotEquals( null, $json );
     }
 }
\ No newline at end of file