@@ -40,7 +40,7 @@ |
||
40 | 40 | */ |
41 | 41 | public function addFields($fieldsArray) |
42 | 42 | { |
43 | - foreach($fieldsArray as $fieldName => $fieldConfig) { |
|
43 | + foreach ($fieldsArray as $fieldName => $fieldConfig) { |
|
44 | 44 | if (is_object($fieldConfig)) { |
45 | 45 | $this->addField($fieldName, $fieldConfig); |
46 | 46 | } else { |
@@ -1,9 +1,9 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Date: 07.12.15 |
|
4 | - * |
|
5 | - * @author Portey Vasil <[email protected]> |
|
6 | - */ |
|
3 | + * Date: 07.12.15 |
|
4 | + * |
|
5 | + * @author Portey Vasil <[email protected]> |
|
6 | + */ |
|
7 | 7 | |
8 | 8 | namespace Youshido\Tests\Type\Config; |
9 | 9 |
@@ -79,7 +79,9 @@ discard block |
||
79 | 79 | |
80 | 80 | private function isArrayOfValues($data) |
81 | 81 | { |
82 | - if (!is_array($data)) return false; |
|
82 | + if (!is_array($data)) { |
|
83 | + return false; |
|
84 | + } |
|
83 | 85 | |
84 | 86 | foreach ($data as $item) { |
85 | 87 | if (!array_key_exists('value', $item)) { |
@@ -92,10 +94,14 @@ discard block |
||
92 | 94 | |
93 | 95 | private function isArrayOfFields($data) |
94 | 96 | { |
95 | - if (!is_array($data)) return false; |
|
97 | + if (!is_array($data)) { |
|
98 | + return false; |
|
99 | + } |
|
96 | 100 | |
97 | 101 | foreach ($data as $name => $item) { |
98 | - if (!$this->isField($item, $name)) return false; |
|
102 | + if (!$this->isField($item, $name)) { |
|
103 | + return false; |
|
104 | + } |
|
99 | 105 | } |
100 | 106 | |
101 | 107 | return true; |
@@ -109,7 +115,9 @@ discard block |
||
109 | 115 | |
110 | 116 | try { |
111 | 117 | /** @todo need to change it to optimize performance */ |
112 | - if (empty($data['name'])) $data['name'] = $name; |
|
118 | + if (empty($data['name'])) { |
|
119 | + $data['name'] = $name; |
|
120 | + } |
|
113 | 121 | |
114 | 122 | $config = new FieldConfig($data); |
115 | 123 | |
@@ -123,10 +131,14 @@ discard block |
||
123 | 131 | |
124 | 132 | private function isArrayOfInputs($data) |
125 | 133 | { |
126 | - if (!is_array($data)) return false; |
|
134 | + if (!is_array($data)) { |
|
135 | + return false; |
|
136 | + } |
|
127 | 137 | |
128 | 138 | foreach ($data as $name => $item) { |
129 | - if (!$this->isInputField($item, $name)) return false; |
|
139 | + if (!$this->isInputField($item, $name)) { |
|
140 | + return false; |
|
141 | + } |
|
130 | 142 | } |
131 | 143 | |
132 | 144 | return true; |
@@ -139,7 +151,9 @@ discard block |
||
139 | 151 | } |
140 | 152 | try { |
141 | 153 | /** @todo need to change it to optimize performance */ |
142 | - if (empty($data['name'])) $data['name'] = $name; |
|
154 | + if (empty($data['name'])) { |
|
155 | + $data['name'] = $name; |
|
156 | + } |
|
143 | 157 | |
144 | 158 | $config = new InputFieldConfig($data); |
145 | 159 |
@@ -22,7 +22,7 @@ |
||
22 | 22 | ->addField('id', TypeMap::TYPE_ID, ['required' => true]) |
23 | 23 | ->addField('name', TypeMap::TYPE_STRING, ['required' => true]) |
24 | 24 | ->addField('friends', new ListType(['item' => new CharacterInterface()]), [ |
25 | - 'resolve' => function($value){ |
|
25 | + 'resolve' => function($value) { |
|
26 | 26 | return $value['friends']; |
27 | 27 | } |
28 | 28 | ]) |
@@ -28,7 +28,7 @@ |
||
28 | 28 | { |
29 | 29 | parent::build($config); |
30 | 30 | |
31 | - $config->getField('friends')->set('resolve', function($droid){ |
|
31 | + $config->getField('friends')->set('resolve', function($droid) { |
|
32 | 32 | return StarWarsData::getFriends($droid); |
33 | 33 | }); |
34 | 34 |
@@ -1,9 +1,9 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Date: 07.12.15 |
|
4 | - * |
|
5 | - * @author Portey Vasil <[email protected]> |
|
6 | - */ |
|
3 | + * Date: 07.12.15 |
|
4 | + * |
|
5 | + * @author Portey Vasil <[email protected]> |
|
6 | + */ |
|
7 | 7 | |
8 | 8 | namespace Youshido\Tests\Type\Config; |
9 | 9 |
@@ -31,7 +31,7 @@ |
||
31 | 31 | ->addField('friends', new ListType([ |
32 | 32 | 'item' => new CharacterInterface() |
33 | 33 | ]), [ |
34 | - 'resolve' => function ($droid) { |
|
34 | + 'resolve' => function($droid) { |
|
35 | 35 | return StarWarsData::getFriends($droid); |
36 | 36 | }, |
37 | 37 | ]) |
@@ -1,9 +1,9 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Date: 07.12.15 |
|
4 | - * |
|
5 | - * @author Portey Vasil <[email protected]> |
|
6 | - */ |
|
3 | + * Date: 07.12.15 |
|
4 | + * |
|
5 | + * @author Portey Vasil <[email protected]> |
|
6 | + */ |
|
7 | 7 | |
8 | 8 | namespace Youshido\Tests\Type\Config; |
9 | 9 |
@@ -33,7 +33,7 @@ |
||
33 | 33 | 'args' => [ |
34 | 34 | 'episode' => ['type' => new EpisodeEnum()] |
35 | 35 | ], |
36 | - 'resolve' => function ($root, $args) { |
|
36 | + 'resolve' => function($root, $args) { |
|
37 | 37 | return StarWarsData::getHero(isset($args['episode']) ? $args['episode'] : null); |
38 | 38 | }, |
39 | 39 | ]) |
@@ -1,9 +1,9 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Date: 07.12.15 |
|
4 | - * |
|
5 | - * @author Portey Vasil <[email protected]> |
|
6 | - */ |
|
3 | + * Date: 07.12.15 |
|
4 | + * |
|
5 | + * @author Portey Vasil <[email protected]> |
|
6 | + */ |
|
7 | 7 | |
8 | 8 | namespace Youshido\Tests\Type\Config; |
9 | 9 |
@@ -1,9 +1,9 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Date: 07.12.15 |
|
4 | - * |
|
5 | - * @author Portey Vasil <[email protected]> |
|
6 | - */ |
|
3 | + * Date: 07.12.15 |
|
4 | + * |
|
5 | + * @author Portey Vasil <[email protected]> |
|
6 | + */ |
|
7 | 7 | |
8 | 8 | namespace Youshido\Tests\Type\Config; |
9 | 9 |
@@ -98,21 +98,21 @@ discard block |
||
98 | 98 | 'friends' => [ |
99 | 99 | [ |
100 | 100 | 'name' => 'Luke Skywalker', |
101 | - 'appearsIn' => ['NEWHOPE', 'EMPIRE', 'JEDI',], |
|
101 | + 'appearsIn' => ['NEWHOPE', 'EMPIRE', 'JEDI', ], |
|
102 | 102 | 'friends' => [ |
103 | - ['name' => 'Han Solo',], |
|
104 | - ['name' => 'Leia Organa',], |
|
105 | - ['name' => 'C-3PO',], |
|
106 | - ['name' => 'R2-D2',], |
|
103 | + ['name' => 'Han Solo', ], |
|
104 | + ['name' => 'Leia Organa', ], |
|
105 | + ['name' => 'C-3PO', ], |
|
106 | + ['name' => 'R2-D2', ], |
|
107 | 107 | ], |
108 | 108 | ], |
109 | 109 | [ |
110 | 110 | 'name' => 'Han Solo', |
111 | 111 | 'appearsIn' => ['NEWHOPE', 'EMPIRE', 'JEDI'], |
112 | 112 | 'friends' => [ |
113 | - ['name' => 'Luke Skywalker',], |
|
113 | + ['name' => 'Luke Skywalker', ], |
|
114 | 114 | ['name' => 'Leia Organa'], |
115 | - ['name' => 'R2-D2',], |
|
115 | + ['name' => 'R2-D2', ], |
|
116 | 116 | ] |
117 | 117 | ], |
118 | 118 | [ |
@@ -120,10 +120,10 @@ discard block |
||
120 | 120 | 'appearsIn' => ['NEWHOPE', 'EMPIRE', 'JEDI'], |
121 | 121 | 'friends' => |
122 | 122 | [ |
123 | - ['name' => 'Luke Skywalker',], |
|
124 | - ['name' => 'Han Solo',], |
|
125 | - ['name' => 'C-3PO',], |
|
126 | - ['name' => 'R2-D2',], |
|
123 | + ['name' => 'Luke Skywalker', ], |
|
124 | + ['name' => 'Han Solo', ], |
|
125 | + ['name' => 'C-3PO', ], |
|
126 | + ['name' => 'R2-D2', ], |
|
127 | 127 | ], |
128 | 128 | ], |
129 | 129 | ], |