cross-solution / YAWIK

module/Core/test/CoreTest/View/Helper/SnippetTest.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
         '@testConstruction' => false,
     ];
 
-    private $inheritance = [ AbstractHelper::class ];
+    private $inheritance = [AbstractHelper::class];
 
     private function getTargetArgs()
     {

module/Core/test/CoreTest/Service/ClearCacheServiceTest.php

Spacing +4 added lines, -4 removed lines

@@ -92,7 +92,7 @@ 
      * @throws \Exception
      * @dataProvider    getTestClearCache
      */
-    public function testClearCache($cacheDir, $expectThrow=false, $message='Test Cache Directory')
+    public function testClearCache($cacheDir, $expectThrow = false, $message = 'Test Cache Directory')
     {
         $options = $this->getMockBuilder(ListenerOptions::class)
             ->disableOriginalConstructor()
@@ -115,9 +115,9 @@ 
     {
         $cacheDir = sys_get_temp_dir().'/yawik/test-cache';
         return [
-            [null,true,'Test with null cache directory'],
-            [sys_get_temp_dir().'/foo',true,'Test with non existent directory'],
-            [$cacheDir,false,'Test with valid cache directory']
+            [null, true, 'Test with null cache directory'],
+            [sys_get_temp_dir().'/foo', true, 'Test with non existent directory'],
+            [$cacheDir, false, 'Test with valid cache directory']
         ];
     }
 

module/Core/test/CoreTest/Listener/LanguageRouteListener/OnRouteCallbackTest.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
     private $target = [
         LanguageRouteListener::class,
-        'mock' => [ 'detectLanguage' => ['return' => 'xx'], 'setLocale' ],
+        'mock' => ['detectLanguage' => ['return' => 'xx'], 'setLocale'],
         'args' => 'getConstructorArgs'
     ];
     

module/Core/test/CoreTest/Listener/AjaxRouteListenerTest.php

Indentation +2 added lines, -2 removed lines

@@ -110,8 +110,8 @@
         $results->push($result1);
         $this->ajaxEventManagerMock->expects($this->once())->method('getEvent')->with('test', $this->target)->willReturn(new AjaxEvent());
         $this->ajaxEventManagerMock->expects($this->once())->method('triggerEventUntil')
-                                   ->with($this->isInstanceOf('\Closure'), $this->isInstanceOf(AjaxEvent::class))
-                                   ->willReturn($results);
+                                    ->with($this->isInstanceOf('\Closure'), $this->isInstanceOf(AjaxEvent::class))
+                                    ->willReturn($results);
 
         $response = $this->target->onRoute($event);
 

Spacing +4 added lines, -4 removed lines

@@ -40,13 +40,13 @@ 
         '@testConstruction' => false,
         '@testInheritance' => '@testUsesTraits',
         '@testDefaultAttributes' => '@testUsesTraits',
-        '@testUsesTraits' => [ 'as_reflection' => true ],
+        '@testUsesTraits' => ['as_reflection' => true],
     ];
 
-    private $traits = [ ListenerAggregateTrait::class ];
+    private $traits = [ListenerAggregateTrait::class];
 
     private $attributes = [
-        'events' => [ [ MvcEvent::EVENT_ROUTE, 'onRoute', 100 ] ],
+        'events' => [[MvcEvent::EVENT_ROUTE, 'onRoute', 100]],
     ];
 
     private function getTargetArgs()
@@ -54,7 +54,7 @@ 
         $this->ajaxEventManagerMock = $this->getMockBuilder(EventManager::class)->disableOriginalConstructor()
             ->setMethods(['getEvent', 'triggerEventUntil'])->getMock();
 
-        return [ $this->ajaxEventManagerMock ];
+        return [$this->ajaxEventManagerMock];
     }
 
     public function testConstruction()

module/Core/test/CoreTest/EventManager/EventManager/BaseTest.php

Spacing +16 added lines, -16 removed lines

@@ -31,7 +31,7 @@ 
      */
     protected $target = '\Core\EventManager\EventManager';
 
-    protected $inheritance = [ '\Zend\EventManager\EventManager', '\Core\EventManager\EventProviderInterface' ];
+    protected $inheritance = ['\Zend\EventManager\EventManager', '\Core\EventManager\EventProviderInterface'];
 
     public function testSetEventPrototype()
     {
@@ -44,23 +44,23 @@ 
     {
         $testTarget = new \stdClass();
         $testTarget2 = new \stdClass();
-        $testParams1 = [ 'params1' => 'value1' ];
-        $testParams2 = [ 'name' => 'test6', 'param2' => 'value2' ];
-        $testParams3 = [ 'name' => 'test8', 'target' => $testTarget, 'param3' => 'value3' ];
-        $expect2     = [ 'param2' => 'value2' ];
-        $expect3     = [ 'param3' => 'value3' ];
+        $testParams1 = ['params1' => 'value1'];
+        $testParams2 = ['name' => 'test6', 'param2' => 'value2'];
+        $testParams3 = ['name' => 'test8', 'target' => $testTarget, 'param3' => 'value3'];
+        $expect2     = ['param2' => 'value2'];
+        $expect3     = ['param3' => 'value3'];
 
         return [
-            [ 'test1', null, [], [ 'name' => 'test1', 'target' => null, 'params' => []]],
-            [ 'test2', $testTarget, [], [ 'name' => 'test2', 'target' => $testTarget, 'params' => []]],
-            [ 'test3', null, $testParams1, [ 'name' => 'test3', 'target' => null, 'params' => $testParams1]],
-            'setParamsAsFirstArg' => [ $testParams1, null, ['ignoreme' => 'yes' ], [ 'name' => null, 'target' => null, 'params' => $testParams1 ]],
-            'setParamsAsSecondArg' => [ 'test5', $testParams1, ['ignoreme' => 'yes' ], ['name' => 'test5', 'target' => null, 'params' => $testParams1 ]],
-            'setNameInParamsAsFirstArg' => [ $testParams2, null, [], [ 'name' => 'test6', 'target' => null, 'params' => $expect2]],
-            'setNameInParamsAsFirstArgAndTarget' => [ $testParams2, $testTarget, ['ignoreme' => 'yes'], [ 'name' => 'test6', 'target' => $testTarget, 'params' => $expect2]],
-            'setNameAndTargetInParams' => [ $testParams3, null, [], [ 'name' => 'test8', 'target' => $testTarget, 'params' => $expect3]],
-            [ 'test9', $testTarget2, $testParams3, ['name' => 'test9', 'target' => $testTarget2, 'params' => $testParams3 ]],
-            [ null, $testParams2, [], ['name' => 'test6', 'target' => null, 'params' => $expect2 ]],
+            ['test1', null, [], ['name' => 'test1', 'target' => null, 'params' => []]],
+            ['test2', $testTarget, [], ['name' => 'test2', 'target' => $testTarget, 'params' => []]],
+            ['test3', null, $testParams1, ['name' => 'test3', 'target' => null, 'params' => $testParams1]],
+            'setParamsAsFirstArg' => [$testParams1, null, ['ignoreme' => 'yes'], ['name' => null, 'target' => null, 'params' => $testParams1]],
+            'setParamsAsSecondArg' => ['test5', $testParams1, ['ignoreme' => 'yes'], ['name' => 'test5', 'target' => null, 'params' => $testParams1]],
+            'setNameInParamsAsFirstArg' => [$testParams2, null, [], ['name' => 'test6', 'target' => null, 'params' => $expect2]],
+            'setNameInParamsAsFirstArgAndTarget' => [$testParams2, $testTarget, ['ignoreme' => 'yes'], ['name' => 'test6', 'target' => $testTarget, 'params' => $expect2]],
+            'setNameAndTargetInParams' => [$testParams3, null, [], ['name' => 'test8', 'target' => $testTarget, 'params' => $expect3]],
+            ['test9', $testTarget2, $testParams3, ['name' => 'test9', 'target' => $testTarget2, 'params' => $testParams3]],
+            [null, $testParams2, [], ['name' => 'test6', 'target' => null, 'params' => $expect2]],
 
         ];
     }

module/Core/test/CoreTestUtils/TestCase/SimpleSetterAndGetterTrait.php

Spacing +3 added lines, -3 removed lines

@@ -21,14 +21,14 @@
         if (!is_object($object)) {
             throw new \InvalidArgumentException('Data Provider first argument should be an object');
         }
-        $setter = 'set' . $propertyName;
-        $getter = 'get' . $propertyName;
+        $setter = 'set'.$propertyName;
+        $getter = 'get'.$propertyName;
 
         call_user_func(array($object, $setter), $propertyValue);
         $this->assertSame(
             $propertyValue,
             call_user_func(array($object, $getter)),
-            '::' . $setter . '() and ::' . $getter . '() should executed properly'
+            '::'.$setter.'() and ::'.$getter.'() should executed properly'
         );
     }
 

module/Core/test/CoreTestUtils/TestCase/TestUsesTraitsTrait.php

Indentation +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     {
         if (!property_exists($this, 'traits') || !property_exists($this, 'target')) {
             throw new \PHPUnit_Framework_Exception(self::class . ': ' . static::class
-                                                   . ' must define the properties $target and $traits.');
+                                                    . ' must define the properties $target and $traits.');
         }
 
         $this->assertUsesTraits($this->traits, $this->target);

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     public function testUsesTraits()
     {
         if (!property_exists($this, 'traits') || !property_exists($this, 'target')) {
-            throw new \PHPUnit_Framework_Exception(self::class . ': ' . static::class
+            throw new \PHPUnit_Framework_Exception(self::class.': '.static::class
                                                    . ' must define the properties $target and $traits.');
         }
 

module/Core/test/CoreTestUtils/TestCase/TestInheritanceTrait.php

Spacing +4 added lines, -4 removed lines

@@ -40,18 +40,18 @@
      */
     public function testInheritance()
     {
-        $errTmpl = __TRAIT__ . ': ' . get_class($this);
+        $errTmpl = __TRAIT__.': '.get_class($this);
 
         if (!property_exists($this, 'inheritance') || !property_exists($this, 'target')) {
-            throw new \PHPUnit_Framework_Exception($errTmpl . ' must define the properties "$inheritance" and "$target"');
+            throw new \PHPUnit_Framework_Exception($errTmpl.' must define the properties "$inheritance" and "$target"');
         }
 
         if (!is_array($this->inheritance)) {
-            throw new \PHPUnit_Framework_Exception($errTmpl . ': Property $inheritance must be an array');
+            throw new \PHPUnit_Framework_Exception($errTmpl.': Property $inheritance must be an array');
         }
 
         if (!is_object($this->target)) {
-            throw new \PHPUnit_Framework_Exception($errTmpl . ': Property $target must be an object');
+            throw new \PHPUnit_Framework_Exception($errTmpl.': Property $target must be an object');
         }
 
         $this->assertInheritance($this->inheritance, $this->target);

module/Core/src/Module.php

Spacing +6 added lines, -6 removed lines

@@ -97,8 +97,8 @@ 
             // assets-install info
             'assets-install [--symlink] [--relative] <target>' => 'Install assets in the given target',
             'The assets-install command will install assets in the given <target> directory. If no option given this command will copy assets into the target.',
-            ['--symlink','This option will install assets using absolute symlink directory'],
-            ['--relative','This option will install assets using relative symlink'],
+            ['--symlink', 'This option will install assets using absolute symlink directory'],
+            ['--relative', 'This option will install assets using relative symlink'],
             ""
         ];
     }
@@ -164,7 +164,7 @@ 
 
         $eventManager->attach(
             MvcEvent::EVENT_DISPATCH_ERROR,
-            function ($event) {
+            function($event) {
                 if ($event instanceof MvcEvent) {
                     $application = $event->getApplication();
                     
@@ -180,7 +180,7 @@ 
         );
         $eventManager->attach(
             MvcEvent::EVENT_DISPATCH,
-            function ($event) use ($eventManager) {
+            function($event) use ($eventManager) {
                 $eventManager->trigger('postDispatch', $event);
             },
             -150
@@ -196,7 +196,7 @@ 
      */
     public function getConfig()
     {
-        $config = include __DIR__ . '/../config/module.config.php';
+        $config = include __DIR__.'/../config/module.config.php';
         return $config;
     }
 
@@ -206,7 +206,7 @@ 
     public function init(ModuleManager $manager)
     {
         $events = $manager->getEventManager();
-        $events->attach(ModuleEvent::EVENT_MERGE_CONFIG, [$this,'onMergeConfig']);
+        $events->attach(ModuleEvent::EVENT_MERGE_CONFIG, [$this, 'onMergeConfig']);
     }
 
     /**