Passed
Push — develop ( ebf599...bfc038 )
by Jens
02:45
created
cloudcontrol/library/components/DocumentComponent.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
 			if (isset($this->parameters['document'])) {
84 84
 				$this->runByDocumentParameter();
85 85
 			} else {
86
-				throw new \Exception('When not using a regex, you need to set the parameter `document` with the path to the document in this sitemap item: ' . $this->matchedSitemapItem->title);
86
+				throw new \Exception('When not using a regex, you need to set the parameter `document` with the path to the document in this sitemap item: '.$this->matchedSitemapItem->title);
87 87
 			}
88 88
 		}
89 89
 
@@ -100,9 +100,9 @@  discard block
 block discarded – undo
100 100
 				$relativeDocumentUri = current($this->matchedSitemapItem->matches[1]);
101 101
 				if (isset($this->parameters['folder'])) {
102 102
 					if (substr($this->parameters['folder'], -1) !== '/') {
103
-						$this->parameters['folder'] = $this->parameters['folder'] . '/';
103
+						$this->parameters['folder'] = $this->parameters['folder'].'/';
104 104
 					}
105
-					$relativeDocumentUri = $this->parameters['folder'] . $relativeDocumentUri;
105
+					$relativeDocumentUri = $this->parameters['folder'].$relativeDocumentUri;
106 106
 				}
107 107
 
108 108
 				$document = $this->storage->getDocumentBySlug($relativeDocumentUri);
Please login to merge, or discard this patch.
cloudcontrol/library/components/CmsComponent.php 1 patch
Spacing   +39 added lines, -39 removed lines patch added patch discarded remove patch
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
 				$whitelistIps = explode(',', $this->parameters['whitelistIps']);
149 149
 				$whitelistIps = array_map("trim", $whitelistIps);
150 150
 				if (!in_array($remoteAddress, $whitelistIps)) {
151
-					throw new \Exception('Ip address ' . $remoteAddress . ' is not on whitelist');
151
+					throw new \Exception('Ip address '.$remoteAddress.' is not on whitelist');
152 152
 				}
153 153
 			}
154 154
 		}
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 				$blacklistIps = explode(',', $this->parameters['blacklistIps']);
164 164
 				$blacklistIps = array_map("trim", $blacklistIps);
165 165
 				if (in_array($remoteAddress, $blacklistIps)) {
166
-					throw new \Exception('Ip address ' . $remoteAddress . ' is on blacklist');
166
+					throw new \Exception('Ip address '.$remoteAddress.' is on blacklist');
167 167
 				}
168 168
 			}
169 169
 		}
@@ -216,7 +216,7 @@  discard block
 block discarded – undo
216 216
 				$this->parameters['mainNavClass'] = 'sitemap';
217 217
 				if (isset($request::$post['title'], $request::$post['template'], $request::$post['component'])) {
218 218
 					$this->storage->addSitemapItem($request::$post);
219
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/sitemap');
219
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/sitemap');
220 220
 					exit;
221 221
 				}
222 222
 			} elseif ($relativeCmsUri == '/sitemap/edit' && isset($request::$get['slug'])) {
@@ -225,13 +225,13 @@  discard block
 block discarded – undo
225 225
 				$sitemapItem = $this->storage->getSitemapItemBySlug($request::$get['slug']);
226 226
 				if (isset($request::$post['title'], $request::$post['template'], $request::$post['component'])) {
227 227
 					$this->storage->saveSitemapItem($request::$get['slug'], $request::$post);
228
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/sitemap');
228
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/sitemap');
229 229
 					exit;
230 230
 				}
231 231
 				$this->parameters['sitemapItem'] = $sitemapItem;
232 232
 			} elseif ($relativeCmsUri == '/sitemap/delete' && isset($request::$get['slug'])) {
233 233
 				$this->storage->deleteSitemapItemBySlug($request::$get['slug']);
234
-				header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/sitemap');
234
+				header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/sitemap');
235 235
 				exit;
236 236
 			}
237 237
 		}
@@ -255,12 +255,12 @@  discard block
 block discarded – undo
255 255
 				$this->parameters['mainNavClass'] = 'images';
256 256
 				if (isset($_FILES['file'])) {
257 257
 					$this->storage->addImage($_FILES['file']);
258
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/images');
258
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/images');
259 259
 					exit;
260 260
 				}
261 261
 			} elseif ($relativeCmsUri == '/images/delete' && isset($request::$get['file'])) {
262 262
 				$this->storage->deleteImageByName($request::$get['file']);
263
-				header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/images');
263
+				header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/images');
264 264
 				exit;
265 265
 			} elseif ($relativeCmsUri == '/images/show' && isset($request::$get['file'])) {
266 266
 				$this->subTemplate = 'cms/images/show';
@@ -301,14 +301,14 @@  discard block
 block discarded – undo
301 301
 				$this->parameters['mainNavClass'] = 'files';
302 302
 				if (isset($_FILES['file'])) {
303 303
 					$this->storage->addFile($_FILES['file']);
304
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/files');
304
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/files');
305 305
 					exit;
306 306
 				}
307 307
 			} elseif ($relativeCmsUri == '/files/get' && isset($request::$get['file'])) {
308 308
 				$this->downloadFile($request::$get['file']);
309 309
 			} elseif ($relativeCmsUri == '/files/delete' && isset($request::$get['file'])) {
310 310
 				$this->storage->deleteFileByName($request::$get['file']);
311
-				header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/files');
311
+				header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/files');
312 312
 				exit;
313 313
 			}
314 314
 		}
@@ -319,21 +319,21 @@  discard block
 block discarded – undo
319 319
 		private function downloadFile($slug)
320 320
 		{
321 321
 			$file = $this->storage->getFileByName($slug);
322
-			$path = realpath(__DIR__ . '/../../www/files/');
323
-			$quoted = sprintf('"%s"', addcslashes(basename($path . '/' . $file->file), '"\\'));
324
-			$size   = filesize($path . '/' . $file->file);
322
+			$path = realpath(__DIR__.'/../../www/files/');
323
+			$quoted = sprintf('"%s"', addcslashes(basename($path.'/'.$file->file), '"\\'));
324
+			$size   = filesize($path.'/'.$file->file);
325 325
 
326 326
 			header('Content-Description: File Transfer');
327
-			header('Content-Type: ' . $file->type);
328
-			header('Content-Disposition: attachment; filename=' . $quoted);
327
+			header('Content-Type: '.$file->type);
328
+			header('Content-Disposition: attachment; filename='.$quoted);
329 329
 			header('Content-Transfer-Encoding: binary');
330 330
 			header('Connection: Keep-Alive');
331 331
 			header('Expires: 0');
332 332
 			header('Cache-Control: must-revalidate, post-check=0, pre-check=0');
333 333
 			header('Pragma: public');
334
-			header('Content-Length: ' . $size);
334
+			header('Content-Length: '.$size);
335 335
 
336
-			readfile($path . '/' . $file->file);
336
+			readfile($path.'/'.$file->file);
337 337
 			exit;
338 338
 		}
339 339
 
@@ -369,7 +369,7 @@  discard block
 block discarded – undo
369 369
 				if (isset($request::$get['documentType'])) {
370 370
 					if (isset($request::$post['title'], $request::$get['documentType'], $request::$get['path'])) {
371 371
 						$this->storage->addDocument($request::$post);
372
-						header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/documents');
372
+						header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/documents');
373 373
 						exit;
374 374
 					}
375 375
 					$this->parameters['documentType'] = $this->storage->getDocumentTypeBySlug($request::$get['documentType'], true);
@@ -383,7 +383,7 @@  discard block
 block discarded – undo
383 383
 				$this->parameters['smallestImage'] = $this->storage->getSmallestImageSet()->slug;
384 384
 				if (isset($request::$post['title'], $request::$get['slug'])) {
385 385
 					$this->storage->saveDocument($request::$post);
386
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/documents');
386
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/documents');
387 387
 					exit;
388 388
 				}
389 389
 				$this->parameters['document'] = $this->storage->getDocumentBySlug($request::$get['slug']);
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
 				die(json_encode($result));
407 407
 			} else if ($relativeCmsUri == '/documents/delete-document' && isset($request::$get['slug'])) {
408 408
 				$this->storage->deleteDocumentBySlug($request::$get['slug']);
409
-				header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/documents');
409
+				header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/documents');
410 410
 				exit;
411 411
 			}
412 412
 		}
@@ -422,7 +422,7 @@  discard block
 block discarded – undo
422 422
 				$this->parameters['mainNavClass'] = 'documents';
423 423
 				if (isset($request::$post['title'], $request::$post['path'])) {
424 424
 					$this->storage->addDocumentFolder($request::$post);
425
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/documents');
425
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/documents');
426 426
 					exit;
427 427
 				}
428 428
 			} else if ($relativeCmsUri == '/documents/edit-folder' && isset($request::$get['slug'])) {
@@ -435,11 +435,11 @@  discard block
 block discarded – undo
435 435
 				array_pop($path);
436 436
 				$path = implode('/', $path);
437 437
 
438
-				$request::$get['path'] = '/' . $path;
438
+				$request::$get['path'] = '/'.$path;
439 439
 
440 440
 				if (isset($request::$post['title'], $request::$post['content'])) {
441 441
 					$this->storage->saveDocumentFolder($request::$post);
442
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/documents');
442
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/documents');
443 443
 					exit;
444 444
 				}
445 445
 
@@ -447,7 +447,7 @@  discard block
 block discarded – undo
447 447
 				$this->parameters['folder'] = $folder;
448 448
 			} else if ($relativeCmsUri == '/documents/delete-folder' && isset($request::$get['slug'])) {
449 449
 				$this->storage->deleteDocumentFolderBySlug($request::$get['slug']);
450
-				header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/documents');
450
+				header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/documents');
451 451
 				exit;
452 452
 			}
453 453
 		}
@@ -467,12 +467,12 @@  discard block
 block discarded – undo
467 467
 				$this->parameters['mainNavClass'] = 'configuration';
468 468
 				if (isset($_POST['username'])) {
469 469
 					$this->storage->addUser($request::$post);
470
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/users');
470
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/users');
471 471
 					exit;
472 472
 				}
473 473
 			} elseif ($relativeCmsUri == '/configuration/users/delete' && isset($request::$get['slug'])) {
474 474
 				$this->storage->deleteUserBySlug($request::$get['slug']);
475
-				header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/users');
475
+				header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/users');
476 476
 				exit;
477 477
 			} elseif ($relativeCmsUri == '/configuration/users/edit' && isset($request::$get['slug'])) {
478 478
 				$this->subTemplate = 'cms/configuration/users-form';
@@ -480,7 +480,7 @@  discard block
 block discarded – undo
480 480
 				$this->parameters['user'] = $this->storage->getUserBySlug($request::$get['slug']);
481 481
 				if (isset($_POST['username'])) {
482 482
 					$this->storage->saveUser($request::$get['slug'], $request::$post);
483
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/users');
483
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/users');
484 484
 					exit;
485 485
 				}
486 486
 			}
@@ -502,7 +502,7 @@  discard block
 block discarded – undo
502 502
 				$bricks = $this->storage->getBricks();
503 503
 				if (isset($request::$post['title'])) {
504 504
 					$this->storage->addDocumentType($request::$post);
505
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/document-types');
505
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/document-types');
506 506
 					exit;
507 507
 				}
508 508
 				$this->parameters['bricks'] = $bricks;
@@ -513,14 +513,14 @@  discard block
 block discarded – undo
513 513
 				$bricks = $this->storage->getBricks();
514 514
 				if (isset($request::$post['title'])) {
515 515
 					$this->storage->saveDocumentType($request::$get['slug'], $request::$post);
516
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/document-types');
516
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/document-types');
517 517
 					exit;
518 518
 				}
519 519
 				$this->parameters['documentType'] = $documentType;
520 520
 				$this->parameters['bricks'] = $bricks;
521 521
 			} elseif ($relativeCmsUri == '/configuration/document-types/delete' && isset($request::$get['slug'])) {
522 522
 				$this->storage->deleteDocumentTypeBySlug($request::$get['slug']);
523
-				header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/document-types');
523
+				header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/document-types');
524 524
 				exit;
525 525
 			}
526 526
 		}
@@ -540,7 +540,7 @@  discard block
 block discarded – undo
540 540
 				$this->parameters['mainNavClass'] = 'configuration';
541 541
 				if (isset($request::$post['title'])) {
542 542
 					$this->storage->addBrick($request::$post);
543
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/bricks');
543
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/bricks');
544 544
 					exit;
545 545
 				}
546 546
 			} elseif ($relativeCmsUri == '/configuration/bricks/edit' && isset($request::$get['slug'])) {
@@ -549,13 +549,13 @@  discard block
 block discarded – undo
549 549
 				$brick = $this->storage->getBrickBySlug($request::$get['slug']);
550 550
 				if (isset($request::$post['title'])) {
551 551
 					$this->storage->saveBrick($request::$get['slug'], $request::$post);
552
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/bricks');
552
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/bricks');
553 553
 					exit;
554 554
 				}
555 555
 				$this->parameters['brick'] = $brick;
556 556
 			} elseif ($relativeCmsUri == '/configuration/bricks/delete' && isset($request::$get['slug'])) {
557 557
 				$this->storage->deleteBrickBySlug($request::$get['slug']);
558
-				header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/bricks');
558
+				header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/bricks');
559 559
 				exit;
560 560
 			} elseif ($relativeCmsUri == '/configuration/image-set') {
561 561
 				$this->subTemplate = 'cms/configuration/image-set';
@@ -580,7 +580,7 @@  discard block
 block discarded – undo
580 580
 				$imageSet = $this->storage->getImageSetBySlug($request::$get['slug']);
581 581
 				if (isset($request::$post['title'])) {
582 582
 					$this->storage->saveImageSet($request::$get['slug'], $request::$post);
583
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/image-set');
583
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/image-set');
584 584
 					exit;
585 585
 				}
586 586
 				$this->parameters['imageSet'] = $imageSet;
@@ -589,12 +589,12 @@  discard block
 block discarded – undo
589 589
 				$this->parameters['mainNavClass'] = 'configuration';
590 590
 				if (isset($request::$post['title'])) {
591 591
 					$this->storage->addImageSet($request::$post);
592
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/image-set');
592
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/image-set');
593 593
 					exit;
594 594
 				}
595 595
 			} elseif ($relativeCmsUri == '/configuration/image-set/delete' && isset($request::$get['slug'])) {
596 596
 				$this->storage->deleteImageSetBySlug($request::$get['slug']);
597
-				header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/image-set');
597
+				header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/image-set');
598 598
 				exit;
599 599
 			}
600 600
 		}
@@ -614,7 +614,7 @@  discard block
 block discarded – undo
614 614
 				$this->parameters['mainNavClass'] = 'configuration';
615 615
 				if (isset($request::$post['title'])) {
616 616
 					$this->storage->addApplicationComponent($request::$post);
617
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/application-components');
617
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/application-components');
618 618
 					exit;
619 619
 				}
620 620
 			} elseif ($relativeCmsUri == '/configuration/application-components/edit' && isset($request::$get['slug'])) {
@@ -623,13 +623,13 @@  discard block
 block discarded – undo
623 623
 				$applicationComponent = $this->storage->getApplicationComponentBySlug($request::$get['slug']);
624 624
 				if (isset($request::$post['title'])) {
625 625
 					$this->storage->saveApplicationComponent($request::$get['slug'], $request::$post);
626
-					header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/application-components');
626
+					header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/application-components');
627 627
 					exit;
628 628
 				}
629 629
 				$this->parameters['applicationComponent'] = $applicationComponent;
630 630
 			} elseif ($relativeCmsUri == '/configuration/application-components/delete' && isset($request::$get['slug'])) {
631 631
 				$this->storage->deleteApplicationComponentBySlug($request::$get['slug']);
632
-				header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix'] . '/configuration/application-components');
632
+				header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix'].'/configuration/application-components');
633 633
 				exit;
634 634
 			}
635 635
 		}
@@ -639,7 +639,7 @@  discard block
 block discarded – undo
639 639
 			if ($relativeCmsUri == '/log-off') {
640 640
 				$_SESSION['cloudcontrol'] = null;
641 641
 				unset($_SESSION['cloudcontrol']);
642
-				header('Location: ' . $request::$subfolders . $this->parameters['cmsPrefix']);
642
+				header('Location: '.$request::$subfolders.$this->parameters['cmsPrefix']);
643 643
 				exit;
644 644
 			}
645 645
 		}
Please login to merge, or discard this patch.