Passed
Push — develop ( 917e37...ca2d13 )
by Jens
02:20
created
cloudcontrol/library/components/cms/SitemapRouting.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
             $cmsComponent->setParameter(CmsComponent::PARAMETER_MAIN_NAV_CLASS, CmsComponent::PARAMETER_SITEMAP);
35 35
             if (isset($request::$post[CmsComponent::POST_PARAMETER_TITLE], $request::$post[CmsComponent::POST_PARAMETER_TEMPLATE], $request::$post[CmsComponent::POST_PARAMETER_COMPONENT])) {
36 36
                 $cmsComponent->storage->addSitemapItem($request::$post);
37
-                header('Location: ' . $request::$subfolders . $cmsComponent->getParameter(CmsComponent::PARAMETER_CMS_PREFIX) . '/sitemap');
37
+                header('Location: '.$request::$subfolders.$cmsComponent->getParameter(CmsComponent::PARAMETER_CMS_PREFIX).'/sitemap');
38 38
                 exit;
39 39
             }
40 40
         } elseif ($relativeCmsUri == '/sitemap/edit' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
@@ -43,13 +43,13 @@  discard block
 block discarded – undo
43 43
             $sitemapItem = $cmsComponent->storage->getSitemapItemBySlug($request::$get[CmsComponent::GET_PARAMETER_SLUG]);
44 44
             if (isset($request::$post[CmsComponent::POST_PARAMETER_TITLE], $request::$post[CmsComponent::POST_PARAMETER_TEMPLATE], $request::$post[CmsComponent::POST_PARAMETER_COMPONENT])) {
45 45
                 $cmsComponent->storage->saveSitemapItem($request::$get[CmsComponent::GET_PARAMETER_SLUG], $request::$post);
46
-                header('Location: ' . $request::$subfolders . $cmsComponent->getParameter(CmsComponent::PARAMETER_CMS_PREFIX) . '/sitemap');
46
+                header('Location: '.$request::$subfolders.$cmsComponent->getParameter(CmsComponent::PARAMETER_CMS_PREFIX).'/sitemap');
47 47
                 exit;
48 48
             }
49 49
             $cmsComponent->setParameter(CmsComponent::PARAMETER_SITEMAP_ITEM, $sitemapItem);
50 50
         } elseif ($relativeCmsUri == '/sitemap/delete' && isset($request::$get[CmsComponent::GET_PARAMETER_SLUG])) {
51 51
             $cmsComponent->storage->deleteSitemapItemBySlug($request::$get[CmsComponent::GET_PARAMETER_SLUG]);
52
-            header('Location: ' . $request::$subfolders . $cmsComponent->getParameter(CmsComponent::PARAMETER_CMS_PREFIX) . '/sitemap');
52
+            header('Location: '.$request::$subfolders.$cmsComponent->getParameter(CmsComponent::PARAMETER_CMS_PREFIX).'/sitemap');
53 53
             exit;
54 54
         }
55 55
     }
Please login to merge, or discard this patch.
cloudcontrol/library/components/CmsComponent.php 1 patch
Spacing   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -192,7 +192,7 @@  discard block
 block discarded – undo
192 192
                 $whitelistIps = explode(',', $this->parameters[self::PARAMETER_WHITELIST_IPS]);
193 193
                 $whitelistIps = array_map("trim", $whitelistIps);
194 194
                 if (!in_array($remoteAddress, $whitelistIps)) {
195
-                    throw new \Exception('Ip address ' . $remoteAddress . ' is not on whitelist');
195
+                    throw new \Exception('Ip address '.$remoteAddress.' is not on whitelist');
196 196
                 }
197 197
             }
198 198
         }
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
                 $blacklistIps = explode(',', $this->parameters[self::PARAMETER_BLACKLIST_IPS]);
208 208
                 $blacklistIps = array_map("trim", $blacklistIps);
209 209
                 if (in_array($remoteAddress, $blacklistIps)) {
210
-                    throw new \Exception('Ip address ' . $remoteAddress . ' is on blacklist');
210
+                    throw new \Exception('Ip address '.$remoteAddress.' is on blacklist');
211 211
                 }
212 212
             }
213 213
         }
@@ -259,14 +259,14 @@  discard block
 block discarded – undo
259 259
                 $this->parameters[self::PARAMETER_MAIN_NAV_CLASS] = self::PARAMETER_FILES;
260 260
                 if (isset($_FILES[self::FILES_PARAMETER_FILE])) {
261 261
                     $this->storage->addFile($_FILES[self::FILES_PARAMETER_FILE]);
262
-                    header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/files');
262
+                    header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/files');
263 263
                     exit;
264 264
                 }
265 265
             } elseif ($relativeCmsUri == '/files/get' && isset($request::$get[self::FILES_PARAMETER_FILE])) {
266 266
                 $this->downloadFile($request::$get[self::FILES_PARAMETER_FILE]);
267 267
             } elseif ($relativeCmsUri == '/files/delete' && isset($request::$get[self::FILES_PARAMETER_FILE])) {
268 268
                 $this->storage->deleteFileByName($request::$get[self::FILES_PARAMETER_FILE]);
269
-                header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/files');
269
+                header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/files');
270 270
                 exit;
271 271
             }
272 272
         }
@@ -277,21 +277,21 @@  discard block
 block discarded – undo
277 277
         private function downloadFile($slug)
278 278
         {
279 279
             $file = $this->storage->getFileByName($slug);
280
-            $path = realpath(__DIR__ . '/../../www/files/');
281
-            $quoted = sprintf('"%s"', addcslashes(basename($path . '/' . $file->file), '"\\'));
282
-            $size = filesize($path . '/' . $file->file);
280
+            $path = realpath(__DIR__.'/../../www/files/');
281
+            $quoted = sprintf('"%s"', addcslashes(basename($path.'/'.$file->file), '"\\'));
282
+            $size = filesize($path.'/'.$file->file);
283 283
 
284 284
             header('Content-Description: File Transfer');
285
-            header('Content-Type: ' . $file->type);
286
-            header('Content-Disposition: attachment; filename=' . $quoted);
285
+            header('Content-Type: '.$file->type);
286
+            header('Content-Disposition: attachment; filename='.$quoted);
287 287
             header('Content-Transfer-Encoding: binary');
288 288
             header('Connection: Keep-Alive');
289 289
             header('Expires: 0');
290 290
             header('Cache-Control: must-revalidate, post-check=0, pre-check=0');
291 291
             header('Pragma: public');
292
-            header('Content-Length: ' . $size);
292
+            header('Content-Length: '.$size);
293 293
 
294
-            readfile($path . '/' . $file->file);
294
+            readfile($path.'/'.$file->file);
295 295
             exit;
296 296
         }
297 297
 
@@ -329,12 +329,12 @@  discard block
 block discarded – undo
329 329
                 $this->parameters[self::PARAMETER_MAIN_NAV_CLASS] = 'configuration';
330 330
                 if (isset($_POST[self::POST_PARAMETER_USERNAME])) {
331 331
                     $this->storage->addUser($request::$post);
332
-                    header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/users');
332
+                    header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/users');
333 333
                     exit;
334 334
                 }
335 335
             } elseif ($relativeCmsUri == '/configuration/users/delete' && isset($request::$get[self::GET_PARAMETER_SLUG])) {
336 336
                 $this->storage->deleteUserBySlug($request::$get[self::GET_PARAMETER_SLUG]);
337
-                header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/users');
337
+                header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/users');
338 338
                 exit;
339 339
             } elseif ($relativeCmsUri == '/configuration/users/edit' && isset($request::$get[self::GET_PARAMETER_SLUG])) {
340 340
                 $this->subTemplate = 'cms/configuration/users-form';
@@ -342,7 +342,7 @@  discard block
 block discarded – undo
342 342
                 $this->parameters[self::PARAMETER_USER] = $this->storage->getUserBySlug($request::$get[self::GET_PARAMETER_SLUG]);
343 343
                 if (isset($_POST[self::POST_PARAMETER_USERNAME])) {
344 344
                     $this->storage->saveUser($request::$get[self::GET_PARAMETER_SLUG], $request::$post);
345
-                    header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/users');
345
+                    header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/users');
346 346
                     exit;
347 347
                 }
348 348
             }
@@ -364,7 +364,7 @@  discard block
 block discarded – undo
364 364
                 $bricks = $this->storage->getBricks();
365 365
                 if (isset($request::$post[self::POST_PARAMETER_TITLE])) {
366 366
                     $this->storage->addDocumentType($request::$post);
367
-                    header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/document-types');
367
+                    header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/document-types');
368 368
                     exit;
369 369
                 }
370 370
                 $this->parameters[self::PARAMETER_BRICKS] = $bricks;
@@ -375,14 +375,14 @@  discard block
 block discarded – undo
375 375
                 $bricks = $this->storage->getBricks();
376 376
                 if (isset($request::$post[self::POST_PARAMETER_TITLE])) {
377 377
                     $this->storage->saveDocumentType($request::$get[self::GET_PARAMETER_SLUG], $request::$post);
378
-                    header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/document-types');
378
+                    header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/document-types');
379 379
                     exit;
380 380
                 }
381 381
                 $this->parameters[self::PARAMETER_DOCUMENT_TYPE] = $documentType;
382 382
                 $this->parameters[self::PARAMETER_BRICKS] = $bricks;
383 383
             } elseif ($relativeCmsUri == '/configuration/document-types/delete' && isset($request::$get[self::GET_PARAMETER_SLUG])) {
384 384
                 $this->storage->deleteDocumentTypeBySlug($request::$get[self::GET_PARAMETER_SLUG]);
385
-                header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/document-types');
385
+                header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/document-types');
386 386
                 exit;
387 387
             }
388 388
         }
@@ -402,7 +402,7 @@  discard block
 block discarded – undo
402 402
                 $this->parameters[self::PARAMETER_MAIN_NAV_CLASS] = 'configuration';
403 403
                 if (isset($request::$post[self::POST_PARAMETER_TITLE])) {
404 404
                     $this->storage->addBrick($request::$post);
405
-                    header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/bricks');
405
+                    header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/bricks');
406 406
                     exit;
407 407
                 }
408 408
             } elseif ($relativeCmsUri == '/configuration/bricks/edit' && isset($request::$get[self::GET_PARAMETER_SLUG])) {
@@ -411,13 +411,13 @@  discard block
 block discarded – undo
411 411
                 $brick = $this->storage->getBrickBySlug($request::$get[self::GET_PARAMETER_SLUG]);
412 412
                 if (isset($request::$post[self::POST_PARAMETER_TITLE])) {
413 413
                     $this->storage->saveBrick($request::$get[self::GET_PARAMETER_SLUG], $request::$post);
414
-                    header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/bricks');
414
+                    header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/bricks');
415 415
                     exit;
416 416
                 }
417 417
                 $this->parameters[self::PARAMETER_BRICK] = $brick;
418 418
             } elseif ($relativeCmsUri == '/configuration/bricks/delete' && isset($request::$get[self::GET_PARAMETER_SLUG])) {
419 419
                 $this->storage->deleteBrickBySlug($request::$get[self::GET_PARAMETER_SLUG]);
420
-                header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/bricks');
420
+                header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/bricks');
421 421
                 exit;
422 422
             } elseif ($relativeCmsUri == '/configuration/image-set') {
423 423
                 $this->subTemplate = 'cms/configuration/image-set';
@@ -442,7 +442,7 @@  discard block
 block discarded – undo
442 442
                 $imageSet = $this->storage->getImageSetBySlug($request::$get[self::GET_PARAMETER_SLUG]);
443 443
                 if (isset($request::$post[self::POST_PARAMETER_TITLE])) {
444 444
                     $this->storage->saveImageSet($request::$get[self::GET_PARAMETER_SLUG], $request::$post);
445
-                    header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/image-set');
445
+                    header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/image-set');
446 446
                     exit;
447 447
                 }
448 448
                 $this->parameters[self::PARAMETER_IMAGE_SET] = $imageSet;
@@ -451,12 +451,12 @@  discard block
 block discarded – undo
451 451
                 $this->parameters[self::PARAMETER_MAIN_NAV_CLASS] = 'configuration';
452 452
                 if (isset($request::$post[self::POST_PARAMETER_TITLE])) {
453 453
                     $this->storage->addImageSet($request::$post);
454
-                    header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/image-set');
454
+                    header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/image-set');
455 455
                     exit;
456 456
                 }
457 457
             } elseif ($relativeCmsUri == '/configuration/image-set/delete' && isset($request::$get[self::GET_PARAMETER_SLUG])) {
458 458
                 $this->storage->deleteImageSetBySlug($request::$get[self::GET_PARAMETER_SLUG]);
459
-                header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/image-set');
459
+                header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/image-set');
460 460
                 exit;
461 461
             }
462 462
         }
@@ -476,7 +476,7 @@  discard block
 block discarded – undo
476 476
                 $this->parameters[self::PARAMETER_MAIN_NAV_CLASS] = 'configuration';
477 477
                 if (isset($request::$post[self::POST_PARAMETER_TITLE])) {
478 478
                     $this->storage->addApplicationComponent($request::$post);
479
-                    header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/application-components');
479
+                    header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/application-components');
480 480
                     exit;
481 481
                 }
482 482
             } elseif ($relativeCmsUri == '/configuration/application-components/edit' && isset($request::$get[self::GET_PARAMETER_SLUG])) {
@@ -485,13 +485,13 @@  discard block
 block discarded – undo
485 485
                 $applicationComponent = $this->storage->getApplicationComponentBySlug($request::$get[self::GET_PARAMETER_SLUG]);
486 486
                 if (isset($request::$post[self::POST_PARAMETER_TITLE])) {
487 487
                     $this->storage->saveApplicationComponent($request::$get[self::GET_PARAMETER_SLUG], $request::$post);
488
-                    header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/application-components');
488
+                    header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/application-components');
489 489
                     exit;
490 490
                 }
491 491
                 $this->parameters['applicationComponent'] = $applicationComponent;
492 492
             } elseif ($relativeCmsUri == '/configuration/application-components/delete' && isset($request::$get[self::GET_PARAMETER_SLUG])) {
493 493
                 $this->storage->deleteApplicationComponentBySlug($request::$get[self::GET_PARAMETER_SLUG]);
494
-                header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX] . '/configuration/application-components');
494
+                header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX].'/configuration/application-components');
495 495
                 exit;
496 496
             }
497 497
         }
@@ -501,7 +501,7 @@  discard block
 block discarded – undo
501 501
             if ($relativeCmsUri == '/log-off') {
502 502
                 $_SESSION[self::SESSION_PARAMETER_CLOUD_CONTROL] = null;
503 503
                 unset($_SESSION[self::SESSION_PARAMETER_CLOUD_CONTROL]);
504
-                header('Location: ' . $request::$subfolders . $this->parameters[self::PARAMETER_CMS_PREFIX]);
504
+                header('Location: '.$request::$subfolders.$this->parameters[self::PARAMETER_CMS_PREFIX]);
505 505
                 exit;
506 506
             }
507 507
         }
Please login to merge, or discard this patch.