x-systems / epesi-filestorage

src/FileStorageCore.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 	
 	public static function boot()
 	{
-		Route::group(['namespace' => 'Epesi\FileStorage'], function(){
+		Route::group(['namespace' => 'Epesi\FileStorage'], function() {
 			Route::get('file', 'FileStorageController@get')->middleware('web', FileStorageAccess::class);
 		});		
 	}

src/Seeds/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);
 				

src/Seeds/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/Models/File.php

Spacing +8 added lines, -8 removed lines

@@ -60,7 +60,7 @@ 
 
     	$file = self::create()->tryLoad($id);
     	
-    	if (! $file->ref('content')['hash']) {
+    	if (!$file->ref('content')['hash']) {
     		throw new FileNotFound('File object does not have corresponding content');
     	}
 
@@ -88,7 +88,7 @@ 
     	if (!$useCache || !isset($cache[$link])) {
     		$file = self::create()->tryLoadBy('link', $link);
     		
-    		$cache[$link] = $file? $file->get('id'): null;
+    		$cache[$link] = $file ? $file->get('id') : null;
     		
     		if (!$cache[$link] && $throwException) {
     			throw new LinkNotFound($link);
@@ -124,7 +124,7 @@ 
     	try {
     		$file = is_numeric($idOrMeta) ? self::retrieve($idOrMeta) : $idOrMeta;
     		
-    		if (! file_exists($file->ref('content')['path'])) {
+    		if (!file_exists($file->ref('content')['path'])) {
     			throw new FileNotFound('Exception - file not found: ' . $file->ref('content')['path']);
     		}
     	} catch (\Exception $exception) {
@@ -149,7 +149,7 @@ 
     {
     	$ids = [];
     	foreach ((array) $files as $filePath) {
-    		if (! is_numeric($filePath)) {
+    		if (!is_numeric($filePath)) {
     			$ids[] = self::store($filePath);
     			
     			continue;
@@ -188,7 +188,7 @@ 
     		return $file['id'];
     	}
     	    	
-    	if (! $content && is_string($fileOrPath)) {
+    	if (!$content && is_string($fileOrPath)) {
     		$content = file_get_contents($fileOrPath);
     		
     		$file = [
@@ -196,7 +196,7 @@ 
     		];
     	}
     	
-    	if (! empty($file['link']) && self::getIdByLink($file['link'], false)) {
+    	if (!empty($file['link']) && self::getIdByLink($file['link'], false)) {
     		throw new LinkDuplicate($file['link']);
     	}
     	
@@ -221,9 +221,9 @@ 
     
     public static function getThumbnailField($model)
     {
-    	if (! $model->thumbnailPossible()) return false;
+    	if (!$model->thumbnailPossible()) return false;
     	
-    	$image = new \Imagick($model->ref('content')['path']  . '[0]');
+    	$image = new \Imagick($model->ref('content')['path'] . '[0]');
     	
     	$image->setImageFormat('jpg');
     	

src/Models/FileRemoteAccess.php

Spacing +2 added lines, -2 removed lines

@@ -44,9 +44,9 @@
     public static function grant($file, $expires = self::DEFAULT_PERIOD)
     {
 		return self::create()->insert([
-				'file_id' => is_numeric($file)? $file: $file->id,
+				'file_id' => is_numeric($file) ? $file : $file->id,
 				'token' => md5(uniqid(rand(), true)),
-				'created_by' => Auth::id()?: 0,
+				'created_by' => Auth::id() ?: 0,
 				'expires_at' => date('Y-m-d H:i:s', strtotime($expires)),
 		]);
     }

src/Models/FileContent.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
 
     protected $appends = ['path'];
     
-    function init(){
+    function init() {
     	parent::init();
     	
     	$this->addFields([
@@ -115,13 +115,13 @@ 
     	
     	$path = self::getStoragePath($hash);
     	
-    	if (! self::storage()->exists($path)) {
+    	if (!self::storage()->exists($path)) {
     		self::storage()->put($path, $content);
     	}
     	
     	$content = self::create()->addCondition('hash', $hash);
 
-    	if (! $content->action('count')->getOne()) {  
+    	if (!$content->action('count')->getOne()) {  
     		return $content->insert([
     				'size' => self::storage()->size($path),
     				'type' => self::storage()->mimeType($path)

src/Integration/Joints/FileStorageAccessJoint.php

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
 	 */
 	final public static function getActionUrls($file, $accessParams = [])
 	{
-		$id = is_numeric($file)? $file: $file['id'];
+		$id = is_numeric($file) ? $file : $file['id'];
 		
 		$urls = [];
 		foreach (static::$allowedActions as $action) {
@@ -66,7 +66,7 @@ 
 	 */
 	final protected function hasUserAccess()
 	{
-		return $this->forUsersOnly? (bool) Auth::id(): true;
+		return $this->forUsersOnly ? (bool) Auth::id() : true;
 	}
 	
 	/**