@@ -38,7 +38,7 @@ |
||
38 | 38 | '@testConstruction' => false, |
39 | 39 | ]; |
40 | 40 | |
41 | - private $inheritance = [ AbstractHelper::class ]; |
|
41 | + private $inheritance = [AbstractHelper::class]; |
|
42 | 42 | |
43 | 43 | private function getTargetArgs() |
44 | 44 | { |
@@ -92,7 +92,7 @@ discard block |
||
92 | 92 | * @throws \Exception |
93 | 93 | * @dataProvider getTestClearCache |
94 | 94 | */ |
95 | - public function testClearCache($cacheDir, $expectThrow=false, $message='Test Cache Directory') |
|
95 | + public function testClearCache($cacheDir, $expectThrow = false, $message = 'Test Cache Directory') |
|
96 | 96 | { |
97 | 97 | $options = $this->getMockBuilder(ListenerOptions::class) |
98 | 98 | ->disableOriginalConstructor() |
@@ -115,9 +115,9 @@ discard block |
||
115 | 115 | { |
116 | 116 | $cacheDir = sys_get_temp_dir().'/yawik/test-cache'; |
117 | 117 | return [ |
118 | - [null,true,'Test with null cache directory'], |
|
119 | - [sys_get_temp_dir().'/foo',true,'Test with non existent directory'], |
|
120 | - [$cacheDir,false,'Test with valid cache directory'] |
|
118 | + [null, true, 'Test with null cache directory'], |
|
119 | + [sys_get_temp_dir().'/foo', true, 'Test with non existent directory'], |
|
120 | + [$cacheDir, false, 'Test with valid cache directory'] |
|
121 | 121 | ]; |
122 | 122 | } |
123 | 123 |
@@ -36,7 +36,7 @@ |
||
36 | 36 | |
37 | 37 | private $target = [ |
38 | 38 | LanguageRouteListener::class, |
39 | - 'mock' => [ 'detectLanguage' => ['return' => 'xx'], 'setLocale' ], |
|
39 | + 'mock' => ['detectLanguage' => ['return' => 'xx'], 'setLocale'], |
|
40 | 40 | 'args' => 'getConstructorArgs' |
41 | 41 | ]; |
42 | 42 |
@@ -110,8 +110,8 @@ |
||
110 | 110 | $results->push($result1); |
111 | 111 | $this->ajaxEventManagerMock->expects($this->once())->method('getEvent')->with('test', $this->target)->willReturn(new AjaxEvent()); |
112 | 112 | $this->ajaxEventManagerMock->expects($this->once())->method('triggerEventUntil') |
113 | - ->with($this->isInstanceOf('\Closure'), $this->isInstanceOf(AjaxEvent::class)) |
|
114 | - ->willReturn($results); |
|
113 | + ->with($this->isInstanceOf('\Closure'), $this->isInstanceOf(AjaxEvent::class)) |
|
114 | + ->willReturn($results); |
|
115 | 115 | |
116 | 116 | $response = $this->target->onRoute($event); |
117 | 117 |
@@ -40,13 +40,13 @@ discard block |
||
40 | 40 | '@testConstruction' => false, |
41 | 41 | '@testInheritance' => '@testUsesTraits', |
42 | 42 | '@testDefaultAttributes' => '@testUsesTraits', |
43 | - '@testUsesTraits' => [ 'as_reflection' => true ], |
|
43 | + '@testUsesTraits' => ['as_reflection' => true], |
|
44 | 44 | ]; |
45 | 45 | |
46 | - private $traits = [ ListenerAggregateTrait::class ]; |
|
46 | + private $traits = [ListenerAggregateTrait::class]; |
|
47 | 47 | |
48 | 48 | private $attributes = [ |
49 | - 'events' => [ [ MvcEvent::EVENT_ROUTE, 'onRoute', 100 ] ], |
|
49 | + 'events' => [[MvcEvent::EVENT_ROUTE, 'onRoute', 100]], |
|
50 | 50 | ]; |
51 | 51 | |
52 | 52 | private function getTargetArgs() |
@@ -54,7 +54,7 @@ discard block |
||
54 | 54 | $this->ajaxEventManagerMock = $this->getMockBuilder(EventManager::class)->disableOriginalConstructor() |
55 | 55 | ->setMethods(['getEvent', 'triggerEventUntil'])->getMock(); |
56 | 56 | |
57 | - return [ $this->ajaxEventManagerMock ]; |
|
57 | + return [$this->ajaxEventManagerMock]; |
|
58 | 58 | } |
59 | 59 | |
60 | 60 | public function testConstruction() |
@@ -21,14 +21,14 @@ |
||
21 | 21 | if (!is_object($object)) { |
22 | 22 | throw new \InvalidArgumentException('Data Provider first argument should be an object'); |
23 | 23 | } |
24 | - $setter = 'set' . $propertyName; |
|
25 | - $getter = 'get' . $propertyName; |
|
24 | + $setter = 'set'.$propertyName; |
|
25 | + $getter = 'get'.$propertyName; |
|
26 | 26 | |
27 | 27 | call_user_func(array($object, $setter), $propertyValue); |
28 | 28 | $this->assertSame( |
29 | 29 | $propertyValue, |
30 | 30 | call_user_func(array($object, $getter)), |
31 | - '::' . $setter . '() and ::' . $getter . '() should executed properly' |
|
31 | + '::'.$setter.'() and ::'.$getter.'() should executed properly' |
|
32 | 32 | ); |
33 | 33 | } |
34 | 34 |
@@ -35,7 +35,7 @@ |
||
35 | 35 | { |
36 | 36 | if (!property_exists($this, 'traits') || !property_exists($this, 'target')) { |
37 | 37 | throw new \PHPUnit_Framework_Exception(self::class . ': ' . static::class |
38 | - . ' must define the properties $target and $traits.'); |
|
38 | + . ' must define the properties $target and $traits.'); |
|
39 | 39 | } |
40 | 40 | |
41 | 41 | $this->assertUsesTraits($this->traits, $this->target); |
@@ -34,7 +34,7 @@ |
||
34 | 34 | public function testUsesTraits() |
35 | 35 | { |
36 | 36 | if (!property_exists($this, 'traits') || !property_exists($this, 'target')) { |
37 | - throw new \PHPUnit_Framework_Exception(self::class . ': ' . static::class |
|
37 | + throw new \PHPUnit_Framework_Exception(self::class.': '.static::class |
|
38 | 38 | . ' must define the properties $target and $traits.'); |
39 | 39 | } |
40 | 40 |
@@ -40,18 +40,18 @@ |
||
40 | 40 | */ |
41 | 41 | public function testInheritance() |
42 | 42 | { |
43 | - $errTmpl = __TRAIT__ . ': ' . get_class($this); |
|
43 | + $errTmpl = __TRAIT__.': '.get_class($this); |
|
44 | 44 | |
45 | 45 | if (!property_exists($this, 'inheritance') || !property_exists($this, 'target')) { |
46 | - throw new \PHPUnit_Framework_Exception($errTmpl . ' must define the properties "$inheritance" and "$target"'); |
|
46 | + throw new \PHPUnit_Framework_Exception($errTmpl.' must define the properties "$inheritance" and "$target"'); |
|
47 | 47 | } |
48 | 48 | |
49 | 49 | if (!is_array($this->inheritance)) { |
50 | - throw new \PHPUnit_Framework_Exception($errTmpl . ': Property $inheritance must be an array'); |
|
50 | + throw new \PHPUnit_Framework_Exception($errTmpl.': Property $inheritance must be an array'); |
|
51 | 51 | } |
52 | 52 | |
53 | 53 | if (!is_object($this->target)) { |
54 | - throw new \PHPUnit_Framework_Exception($errTmpl . ': Property $target must be an object'); |
|
54 | + throw new \PHPUnit_Framework_Exception($errTmpl.': Property $target must be an object'); |
|
55 | 55 | } |
56 | 56 | |
57 | 57 | $this->assertInheritance($this->inheritance, $this->target); |
@@ -97,8 +97,8 @@ discard block |
||
97 | 97 | // assets-install info |
98 | 98 | 'assets-install [--symlink] [--relative] <target>' => 'Install assets in the given target', |
99 | 99 | 'The assets-install command will install assets in the given <target> directory. If no option given this command will copy assets into the target.', |
100 | - ['--symlink','This option will install assets using absolute symlink directory'], |
|
101 | - ['--relative','This option will install assets using relative symlink'], |
|
100 | + ['--symlink', 'This option will install assets using absolute symlink directory'], |
|
101 | + ['--relative', 'This option will install assets using relative symlink'], |
|
102 | 102 | "" |
103 | 103 | ]; |
104 | 104 | } |
@@ -164,7 +164,7 @@ discard block |
||
164 | 164 | |
165 | 165 | $eventManager->attach( |
166 | 166 | MvcEvent::EVENT_DISPATCH_ERROR, |
167 | - function ($event) { |
|
167 | + function($event) { |
|
168 | 168 | if ($event instanceof MvcEvent) { |
169 | 169 | $application = $event->getApplication(); |
170 | 170 | |
@@ -180,7 +180,7 @@ discard block |
||
180 | 180 | ); |
181 | 181 | $eventManager->attach( |
182 | 182 | MvcEvent::EVENT_DISPATCH, |
183 | - function ($event) use ($eventManager) { |
|
183 | + function($event) use ($eventManager) { |
|
184 | 184 | $eventManager->trigger('postDispatch', $event); |
185 | 185 | }, |
186 | 186 | -150 |
@@ -196,7 +196,7 @@ discard block |
||
196 | 196 | */ |
197 | 197 | public function getConfig() |
198 | 198 | { |
199 | - $config = include __DIR__ . '/../config/module.config.php'; |
|
199 | + $config = include __DIR__.'/../config/module.config.php'; |
|
200 | 200 | return $config; |
201 | 201 | } |
202 | 202 | |
@@ -206,7 +206,7 @@ discard block |
||
206 | 206 | public function init(ModuleManager $manager) |
207 | 207 | { |
208 | 208 | $events = $manager->getEventManager(); |
209 | - $events->attach(ModuleEvent::EVENT_MERGE_CONFIG, [$this,'onMergeConfig']); |
|
209 | + $events->attach(ModuleEvent::EVENT_MERGE_CONFIG, [$this, 'onMergeConfig']); |
|
210 | 210 | } |
211 | 211 | |
212 | 212 | /** |
@@ -103,8 +103,8 @@ |
||
103 | 103 | ->getMockBuilder('\Core\Form\Form') |
104 | 104 | ->disableOriginalConstructor() |
105 | 105 | ->setMethods(['setIsDisableCapable', 'setIsDisableElementsCapable', |
106 | - 'setIsDescriptionsEnabled', 'setDescription', 'setParam', |
|
107 | - 'setLabel', 'get']) |
|
106 | + 'setIsDescriptionsEnabled', 'setDescription', 'setParam', |
|
107 | + 'setLabel', 'get']) |
|
108 | 108 | ->getMock(); |
109 | 109 | |
110 | 110 | $form->expects($this->once())->method('setIsDisableCapable')->with(false)->will($this->returnSelf()); |
@@ -38,7 +38,7 @@ discard block |
||
38 | 38 | '@testDefaultPropertyValues' => AttachmentsFormFactory::class, |
39 | 39 | ]; |
40 | 40 | |
41 | - private $inheritance = [ FileUploadFactory::class ]; |
|
41 | + private $inheritance = [FileUploadFactory::class]; |
|
42 | 42 | |
43 | 43 | private $attributes = [ |
44 | 44 | 'fileElement' => 'Core/FileUpload', |
@@ -61,8 +61,8 @@ discard block |
||
61 | 61 | public function provideTypes() |
62 | 62 | { |
63 | 63 | return [ |
64 | - [ null ], |
|
65 | - [ ['text/plain'] ], |
|
64 | + [null], |
|
65 | + [['text/plain']], |
|
66 | 66 | ]; |
67 | 67 | } |
68 | 68 |