Passed
Push — main ( 6c6690...0dcad8 )
by Thierry
03:45
created
jaxon-annotations/src/AnnotationReader.php 2 patches
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.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -232,8 +232,7 @@
 block discarded – undo
232 232
             }
233 233
 
234 234
             return $this->xMetadata;
235
-        }
236
-        catch(AnnotationException $e)
235
+        } catch(AnnotationException $e)
237 236
         {
238 237
             throw new SetupException($e->getMessage());
239 238
         }
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/Validator.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
     {
98 98
         $xDefault = $this->xConfigManager->getOption('upload.default.' . $sProperty);
99 99
         $aAllowed = $this->xConfigManager->getOption('upload.files.' . $sVarName . '.' . $sProperty, $xDefault);
100
-        if(is_array($aAllowed) && !in_array($sValue, $aAllowed))
100
+        if (is_array($aAllowed) && !in_array($sValue, $aAllowed))
101 101
         {
102 102
             $this->sErrorMessage = $this->xTranslator->trans('errors.upload.' . $sField, [$sField => $sValue]);
103 103
             return false;
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
     {
119 119
         $xDefault = $this->xConfigManager->getOption('upload.default.' . $sProperty, 0);
120 120
         $nSize = $this->xConfigManager->getOption('upload.files.' . $sVarName . '.' . $sProperty, $xDefault);
121
-        if($nSize > 0 && (
121
+        if ($nSize > 0 && (
122 122
             ($sProperty == 'max-size' && $nFileSize > $nSize) ||
123 123
             ($sProperty == 'min-size' && $nFileSize < $nSize)))
124 124
         {
@@ -140,25 +140,25 @@  discard block
 block discarded – undo
140 140
     {
141 141
         $this->sErrorMessage = '';
142 142
         // Verify the file extension
143
-        if(!$this->validateFileProperty($sVarName, $xFile->type(), 'types', 'type'))
143
+        if (!$this->validateFileProperty($sVarName, $xFile->type(), 'types', 'type'))
144 144
         {
145 145
             return false;
146 146
         }
147 147
 
148 148
         // Verify the file extension
149
-        if(!$this->validateFileProperty($sVarName, $xFile->extension(), 'extensions', 'extension'))
149
+        if (!$this->validateFileProperty($sVarName, $xFile->extension(), 'extensions', 'extension'))
150 150
         {
151 151
             return false;
152 152
         }
153 153
 
154 154
         // Verify the max size
155
-        if(!$this->validateFileSize($sVarName, $xFile->size(), 'max-size'))
155
+        if (!$this->validateFileSize($sVarName, $xFile->size(), 'max-size'))
156 156
         {
157 157
             return false;
158 158
         }
159 159
 
160 160
         // Verify the min size
161
-        if(!$this->validateFileSize($sVarName, $xFile->size(), 'min-size'))
161
+        if (!$this->validateFileSize($sVarName, $xFile->size(), 'min-size'))
162 162
         {
163 163
             return false;
164 164
         }
Please login to merge, or discard this patch.
jaxon-upload/src/Manager/UploadManager.php 2 patches
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -138,8 +138,7 @@  discard block
 block discarded – undo
138 138
                 throw new RequestException($this->xTranslator->trans('errors.upload.access'));
139 139
             }
140 140
             return $sUploadDir;
141
-        }
142
-        catch(FilesystemException $e)
141
+        } catch(FilesystemException $e)
143 142
         {
144 143
             $this->xLogger->error('Filesystem error.', ['message' => $e->getMessage()]);
145 144
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
@@ -238,8 +237,7 @@  discard block
 block discarded – undo
238 237
             {
239 238
                 $aFiles['user']->filesystem()->write($aFiles['user']->path(), $aFiles['temp']->getStream());
240 239
             }
241
-        }
242
-        catch(FilesystemException $e)
240
+        } catch(FilesystemException $e)
243 241
         {
244 242
             $this->xLogger->error('Filesystem error.', ['message' => $e->getMessage()]);
245 243
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
Please login to merge, or discard this patch.
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -133,13 +133,13 @@  discard block
 block discarded – undo
133 133
         try
134 134
         {
135 135
             $xFilesystem->createDirectory($sUploadDir);
136
-            if(!$xFilesystem->directoryExists($sUploadDir))
136
+            if (!$xFilesystem->directoryExists($sUploadDir))
137 137
             {
138 138
                 throw new RequestException($this->xTranslator->trans('errors.upload.access'));
139 139
             }
140 140
             return $sUploadDir;
141 141
         }
142
-        catch(FilesystemException $e)
142
+        catch (FilesystemException $e)
143 143
         {
144 144
             $this->xLogger->error('Filesystem error.', ['message' => $e->getMessage()]);
145 145
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
     {
174 174
         // Check the uploaded file validity
175 175
         $nErrorCode = $xHttpFile->getError();
176
-        if($nErrorCode !== UPLOAD_ERR_OK)
176
+        if ($nErrorCode !== UPLOAD_ERR_OK)
177 177
         {
178 178
             $this->xLogger->error('File upload error.', [
179 179
                 'code' => $nErrorCode,
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 
185 185
         // Filename without the extension. Needs to be sanitized.
186 186
         $sName = pathinfo($xHttpFile->getClientFilename(), PATHINFO_FILENAME);
187
-        if($this->cNameSanitizer !== null)
187
+        if ($this->cNameSanitizer !== null)
188 188
         {
189 189
             $sName = (string)call_user_func($this->cNameSanitizer, $sName, $sField, $this->sUploadFieldId);
190 190
         }
@@ -192,7 +192,7 @@  discard block
 block discarded – undo
192 192
         // Set the user file data
193 193
         $xFile = File::fromHttpFile($this->xFileStorage->filesystem($sField), $xHttpFile, $sUploadDir, $sName);
194 194
         // Verify file validity (format, size)
195
-        if(!$this->xValidator->validateUploadedFile($sField, $xFile))
195
+        if (!$this->xValidator->validateUploadedFile($sField, $xFile))
196 196
         {
197 197
             throw new RequestException($this->xValidator->getErrorMessage());
198 198
         }
@@ -217,16 +217,16 @@  discard block
 block discarded – undo
217 217
 
218 218
         $aUserFiles = [];
219 219
         $this->aAllFiles = []; // A flat list of all uploaded files
220
-        foreach($aTempFiles as $sField => $aFiles)
220
+        foreach ($aTempFiles as $sField => $aFiles)
221 221
         {
222 222
             $aUserFiles[$sField] = [];
223 223
             // Get the path to the upload dir
224 224
             $sUploadDir = $this->getUploadDir($sField);
225
-            if(!is_array($aFiles))
225
+            if (!is_array($aFiles))
226 226
             {
227 227
                 $aFiles = [$aFiles];
228 228
             }
229
-            foreach($aFiles as $xHttpFile)
229
+            foreach ($aFiles as $xHttpFile)
230 230
             {
231 231
                 $aUserFiles[$sField][] = $this->makeUploadedFile($xHttpFile, $sUploadDir, $sField);
232 232
             }
@@ -234,12 +234,12 @@  discard block
 block discarded – undo
234 234
         // Copy the uploaded files from the temp dir to the user dir
235 235
         try
236 236
         {
237
-            foreach($this->aAllFiles as $aFiles)
237
+            foreach ($this->aAllFiles as $aFiles)
238 238
             {
239 239
                 $aFiles['user']->filesystem()->write($aFiles['user']->path(), $aFiles['temp']->getStream());
240 240
             }
241 241
         }
242
-        catch(FilesystemException $e)
242
+        catch (FilesystemException $e)
243 243
         {
244 244
             $this->xLogger->error('Filesystem error.', ['message' => $e->getMessage()]);
245 245
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
Please login to merge, or discard this patch.
jaxon-core/src/App/Metadata/Data/DatabagData.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
     protected function validateName(string $sName): void
52 52
     {
53
-        if(preg_match('/^[a-zA-Z][a-zA-Z0-9_\-\.]*$/', $sName) > 0)
53
+        if (preg_match('/^[a-zA-Z][a-zA-Z0-9_\-\.]*$/', $sName) > 0)
54 54
         {
55 55
             return;
56 56
         }
Please login to merge, or discard this patch.
jaxon-core/src/App/Metadata/Data/HookData.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
      */
55 55
     protected function validateMethod(string $sMethod): void
56 56
     {
57
-        if(preg_match('/^[a-zA-Z][a-zA-Z0-9_]*$/', $sMethod) > 0)
57
+        if (preg_match('/^[a-zA-Z][a-zA-Z0-9_]*$/', $sMethod) > 0)
58 58
         {
59 59
             return;
60 60
         }
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
     public function encode(string $sVarName): array
82 82
     {
83 83
         $aCalls = [];
84
-        foreach($this->aCalls as $sMethod => $aParams)
84
+        foreach ($this->aCalls as $sMethod => $aParams)
85 85
         {
86 86
             $sParams = addslashes(json_encode($aParams));
87 87
             $aCalls[] = "{$sVarName}->addCall('$sMethod', json_decode(\"$sParams\", true));";
Please login to merge, or discard this patch.
jaxon-core/src/Plugin/AbstractRequestPlugin.php 1 patch
Switch Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -30,16 +30,16 @@
 block discarded – undo
30 30
 
31 31
 abstract class AbstractRequestPlugin extends AbstractPlugin implements CallableRegistryInterface, RequestHandlerInterface
32 32
 {
33
-    /**
33
+/**
34 34
      * @var Target
35 35
      */
36
-    protected $xTarget = null;
36
+protected $xTarget = null;
37 37
 
38
-    /**
38
+/**
39 39
      * @inheritDoc
40 40
      */
41
-    public function getTarget(): ?Target
42
-    {
43
-        return $this->xTarget;
44
-    }
41
+public function getTarget(): ?Target
42
+{
43
+return $this->xTarget;
44
+}
45 45
 }
Please login to merge, or discard this patch.
jaxon-core/src/Plugin/AbstractPackage.php 1 patch
Switch Indentation   +40 added lines, -40 removed lines patch added patch discarded remove patch
@@ -19,46 +19,46 @@  discard block
 block discarded – undo
19 19
 
20 20
 abstract class AbstractPackage extends AbstractCodeGenerator
21 21
 {
22
-    /**
22
+/**
23 23
      * The configuration options of the package
24 24
      *
25 25
      * @var Config
26 26
      */
27
-    protected $xPkgConfig;
27
+protected $xPkgConfig;
28 28
 
29
-    /**
29
+/**
30 30
      * The view renderer
31 31
      *
32 32
      * @var ViewRenderer
33 33
      */
34
-    protected $xRenderer;
34
+protected $xRenderer;
35 35
 
36
-    /**
36
+/**
37 37
      * Get the path to the config file, or the config options in an array.
38 38
      *
39 39
      * @return string|array
40 40
      */
41
-    abstract public static function config(): string|array;
41
+abstract public static function config(): string|array;
42 42
 
43
-    /**
43
+/**
44 44
      * Get the package config object
45 45
      *
46 46
      * @return Config
47 47
      */
48
-    final public function getConfig(): Config
49
-    {
50
-        return $this->xPkgConfig;
51
-    }
48
+final public function getConfig(): Config
49
+{
50
+return $this->xPkgConfig;
51
+}
52 52
 
53
-    /**
53
+/**
54 54
      * This method is automatically called after the package instance is created and configured.
55 55
      *
56 56
      * @return void
57 57
      */
58
-    protected function init(): void
59
-    {}
58
+protected function init(): void
59
+{}
60 60
 
61
-    /**
61
+/**
62 62
      * Get the value of a given package option
63 63
      *
64 64
      * @param string $sOption    The option name
@@ -66,49 +66,49 @@  discard block
 block discarded – undo
66 66
      *
67 67
      * @return mixed
68 68
      */
69
-    final public function getOption(string $sOption, $xDefault = null): mixed
70
-    {
71
-        return $this->xPkgConfig->getOption($sOption, $xDefault);
72
-    }
69
+final public function getOption(string $sOption, $xDefault = null): mixed
70
+{
71
+return $this->xPkgConfig->getOption($sOption, $xDefault);
72
+}
73 73
 
74
-    /**
74
+/**
75 75
      * Get the view renderer
76 76
      *
77 77
      * @return ViewRenderer
78 78
      */
79
-    final public function view(): ViewRenderer
80
-    {
81
-        return $this->xRenderer;
82
-    }
79
+final public function view(): ViewRenderer
80
+{
81
+return $this->xRenderer;
82
+}
83 83
 
84
-    /**
84
+/**
85 85
      * @inheritDoc
86 86
      */
87
-    final public function getHash(): string
88
-    {
89
-        // Packages do not generate hash on their own. So we make this method final.
90
-        return '';
91
-    }
87
+final public function getHash(): string
88
+{
89
+// Packages do not generate hash on their own. So we make this method final.
90
+return '';
91
+}
92 92
 
93
-    /**
93
+/**
94 94
      * Get the HTML code of the package home page
95 95
      *
96 96
      * @return string|Stringable
97 97
      */
98
-    public function getHtml(): string|Stringable
99
-    {
100
-        return '';
101
-    }
98
+public function getHtml(): string|Stringable
99
+{
100
+return '';
101
+}
102 102
 
103
-    /**
103
+/**
104 104
      * Get the HTML code of the package home page
105 105
      *
106 106
      * This method is an alias for getHtml().
107 107
      *
108 108
      * @return string|Stringable
109 109
      */
110
-    public function html(): string|Stringable
111
-    {
112
-        return $this->getHtml();
113
-    }
110
+public function html(): string|Stringable
111
+{
112
+return $this->getHtml();
113
+}
114 114
 }
Please login to merge, or discard this patch.
jaxon-core/src/Plugin/RequestHandlerInterface.php 1 patch
Switch Indentation   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -7,21 +7,21 @@  discard block
 block discarded – undo
7 7
 
8 8
 interface RequestHandlerInterface
9 9
 {
10
-    /**
10
+/**
11 11
      * Get the target function or class and method
12 12
      *
13 13
      * @return Target|null
14 14
      */
15
-    public function getTarget(): ?Target;
15
+public function getTarget(): ?Target;
16 16
 
17
-    /**
17
+/**
18 18
      * @param ServerRequestInterface $xRequest
19 19
      *
20 20
      * @return Target
21 21
      */
22
-    public function setTarget(ServerRequestInterface $xRequest): Target;
22
+public function setTarget(ServerRequestInterface $xRequest): Target;
23 23
 
24
-    /**
24
+/**
25 25
      * Check if this plugin can process the current request
26 26
      *
27 27
      * Called by the <Jaxon\Plugin\RequestManager> when a request has been received to determine
@@ -31,9 +31,9 @@  discard block
 block discarded – undo
31 31
      *
32 32
      * @return bool
33 33
      */
34
-    public static function canProcessRequest(ServerRequestInterface $xRequest): bool;
34
+public static function canProcessRequest(ServerRequestInterface $xRequest): bool;
35 35
 
36
-    /**
36
+/**
37 37
      * Process the current request
38 38
      *
39 39
      * Called by the <Jaxon\Plugin\RequestManager> when a request is being processed.
@@ -42,5 +42,5 @@  discard block
 block discarded – undo
42 42
      *
43 43
      * @return void
44 44
      */
45
-    public function processRequest(): void;
45
+public function processRequest(): void;
46 46
 }
Please login to merge, or discard this patch.