@@ -32,14 +32,14 @@ |
||
| 32 | 32 | |
| 33 | 33 | private $target = ManagementForm::class; |
| 34 | 34 | |
| 35 | - private $inheritance = [ SummaryForm::class, HeadscriptProviderInterface::class ]; |
|
| 35 | + private $inheritance = [SummaryForm::class, HeadscriptProviderInterface::class]; |
|
| 36 | 36 | |
| 37 | 37 | private $attributes = [ |
| 38 | 38 | 'baseFieldset' => 'Core/Tree/ManagementFieldset', |
| 39 | - 'attributes' => [ 'method' => 'POST', 'class' => 'yk-tree-management-form' ], |
|
| 39 | + 'attributes' => ['method' => 'POST', 'class' => 'yk-tree-management-form'], |
|
| 40 | 40 | ]; |
| 41 | 41 | |
| 42 | 42 | private $properties = [ |
| 43 | - [ 'headscripts', ['value' => ['testHeadScript'], 'default' => [ 'modules/Core/js/html.sortable.min.js', 'modules/Core/js/forms.tree-management.js' ]] ] |
|
| 43 | + ['headscripts', ['value' => ['testHeadScript'], 'default' => ['modules/Core/js/html.sortable.min.js', 'modules/Core/js/forms.tree-management.js']]] |
|
| 44 | 44 | ]; |
| 45 | 45 | } |
@@ -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 | |