Completed
Branch develop (8f72cf)
by John
07:52
created
Alpha/Util/Logging/KPI.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
 
168 168
         $this->endTime = microtime(true);
169 169
 
170
-        $this->duration = $this->endTime - $this->startTime;
170
+        $this->duration = $this->endTime-$this->startTime;
171 171
 
172 172
         $logfile = new LogProviderFile();
173 173
         $logfile->setPath($config->get('app.file.store.dir').'logs/kpi-'.$this->name->getValue().'.csv');
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
 
190 190
         $this->endTime = microtime(true);
191 191
 
192
-        $this->duration = $this->endTime - $this->startTime;
192
+        $this->duration = $this->endTime-$this->startTime;
193 193
 
194 194
         $logfile = new LogProviderFile();
195 195
         $logfile->setPath($config->get('app.file.store.dir').'logs/kpi-'.$this->name->getValue().'.csv');
Please login to merge, or discard this patch.
Alpha/Util/Extension/TCPDFFacade.php 2 patches
Indentation   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -238,14 +238,14 @@  discard block
 block discarded – undo
238 238
         self::$logger->debug('<<__construct()');
239 239
     }
240 240
 
241
-     /**
242
-      * Facade method which will invoke our custom markdown class rather than the standard one.
243
-      *
244
-      * @since 1.0
245
-      */
246
-     private function markdown($text, $attachURL = '')
247
-     {
248
-         $config = ConfigProvider::getInstance();
241
+        /**
242
+         * Facade method which will invoke our custom markdown class rather than the standard one.
243
+         *
244
+         * @since 1.0
245
+         */
246
+        private function markdown($text, $attachURL = '')
247
+        {
248
+            $config = ConfigProvider::getInstance();
249 249
 
250 250
         /*
251 251
          * Initialize the parser and return the result of its transform method.
@@ -253,9 +253,9 @@  discard block
 block discarded – undo
253 253
          */
254 254
         static $parser;
255 255
 
256
-         if (!isset($parser)) {
257
-             $parser = new \Alpha\Util\Extension\Markdown();
258
-         }
256
+            if (!isset($parser)) {
257
+                $parser = new \Alpha\Util\Extension\Markdown();
258
+            }
259 259
 
260 260
         /*
261 261
          * Replace all instances of $sysURL in the text with the app.url setting from config
@@ -265,7 +265,7 @@  discard block
 block discarded – undo
265 265
 
266 266
         // transform text using parser.
267 267
         return $parser->transform($text);
268
-     }
268
+        }
269 269
 
270 270
     /**
271 271
      * Fetter for the content.
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
         foreach ($attachments[0] as $attachmentURL) {
157 157
             $start = mb_strpos($attachmentURL, '/');
158 158
             $end = mb_strrpos($attachmentURL, '"');
159
-            $fileName = mb_substr($attachmentURL, $start + 1, $end - ($start + 1));
159
+            $fileName = mb_substr($attachmentURL, $start+1, $end-($start+1));
160 160
 
161 161
             if (method_exists($this->BO, 'getAttachmentSecureURL')) {
162 162
                 $this->content = str_replace($attachmentURL, 'href='.$this->BO->getAttachmentSecureURL($fileName), $this->content);
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
         foreach ($attachments[0] as $attachmentURL) {
171 171
             $start = mb_strpos($attachmentURL, '/');
172 172
             $end = mb_strrpos($attachmentURL, '" alt');
173
-            $fileName = mb_substr($attachmentURL, $start + 1, $end - ($start + 1));
173
+            $fileName = mb_substr($attachmentURL, $start+1, $end-($start+1));
174 174
 
175 175
             if ($config->get('cms.images.widget')) {
176 176
                 // get the details of the source image
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
                     $type = 'png';
186 186
                 }
187 187
 
188
-                $img = new Image($path, $image_details[0], $image_details[1], $type, 0.95, false, (boolean) $config->get('cms.images.widget.secure'));
188
+                $img = new Image($path, $image_details[0], $image_details[1], $type, 0.95, false, (boolean)$config->get('cms.images.widget.secure'));
189 189
                 $this->content = str_replace($attachmentURL, $img->renderHTMLLink(), $this->content);
190 190
             } else {
191 191
                 // render a normal image link to the ViewAttachment controller
Please login to merge, or discard this patch.
Alpha/Util/Extension/TCPDF.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -94,9 +94,9 @@
 block discarded – undo
94 94
 
95 95
         try {
96 96
             if (mb_strpos($file, '/tk/') !== false) {
97
-                $start = mb_strpos($file, '/tk/') + 3;
97
+                $start = mb_strpos($file, '/tk/')+3;
98 98
                 $end = mb_strlen($file);
99
-                $tk = mb_substr($file, $start + 1, $end - ($start + 1));
99
+                $tk = mb_substr($file, $start+1, $end-($start+1));
100 100
                 $decoded = FrontController::getDecodeQueryParams($tk);
101 101
 
102 102
                 parent::Image($decoded['source'], $x, $y, $w, $h, $type, $link, $align, $resize, $dpi, $palign, $ismask, $imgmask, $border);
Please login to merge, or discard this patch.
Alpha/Util/Graph/TreeGraph.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
             $leftSibling = $node->getLeftSibling();
213 213
 
214 214
             if (isset($leftSibling)) {
215
-                $node->setOffset($leftSibling->getOffset() + $leftSibling->getWidth() + $this->colSpace);
215
+                $node->setOffset($leftSibling->getOffset()+$leftSibling->getWidth()+$this->colSpace);
216 216
             } else {
217 217
                 $node->setOffset(0);
218 218
             }
@@ -220,16 +220,16 @@  discard block
 block discarded – undo
220 220
             $childCount = $node->childCount();
221 221
 
222 222
             for ($i = 0; $i < $childCount; ++$i) {
223
-                $this->firstPass($node->getChildAt($i), $level + 1);
223
+                $this->firstPass($node->getChildAt($i), $level+1);
224 224
             }
225 225
 
226 226
             $midPoint = $node->getChildrenCenter();
227
-            $midPoint -= $node->getWidth() / 2;
227
+            $midPoint -= $node->getWidth()/2;
228 228
             $leftSibling = $node->getLeftSibling();
229 229
 
230 230
             if (isset($leftSibling)) {
231
-                $node->setOffset($leftSibling->getOffset() + $leftSibling->getWidth() + $this->colSpace);
232
-                $node->setModifier($node->getOffset() - $midPoint);
231
+                $node->setOffset($leftSibling->getOffset()+$leftSibling->getWidth()+$this->colSpace);
232
+                $node->setModifier($node->getOffset()-$midPoint);
233 233
 
234 234
                 $this->layout($node, $level);
235 235
             } else {
@@ -237,7 +237,7 @@  discard block
 block discarded – undo
237 237
             }
238 238
         }
239 239
 
240
-        self::$logger->debug('Memory usage at first scan ['.((memory_get_usage(true) / 1024) / 1024).' MB]');
240
+        self::$logger->debug('Memory usage at first scan ['.((memory_get_usage(true)/1024)/1024).' MB]');
241 241
     }
242 242
 
243 243
     /**
@@ -252,17 +252,17 @@  discard block
 block discarded – undo
252 252
      */
253 253
     private function secondPass($node, $level, $x = 0, $y = 0)
254 254
     {
255
-        $nodeX = $node->getOffset() + $x;
255
+        $nodeX = $node->getOffset()+$x;
256 256
         $nodeY = $y;
257 257
 
258 258
         $node->setX($nodeX);
259 259
         $node->setY($nodeY);
260 260
 
261
-        $this->height = ($this->height > $node->getY() + $node->getWidth()) ? $this->height : $node->getY() + $node->getWidth();
262
-        $this->width = ($this->width > $nodeX + $node->getWidth()) ? $this->width : $nodeX + $node->getWidth() + 10;
261
+        $this->height = ($this->height > $node->getY()+$node->getWidth()) ? $this->height : $node->getY()+$node->getWidth();
262
+        $this->width = ($this->width > $nodeX+$node->getWidth()) ? $this->width : $nodeX+$node->getWidth()+10;
263 263
 
264 264
         if ($node->childCount() > 0) {
265
-            $this->secondPass($node->getChildAt(0), $level + 1, $x + $node->getModifier(), $y + $node->getHeight() + $this->rowSpace);
265
+            $this->secondPass($node->getChildAt(0), $level+1, $x+$node->getModifier(), $y+$node->getHeight()+$this->rowSpace);
266 266
         }
267 267
 
268 268
         $rightSibling = $node->getRightSibling();
@@ -271,7 +271,7 @@  discard block
 block discarded – undo
271 271
             $this->secondPass($rightSibling, $level, $x, $y);
272 272
         }
273 273
 
274
-        self::$logger->debug('Memory usage at second scan ['.((memory_get_usage(true) / 1024) / 1024).' MB]');
274
+        self::$logger->debug('Memory usage at second scan ['.((memory_get_usage(true)/1024)/1024).' MB]');
275 275
     }
276 276
 
277 277
     /**
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
                 $modifierSumLeft += $leftAncestor->getModifier();
301 301
             }
302 302
 
303
-            $totalGap = ($firstChildLeftNeighbour->getOffset() + $modifierSumLeft + $firstChildLeftNeighbour->getWidth() + $this->branchSpace) - ($firstChild->getOffset() + $modifierSumRight);
303
+            $totalGap = ($firstChildLeftNeighbour->getOffset()+$modifierSumLeft+$firstChildLeftNeighbour->getWidth()+$this->branchSpace)-($firstChild->getOffset()+$modifierSumRight);
304 304
 
305 305
             if ($totalGap > 0) {
306 306
                 $subTree = $node;
@@ -312,14 +312,14 @@  discard block
 block discarded – undo
312 312
                 }
313 313
 
314 314
                 $subTreeMove = $node;
315
-                $singleGap = $totalGap / $subTreesCount;
315
+                $singleGap = $totalGap/$subTreesCount;
316 316
 
317 317
                 while (isset($subTreeMove) && $subTreeMove !== $leftAncestor) {
318 318
                     $subTreeMove = $subTreeMove->getLeftSibling();
319 319
 
320 320
                     if (isset($subTreeMove)) {
321
-                        $subTreeMove->setOffset($subTreeMove->getOffset() + $totalGap);
322
-                        $subTreeMove->setModifier($subTreeMove->getModifier() + $totalGap);
321
+                        $subTreeMove->setOffset($subTreeMove->getOffset()+$totalGap);
322
+                        $subTreeMove->setModifier($subTreeMove->getModifier()+$totalGap);
323 323
                         $totalGap -= $singleGap;
324 324
                     }
325 325
                 }
@@ -383,7 +383,7 @@  discard block
 block discarded – undo
383 383
         for ($i = 0; $i < $childCount; ++$i) {
384 384
             $child = $node->getChildAt($i);
385 385
 
386
-            $leftmostDescendant = $this->getLeftmost($child, $level + 1, $maxlevel);
386
+            $leftmostDescendant = $this->getLeftmost($child, $level+1, $maxlevel);
387 387
 
388 388
             if (isset($leftmostDescendant)) {
389 389
                 return $leftmostDescendant;
@@ -451,7 +451,7 @@  discard block
 block discarded – undo
451 451
             $this->render();
452 452
         }
453 453
 
454
-        if (isset($this->nodes[$this->position + 1])) {
454
+        if (isset($this->nodes[$this->position+1])) {
455 455
             ++$this->position;
456 456
 
457 457
             return $this->nodes[$this->position];
@@ -469,7 +469,7 @@  discard block
 block discarded – undo
469 469
      */
470 470
     public function hasNext()
471 471
     {
472
-        if (isset($this->nodes[$this->position + 1])) {
472
+        if (isset($this->nodes[$this->position+1])) {
473 473
             return true;
474 474
         } else {
475 475
             return false;
Please login to merge, or discard this patch.
Alpha/Util/Http/PHPServerUtils.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -97,7 +97,7 @@
 block discarded – undo
97 97
             // Execute the command and store the process ID
98 98
             $output = array();
99 99
             exec($command, $output);
100
-            $pid = (int) $output[0];
100
+            $pid = (int)$output[0];
101 101
         }
102 102
 
103 103
         if (!isset($pid)) {
Please login to merge, or discard this patch.
Alpha/Model/ActiveRecordProviderMySQL.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1033,7 +1033,7 @@  discard block
 block discarded – undo
1033 1033
         } else {
1034 1034
             // there has been an error, so decrement the version number back
1035 1035
             $temp = $this->BO->getVersionNumber()->getValue();
1036
-            $this->BO->set('version_num', $temp - 1);
1036
+            $this->BO->set('version_num', $temp-1);
1037 1037
 
1038 1038
             // check for unique violations
1039 1039
             if (self::getConnection()->errno == '1062') {
@@ -1065,7 +1065,7 @@  discard block
 block discarded – undo
1065 1065
         $this->BO->setLastQuery($sqlQuery);
1066 1066
         $stmt = self::getConnection()->stmt_init();
1067 1067
 
1068
-        $newVersionNumber = $this->BO->getVersionNumber()->getValue() + 1;
1068
+        $newVersionNumber = $this->BO->getVersionNumber()->getValue()+1;
1069 1069
 
1070 1070
         if ($stmt->prepare($sqlQuery)) {
1071 1071
             if ($this->BO->getPropObject($attribute) instanceof Integer) {
@@ -2441,7 +2441,7 @@  discard block
 block discarded – undo
2441 2441
 
2442 2442
                     if ($propName == 'version_num') {
2443 2443
                         $temp = $this->BO->getVersionNumber()->getValue();
2444
-                        $this->BO->set('version_num', $temp + 1);
2444
+                        $this->BO->set('version_num', $temp+1);
2445 2445
                         $bindingsTypes .= 'i';
2446 2446
                         array_push($params, $this->BO->getVersionNumber()->getValue());
2447 2447
                     }
@@ -2542,7 +2542,7 @@  discard block
 block discarded – undo
2542 2542
         $singleQuote1 = mb_strpos($error, "'");
2543 2543
         $singleQuote2 = mb_strrpos($error, "'");
2544 2544
 
2545
-        $value = mb_substr($error, $singleQuote1, ($singleQuote2 - $singleQuote1) + 1);
2545
+        $value = mb_substr($error, $singleQuote1, ($singleQuote2-$singleQuote1)+1);
2546 2546
         self::$logger->debug('<<findOffendingValue ['.$value.'])');
2547 2547
 
2548 2548
         return $value;
Please login to merge, or discard this patch.
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -2363,11 +2363,11 @@
 block discarded – undo
2363 2363
      * parameters.  If arrays of attribute names and values are provided, only those will be bound to
2364 2364
      * the supplied statement.
2365 2365
      *
2366
-     * @param mysqli_stmt $stmt The SQL statement to bind to.
2366
+     * @param \mysqli_stmt $stmt The SQL statement to bind to.
2367 2367
      * @param array Optional array of BO attributes.
2368 2368
      * @param array Optional array of BO values.
2369 2369
      *
2370
-     * @return mysqli_stmt
2370
+     * @return \mysqli_stmt
2371 2371
      *
2372 2372
      * @since 1.1
2373 2373
      */
Please login to merge, or discard this patch.
Alpha/Model/ActiveRecordProviderSQLite.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -981,8 +981,8 @@  discard block
 block discarded – undo
981 981
                 $stmt->bindValue(':OID', $this->BO->getOID(), SQLITE3_INTEGER);
982 982
 
983 983
                 $temp = $this->BO->getVersionNumber()->getValue();
984
-                $this->BO->set('version_num', $temp + 1);
985
-                $stmt->bindValue(':version_num', $temp + 1, SQLITE3_INTEGER);
984
+                $this->BO->set('version_num', $temp+1);
985
+                $stmt->bindValue(':version_num', $temp+1, SQLITE3_INTEGER);
986 986
 
987 987
                 $stmt->execute();
988 988
             } else {
@@ -1065,7 +1065,7 @@  discard block
 block discarded – undo
1065 1065
         } else {
1066 1066
             // there has been an error, so decrement the version number back
1067 1067
             $temp = $this->BO->getVersionNumber()->getValue();
1068
-            $this->BO->set('version_num', $temp - 1);
1068
+            $this->BO->set('version_num', $temp-1);
1069 1069
 
1070 1070
             throw new FailedSaveException('Failed to save object, SQLite error is ['.self::getLastDatabaseError().'], query ['.$this->BO->getLastQuery().']');
1071 1071
         }
@@ -1090,7 +1090,7 @@  discard block
 block discarded – undo
1090 1090
         $this->BO->setLastQuery($sqlQuery);
1091 1091
         $stmt = self::getConnection()->prepare($sqlQuery);
1092 1092
 
1093
-        $newVersionNumber = $this->BO->getVersionNumber()->getValue() + 1;
1093
+        $newVersionNumber = $this->BO->getVersionNumber()->getValue()+1;
1094 1094
 
1095 1095
         if ($stmt instanceof SQLite3Stmt) {
1096 1096
             if ($this->BO->getPropObject($attribute) instanceof Integer) {
Please login to merge, or discard this patch.
Alpha/Model/Type/Sequence.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -174,7 +174,7 @@
 block discarded – undo
174 174
             $this->set('sequence', 0);
175 175
         }
176 176
 
177
-        $this->set('sequence', $this->get('sequence') + 1);
177
+        $this->set('sequence', $this->get('sequence')+1);
178 178
         $this->save();
179 179
     }
180 180
 
Please login to merge, or discard this patch.
Alpha/Util/Http/Request.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -628,7 +628,7 @@  discard block
 block discarded – undo
628 628
             $URI = mb_substr($this->URI, 0, mb_strpos($this->URI, '?'));
629 629
 
630 630
             // let's take this opportunity to pass query string params to $this->params
631
-            $queryString = mb_substr($this->URI, (mb_strpos($this->URI, '?') + 1));
631
+            $queryString = mb_substr($this->URI, (mb_strpos($this->URI, '?')+1));
632 632
             $this->queryString = $queryString;
633 633
             parse_str($queryString, $this->params);
634 634
         } else {
@@ -642,7 +642,7 @@  discard block
 block discarded – undo
642 642
             $name = $paramNames[$i];
643 643
 
644 644
             if (!isset($this->params[trim($name, '{}')])) {
645
-                if (isset($paramValues[$i]) && substr($name, 0, 1) == '{' && substr($name, strlen($name) - 1, 1) == '}') {
645
+                if (isset($paramValues[$i]) && substr($name, 0, 1) == '{' && substr($name, strlen($name)-1, 1) == '}') {
646 646
                     $this->params[trim($name, '{}')] = $paramValues[$i];
647 647
                 }
648 648
                 if (!isset($paramValues[$i]) && isset($defaultParams[trim($name, '{}')])) {
Please login to merge, or discard this patch.
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
      * @param string $key     The key to search for
301 301
      * @param mixed  $default If key is not found, return this instead
302 302
      *
303
-     * @return mixed
303
+     * @return string
304 304
      *
305 305
      * @since 2.0
306 306
      */
@@ -391,7 +391,7 @@  discard block
 block discarded – undo
391 391
      * @param string $key     The key to search for
392 392
      * @param mixed  $default If key is not found, return this instead
393 393
      *
394
-     * @return mixed
394
+     * @return string
395 395
      *
396 396
      * @since 2.0
397 397
      */
Please login to merge, or discard this patch.