Passed
Push — master ( 1f88f7...9aeaca )
by Mike
02:51
created
src/Xervice/DataProvider/Generator/DataProviderGenerator.php 2 patches
Indentation   +37 added lines, -37 removed lines patch added patch discarded remove patch
@@ -90,10 +90,10 @@  discard block
 block discarded – undo
90 90
     private function addGetter(ClassType $dataProvider, $element): void
91 91
     {
92 92
         $dataProvider->addMethod('get' . $element['name'])
93
-                     ->addComment('@return ' . $element['type'])
94
-                     ->setVisibility('public')
95
-                     ->setBody('return $this->' . $element['name'] . ';')
96
-                     ->setReturnType($this->getTypeHint($element['type']));
93
+                        ->addComment('@return ' . $element['type'])
94
+                        ->setVisibility('public')
95
+                        ->setBody('return $this->' . $element['name'] . ';')
96
+                        ->setReturnType($this->getTypeHint($element['type']));
97 97
     }
98 98
 
99 99
     /**
@@ -103,9 +103,9 @@  discard block
 block discarded – undo
103 103
     private function addUnsetter(ClassType $dataProvider, $element): void
104 104
     {
105 105
         $dataProvider->addMethod('unset' . $element['name'])
106
-                     ->addComment('@return ' . $dataProvider->getName())
107
-                     ->setVisibility('public')
108
-                     ->setBody('$this->' . $element['name'] . ' = null;' . PHP_EOL . PHP_EOL . 'return $this;');
106
+                        ->addComment('@return ' . $dataProvider->getName())
107
+                        ->setVisibility('public')
108
+                        ->setBody('$this->' . $element['name'] . ' = null;' . PHP_EOL . PHP_EOL . 'return $this;');
109 109
     }
110 110
 
111 111
     /**
@@ -115,11 +115,11 @@  discard block
 block discarded – undo
115 115
     private function addHas(ClassType $dataProvider, $element): void
116 116
     {
117 117
         $dataProvider->addMethod('has' . $element['name'])
118
-                     ->addComment('@return bool')
119
-                     ->setVisibility('public')
120
-                     ->setBody(
121
-                         'return ($this->' . $element['name'] . ' !== null && $this->' . $element['name'] . ' !== []);'
122
-                     );
118
+                        ->addComment('@return bool')
119
+                        ->setVisibility('public')
120
+                        ->setBody(
121
+                            'return ($this->' . $element['name'] . ' !== null && $this->' . $element['name'] . ' !== []);'
122
+                        );
123 123
     }
124 124
 
125 125
     /**
@@ -129,16 +129,16 @@  discard block
 block discarded – undo
129 129
     private function addSetter(ClassType $dataProvider, $element): void
130 130
     {
131 131
         $setter = $dataProvider->addMethod('set' . $element['name'])
132
-                               ->addComment(
133
-                                   '@param ' . $element['type'] . ' $'
134
-                                   . $element['name']
135
-                               )
136
-                               ->addComment('@return ' . $dataProvider->getName())
137
-                               ->setVisibility('public')
138
-                               ->setBody(
139
-                                   '$this->' . $element['name'] . ' = $' . $element['name'] . ';' . PHP_EOL . PHP_EOL
140
-                                   . 'return $this;'
141
-                               );
132
+                                ->addComment(
133
+                                    '@param ' . $element['type'] . ' $'
134
+                                    . $element['name']
135
+                                )
136
+                                ->addComment('@return ' . $dataProvider->getName())
137
+                                ->setVisibility('public')
138
+                                ->setBody(
139
+                                    '$this->' . $element['name'] . ' = $' . $element['name'] . ';' . PHP_EOL . PHP_EOL
140
+                                    . 'return $this;'
141
+                                );
142 142
 
143 143
         $param = $setter->addParameter($element['name'])
144 144
                         ->setTypeHint($this->getTypeHint($element['type']));
@@ -160,17 +160,17 @@  discard block
 block discarded – undo
160 160
     {
161 161
         if (isset($element['singleton']) && $element['singleton'] !== '') {
162 162
             $singleSetter = $dataProvider->addMethod('add' . $element['singleton'])
163
-                                         ->addComment(
164
-                                             '@param ' . $element['singleton_type'] . ' $'
165
-                                             . $element['singleton']
166
-                                         )
167
-                                         ->setVisibility('public')
168
-                                         ->setBody(
169
-                                             '$this->' . $element['name'] . '[] = $' . $element['singleton'] . ';'
170
-                                         );
163
+                                            ->addComment(
164
+                                                '@param ' . $element['singleton_type'] . ' $'
165
+                                                . $element['singleton']
166
+                                            )
167
+                                            ->setVisibility('public')
168
+                                            ->setBody(
169
+                                                '$this->' . $element['name'] . '[] = $' . $element['singleton'] . ';'
170
+                                            );
171 171
 
172 172
             $singleSetter->addParameter($element['singleton'])
173
-                         ->setTypeHint($element['singleton_type']);
173
+                            ->setTypeHint($element['singleton_type']);
174 174
         }
175 175
     }
176 176
 
@@ -181,8 +181,8 @@  discard block
 block discarded – undo
181 181
     private function addProperty(ClassType $dataProvider, $element): void
182 182
     {
183 183
         $property = $dataProvider->addProperty($element['name'])
184
-                                 ->setVisibility('protected')
185
-                                 ->addComment('@var ' . $element['type']);
184
+                                    ->setVisibility('protected')
185
+                                    ->addComment('@var ' . $element['type']);
186 186
 
187 187
         if ($element['default']) {
188 188
             $property->setValue($element['default']);
@@ -213,10 +213,10 @@  discard block
 block discarded – undo
213 213
     private function addElementsGetter($dataProvider, $elements): void
214 214
     {
215 215
         $dataProvider->addMethod('getElements')
216
-                     ->setReturnType('array')
217
-                     ->setVisibility('protected')
218
-                     ->addComment('@return array')
219
-                     ->setBody('return ' . var_export($elements, true) . ';');
216
+                        ->setReturnType('array')
217
+                        ->setVisibility('protected')
218
+                        ->addComment('@return array')
219
+                        ->setBody('return ' . var_export($elements, true) . ';');
220 220
     }
221 221
 
222 222
     /**
Please login to merge, or discard this patch.
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -146,8 +146,7 @@  discard block
 block discarded – undo
146 146
             $default = $element['default'];
147 147
             settype($default, $this->getTypeHint($element['type']));
148 148
             $param->setDefaultValue($default);
149
-        }
150
-        elseif ($element['allownull']) {
149
+        } elseif ($element['allownull']) {
151 150
             $param->setDefaultValue(null);
152 151
         }
153 152
     }
@@ -186,8 +185,7 @@  discard block
 block discarded – undo
186 185
 
187 186
         if ($element['default']) {
188 187
             $property->setValue($element['default']);
189
-        }
190
-        elseif (strpos($element['type'], '[]') !== false) {
188
+        } elseif (strpos($element['type'], '[]') !== false) {
191 189
             $property->setValue([]);
192 190
         }
193 191
     }
Please login to merge, or discard this patch.
src/Xervice/DataProvider/DataProvider/AbstractDataProvider.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -60,14 +60,12 @@
 block discarded – undo
60 60
                 $getMethod = 'get' . $fieldname;
61 61
                 if ($element['is_dataprovider'] && $provider->{$getMethod}() instanceof DataProviderInterface) {
62 62
                     $data[$fieldname] = $this->convertToArray($provider->{$getMethod}());
63
-                }
64
-                elseif ($element['is_collection']) {
63
+                } elseif ($element['is_collection']) {
65 64
                     $data[$fieldname] = [];
66 65
                     foreach ($provider->{$getMethod}() as $child) {
67 66
                         $data[$fieldname][] = $child->toArray();
68 67
                     }
69
-                }
70
-                else {
68
+                } else {
71 69
                     $data[$fieldname] = $provider->{$getMethod}();
72 70
                 }
73 71
             }
Please login to merge, or discard this patch.