@@ -40,12 +40,12 @@ |
||
40 | 40 | */ |
41 | 41 | private $target = [ |
42 | 42 | ManagementFieldset::class, |
43 | - '@testInitialize' => [ 'mock' => ['add'] ], |
|
43 | + '@testInitialize' => ['mock' => ['add']], |
|
44 | 44 | ]; |
45 | 45 | |
46 | - private $inheritance = [ Fieldset::class, ViewPartialProviderInterface::class ]; |
|
46 | + private $inheritance = [Fieldset::class, ViewPartialProviderInterface::class]; |
|
47 | 47 | |
48 | - private $traits = [ ViewPartialProviderTrait::class ]; |
|
48 | + private $traits = [ViewPartialProviderTrait::class]; |
|
49 | 49 | |
50 | 50 | private $attributes = [ |
51 | 51 | 'defaultPartial' => 'core/form/tree-manage', |
@@ -32,7 +32,7 @@ |
||
32 | 32 | |
33 | 33 | private $target = Select::class; |
34 | 34 | |
35 | - private $inheritance = [ ZfSelect::class, HydratorStrategyProviderInterface::class ]; |
|
35 | + private $inheritance = [ZfSelect::class, HydratorStrategyProviderInterface::class]; |
|
36 | 36 | |
37 | - private $traits = [ HydratorStrategyProviderTrait::class ]; |
|
37 | + private $traits = [HydratorStrategyProviderTrait::class]; |
|
38 | 38 | } |
@@ -37,7 +37,7 @@ discard block |
||
37 | 37 | '@testInitializesItself' => 'setupMock' |
38 | 38 | ]; |
39 | 39 | |
40 | - private $inheritance = [ Fieldset::class, ViewPartialProviderInterface::class, InputFilterProviderInterface::class]; |
|
40 | + private $inheritance = [Fieldset::class, ViewPartialProviderInterface::class, InputFilterProviderInterface::class]; |
|
41 | 41 | |
42 | 42 | private $traits = [ViewPartialProviderTrait::class]; |
43 | 43 | |
@@ -77,13 +77,13 @@ discard block |
||
77 | 77 | 'name' => [ |
78 | 78 | 'required' => true, |
79 | 79 | 'filters' => [ |
80 | - [ 'name' => 'StringTrim' ], |
|
80 | + ['name' => 'StringTrim'], |
|
81 | 81 | ], |
82 | 82 | ], |
83 | 83 | 'value' => [ |
84 | 84 | 'required' => false, |
85 | 85 | 'filters' => [ |
86 | - [ 'name' => 'StringTrim' ], |
|
86 | + ['name' => 'StringTrim'], |
|
87 | 87 | ], |
88 | 88 | ], |
89 | 89 | ]; |
@@ -51,19 +51,19 @@ |
||
51 | 51 | { |
52 | 52 | $mock = $this->getMockBuilder(AddItemFieldset::class) |
53 | 53 | ->setMethods(['setObject', 'add']) |
54 | - ->disableOriginalConstructor()->getMock(); |
|
54 | + ->disableOriginalConstructor()->getMock(); |
|
55 | 55 | |
56 | 56 | $mock->expects($this->once())->method('setObject')->with($this->isInstanceOf(\ArrayObject::class)); |
57 | 57 | |
58 | 58 | $mock->expects($this->exactly(6))->method('add') |
59 | - ->withConsecutive( |
|
60 | - [['name' => 'id', 'type' => 'Hidden']], |
|
61 | - [['name' => 'current', 'type' => 'Hidden']], |
|
62 | - [['name' => 'do', 'type' => 'Hidden']], |
|
63 | - [['name' => 'name', 'type' => 'Text', 'options' => ['label' => 'Name'], 'attributes' => ['required' => 'required']]], |
|
64 | - [['name' => 'value', 'type' => 'Text', 'options' => ['label' => 'Value']]], |
|
65 | - [['name' => 'priority', 'type' => 'Text']] |
|
66 | - ); |
|
59 | + ->withConsecutive( |
|
60 | + [['name' => 'id', 'type' => 'Hidden']], |
|
61 | + [['name' => 'current', 'type' => 'Hidden']], |
|
62 | + [['name' => 'do', 'type' => 'Hidden']], |
|
63 | + [['name' => 'name', 'type' => 'Text', 'options' => ['label' => 'Name'], 'attributes' => ['required' => 'required']]], |
|
64 | + [['name' => 'value', 'type' => 'Text', 'options' => ['label' => 'Value']]], |
|
65 | + [['name' => 'priority', 'type' => 'Text']] |
|
66 | + ); |
|
67 | 67 | |
68 | 68 | return $mock; |
69 | 69 | } |
@@ -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 |
@@ -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 | /** |