Passed
Push — main ( c6b105...31af8b )
by Thierry
07:02
created
jaxon-annotations/src/Annotation/ContainerAnnotation.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -122,38 +122,38 @@  discard block
 block discarded – undo
122 122
         $value = $this->properties['__value__'];
123 123
         $aParams = preg_split('/[\s]+/', $value, 3);
124 124
         $nParamCount = count($aParams);
125
-        if($nParamCount === 1)
125
+        if ($nParamCount === 1)
126 126
         {
127 127
             // For a property, the only parameter is the class. Otherwise, it is the attribute.
128
-            if(self::$xReader->annotationIsOnProperty())
128
+            if (self::$xReader->annotationIsOnProperty())
129 129
             {
130
-                if(substr($aParams[0], 0, 1) === '$')
130
+                if (substr($aParams[0], 0, 1) === '$')
131 131
                 {
132 132
                     throw new AnnotationException('The only property of the @di annotation must be a class name');
133 133
                 }
134 134
                 $this->sClass = $this->getFullClassName($aParams[0]);
135 135
                 return;
136 136
             }
137
-            if(substr($aParams[0], 0, 1) !== '$')
137
+            if (substr($aParams[0], 0, 1) !== '$')
138 138
             {
139 139
                 throw new AnnotationException('The only property of the @di annotation must be a var name');
140 140
             }
141 141
             $this->sAttr = substr($aParams[0], 1);
142 142
             return;
143 143
         }
144
-        if($nParamCount === 2)
144
+        if ($nParamCount === 2)
145 145
         {
146 146
             // For a property, having 2 parameters is not allowed.
147
-            if(self::$xReader->annotationIsOnProperty())
147
+            if (self::$xReader->annotationIsOnProperty())
148 148
             {
149 149
                 throw new AnnotationException('The @di annotation accepts only one property on a class attribute');
150 150
             }
151 151
 
152
-            if(substr($aParams[0], 0, 1) !== '$')
152
+            if (substr($aParams[0], 0, 1) !== '$')
153 153
             {
154 154
                 throw new AnnotationException('The only property of the @di annotation must be a var name');
155 155
             }
156
-            if(substr($aParams[1], 0, 1) === '$')
156
+            if (substr($aParams[1], 0, 1) === '$')
157 157
             {
158 158
                 throw new AnnotationException('The first property of the @di annotation must be a class name');
159 159
             }
@@ -185,26 +185,26 @@  discard block
 block discarded – undo
185 185
      */
186 186
     private function parseProperties()
187 187
     {
188
-        if(!$this->checkPropertiesNames())
188
+        if (!$this->checkPropertiesNames())
189 189
         {
190 190
             throw new AnnotationException('The @di annotation accepts only "attr" or "class" as properties');
191 191
         }
192 192
 
193
-        if(isset($this->properties['attr']))
193
+        if (isset($this->properties['attr']))
194 194
         {
195
-            if(self::$xReader->annotationIsOnProperty())
195
+            if (self::$xReader->annotationIsOnProperty())
196 196
             {
197 197
                 throw new AnnotationException('The @di annotation does not allow the "attr" property on class attributes');
198 198
             }
199
-            if(!is_string($this->properties['attr']))
199
+            if (!is_string($this->properties['attr']))
200 200
             {
201 201
                 throw new AnnotationException('The @di annotation requires a property "attr" of type string');
202 202
             }
203 203
             $this->sAttr = $this->properties['attr'];
204 204
         }
205
-        if(isset($this->properties['class']))
205
+        if (isset($this->properties['class']))
206 206
         {
207
-            if(!is_string($this->properties['class']))
207
+            if (!is_string($this->properties['class']))
208 208
             {
209 209
                 throw new AnnotationException('The @di annotation requires a property "class" of type string');
210 210
             }
@@ -221,9 +221,9 @@  discard block
 block discarded – undo
221 221
 
222 222
         // The type in the @di annotations can be set from the values in the @var annotations
223 223
         $aPropTypes = self::$xReader->getPropTypes();
224
-        if($this->sClass === '')
224
+        if ($this->sClass === '')
225 225
         {
226
-            if(!isset($aPropTypes[$this->sAttr]))
226
+            if (!isset($aPropTypes[$this->sAttr]))
227 227
             {
228 228
                 throw new AnnotationException('No class defined for @di on attribute "' .
229 229
                     $this->sAttr . '".');
Please login to merge, or discard this patch.
jaxon-annotations/src/Annotation/CallbackAnnotation.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@
 block discarded – undo
50 50
      */
51 51
     public function initAnnotation(array $properties)
52 52
     {
53
-        if(count($properties) !== 1 || !isset($properties['name']) || !is_string($properties['name']))
53
+        if (count($properties) !== 1 || !isset($properties['name']) || !is_string($properties['name']))
54 54
         {
55 55
             throw new AnnotationException('the @callback annotation requires a single string as property');
56 56
         }
Please login to merge, or discard this patch.
jaxon-annotations/src/Annotation/ExcludeAnnotation.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
      */
47 47
     public function initAnnotation(array $properties)
48 48
     {
49
-        if(count($properties) !== 0 && (count($properties) !== 1
49
+        if (count($properties) !== 0 && (count($properties) !== 1
50 50
             || !isset($properties[0]) || !is_bool($properties[0])))
51 51
         {
52 52
             throw new AnnotationException('the @exclude annotation requires a single boolean or no property');
Please login to merge, or discard this patch.
jaxon-annotations/src/Annotation/UploadAnnotation.php 1 patch
Spacing   +2 added lines, -3 removed lines patch added patch discarded remove patch
@@ -41,8 +41,7 @@  discard block
 block discarded – undo
41 41
     public static function parseAnnotation($value)
42 42
     {
43 43
         $aParams = preg_split('/[\s]+/', $value, 2);
44
-        return count($aParams) === 1 ? ['field' => $aParams[0]] :
45
-            ['field' => $aParams[0], 'extra' => $aParams[1]];
44
+        return count($aParams) === 1 ? ['field' => $aParams[0]] : ['field' => $aParams[0], 'extra' => $aParams[1]];
46 45
     }
47 46
 
48 47
     /**
@@ -51,7 +50,7 @@  discard block
 block discarded – undo
51 50
      */
52 51
     public function initAnnotation(array $properties)
53 52
     {
54
-        if(count($properties) != 1 || !isset($properties['field']) ||
53
+        if (count($properties) != 1 || !isset($properties['field']) ||
55 54
             !is_string($properties['field']))
56 55
         {
57 56
             throw new AnnotationException('The @upload annotation requires a property "field" of type string');
Please login to merge, or discard this patch.
jaxon-annotations/src/Annotation/HookAnnotation.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
     public static function parseAnnotation($value)
49 49
     {
50 50
         $aParams = preg_split('/[\s]+/', $value, 2);
51
-        if(count($aParams) === 1)
51
+        if (count($aParams) === 1)
52 52
         {
53 53
             return ['call' => rtrim($aParams[0])];
54 54
         }
@@ -62,23 +62,23 @@  discard block
 block discarded – undo
62 62
      */
63 63
     public function initAnnotation(array $properties)
64 64
     {
65
-        if(!isset($properties['call']) || !is_string($properties['call']))
65
+        if (!isset($properties['call']) || !is_string($properties['call']))
66 66
         {
67 67
             throw new AnnotationException('The @' . $this->getType() .
68 68
                 ' annotation requires a property "call" of type string');
69 69
         }
70
-        foreach(array_keys($properties) as $propName)
70
+        foreach (array_keys($properties) as $propName)
71 71
         {
72
-            if($propName !== 'call' && $propName !== 'with')
72
+            if ($propName !== 'call' && $propName !== 'with')
73 73
             {
74 74
                 throw new AnnotationException('Unknown property "' . $propName .
75 75
                     '" in the @' . $this->getType() . ' annotation');
76 76
             }
77 77
         }
78 78
         // Cannot use isset here, because it will return false in case $properties['with'] === null
79
-        if(array_key_exists('with', $properties))
79
+        if (array_key_exists('with', $properties))
80 80
         {
81
-            if(!is_array($properties['with']))
81
+            if (!is_array($properties['with']))
82 82
             {
83 83
                 throw new AnnotationException('The "with" property of the @' .
84 84
                     $this->getType() . ' annotation must be of type array');
Please login to merge, or discard this patch.
jaxon-annotations/src/Annotation/DataBagAnnotation.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@
 block discarded – undo
50 50
      */
51 51
     public function initAnnotation(array $properties)
52 52
     {
53
-        if(count($properties) !== 1 || !isset($properties['name']) || !is_string($properties['name']))
53
+        if (count($properties) !== 1 || !isset($properties['name']) || !is_string($properties['name']))
54 54
         {
55 55
             throw new AnnotationException('The @databag annotation requires a property "name" of type string');
56 56
         }
Please login to merge, or discard this patch.
jaxon-annotations/src/AnnotationReader.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -114,21 +114,21 @@  discard block
 block discarded – undo
114 114
             fn($xAnnotation) => is_a($xAnnotation, AbstractAnnotation::class)
115 115
         );
116 116
         // First check if the class is excluded.
117
-        foreach($aAnnotations as $xAnnotation)
117
+        foreach ($aAnnotations as $xAnnotation)
118 118
         {
119
-            if(is_a($xAnnotation, ExcludeAnnotation::class))
119
+            if (is_a($xAnnotation, ExcludeAnnotation::class))
120 120
             {
121 121
                 $xAnnotation->saveValue($this->xMetadata);
122 122
             }
123 123
         }
124
-        if($this->xMetadata->isExcluded())
124
+        if ($this->xMetadata->isExcluded())
125 125
         {
126 126
             return;
127 127
         }
128 128
 
129
-        foreach($aAnnotations as $xAnnotation)
129
+        foreach ($aAnnotations as $xAnnotation)
130 130
         {
131
-            if(!is_a($xAnnotation, ExcludeAnnotation::class))
131
+            if (!is_a($xAnnotation, ExcludeAnnotation::class))
132 132
             {
133 133
                 $xAnnotation->saveValue($this->xMetadata);
134 134
             }
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
             $this->xManager->getPropertyAnnotations($sClass, $sProperty),
151 151
             function($xAnnotation) use($sProperty) {
152 152
                 // Save the property type
153
-                if(is_a($xAnnotation, VarAnnotation::class))
153
+                if (is_a($xAnnotation, VarAnnotation::class))
154 154
                 {
155 155
                     $this->aPropTypes[$sProperty] = $xAnnotation->type;
156 156
                 }
@@ -158,12 +158,12 @@  discard block
 block discarded – undo
158 158
                 return is_a($xAnnotation, ContainerAnnotation::class);
159 159
             }
160 160
         );
161
-        if(count($aAnnotations) > 1)
161
+        if (count($aAnnotations) > 1)
162 162
         {
163 163
             throw new AnnotationException('Only one @di annotation is allowed on a property');
164 164
         }
165 165
 
166
-        foreach($aAnnotations as $xAnnotation)
166
+        foreach ($aAnnotations as $xAnnotation)
167 167
         {
168 168
             $xAnnotation->setAttr($sProperty);
169 169
             $xAnnotation->saveValue($this->xMetadata);
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
             $this->xManager->getMethodAnnotations($sClass, $sMethod),
186 186
             fn($xAnnotation) => is_a($xAnnotation, AbstractAnnotation::class)
187 187
         );
188
-        foreach($aAnnotations as $xAnnotation)
188
+        foreach ($aAnnotations as $xAnnotation)
189 189
         {
190 190
             $xAnnotation->saveValue($this->xMetadata, $sMethod);
191 191
         }
@@ -208,7 +208,7 @@  discard block
 block discarded – undo
208 208
             $this->sCurrMemberType = AnnotationManager::MEMBER_CLASS;
209 209
 
210 210
             $this->getClassAttrs($sClass);
211
-            if($this->xMetadata->isExcluded())
211
+            if ($this->xMetadata->isExcluded())
212 212
             {
213 213
                 // The entire class is not to be exported.
214 214
                 return $this->xMetadata;
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
             $this->sCurrMemberType = AnnotationManager::MEMBER_PROPERTY;
219 219
 
220 220
             // Properties annotations
221
-            foreach($xInput->getProperties() as $sProperty)
221
+            foreach ($xInput->getProperties() as $sProperty)
222 222
             {
223 223
                 $this->getPropertyAttrs($sClass, $sProperty);
224 224
             }
@@ -226,14 +226,14 @@  discard block
 block discarded – undo
226 226
             // Processing methods annotations
227 227
             $this->sCurrMemberType = AnnotationManager::MEMBER_METHOD;
228 228
 
229
-            foreach($xInput->getMethods() as $sMethod)
229
+            foreach ($xInput->getMethods() as $sMethod)
230 230
             {
231 231
                 $this->getMethodAttrs($sClass, $sMethod);
232 232
             }
233 233
 
234 234
             return $this->xMetadata;
235 235
         }
236
-        catch(AnnotationException $e)
236
+        catch (AnnotationException $e)
237 237
         {
238 238
             throw new SetupException($e->getMessage());
239 239
         }
Please login to merge, or discard this patch.
jaxon-upload/src/register.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 {
25 25
     $jaxon = jaxon();
26 26
     $di = $jaxon->di();
27
-    if($di->h(UploadHandler::class))
27
+    if ($di->h(UploadHandler::class))
28 28
     {
29 29
         return;
30 30
     }
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
         $xUploadHandler = $di->g(UploadHandler::class);
77 77
         // The HTTP request
78 78
         $xRequest = $di->getRequest();
79
-        if($xUploadHandler->canProcessRequest($xRequest))
79
+        if ($xUploadHandler->canProcessRequest($xRequest))
80 80
         {
81 81
             $xUploadHandler->processRequest($xRequest);
82 82
         }
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 {
93 93
     $jaxon = jaxon();
94 94
     $jaxon->callback()->boot(function() use($jaxon) {
95
-        if($jaxon->getOption('core.upload.enabled'))
95
+        if ($jaxon->getOption('core.upload.enabled'))
96 96
         {
97 97
             registerUpload();
98 98
         }
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 function register()
103 103
 {
104 104
     // Do nothing if running in cli.
105
-    if(php_sapi_name() !== 'cli')
105
+    if (php_sapi_name() !== 'cli')
106 106
     {
107 107
         _register();
108 108
     };
Please login to merge, or discard this patch.
jaxon-upload/src/Manager/FileStorage.php 1 patch
Spacing   +5 added lines, -6 removed lines patch added patch discarded remove patch
@@ -80,8 +80,7 @@  discard block
 block discarded – undo
80 80
     {
81 81
         // Local file system adapter
82 82
         $this->registerAdapter('local', function(string $sRootDir, $xOptions) {
83
-            return empty($xOptions) ? new LocalFilesystemAdapter($sRootDir) :
84
-                new LocalFilesystemAdapter($sRootDir, $xOptions);
83
+            return empty($xOptions) ? new LocalFilesystemAdapter($sRootDir) : new LocalFilesystemAdapter($sRootDir, $xOptions);
85 84
         });
86 85
 
87 86
         // In memory file system adapter
@@ -144,7 +143,7 @@  discard block
 block discarded – undo
144 143
     public function filesystem(string $sField = ''): Filesystem
145 144
     {
146 145
         $sField = trim($sField);
147
-        if(isset($this->aFilesystems[$sField]))
146
+        if (isset($this->aFilesystems[$sField]))
148 147
         {
149 148
             return $this->aFilesystems[$sField];
150 149
         }
@@ -154,18 +153,18 @@  discard block
 block discarded – undo
154 153
         $sRootDir = $this->xConfigManager->getOption('upload.default.dir', '');
155 154
         $aOptions = $this->xConfigManager->getOption('upload.default.options');
156 155
         $sConfigKey = "upload.files.$sField";
157
-        if($sField !== '' && $this->xConfigManager->hasOption($sConfigKey))
156
+        if ($sField !== '' && $this->xConfigManager->hasOption($sConfigKey))
158 157
         {
159 158
             $sStorage = $this->xConfigManager->getOption("$sConfigKey.storage", $sStorage);
160 159
             $sRootDir = $this->xConfigManager->getOption("$sConfigKey.dir", $sRootDir);
161 160
             $aOptions = $this->xConfigManager->getOption("$sConfigKey.options", $aOptions);
162 161
         }
163 162
 
164
-        if(!is_string($sRootDir))
163
+        if (!is_string($sRootDir))
165 164
         {
166 165
             throw new RequestException($this->xTranslator->trans('errors.upload.dir'));
167 166
         }
168
-        if(!isset($this->aAdapters[$sStorage]))
167
+        if (!isset($this->aAdapters[$sStorage]))
169 168
         {
170 169
             throw new RequestException($this->xTranslator->trans('errors.upload.adapter'));
171 170
         }
Please login to merge, or discard this patch.