jenskooij / cloudcontrol

src/storage/storage/ImagesStorage.php

Doc Comments +4 added lines

@@ -13,6 +13,10 @@
 {
     protected $imagesDir;
 
+    /**
+     * @param \CloudControl\Cms\storage\Repository $repository
+     * @param string $imagesDir
+     */
     public function __construct($repository, $imagesDir)
     {
         parent::__construct($repository);

src/cc/Application.php

Doc Comments +5 added lines, -2 removed lines

@@ -83,6 +83,9 @@ 
             $this->storage = new Storage($this->config->rootDir . DIRECTORY_SEPARATOR . $this->config->storageDir, $this->config->imagesDir, $this->config->filesDir);
         }
 
+        /**
+         * @param Request $request
+         */
         private function redirectMatching($request)
         {
             $redirects = $this->storage->getRedirects()->getRedirects();
@@ -112,7 +115,7 @@ 
          * Loop through sitemap items and see if one matches the requestUri.
          * If it does, add it tot the matchedSitemapItems array
          *
-         * @param $request
+         * @param Request $request
          */
         private function sitemapMatching($request)
         {
@@ -177,7 +180,7 @@ 
          * @param array $parameters
          * @param \stdClass|null $matchedSitemapItem
          *
-         * @return mixed
+         * @return Component
          * @throws \Exception
          */
         private function getComponentObject($class = '', $template = '', $parameters = array(), $matchedSitemapItem)

src/templates/documents/brick.php

Indentation +13 added lines, -13 removed lines

@@ -12,11 +12,11 @@ 
 			<div class="form-element">
 				<label for="<?=$field->slug?>"><?=$field->title?></label>
 				<? if (isset($dynamicBrick)) :
-					$fieldSlug = $field->slug;
-					$value = isset($dynamicBrick->fields->$fieldSlug) ? current($dynamicBrick->fields->$fieldSlug) : '';
-				else :
-					$value = '';
-				endif ?>
+                    $fieldSlug = $field->slug;
+                    $value = isset($dynamicBrick->fields->$fieldSlug) ? current($dynamicBrick->fields->$fieldSlug) : '';
+                else :
+                    $value = '';
+                endif ?>
 				<? if ($field->multiple == true && $field->type != 'Rich Text') : ?>
 				<ul class="grid-wrapper sortable">
 					<li class="grid-container">
@@ -44,10 +44,10 @@ 
 						</div>
 					</li>
 					<? if (isset($dynamicBrick)) :
-						$fieldSlug = $field->slug;
-						$iterable = isset($dynamicBrick->fields->$fieldSlug) ? $dynamicBrick->fields->$fieldSlug : array();
-						array_shift($iterable);
-						?>
+                        $fieldSlug = $field->slug;
+                        $iterable = isset($dynamicBrick->fields->$fieldSlug) ? $dynamicBrick->fields->$fieldSlug : array();
+                        array_shift($iterable);
+                        ?>
 						<? foreach ($iterable as $value) : ?>
 
 						<li class="grid-container">
@@ -70,10 +70,10 @@ 
 				<a class="btn js-addmultiple">+</a>
 				<? elseif ($field->multiple == true) : ?>
 					<? if (isset($dynamicBrick)) :
-						$fieldSlug = $field->slug;
-						$iterable = isset($dynamicBrick->fields->$fieldSlug) ? $dynamicBrick->fields->$fieldSlug : array();
-						array_shift($iterable);
-						?>
+                        $fieldSlug = $field->slug;
+                        $iterable = isset($dynamicBrick->fields->$fieldSlug) ? $dynamicBrick->fields->$fieldSlug : array();
+                        array_shift($iterable);
+                        ?>
 						<? foreach ($iterable as $value) : ?>
 
 						<li>

Braces +3 added lines, -1 removed lines

@@ -14,8 +14,10 @@
 				<? if (isset($dynamicBrick)) :
 					$fieldSlug = $field->slug;
 					$value = isset($dynamicBrick->fields->$fieldSlug) ? current($dynamicBrick->fields->$fieldSlug) : '';
-				else :
+				else {
+				    :
 					$value = '';
+				}
 				endif ?>
 				<? if ($field->multiple == true && $field->type != 'Rich Text') : ?>
 				<ul class="grid-wrapper sortable">

Spacing +3 added lines, -3 removed lines

@@ -63,7 +63,7 @@ 
 								</div>
 							</div>
 						</li>
-						<?$value='';?>
+						<?$value = ''; ?>
 						<? endforeach ?>
 					<? endif ?>
 				</ul>
@@ -83,7 +83,7 @@ 
 							<? include(__DIR__ . '/fieldTypes/' . str_replace(' ', '-', $field->type) . '.php') ?>
 							</div>
 						</li>
-						<?$value='';?>
+						<?$value = ''; ?>
 					<? endforeach ?>
 					<? endif ?>
 					</div>
@@ -92,5 +92,5 @@ 
 				<a class="btn js-addrtemultiple">+</a>
 				<? endif ?>
 			</div>
-			<?$value='';?>
+			<?$value = ''; ?>
 		<? endforeach ?>

src/templates/documents/document-form-form.php

Indentation +37 added lines, -37 removed lines

@@ -22,11 +22,11 @@ 
 				<div class="form-element">
 					<label for="<?=$field->slug?>"><?=$field->title?></label>
 					<? if (isset($document)) :
-						$fieldSlug = $field->slug;
-						$value = isset($document->fields->$fieldSlug) ? current($document->fields->$fieldSlug) : '';
-					else :
-						$value = '';
-					endif ?>
+                        $fieldSlug = $field->slug;
+                        $value = isset($document->fields->$fieldSlug) ? current($document->fields->$fieldSlug) : '';
+                    else :
+                        $value = '';
+                    endif ?>
 					<? if ($field->multiple == true && $field->type != 'Rich Text') : ?>
 					<ul class="grid-wrapper sortable">
 						<li class="grid-container">
@@ -54,10 +54,10 @@ 
 							</div>
 						</li>
 						<? if (isset($document)) :
-							$fieldSlug = $field->slug;
-							$iterable = isset($document->fields->$fieldSlug) ? $document->fields->$fieldSlug : array();
-							array_shift($iterable);
-							?>
+                            $fieldSlug = $field->slug;
+                            $iterable = isset($document->fields->$fieldSlug) ? $document->fields->$fieldSlug : array();
+                            array_shift($iterable);
+                            ?>
 							<? foreach ($iterable as $value) : ?>
 
 							<li class="grid-container">
@@ -80,10 +80,10 @@ 
 					<a class="btn js-addmultiple">+</a>
 					<? elseif ($field->multiple == true) : ?>
 						<? if (isset($document)) :
-							$fieldSlug = $field->slug;
-							$iterable = isset($document->fields->$fieldSlug) ? $document->fields->$fieldSlug : array();
-							array_shift($iterable);
-							?>
+                            $fieldSlug = $field->slug;
+                            $iterable = isset($document->fields->$fieldSlug) ? $document->fields->$fieldSlug : array();
+                            array_shift($iterable);
+                            ?>
 							<? foreach ($iterable as $value) : ?>
 
 							<li>
@@ -118,10 +118,10 @@ 
 							<? foreach ($document->bricks as $currentBrickSlug => $brickArray) : ?>
 								<? foreach ($brickArray as $dynamicBrick) : ?>
 									<? foreach ($bricks as $brick) :
-										if (is_object($dynamicBrick) && isset($dynamicBrick->type) && $brick->slug === $dynamicBrick->type && $currentBrickSlug === $myBrickSlug) {
-											break;
-										}
-									endforeach ?>
+                                        if (is_object($dynamicBrick) && isset($dynamicBrick->type) && $brick->slug === $dynamicBrick->type && $currentBrickSlug === $myBrickSlug) {
+                                            break;
+                                        }
+                                    endforeach ?>
 									<? if (is_object($dynamicBrick) && isset($dynamicBrick->type) && $brick->slug === $dynamicBrick->type && $currentBrickSlug === $myBrickSlug) : ?>
 									<li class="brick form-element">
 
@@ -142,12 +142,12 @@ 
 						<div class="form-element">
 						<label for="<?=$field->slug?>"><?=$field->title?></label>
 						<? if (isset($document)) :
-							$brickSlug = $brick->slug;
-							$fieldSlug = $field->slug;
-							$value = isset($document->bricks->$brickSlug->fields->$fieldSlug) ? current($document->bricks->$brickSlug->fields->$fieldSlug) : '';
-						else :
-							$value = '';
-						endif ?>
+                            $brickSlug = $brick->slug;
+                            $fieldSlug = $field->slug;
+                            $value = isset($document->bricks->$brickSlug->fields->$fieldSlug) ? current($document->bricks->$brickSlug->fields->$fieldSlug) : '';
+                        else :
+                            $value = '';
+                        endif ?>
 						<? if ($field->multiple == true && $field->type != 'Rich Text') : ?>
 						<ul class="grid-wrapper sortable">
 							<li class="grid-container">
@@ -175,11 +175,11 @@ 
 								</div>
 							</li>
 							<? if (isset($document)) :
-								$brickSlug = $brick->slug;
-								$fieldSlug = $field->slug;
-								$iterable = isset($document->bricks->$brickSlug->fields->$fieldSlug) ? $document->bricks->$brickSlug->fields->$fieldSlug : array();
-								array_shift($iterable);
-								?>
+                                $brickSlug = $brick->slug;
+                                $fieldSlug = $field->slug;
+                                $iterable = isset($document->bricks->$brickSlug->fields->$fieldSlug) ? $document->bricks->$brickSlug->fields->$fieldSlug : array();
+                                array_shift($iterable);
+                                ?>
 								<? foreach ($iterable as $value) : ?>
 
 								<li class="grid-container">
@@ -202,11 +202,11 @@ 
 						<a class="btn js-addmultiple">+</a>
 						<? elseif ($field->multiple == true) : ?>
 							<? if (isset($document)) :
-								$brickSlug = $brick->slug;
-								$fieldSlug = $field->slug;
-								$iterable = isset($document->bricks->$brickSlug->fields->$fieldSlug) ? $document->bricks->$brickSlug->fields->$fieldSlug : array();
-								array_shift($iterable);
-								?>
+                                $brickSlug = $brick->slug;
+                                $fieldSlug = $field->slug;
+                                $iterable = isset($document->bricks->$brickSlug->fields->$fieldSlug) ? $document->bricks->$brickSlug->fields->$fieldSlug : array();
+                                array_shift($iterable);
+                                ?>
 								<? foreach ($iterable as $value) : ?>
 
 								<li>
@@ -246,10 +246,10 @@ 
 				<? if (isset($document)) : ?>
 					<? foreach ($document->dynamicBricks as $dynamicBrick) : ?>
 						<? foreach ($bricks as $brick) :
-						if ($brick->slug == $dynamicBrick->type) {
-							break;
-						}
-						endforeach ?>
+                        if ($brick->slug == $dynamicBrick->type) {
+                            break;
+                        }
+                        endforeach ?>
 						<li class="brick form-element">
 							<label><?=$brick->title?></label>
 							<?$static = false; ?>

Braces +16 added lines, -6 removed lines

@@ -1,7 +1,8 @@ 
 <form method="<?= isset($documentType) ? 'post' : 'get' ?>" onsubmit="return processRtes();" action="<?= isset($formId) ? '#' . $formId : '' ?>">
     <? if (!isset($hideTitleAndState)) : ?>
       <input type="hidden" name="path" value="<?= $request::$get['path'] ?>"/>
-    <? else : ?>
+    <? else {
+    : ?>
       <input type="hidden" name="formId" value="<?= $formId ?>"/>
       <a name="<?= $formId ?>"></a>
     <? endif ?>
@@ -17,15 +18,19 @@ 
                     <input required="required" value="<?=isset($document) ? $document->title : '' ?>" type="text" id="title" name="title" placeholder="Title" />
                 </div>
 			<? endif ?>
-			<?$fieldPrefix='fields';?>
+			<?$fieldPrefix='fields';
+}
+?>
 			<? foreach ($documentType->fields as $field) : ?>
 				<div class="form-element">
 					<label for="<?=$field->slug?>"><?=$field->title?></label>
 					<? if (isset($document)) :
 						$fieldSlug = $field->slug;
 						$value = isset($document->fields->$fieldSlug) ? current($document->fields->$fieldSlug) : '';
-					else :
+					else {
+					    :
 						$value = '';
+					}
 					endif ?>
 					<? if ($field->multiple == true && $field->type != 'Rich Text') : ?>
 					<ul class="grid-wrapper sortable">
@@ -135,8 +140,11 @@ 
 					</ul>
 					<a class="btn" onclick="addDynamicBrick(this, 'true', 'newBrickDropzone_<?=$static_brick_nr?>');">+</a>
 					<?$static_brick_nr += 1?>
-				<? else : ?>
-					<?$fieldPrefix='bricks[' . $brick->slug . '][fields]';?>
+				<? else {
+    : ?>
+					<?$fieldPrefix='bricks[' . $brick->slug . '][fields]';
+}
+?>
 					<input type="hidden" name="bricks[<?=$brick->slug?>][type]" value="<?=$brick->brickSlug?>" />
 					<? foreach ($brick->structure->fields as $field) : ?>
 						<div class="form-element">
@@ -145,8 +153,10 @@ 
 							$brickSlug = $brick->slug;
 							$fieldSlug = $field->slug;
 							$value = isset($document->bricks->$brickSlug->fields->$fieldSlug) ? current($document->bricks->$brickSlug->fields->$fieldSlug) : '';
-						else :
+						else {
+						    :
 							$value = '';
+						}
 						endif ?>
 						<? if ($field->multiple == true && $field->type != 'Rich Text') : ?>
 						<ul class="grid-wrapper sortable">

Spacing +11 added lines, -11 removed lines

@@ -17,7 +17,7 @@ 
                     <input required="required" value="<?=isset($document) ? $document->title : '' ?>" type="text" id="title" name="title" placeholder="Title" />
                 </div>
 			<? endif ?>
-			<?$fieldPrefix='fields';?>
+			<?$fieldPrefix = 'fields'; ?>
 			<? foreach ($documentType->fields as $field) : ?>
 				<div class="form-element">
 					<label for="<?=$field->slug?>"><?=$field->title?></label>
@@ -73,7 +73,7 @@ 
 									</div>
 								</div>
 							</li>
-							<?$value='';?>
+							<?$value = ''; ?>
 							<? endforeach ?>
 						<? endif ?>
 					</ul>
@@ -93,7 +93,7 @@ 
 								<? include(__DIR__ . '/fieldTypes/' . str_replace(' ', '-', $field->type) . '.php') ?>
 								</div>
 							</li>
-							<?$value='';?>
+							<?$value = ''; ?>
 						<? endforeach ?>
 						<? endif ?>
 						</div>
@@ -102,17 +102,17 @@ 
 					<a class="btn js-addrtemultiple">+</a>
 					<? endif ?>
 				</div>
-				<?$value='';?>
+				<?$value = ''; ?>
 			<? endforeach ?>
 			<hr />
-			<? $static_brick_nr = 0;?>
+			<? $static_brick_nr = 0; ?>
 			<? foreach ($documentType->bricks as $brick) : ?>
 			<div class="brick">
 				<label><?=$brick->title?></label>
 				<? if ($brick->multiple == 'true') : ?>
 					<input type="hidden" value="<?=$brick->brickSlug?>"/>
 					<input type="hidden" value="<?=$brick->slug?>"/>
-					<?$myBrickSlug=$brick->slug;?>
+					<?$myBrickSlug = $brick->slug; ?>
 					<ul id="newBrickDropzone_<?=$static_brick_nr?>" class="dynamicBricks sortable">
 						<? if (isset($document)) : ?>
 							<? foreach ($document->bricks as $currentBrickSlug => $brickArray) : ?>
@@ -136,7 +136,7 @@ 
 					<a class="btn" onclick="addDynamicBrick(this, 'true', 'newBrickDropzone_<?=$static_brick_nr?>');">+</a>
 					<?$static_brick_nr += 1?>
 				<? else : ?>
-					<?$fieldPrefix='bricks[' . $brick->slug . '][fields]';?>
+					<?$fieldPrefix = 'bricks[' . $brick->slug . '][fields]'; ?>
 					<input type="hidden" name="bricks[<?=$brick->slug?>][type]" value="<?=$brick->brickSlug?>" />
 					<? foreach ($brick->structure->fields as $field) : ?>
 						<div class="form-element">
@@ -195,7 +195,7 @@ 
 										</div>
 									</div>
 								</li>
-								<?$value='';?>
+								<?$value = ''; ?>
 								<? endforeach ?>
 							<? endif ?>
 						</ul>
@@ -216,7 +216,7 @@ 
 									<? include(__DIR__ . '/fieldTypes/' . str_replace(' ', '-', $field->type) . '.php') ?>
 									</div>
 								</li>
-								<?$value='';?>
+								<?$value = ''; ?>
 							<? endforeach ?>
 							<? endif ?>
 							</div>
@@ -225,12 +225,12 @@ 
 						<a class="btn js-addrtemultiple">+</a>
 						<? endif ?>
 					</div>
-					<?$value='';?>
+					<?$value = ''; ?>
 				<? endforeach ?>
 				<? endif ?>
 				</div>
 				<hr />
-			<? endforeach;?>
+			<? endforeach; ?>
 
 
 

src/components/cms/FilesRouting.php

Doc Comments +2 added lines, -2 removed lines

@@ -68,7 +68,7 @@ 
     }
 
     /**
-     * @param $request
+     * @param Request $request
      * @param CmsComponent $cmsComponent
      */
     private function newRoute($request, $cmsComponent)
@@ -83,7 +83,7 @@ 
     }
 
     /**
-     * @param $request
+     * @param Request $request
      * @param CmsComponent $cmsComponent
      */
     private function deleteRoute($request, $cmsComponent)

src/images/Image.php

Doc Comments +9 added lines, -9 removed lines

@@ -76,7 +76,7 @@ 
          * @param string $imagePath
          * @param bool $getExtension
          *
-         * @return bool|int|string
+         * @return integer
          */
         public function getImageMimeType($imagePath, $getExtension = false)
         {
@@ -149,7 +149,7 @@ 
         }
 
         /**
-         * @param $pathToBitmapFile
+         * @param string $pathToBitmapFile
          *
          * @return string
          */
@@ -165,7 +165,7 @@ 
         }
 
         /**
-         * @param $header
+         * @param string $header
          *
          * @return array
          */
@@ -195,14 +195,14 @@ 
          * Loop through the data in the body of the bitmap
          * file and calculate each individual pixel based on the
          * bytes
-         * @param $bodySize
-         * @param $x
+         * @param integer $bodySize
+         * @param integer $x
          * @param $width
-         * @param $usePadding
-         * @param $y
+         * @param boolean $usePadding
+         * @param integer $y
          * @param $height
-         * @param $body
-         * @param $image
+         * @param string $body
+         * @param resource $image
          */
         private function loopThroughBodyAndCalculatePixels($bodySize, $x, $width, $usePadding, $y, $height, $body, $image)
         {

src/storage/storage/FilesStorage.php

Doc Comments +5 added lines, -1 removed lines

@@ -10,6 +10,10 @@ 
 {
     protected $filesDir;
 
+    /**
+     * @param \CloudControl\Cms\storage\Repository $repository
+     * @param string $filesDir
+     */
     public function __construct($repository, $filesDir)
     {
         parent::__construct($repository);
@@ -102,7 +106,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getFilesDir()
     {

src/components/BaseComponent.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         {
             $this->template = $template;
             $this->request = $request;
-            $this->parameters = (array)$parameters;
+            $this->parameters = (array) $parameters;
             $this->matchedSitemapItem = $matchedSitemapItem;
         }
 

Doc Comments +2 added lines, -1 removed lines

@@ -88,6 +88,7 @@ 
          *
          * @param bool $obClean
          * @param null | Application $application
+         * @param Application $application
          *
          * @return string
          * @throws \Exception
@@ -157,7 +158,7 @@ 
         }
 
         /**
-         * @param $template
+         * @param string $template
          * @param $application
          * @return string
          */

src/components/FormComponent.php

Spacing +1 added lines, -1 removed lines

@@ -163,7 +163,7 @@
         if (isset($_SESSION[self::SESSION_PARAMETER_FORM_COMPONENT][$this->formParameterName][self::PARAMETER_FORM_ID])) {
             $this->formId = $_SESSION[self::SESSION_PARAMETER_FORM_COMPONENT][$this->formParameterName][self::PARAMETER_FORM_ID];
         } else {
-            $_SESSION[self::SESSION_PARAMETER_FORM_COMPONENT][$this->formParameterName][self::PARAMETER_FORM_ID] = (string)microtime(true);
+            $_SESSION[self::SESSION_PARAMETER_FORM_COMPONENT][$this->formParameterName][self::PARAMETER_FORM_ID] = (string) microtime(true);
             $_SESSION[self::SESSION_PARAMETER_FORM_COMPONENT][$this->formParameterName]['submitted'] = false;
             $this->formId = $_SESSION[self::SESSION_PARAMETER_FORM_COMPONENT][$this->formParameterName][self::PARAMETER_FORM_ID];
         }

Doc Comments +2 added lines, -2 removed lines

@@ -149,7 +149,7 @@ 
      * submitting the form
      *
      * @param $postValues
-     * @param $storage
+     * @param Storage $storage
      */
     protected function postSubmit($postValues, $storage)
     {
@@ -320,7 +320,7 @@ 
     }
 
     /**
-     * @param $form
+     * @param string|null $form
      */
     private function setFormParameter($form)
     {