jenskooij / cloudcontrol

cloudcontrol/library/storage/JsonStorage.php

Doc Comments +4 added lines, -1 removed lines

@@ -17,7 +17,7 @@ 
 		/**
 		 * JsonStorage constructor.
 		 *
-		 * @param $storagePath
+		 * @param string $storagePath
 		 */
 		public function __construct($storagePath)
 		{
@@ -837,6 +837,9 @@ 
 			}
 		}
 
+		/**
+		 * @param string $path
+		 */
 		private function validateFilename($filename, $path)
 		{
 			$fileParts = explode('.', $filename);

Indentation +3 added lines, -3 removed lines

@@ -786,7 +786,7 @@ 
 		/**
 		 * @param $filename
 		 * @return null
-         */
+		 */
 		public function getImageByName($filename)
 		{
 			$images = $this->getImages();
@@ -876,7 +876,7 @@ 
 		/**
 		 * @param $filename
 		 * @return null
-         */
+		 */
 		public function getFileByName($filename)
 		{
 			$files = $this->getFiles();
@@ -891,7 +891,7 @@ 
 		/**
 		 * @param $filename
 		 * @throws \Exception
-         */
+		 */
 		public function deleteFileByName($filename)
 		{
 			$destinationPath = realpath(__DIR__ . '/../../www/files/');

Spacing +33 added lines, -33 removed lines

@@ -33,7 +33,7 @@ 
 		 */
 		private function config()
 		{
-			$storagePath = __DIR__ . $this->storagePath;
+			$storagePath = __DIR__.$this->storagePath;
 			if (realpath($storagePath) !== false) {
 				$jsonString = file_get_contents($storagePath);
 				$this->repository = json_decode($jsonString);
@@ -183,7 +183,7 @@ 
 		 */
 		public function getDocumentBySlug($slug)
 		{
-			$documentContainer = $this->getDocumentContainerByPath('/' . $slug);
+			$documentContainer = $this->getDocumentContainerByPath('/'.$slug);
 			$indices = $documentContainer['indices'];
 
 			if ($indices === null) {
@@ -228,7 +228,7 @@ 
 				// Check for duplicates
 				foreach ($this->repository->documents as $index => $document) {
 					if (end($indices) !== $index && $document->slug == $documentFolderObject->slug && $document->type == 'document') {
-						throw new \Exception('Duplicate slug: ' . $document->slug . ' in folder ' . $postValues['path']);
+						throw new \Exception('Duplicate slug: '.$document->slug.' in folder '.$postValues['path']);
 					}
 				}
 				$this->repository->documents[end($indices)] = $documentFolderObject;
@@ -236,10 +236,10 @@ 
 				// Check for duplicates
 				foreach ($previousFolder->content as $index => $document) {
 					if (end($indices) !== $index && $document->slug == $documentFolderObject->slug && $document->type == 'document') {
-						throw new \Exception('Duplicate slug: ' . $document->slug . ' in folder ' . $postValues['path']);
+						throw new \Exception('Duplicate slug: '.$document->slug.' in folder '.$postValues['path']);
 					}
 				}
-				$previousFolder->content[end($indices)] = $documentFolderObject ;
+				$previousFolder->content[end($indices)] = $documentFolderObject;
 			}
 
 			$this->save();
@@ -253,7 +253,7 @@ 
 				foreach ($this->repository->documents as $document) {
 					if ($document->slug == $documentFolderObject->slug && $document->type == 'document') {
 						// TODO make it so it doesnt throw an exception, but instead shows a warning
-						throw new \Exception('Duplicate slug: ' . $document->slug . ' in folder ' . $postValues['path']);
+						throw new \Exception('Duplicate slug: '.$document->slug.' in folder '.$postValues['path']);
 					}
 				}
 				$this->repository->documents[] = $documentFolderObject;
@@ -264,7 +264,7 @@ 
 					foreach ($containerFolder->content as $document) {
 						if ($document->slug == $documentFolderObject->slug && $document->type == 'document') {
 							// TODO make it so it doesnt throw an exception, but instead shows a warning
-							throw new \Exception('Duplicate slug: ' . $document->slug . ' in folder ' . $postValues['path']);
+							throw new \Exception('Duplicate slug: '.$document->slug.' in folder '.$postValues['path']);
 						}
 					}
 				}
@@ -380,7 +380,7 @@ 
 				foreach ($this->repository->documents as $document) {
 					if ($document->slug == $documentFolderObject->slug && $document->type == 'folder') {
 						// TODO make it so it doesnt throw an exception, but instead shows a warning
-						throw new \Exception('Duplicate slug: ' . $document->slug . ' in folder ' . $postValues['path']);
+						throw new \Exception('Duplicate slug: '.$document->slug.' in folder '.$postValues['path']);
 					}
 				}
 				$this->repository->documents[] = $documentFolderObject;
@@ -401,7 +401,7 @@ 
 				foreach ($containerFolder->content as $document) {
 					if ($document->slug == $documentFolderObject->slug && $document->type == 'folder') {
 						// TODO make it so it doesnt throw an exception, but instead shows a warning
-						throw new \Exception('Duplicate slug: ' . $document->slug . ' in folder ' . $postValues['path']);
+						throw new \Exception('Duplicate slug: '.$document->slug.' in folder '.$postValues['path']);
 					}
 				}
 				$folder->content[] = $documentFolderObject;
@@ -453,12 +453,12 @@ 
 		 */
 		public function getDocumentFolderBySlug($slug)
 		{
-			$documentContainer = $this->getDocumentContainerByPath('/' . $slug);
+			$documentContainer = $this->getDocumentContainerByPath('/'.$slug);
 			$indices = $documentContainer['indices'];
 
 			$folder = $this->repository->documents;
 			if ($indices === null) {
-				throw new \Exception('Can\'t find folder with slug `' . $slug . '`');
+				throw new \Exception('Can\'t find folder with slug `'.$slug.'`');
 			}
 			foreach ($indices as $index) {
 				if ($folder === $this->repository->documents) {
@@ -500,7 +500,7 @@ 
 				// Check for duplicates
 				foreach ($this->repository->documents as $index => $document) {
 					if (end($indices) !== $index && $document->slug == $documentFolderObject->slug && $document->type == 'folder') {
-						throw new \Exception('Duplicate slug: ' . $document->slug . ' in folder ' . $postValues['path']);
+						throw new \Exception('Duplicate slug: '.$document->slug.' in folder '.$postValues['path']);
 					}
 				}
 				$this->repository->documents[end($indices)] = $documentFolderObject;
@@ -508,10 +508,10 @@ 
 				// Check for duplicates
 				foreach ($previousFolder->content as $index => $document) {
 					if (end($indices) !== $index && $document->slug == $documentFolderObject->slug && $document->type == 'folder') {
-						throw new \Exception('Duplicate slug: ' . $document->slug . ' in folder ' . $postValues['path']);
+						throw new \Exception('Duplicate slug: '.$document->slug.' in folder '.$postValues['path']);
 					}
 				}
-				$previousFolder->content[end($indices)] = $documentFolderObject ;
+				$previousFolder->content[end($indices)] = $documentFolderObject;
 			}
 
 			$this->save();
@@ -580,7 +580,7 @@ 
 					'previousDocument' => $previousDocument
 				);
 			} else {
-				throw new \Exception('Invalid path: ' . $path);
+				throw new \Exception('Invalid path: '.$path);
 			}
 		}
 
@@ -765,13 +765,13 @@ 
 
 		public function addImage($postValues)
 		{
-			$destinationPath = realpath(__DIR__ . '/../../www/images/');
+			$destinationPath = realpath(__DIR__.'/../../www/images/');
 
 			$filename = $this->validateFilename($postValues['name'], $destinationPath);
-			$destination = $destinationPath . '/' . $filename;
+			$destination = $destinationPath.'/'.$filename;
 
 			if ($postValues['error'] != '0') {
-				throw new \Exception('Error uploading file. Error code: ' . $postValues['error']);
+				throw new \Exception('Error uploading file. Error code: '.$postValues['error']);
 			}
 
 			if (move_uploaded_file($postValues['tmp_name'], $destination)) {
@@ -793,14 +793,14 @@ 
 
 		public function deleteImageByName($filename)
 		{
-			$destinationPath = realpath(__DIR__ . '/../../www/images/');
+			$destinationPath = realpath(__DIR__.'/../../www/images/');
 
 			$images = $this->getImages();
 
 			foreach ($images as $key => $image) {
 				if ($image->file == $filename) {
 					foreach ($image->set as $imageSetFilename) {
-						$destination = $destinationPath . '/' . $imageSetFilename;
+						$destination = $destinationPath.'/'.$imageSetFilename;
 						if (file_exists($destination)) {
 							unlink($destination);
 						} else {
@@ -842,7 +842,7 @@ 
 		 */
 		public function getFiles()
 		{
-			$files =  $this->repository->files;
+			$files = $this->repository->files;
 			usort($files, array($this, 'compareFiles'));
 			return $files;
 		}
@@ -854,13 +854,13 @@ 
 
 		public function addFile($postValues)
 		{
-			$destinationPath = realpath(__DIR__ . '/../../www/files/');
+			$destinationPath = realpath(__DIR__.'/../../www/files/');
 
 			$filename = $this->validateFilename($postValues['name'], $destinationPath);
-			$destination = $destinationPath . '/' . $filename;
+			$destination = $destinationPath.'/'.$filename;
 
 			if ($postValues['error'] != '0') {
-				throw new \Exception('Error uploading file. Error code: ' . $postValues['error']);
+				throw new \Exception('Error uploading file. Error code: '.$postValues['error']);
 			}
 
 			if (move_uploaded_file($postValues['tmp_name'], $destination)) {
@@ -884,23 +884,23 @@ 
 				array_pop($fileParts);
 				$fileNameWithoutExtension = implode('-', $fileParts);
 				$fileNameWithoutExtension = slugify($fileNameWithoutExtension);
-				$filename = $fileNameWithoutExtension . '.' . $extension;
+				$filename = $fileNameWithoutExtension.'.'.$extension;
 			} else {
 				$filename = slugify($filename);
 			}
 
-			if (file_exists($path . '/' . $filename)) {
+			if (file_exists($path.'/'.$filename)) {
 				$fileParts = explode('.', $filename);
 				if (count($fileParts) > 1) {
 					$extension = end($fileParts);
 					array_pop($fileParts);
 					$fileNameWithoutExtension = implode('-', $fileParts);
 					$fileNameWithoutExtension .= '-copy';
-					$filename = $fileNameWithoutExtension . '.' . $extension;
+					$filename = $fileNameWithoutExtension.'.'.$extension;
 				} else {
 					$filename .= '-copy';
 				}
-				return $this->validateFilename($filename,$path);
+				return $this->validateFilename($filename, $path);
 			}
 			return $filename;
 		}
@@ -926,8 +926,8 @@ 
          */
 		public function deleteFileByName($filename)
 		{
-			$destinationPath = realpath(__DIR__ . '/../../www/files/');
-			$destination = $destinationPath . '/' . $filename;
+			$destinationPath = realpath(__DIR__.'/../../www/files/');
+			$destination = $destinationPath.'/'.$filename;
 
 			if (file_exists($destination)) {
 				$files = $this->getFiles();
@@ -1223,12 +1223,12 @@ 
 		 * @throws \Exception
 		 */
 		private function save() {
-			$storagePath = __DIR__ . $this->storagePath;
+			$storagePath = __DIR__.$this->storagePath;
 			if (realpath($storagePath) !== false) {
-				copy($storagePath, $storagePath . '.bak');
+				copy($storagePath, $storagePath.'.bak');
 				file_put_contents($storagePath, json_encode($this->repository));
 			} else {
-				throw new \Exception('Couldnt find storagePath ' . $storagePath);
+				throw new \Exception('Couldnt find storagePath '.$storagePath);
 			}
 		}
 

cloudcontrol/templates/cms/documents/brick.php

Braces +3 added lines, -1 removed lines

@@ -9,8 +9,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 +8 added lines, -8 removed lines

@@ -3,9 +3,9 @@ 
 	<a class="btn move ui-sortable-handle"><i class="fa fa-arrows-v"></i></a>
 </div>
 <?if ($static == true) {
-	$fieldPrefix = 'bricks[' . $myBrickSlug . '][' . str_replace('.', '', str_replace(' ', '', microtime())) . '][fields]';
+	$fieldPrefix = 'bricks['.$myBrickSlug.']['.str_replace('.', '', str_replace(' ', '', microtime())).'][fields]';
 } else {
-	$fieldPrefix = 'dynamicBricks[' . $brick->slug . '][' . str_replace('.', '', str_replace(' ', '', microtime())) . ']';
+	$fieldPrefix = 'dynamicBricks['.$brick->slug.']['.str_replace('.', '', str_replace(' ', '', microtime())).']';
 }?>
 <? foreach ($brick->fields as $field) : ?>
 			<div class="form-element">
@@ -30,7 +30,7 @@ 
 							<a class="btn move ui-sortable-handle"><i class="fa fa-arrows-v"></i></a>
 							<div class="form-element">
 				<? endif ?>
-					<? include(__DIR__ . '/fieldTypes/' . str_replace(' ', '-', $field->type) . '.php') ?>
+					<? include(__DIR__.'/fieldTypes/'.str_replace(' ', '-', $field->type).'.php') ?>
 				<? if ($field->multiple == true && $field->type != 'Rich Text') : ?>
 
 							</div>
@@ -52,7 +52,7 @@ 
 						<li class="grid-container">
 							<div class="grid-box-10">
 								<div class="grid-inner form-element">
-									<? include(__DIR__ . '/fieldTypes/' . str_replace(' ', '-', $field->type) . '.php') ?>
+									<? include(__DIR__.'/fieldTypes/'.str_replace(' ', '-', $field->type).'.php') ?>
 								</div>
 							</div>
 							<div class="grid-box-2">
@@ -62,7 +62,7 @@ 
 								</div>
 							</div>
 						</li>
-						<?$value='';?>
+						<?$value = ''; ?>
 						<? endforeach ?>
 					<? endif ?>
 				</ul>
@@ -79,10 +79,10 @@ 
 							<a class="btn error js-deletemultiple"><i class="fa fa-times"></i></a>
 							<a class="btn move ui-sortable-handle"><i class="fa fa-arrows-v"></i></a>
 							<div class="form-element">
-							<? include(__DIR__ . '/fieldTypes/' . str_replace(' ', '-', $field->type) . '.php') ?>
+							<? include(__DIR__.'/fieldTypes/'.str_replace(' ', '-', $field->type).'.php') ?>
 							</div>
 						</li>
-						<?$value='';?>
+						<?$value = ''; ?>
 					<? endforeach ?>
 					<? endif ?>
 					</div>
@@ -91,5 +91,5 @@ 
 				<a class="btn js-addrtemultiple">+</a>
 				<? endif ?>
 			</div>
-			<?$value='';?>
+			<?$value = ''; ?>
 		<? endforeach ?>

cloudcontrol/templates/cms/documents/fieldTypes/Image.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 	<div class="grid-container">
 		<div class="grid-box-2">
 			<div class="grid-inner">
-				<div<?=isset($value) ? ' style="background-image:url(\'' . $request::$subfolders . 'images/' . $value . '\');"' : '' ?> class="selected-image" id="<?=$field->slug?>_selectedImage"></div>
+				<div<?=isset($value) ? ' style="background-image:url(\''.$request::$subfolders.'images/'.$value.'\');"' : '' ?> class="selected-image" id="<?=$field->slug?>_selectedImage"></div>
 			</div>
 		</div>
 		<div class="grid-box-10">

cloudcontrol/templates/cms/images/show.php

Spacing +4 added lines, -4 removed lines

@@ -10,22 +10,22 @@
 	<div class="show-image">
 		<label>File</label>
 		<div class="value">
-			<?=isset($image)? $image->file : '' ?>
+			<?=isset($image) ? $image->file : '' ?>
 		</div>
 		<label>Type</label>
 		<div class="value">
-			<?=isset($image)? $image->type : '' ?>
+			<?=isset($image) ? $image->type : '' ?>
 		</div>
 		<label>Size</label>
 		<div class="value">
-			<?=isset($image)? humanFileSize($image->size) : '' ?>
+			<?=isset($image) ? humanFileSize($image->size) : '' ?>
 		</div>
 		<label>Set</label>
 		<? if (isset($image)) : ?>
 			<? foreach ($image->set as $key => $set) : ?>
 				<div class="sets">
 					<label><?=$key?></label>
-					<img src="<?=\library\cc\Request::$subfolders . 'images/' . $set?>" />
+					<img src="<?=\library\cc\Request::$subfolders.'images/'.$set?>" />
 				</div>
 			<? endforeach ?>
 		<? endif ?>

cloudcontrol/templates/cms/documents.php

Spacing +12 added lines, -12 removed lines

@@ -1,4 +1,4 @@ 
-<script>window.onload=function(){History.init();History.replaceState(null, 'Cloud Control CMS', '/<?=$request::$subfolders . $cmsPrefix?>/documents?path=/');};</script>
+<script>window.onload=function(){History.init();History.replaceState(null, 'Cloud Control CMS', '/<?=$request::$subfolders.$cmsPrefix?>/documents?path=/');};</script>
 <section class="documents">
 	<h2><i class="fa fa-file-text-o"></i> Documents</h2>
 	<nav class="actions">
@@ -24,9 +24,9 @@ 
 			<? foreach ($documents as $document) : ?>
 				<li class="grid-container">
 					<? if ($document->type == 'document') : ?>
-						<?renderDocument($document, $cmsPrefix);?>
+						<?renderDocument($document, $cmsPrefix); ?>
 					<? elseif ($document->type == 'folder') : ?>
-						<?renderFolder($document, $cmsPrefix, '', true);?>
+						<?renderFolder($document, $cmsPrefix, '', true); ?>
 					<? endif ?>
 				</li>
 
@@ -37,7 +37,7 @@ 
 <? function renderDocument($document, $cmsPrefix, $slugPrefix = '') {?>
 <div class="grid-box-10">
 	<h3>
-		<a class="btn documentTitle" href="<?=\library\cc\Request::$subfolders?><?=$cmsPrefix?>/documents/edit-document?slug=<?=$slugPrefix . $document->slug?>" title="Edit">
+		<a class="btn documentTitle" href="<?=\library\cc\Request::$subfolders?><?=$cmsPrefix?>/documents/edit-document?slug=<?=$slugPrefix.$document->slug?>" title="Edit">
 			<i class="fa fa-file-text-o"></i> <?=$document->title?>
 		</a>
 		<small class="small state <?=strtolower($document->state)?>"><?=ucfirst($document->state)?></small>
@@ -53,29 +53,29 @@ 
 	</h3>
 </div>
 <div class="documentActions grid-box-2">
-	<a class="btn" href="<?=\library\cc\Request::$subfolders?><?=$cmsPrefix?>/documents/edit-document?slug=<?=$slugPrefix . $document->slug?>" title="Edit"><i class="fa fa-pencil"></i></a>
-	<a onclick="return confirm('Are you sure you want to delete this item?');" class="btn error" href="<?=\library\cc\Request::$subfolders?><?=$cmsPrefix?>/documents/delete-document?slug=<?=$slugPrefix . $document->slug?>" title="Delete"><i class="fa fa-times"></i></a>
+	<a class="btn" href="<?=\library\cc\Request::$subfolders?><?=$cmsPrefix?>/documents/edit-document?slug=<?=$slugPrefix.$document->slug?>" title="Edit"><i class="fa fa-pencil"></i></a>
+	<a onclick="return confirm('Are you sure you want to delete this item?');" class="btn error" href="<?=\library\cc\Request::$subfolders?><?=$cmsPrefix?>/documents/delete-document?slug=<?=$slugPrefix.$document->slug?>" title="Delete"><i class="fa fa-times"></i></a>
 </div>
 <?}?>
-<? function renderFolder($document, $cmsPrefix, $slugPrefix ='', $root = false) {?>
+<? function renderFolder($document, $cmsPrefix, $slugPrefix = '', $root = false) {?>
 <div class="grid-box-8">
 	<h3>
-		<a class="btn documentTitle openFolder" data-slug="<?=$slugPrefix . $document->slug?>" title="Open">
+		<a class="btn documentTitle openFolder" data-slug="<?=$slugPrefix.$document->slug?>" title="Open">
 			<i class="fa fa-folder-o "></i> <?=$document->title?>
 		</a>
 	</h3>
 </div>
 <div class="documentActions grid-box-4">
-	<a class="btn" href="<?=\library\cc\Request::$subfolders?><?=$cmsPrefix?>/documents/edit-folder?slug=<?=$slugPrefix . $document->slug?>" title="Edit"><i class="fa fa-pencil"></i></a>
-	<a onclick="return confirm('Are you sure you want to delete this item?');" class="btn error" href="<?=\library\cc\Request::$subfolders?><?=$cmsPrefix?>/documents/delete-folder?slug=<?=$slugPrefix . $document->slug?>" title="Delete"><i class="fa fa-times"></i></a>
+	<a class="btn" href="<?=\library\cc\Request::$subfolders?><?=$cmsPrefix?>/documents/edit-folder?slug=<?=$slugPrefix.$document->slug?>" title="Edit"><i class="fa fa-pencil"></i></a>
+	<a onclick="return confirm('Are you sure you want to delete this item?');" class="btn error" href="<?=\library\cc\Request::$subfolders?><?=$cmsPrefix?>/documents/delete-folder?slug=<?=$slugPrefix.$document->slug?>" title="Delete"><i class="fa fa-times"></i></a>
 </div>
 <ul class="documents grid-wrapper nested<?=$root ? ' root' : '' ?>">
 	<? foreach ($document->content as $subDocument) : ?>
 		<li class="grid-container">
 			<? if ($subDocument->type == 'document') : ?>
-				<?renderDocument($subDocument, $cmsPrefix, $slugPrefix . $document->slug . '/');?>
+				<?renderDocument($subDocument, $cmsPrefix, $slugPrefix.$document->slug.'/'); ?>
 			<? elseif ($subDocument->type == 'folder') : ?>
-				<?renderFolder($subDocument, $cmsPrefix, $slugPrefix . $document->slug . '/');?>
+				<?renderFolder($subDocument, $cmsPrefix, $slugPrefix.$document->slug.'/'); ?>
 			<? endif ?>
 		</li>
 	<? endforeach ?>

cloudcontrol/templates/cms.php

Spacing +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
 				</li>
 				<? $nrOfMenuItems += 1 ?>
 				<? endif ?>
-				<li class="grid-box-<?=6 + (5-$nrOfMenuItems)?> log-off-box">
+				<li class="grid-box-<?=6 + (5 - $nrOfMenuItems)?> log-off-box">
 					<a class="btn log-off grid-inner" href="<?=$request::$subfolders?><?=$cmsPrefix?>/log-off">
 						<i class="fa fa-power-off"></i>
 						<span>Log off</span>
@@ -86,7 +86,7 @@ 
 	</main>
 <script>
 var subfolders = '<?=$request::$subfolders?>',
-	cmsSubfolders = '<?=$request::$subfolders . $cmsPrefix?>';
+	cmsSubfolders = '<?=$request::$subfolders.$cmsPrefix?>';
 </script>
 <script src="<?=$request::$subfolders?>js/cms.js"></script>
 </body>

cloudcontrol/library/images/Image.php

Braces +10 added lines, -8 removed lines

@@ -59,8 +59,7 @@ 
 				return imagejpeg($imageResource, $path, $quality);
 			} elseif ($mimeTypeConstantValue == IMAGETYPE_PNG) {
 				return imagepng($imageResource, $path, (intval($quality / 10) -1));
-			}
-			else {
+			} else {
 				throw new \Exception('Not a valid mimetypeconstant given see function documentation');
 			}
 			return false;
@@ -101,8 +100,9 @@ 
 			//    Load the image into a string 
 			$file    =    fopen($p_sFile,"rb"); 
 			$read    =    fread($file,10); 
-			while(!feof($file)&&($read<>"")) 
-				$read    .=    fread($file,1024); 
+			while(!feof($file)&&($read<>"")) {
+							$read    .=    fread($file,1024);
+			}
 			
 			$temp    =    unpack("H*",$read); 
 			$hex    =    $temp[1]; 
@@ -153,8 +153,9 @@ 
 				{ 
 					//    If padding needed, ignore image-padding 
 					//    Shift i to the ending of the current 32-bit-block 
-					if ($usePadding) 
-						$i    +=    $width%4; 
+					if ($usePadding) {
+											$i    +=    $width%4;
+					}
 					
 					//    Reset horizontal position 
 					$x    =    0; 
@@ -163,8 +164,9 @@ 
 					$y++; 
 					
 					//    Reached the image-height? Break the for-loop 
-					if ($y>$height) 
-						break; 
+					if ($y>$height) {
+											break;
+					}
 				} 
 				
 				//    Calculation of the RGB-pixel (defined as BGR in image-data) 

Spacing +36 added lines, -36 removed lines

@@ -32,10 +32,10 @@ 
 					$imageContent = file_get_contents($imageContainer);
 					$this->_imageResource = imagecreatefromstring($imageContent);
 				}
-			} elseif(is_string($imageContainer)) {
+			} elseif (is_string($imageContainer)) {
 				$this->_imageResource = imagecreatefromstring($imageContainer);
 			} else {
-				throw new \Exception('Could not create image resource, accepted inputs are: "resource of type (gd)", path_to_image and "string". <br /><pre>' . var_export($imageContainer, true) . '</pre>');
+				throw new \Exception('Could not create image resource, accepted inputs are: "resource of type (gd)", path_to_image and "string". <br /><pre>'.var_export($imageContainer, true).'</pre>');
 			}
 		}
 		
@@ -55,10 +55,10 @@ 
 			
 			if ($mimeTypeConstantValue == IMAGETYPE_GIF) {
 				return imagegif($imageResource, $path);
-			} elseif ($mimeTypeConstantValue == IMAGETYPE_JPEG)  {
+			} elseif ($mimeTypeConstantValue == IMAGETYPE_JPEG) {
 				return imagejpeg($imageResource, $path, $quality);
 			} elseif ($mimeTypeConstantValue == IMAGETYPE_PNG) {
-				return imagepng($imageResource, $path, (intval($quality / 10) -1));
+				return imagepng($imageResource, $path, (intval($quality / 10) - 1));
 			}
 			else {
 				throw new \Exception('Not a valid mimetypeconstant given see function documentation');
@@ -79,7 +79,7 @@ 
 			if (function_exists('exif_imagetype')) {
 				$exif = exif_imagetype($imagePath);
 			} else {
-				if ((list($width, $height, $type, $attr) = getimagesize($imagePath)) !== false ) {
+				if ((list($width, $height, $type, $attr) = getimagesize($imagePath)) !== false) {
 					$exif = $type;
 				} else {
 					$exif = false;
@@ -99,86 +99,86 @@ 
 		public function CreateImageFromBmp($p_sFile)
 		{
 			//    Load the image into a string 
-			$file    =    fopen($p_sFile,"rb"); 
-			$read    =    fread($file,10); 
-			while(!feof($file)&&($read<>"")) 
-				$read    .=    fread($file,1024); 
+			$file    = fopen($p_sFile, "rb"); 
+			$read    = fread($file, 10); 
+			while (!feof($file) && ($read <> "")) 
+				$read .= fread($file, 1024); 
 			
-			$temp    =    unpack("H*",$read); 
-			$hex    =    $temp[1]; 
-			$header    =    substr($hex,0,108);
-			$width=null;
-			$height=null;
+			$temp = unpack("H*", $read); 
+			$hex = $temp[1]; 
+			$header = substr($hex, 0, 108);
+			$width = null;
+			$height = null;
 			
 			//    Process the header 
 			//    Structure: http://www.fastgraph.com/help/bmp_header_format.html 
-			if (substr($header,0,4)=="424d") 
+			if (substr($header, 0, 4) == "424d") 
 			{ 
 				//    Cut it in parts of 2 bytes 
-				$header_parts    =    str_split($header,2); 
+				$header_parts = str_split($header, 2); 
 				
 				//    Get the width        4 bytes 
-				$width            =    hexdec($header_parts[19].$header_parts[18]); 
+				$width = hexdec($header_parts[19].$header_parts[18]); 
 				
 				//    Get the height        4 bytes 
-				$height            =    hexdec($header_parts[23].$header_parts[22]); 
+				$height = hexdec($header_parts[23].$header_parts[22]); 
 				
 				//    Unset the header params 
 				unset($header_parts); 
 			} 
 			
 			//    Define starting X and Y 
-			$x                =    0; 
-			$y                =    1; 
+			$x                = 0; 
+			$y                = 1; 
 			
 			//    Create newimage 
-			$image            =    imagecreatetruecolor($width,$height); 
+			$image            = imagecreatetruecolor($width, $height); 
 			
 			//    Grab the body from the image 
-			$body            =    substr($hex,108); 
+			$body = substr($hex, 108); 
 
 			//    Calculate if padding at the end-line is needed 
 			//    Divided by two to keep overview. 
 			//    1 byte = 2 HEX-chars 
-			$body_size        =    (strlen($body)/2); 
-			$header_size    =    ($width*$height); 
+			$body_size = (strlen($body) / 2); 
+			$header_size = ($width * $height); 
 
 			//    Use end-line padding? Only when needed 
-			$usePadding        =    ($body_size>($header_size*3)+4); 
+			$usePadding = ($body_size > ($header_size * 3) + 4); 
 			
 			//    Using a for-loop with index-calculation instaid of str_split to avoid large memory consumption 
 			//    Calculate the next DWORD-position in the body 
-			for ($i=0;$i<$body_size;$i+=3) 
+			for ($i = 0; $i < $body_size; $i += 3) 
 			{ 
 				//    Calculate line-ending and padding 
-				if ($x>=$width) 
+				if ($x >= $width) 
 				{ 
 					//    If padding needed, ignore image-padding 
 					//    Shift i to the ending of the current 32-bit-block 
 					if ($usePadding) 
-						$i    +=    $width%4; 
+						$i += $width % 4; 
 					
 					//    Reset horizontal position 
-					$x    =    0; 
+					$x = 0; 
 					
 					//    Raise the height-position (bottom-up) 
 					$y++; 
 					
 					//    Reached the image-height? Break the for-loop 
-					if ($y>$height) 
+					if ($y > $height) 
 						break; 
 				} 
 				
 				//    Calculation of the RGB-pixel (defined as BGR in image-data) 
 				//    Define $i_pos as absolute position in the body 
-				$i_pos    =    $i*2; 
-				$r        =    hexdec($body[$i_pos+4].$body[$i_pos+5]); 
-				$g        =    hexdec($body[$i_pos+2].$body[$i_pos+3]); 
-				$b        =    hexdec($body[$i_pos].$body[$i_pos+1]); 
+				$i_pos    = $i * 2; 
+				$r        = hexdec($body[$i_pos + 4].$body[$i_pos + 5]); 
+				$g        = hexdec($body[$i_pos + 2].$body[$i_pos + 3]); 
+				$b        = hexdec($body[$i_pos].$body[$i_pos + 1]); 
 				
 				//    Calculate and draw the pixel 
-				$color    =    imagecolorallocate($image,$r,$g,$b); 
-				imagesetpixel($image,$x,$height-$y,$color); 
+				$color    = imagecolorallocate($image, $r, $g, $b); 
+				imagesetpixel($image, $x, $height - $y, $color); 
 				
 				//    Raise the horizontal position 
 				$x++; 

Doc Comments +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
 		 * @see http://www.php.net/manual/en/function.image-type-to-mime-type.php
 		 * @param string $imagePath
 		 * @param bool $getExtension
-		 * @return bool|int|string
+		 * @return integer
 		 */
 		public function GetImageMimeType($imagePath, $getExtension = false)
 		{

cloudcontrol/library/images/methods/Watermark.php

Braces +9 added lines, -3 removed lines

@@ -61,7 +61,9 @@ 
 		 */
 		protected function calculateX($imageResource)
 		{
-			if (intval($this->_x) === $this->_x) return $this->_x;
+			if (intval($this->_x) === $this->_x) {
+				return $this->_x;
+			}
 			
 			$x = strtolower($this->_x);
 			
@@ -86,7 +88,9 @@ 
 		 */
 		public function calculateY($imageResource)
 		{
-			if (intval($this->_y) === $this->_y) return $this->_y;
+			if (intval($this->_y) === $this->_y) {
+				return $this->_y;
+			}
 		
 			$y = strtolower($this->_y);
 			
@@ -123,7 +127,9 @@ 
 		 */
 		public function GetWatermark()
 		{
-			if ($this->_watermark == null) throw new \Exception('A watermark is not set. Please supply a \library\image\Image using $this->SetWatermark');
+			if ($this->_watermark == null) {
+				throw new \Exception('A watermark is not set. Please supply a \library\image\Image using $this->SetWatermark');
+			}
 			return $this->_watermark;
 		}
 		

cloudcontrol/library/images/methods/Resize.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
 			
 			// Define the ratio and adjust the width and height
 			if ($this->_preserveAspectRatio) {
-				$ratio = min($this->_width/$originalWidth, $this->_height/$originalHeight);
+				$ratio = min($this->_width / $originalWidth, $this->_height / $originalHeight);
 				$this->_width = $originalWidth * $ratio;
 				$this->_height = $originalHeight * $ratio;
 			}