@@ -42,7 +42,7 @@ discard block |
||
42 | 42 | */ |
43 | 43 | private $target = 'getTarget'; |
44 | 44 | |
45 | - private $inheritance = [ StrategyInterface::class ]; |
|
45 | + private $inheritance = [StrategyInterface::class]; |
|
46 | 46 | |
47 | 47 | public function propertiesProvider() |
48 | 48 | { |
@@ -51,7 +51,7 @@ discard block |
||
51 | 51 | ['treeRoot', new Node()], |
52 | 52 | ['allowSelectMultipleItems', ['default' => false, 'value' => true, 'getter_method' => '*']], |
53 | 53 | ['allowSelectMultipleItems', ['value' => false, 'getter_method' => '*']], |
54 | - ['allowSelectMultipleItems', ['value' => function () { |
|
54 | + ['allowSelectMultipleItems', ['value' => function() { |
|
55 | 55 | return true; |
56 | 56 | }, 'getter_method' => '*', 'expect' => true]], |
57 | 57 |
@@ -134,8 +134,8 @@ discard block |
||
134 | 134 | public function provideSetFormData() |
135 | 135 | { |
136 | 136 | return [ |
137 | - ['fieldname','input',true,true ], |
|
138 | - ['fieldname', 'foobar', false, false ], |
|
137 | + ['fieldname', 'input', true, true], |
|
138 | + ['fieldname', 'foobar', false, false], |
|
139 | 139 | ]; |
140 | 140 | } |
141 | 141 | |
@@ -192,10 +192,10 @@ discard block |
||
192 | 192 | |
193 | 193 | |
194 | 194 | return [ |
195 | - [$e['application'], null , $e['application'] ], |
|
196 | - [$e['job'], 'job' , $e['job'] ], |
|
197 | - [$e['user'], null , $e['user'] ], |
|
198 | - [$e['organization'], null , $e['organization'] ], |
|
195 | + [$e['application'], null, $e['application']], |
|
196 | + [$e['job'], 'job', $e['job']], |
|
197 | + [$e['user'], null, $e['user']], |
|
198 | + [$e['organization'], null, $e['organization']], |
|
199 | 199 | ]; |
200 | 200 | } |
201 | 201 | |
@@ -213,7 +213,7 @@ discard block |
||
213 | 213 | $childName = 'child'; |
214 | 214 | $child = new Container($childName); |
215 | 215 | $this->target->setForm($childName, $child); |
216 | - $this->assertSame($childName .'.' . $name, $child->formatAction($name)); |
|
216 | + $this->assertSame($childName.'.'.$name, $child->formatAction($name)); |
|
217 | 217 | } |
218 | 218 | |
219 | 219 | public function testGetActionFor() |
@@ -36,41 +36,41 @@ |
||
36 | 36 | */ |
37 | 37 | protected $target = '\Core\Form\Event\FormEvent'; |
38 | 38 | |
39 | - protected $inheritance = [ '\Zend\EventManager\Event' ]; |
|
39 | + protected $inheritance = ['\Zend\EventManager\Event']; |
|
40 | 40 | |
41 | 41 | public function propertiesProvider() |
42 | 42 | { |
43 | 43 | $form = new Form(); |
44 | 44 | $container = new Container(); |
45 | 45 | return [ |
46 | - [ 'Form', '@\Zend\Form\Form' ], |
|
47 | - [ 'Form', '@\Core\Form\Container' ], |
|
48 | - [ 'Form', [ 'value' => 'invalid', 'setter_exception' => ['\InvalidArgumentException', 'Form must either' ] ] ], |
|
49 | - [ 'Target', 'TestTarget' ], |
|
50 | - [ 'Target', [ |
|
46 | + ['Form', '@\Zend\Form\Form'], |
|
47 | + ['Form', '@\Core\Form\Container'], |
|
48 | + ['Form', ['value' => 'invalid', 'setter_exception' => ['\InvalidArgumentException', 'Form must either']]], |
|
49 | + ['Target', 'TestTarget'], |
|
50 | + ['Target', [ |
|
51 | 51 | 'value' => $form, |
52 | - 'expect_property' => [ 'form', $form ] |
|
52 | + 'expect_property' => ['form', $form] |
|
53 | 53 | ]], |
54 | - [ 'Target', [ |
|
54 | + ['Target', [ |
|
55 | 55 | 'value' => $container, |
56 | - 'expect_property' => [ 'form', $container ], |
|
56 | + 'expect_property' => ['form', $container], |
|
57 | 57 | ]], |
58 | - [ 'Target', [ |
|
58 | + ['Target', [ |
|
59 | 59 | 'value' => 'NonFormOrContainer', |
60 | - 'expect_property' => [ 'form', null ], |
|
60 | + 'expect_property' => ['form', null], |
|
61 | 61 | ]], |
62 | - [ 'Params', [ 'value' => [ 'param1' => 'value1' ]]], |
|
63 | - [ 'Params', [ |
|
64 | - 'value' => [ 'form' => $form ], |
|
65 | - 'expect_property' => [ 'form', $form ] |
|
62 | + ['Params', ['value' => ['param1' => 'value1']]], |
|
63 | + ['Params', [ |
|
64 | + 'value' => ['form' => $form], |
|
65 | + 'expect_property' => ['form', $form] |
|
66 | 66 | ]], |
67 | - [ 'Params', [ |
|
68 | - 'value' => [ 'form' => $container ], |
|
69 | - 'expect_property' => [ 'form', $container ] |
|
67 | + ['Params', [ |
|
68 | + 'value' => ['form' => $container], |
|
69 | + 'expect_property' => ['form', $container] |
|
70 | 70 | ]], |
71 | - [ 'Params', [ |
|
72 | - 'value' => [ 'form' => 'notAnFormInstance' ], |
|
73 | - 'expect_property' => [ 'form', null ] |
|
71 | + ['Params', [ |
|
72 | + 'value' => ['form' => 'notAnFormInstance'], |
|
73 | + 'expect_property' => ['form', null] |
|
74 | 74 | ]] |
75 | 75 | ]; |
76 | 76 | } |
@@ -42,9 +42,9 @@ |
||
42 | 42 | |
43 | 43 | public function testSetGetFormId() |
44 | 44 | { |
45 | - $id=123; |
|
45 | + $id = 123; |
|
46 | 46 | $this->target->setFormId($id); |
47 | - $this->assertSame($this->target->getFormId(), (string)$id); |
|
47 | + $this->assertSame($this->target->getFormId(), (string) $id); |
|
48 | 48 | } |
49 | 49 | |
50 | 50 | public function testSetAttribute() |
@@ -52,8 +52,8 @@ discard block |
||
52 | 52 | public function providerRenderMode() |
53 | 53 | { |
54 | 54 | return [ |
55 | - [SummaryFormInterface::RENDER_FORM,SummaryFormInterface::RENDER_FORM], |
|
56 | - [SummaryFormInterface::RENDER_SUMMARY,SummaryFormInterface::RENDER_SUMMARY], |
|
55 | + [SummaryFormInterface::RENDER_FORM, SummaryFormInterface::RENDER_FORM], |
|
56 | + [SummaryFormInterface::RENDER_SUMMARY, SummaryFormInterface::RENDER_SUMMARY], |
|
57 | 57 | [SummaryFormInterface::RENDER_ALL, SummaryFormInterface::RENDER_ALL], |
58 | 58 | ]; |
59 | 59 | } |
@@ -75,8 +75,8 @@ discard block |
||
75 | 75 | public function providerDisplayMode() |
76 | 76 | { |
77 | 77 | return [ |
78 | - [SummaryFormInterface::DISPLAY_FORM,SummaryFormInterface::DISPLAY_FORM], |
|
79 | - [SummaryFormInterface::DISPLAY_SUMMARY,SummaryFormInterface::DISPLAY_SUMMARY], |
|
78 | + [SummaryFormInterface::DISPLAY_FORM, SummaryFormInterface::DISPLAY_FORM], |
|
79 | + [SummaryFormInterface::DISPLAY_SUMMARY, SummaryFormInterface::DISPLAY_SUMMARY], |
|
80 | 80 | ]; |
81 | 81 | } |
82 | 82 | |
@@ -94,7 +94,7 @@ discard block |
||
94 | 94 | */ |
95 | 95 | public function testSetOptions($input, $expected) |
96 | 96 | { |
97 | - $this->target->setOptions(['display_mode' => $input ]); |
|
97 | + $this->target->setOptions(['display_mode' => $input]); |
|
98 | 98 | $this->assertSame($this->target->getDisplayMode(), $expected); |
99 | 99 | } |
100 | 100 | } |
@@ -31,7 +31,7 @@ discard block |
||
31 | 31 | $fs->setName('myField'); |
32 | 32 | $target = new Form(); |
33 | 33 | $target->add($fs); |
34 | - $this->target=$target; |
|
34 | + $this->target = $target; |
|
35 | 35 | } |
36 | 36 | |
37 | 37 | public function testConstructor() |
@@ -66,9 +66,9 @@ discard block |
||
66 | 66 | public function testSetDescriptionWithAdditionalParams() |
67 | 67 | { |
68 | 68 | $input = "this is my description"; |
69 | - $this->target->setDescription($input, ['p1','p2']); |
|
69 | + $this->target->setDescription($input, ['p1', 'p2']); |
|
70 | 70 | $this->assertAttributeSame(['description' => $input, |
71 | - 'description_params' => ['p1','p2']], 'options', $this->target); |
|
71 | + 'description_params' => ['p1', 'p2']], 'options', $this->target); |
|
72 | 72 | } |
73 | 73 | |
74 | 74 | public function testSetGetIsDisableCapable() |
@@ -134,7 +134,7 @@ discard block |
||
134 | 134 | $this->target->addClass($classes); |
135 | 135 | } |
136 | 136 | |
137 | - $x=$this->target->getAttribute('class'); |
|
137 | + $x = $this->target->getAttribute('class'); |
|
138 | 138 | $this->assertEquals($x, $expected); |
139 | 139 | } |
140 | 140 |
@@ -32,27 +32,27 @@ |
||
32 | 32 | public function propertiesProvider() |
33 | 33 | { |
34 | 34 | return [ |
35 | - [ 'summaryEmpty', [ |
|
35 | + ['summaryEmpty', [ |
|
36 | 36 | 'ignore_setter' => true, |
37 | - 'pre' => function () { |
|
37 | + 'pre' => function() { |
|
38 | 38 | $this->target->add(['name' => 'test', 'type' => 'text', 'attributes' => ['value'=>'test']]); |
39 | 39 | }, |
40 | 40 | 'value' => false, |
41 | 41 | 'getter_method' => 'is*' |
42 | 42 | ]], |
43 | - [ 'summaryEmpty', [ |
|
43 | + ['summaryEmpty', [ |
|
44 | 44 | 'ignore_setter' => true, |
45 | - 'pre' => function () { |
|
45 | + 'pre' => function() { |
|
46 | 46 | $this->target->add(['name' => 'test', 'type' => 'text']); |
47 | 47 | }, |
48 | 48 | 'value' => true, |
49 | 49 | 'getter_method' => 'is*', |
50 | 50 | ]], |
51 | - [ 'emptySummaryNotice', [ |
|
51 | + ['emptySummaryNotice', [ |
|
52 | 52 | 'value' => 'notice', |
53 | 53 | 'default' => null |
54 | 54 | ]], |
55 | - [ 'emptySummaryNotice', [ |
|
55 | + ['emptySummaryNotice', [ |
|
56 | 56 | 'value' => 'notice', |
57 | 57 | 'default' => 'defaultNotice', |
58 | 58 | ]], |
@@ -126,7 +126,7 @@ discard block |
||
126 | 126 | $this->assertIsString($result); |
127 | 127 | $this->assertStringStartsWith('<input', $result); |
128 | 128 | $this->assertContains('type="file"', $result); |
129 | - $this->assertContains('name="' . $element->getName() . '"', $result); |
|
129 | + $this->assertContains('name="'.$element->getName().'"', $result); |
|
130 | 130 | } |
131 | 131 | |
132 | 132 | /** |
@@ -223,7 +223,7 @@ discard block |
||
223 | 223 | $fileUploadHelper->expects($this->once()) |
224 | 224 | ->method('renderMarkup') |
225 | 225 | ->with($this->equalTo($element)) |
226 | - ->willReturn($markupOutput . $fileElementPlaceholder); |
|
226 | + ->willReturn($markupOutput.$fileElementPlaceholder); |
|
227 | 227 | $fileUploadHelper->expects($this->once()) |
228 | 228 | ->method('renderFileElement') |
229 | 229 | ->with($this->equalTo($element)) |
@@ -33,9 +33,9 @@ |
||
33 | 33 | |
34 | 34 | private $target = MetaDataFieldset::class; |
35 | 35 | |
36 | - private $inheritance = [ Fieldset::class ]; |
|
36 | + private $inheritance = [Fieldset::class]; |
|
37 | 37 | |
38 | 38 | private $properties = [ |
39 | - [ 'hydrator', ['default@' => MetaDataHydrator::class, '@value' => EntityHydrator::class]] |
|
39 | + ['hydrator', ['default@' => MetaDataHydrator::class, '@value' => EntityHydrator::class]] |
|
40 | 40 | ]; |
41 | 41 | } |