x-systems / epesi-filestorage

src/View/FileView.php

Spacing +11 added lines, -11 removed lines

@@ -34,7 +34,7 @@ 
 	
 	protected function setPeriodSelection()
 	{
-		if (! $this->periodSelection) {
+		if (!$this->periodSelection) {
 			$this->periodSelection = [];
 			foreach ([1, 2, 3, 4] as $count) {
 				$this->periodSelection[$count . ' weeks'] = trans_choice('{1} :count week |[2,*] :count weeks', $count);
@@ -42,7 +42,7 @@ 
 		}
 		
 		foreach ($this->periodSelection as $key => $period) {
-			$default = $key == $this->defaultPeriod? ' (' . __('Default') . ')': '';
+			$default = $key == $this->defaultPeriod ? ' (' . __('Default') . ')' : '';
 			
 			$this->periodSelection[$key] = $period . $default;
 		}
@@ -50,7 +50,7 @@ 
 	
 	protected function addContents()
 	{
-		if (! $this->file) {
+		if (!$this->file) {
 			$this->set(__('Wrong parameters for file'));
 			return;
 		}
@@ -70,13 +70,13 @@ 
 	
 	protected function addFileDetails($container = null)
 	{
-		$container = $container?: $this;
+		$container = $container ?: $this;
 		
 		$container->add(new FileDetailsLister($this->file));
 	}
 	
 	protected function addFileRemoteLinks($container = null) {
-		$container = $container?: $this;
+		$container = $container ?: $this;
 		
 		foreach ($this->file->userActiveLinks() as $link) {
 			$container->add(['View', __('Remote access link expiring :expiry', ['expiry' => $link['expires_at']->format('Y-m-d H:i:s')])]);
@@ -90,7 +90,7 @@ 
 		$container->js(true, new jsExpression('new ClipboardJS(".copy-button")'));
 		
 		$container->on('click', '.delete-link', $this->add(['jsCallback', 'postTrigger' => 'link'])->set(function($j, $linkId) {
-			if (! $link = FileRemoteAccess::create()->load($linkId)) return;
+			if (!$link = FileRemoteAccess::create()->load($linkId)) return;
 			
 			$link->delete();
 			
@@ -101,19 +101,19 @@ 
 	protected function addFileControlButtons() {
 		$urls = FileStorageAccessJoint::getActionUrls($this->file);
 		
-		if (! $this->actionDisabled('preview') && ($urls['preview']?? null)) {
+		if (!$this->actionDisabled('preview') && ($urls['preview']?? null)) {
 			$this->add(['Button', 'class' => ['basic'], 'icon' => 'file alternate outline'])->set(__('View'))->link($urls['preview'])->setAttr(['target' => '_blank']);
 		}
 		
-		if (! $this->actionDisabled('download') && ($urls['download']?? null)) {
+		if (!$this->actionDisabled('download') && ($urls['download']?? null)) {
 			$this->add(['Button', 'class' => ['basic'], 'icon' => 'file download'])->set(__('Download'))->link($urls['download']);
 		}
 		
-		if (! $this->actionDisabled('history')) {
+		if (!$this->actionDisabled('history')) {
 			$this->add(['Button', 'class' => ['basic'], 'icon' => 'history'])->set(__('Access History'))->link(url('view/filestorage:file-access-history/body') . '?' . http_build_query(['id' => $this->file->id]));
 		}
 		
-		if (! $this->actionDisabled('remote')) {
+		if (!$this->actionDisabled('remote')) {
 			$linkControl = $this->add(['View', 'ui' => 'basic buttons']);
 			
 			$linkButton = $linkControl->add(['Button', 'class' => ['basic'], 'icon' => 'linkify'])->set(__('Get Remote Link'));
@@ -123,7 +123,7 @@ 
 			$dropdown->setSource($this->periodSelection);
 			
 			$dropdown->onChange(function($value) {
-				$period = array_search($value, $this->periodSelection)?: $this->defaultPeriod;
+				$period = array_search($value, $this->periodSelection) ?: $this->defaultPeriod;
 				
 				FileRemoteAccess::grant($this->file, $period);
 				

Braces +3 added lines, -1 removed lines

@@ -90,7 +90,9 @@
 		$container->js(true, new jsExpression('new ClipboardJS(".copy-button")'));
 		
 		$container->on('click', '.delete-link', $this->add(['jsCallback', 'postTrigger' => 'link'])->set(function($j, $linkId) {
-			if (! $link = FileRemoteAccess::create()->load($linkId)) return;
+			if (! $link = FileRemoteAccess::create()->load($linkId)) {
+				return;
+			}
 			
 			$link->delete();
 			

src/View/FileDetailsLister.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
 	
 	public function getDefaultSource()
 	{
-		return $this->file? [
+		return $this->file ? [
 				'name' => [
 						'title'=> __('Name'),
 						'descr'=> $this->file['name']
@@ -50,6 +50,6 @@ 
 						'title'=> __('Stored At'),
 						'descr'=> $this->file['created_at']
 				]
-		]: [];
+		] : [];
 	}
 }
\ No newline at end of file

src/FileStorageUI.php

Spacing +5 added lines, -5 removed lines

@@ -13,7 +13,7 @@ 
 		$fileExists = File::exists($id);
 		
 		if ($icon) {
-			$icon_file = $fileExists ? 'z-attach.png': 'z-attach-off.png';
+			$icon_file = $fileExists ? 'z-attach.png' : 'z-attach-off.png';
 			$img_src = Base_ThemeCommon::get_template_file(self::module_name(), $icon_file);
 			$icon_img = '<img src="' . $img_src . '" style="vertical-align:bottom">';
 		}
@@ -23,12 +23,12 @@ 
 		
 		$file = File::get($id);
 				
-		if (! $fileName = $label) {
+		if (!$fileName = $label) {
 			$fileName = ($file['name']?? '') ?: '[' . __('missing file name') . ']';
 		}
 		
-		if ($nolink || ! $file) {
-			return $fileName . ($fileExists ? '': ' [' . __('missing file') . ']');
+		if ($nolink || !$file) {
+			return $fileName . ($fileExists ? '' : ' [' . __('missing file') . ']');
 		}
 		
 		$link_href = '';
@@ -49,6 +49,6 @@ 
 		
 		$ret = '<a ' . $link_href . '>' . $icon_img . '<span class="file_name">' . $fileName . '</span></a>';
 		
-		return $inline? $ret: '<div class="file_link">'.$ret.'</div>';
+		return $inline ? $ret : '<div class="file_link">' . $ret . '</div>';
 	}
 }

Braces +1 added lines, -2 removed lines

@@ -16,8 +16,7 @@
 			$icon_file = $fileExists ? 'z-attach.png': 'z-attach-off.png';
 			$img_src = Base_ThemeCommon::get_template_file(self::module_name(), $icon_file);
 			$icon_img = '<img src="' . $img_src . '" style="vertical-align:bottom">';
-		}
-		else {
+		} else {
 			$icon_img = '';
 		}