Applied fixes from StyleCI

Rhincodon · Nov 20, 2016 · 560371b · 560371b
1 parent 7f988f1
commit 560371b
Show file tree

Hide file tree

Showing 9 changed files with 22 additions and 18 deletions.
diff --git a/src/Commands/FillSheet.php b/src/Commands/FillSheet.php
@@ -38,8 +38,9 @@ public function handle()
     {
         $modelType = $this->argument('modelType');
 
-        if (empty(config('laravel-eloquent-spreadsheets')["sync_models"][$modelType])) {
+        if (empty(config('laravel-eloquent-spreadsheets')['sync_models'][$modelType])) {
             $this->warn('Model not registered in config');
+
             return;
         }
 
@@ -50,6 +51,6 @@ public function handle()
             $this->info("Dispatched model with ID: {$modelItem->id}");
         }
 
-        $this->info("Dispatching completed");
+        $this->info('Dispatching completed');
     }
 }
diff --git a/src/Commands/Sync.php b/src/Commands/Sync.php
@@ -43,8 +43,9 @@ public function handle()
     {
         $modelType = $this->argument('modelType');
 
-        if (empty(config('laravel-eloquent-spreadsheets')["sync_models"][$modelType])) {
+        if (empty(config('laravel-eloquent-spreadsheets')['sync_models'][$modelType])) {
             $this->warn('Model not registered in config');
+
             return;
         }
 
@@ -60,12 +61,13 @@ public function handle()
 
         if (empty($values)) {
             $this->warn('Sheet is empty');
+
             return;
         }
 
         foreach ($values as $value) {
             if (
-                !is_array($value) ||
+                ! is_array($value) ||
                 empty($value[getLettersToIdsTable($this->startColumn, $this->endColumn)[$this->idColumn]])
             ) {
                 continue;
@@ -74,7 +76,7 @@ public function handle()
             $id = $value[getLettersToIdsTable($this->startColumn, $this->endColumn)[$this->idColumn]];
             $modelItem = $model->whereId($id)->first();
 
-            if (!$modelItem) {
+            if (! $modelItem) {
                 continue;
             }
 
@@ -107,7 +109,7 @@ private function updateModel($modelItem, $value)
 
     private function prepareConfig($modelType)
     {
-        $this->config = config('laravel-eloquent-spreadsheets')["sync_models"][$modelType];
+        $this->config = config('laravel-eloquent-spreadsheets')['sync_models'][$modelType];
 
         asort($this->config['sync_attributes']);
         $this->startColumn = array_first($this->config['sync_attributes']);

diff --git a/src/EloquentSpreadsheetsServiceProvider.php b/src/EloquentSpreadsheetsServiceProvider.php
@@ -14,7 +14,7 @@ class EloquentSpreadsheetsServiceProvider extends ServiceProvider
     public function boot()
     {
         $this->publishes([
-            __DIR__ . '/../config/laravel-eloquent-spreadsheets.php' => config_path('laravel-eloquent-spreadsheets.php'),
+            __DIR__.'/../config/laravel-eloquent-spreadsheets.php' => config_path('laravel-eloquent-spreadsheets.php'),
         ], 'config');
 
         $this->bootObservers();
@@ -26,10 +26,10 @@ public function boot()
     public function register()
     {
         $this->mergeConfigFrom(
-            __DIR__ . '/../config/laravel-eloquent-spreadsheets.php', 'laravel-eloquent-spreadsheets'
+            __DIR__.'/../config/laravel-eloquent-spreadsheets.php', 'laravel-eloquent-spreadsheets'
         );
 
-        $this->app->singleton(SpreadsheetService::class, function() {
+        $this->app->singleton(SpreadsheetService::class, function () {
             $client = (new SpreadsheetClient(config('laravel-eloquent-spreadsheets.credentials_path')))->client();
 
             return new SpreadsheetService($client);

diff --git a/src/Jobs/Traits/Saveable.php b/src/Jobs/Traits/Saveable.php
@@ -33,7 +33,7 @@ public function getModelSyncedData()
         }
 
         foreach (range(0, count(range($this->startColumn, $this->endColumn)) - 1) as $key) {
-            if (!empty($result[$key])) {
+            if (! empty($result[$key])) {
                 continue;
             }
 
@@ -64,4 +64,4 @@ public function insertModelToSheet($rowToInsert)
             $batch
         );
     }
-}
+}
diff --git a/src/Jobs/UpdateModelInSpreadsheet.php b/src/Jobs/UpdateModelInSpreadsheet.php
@@ -46,6 +46,7 @@ public function handle()
 
         if ($rowToInsert === false) {
             dispatch(new CreateModelInSpreadsheet($this->model, $this->config));
+
             return;
         }
 

diff --git a/src/ModelObserver.php b/src/ModelObserver.php
@@ -35,6 +35,6 @@ public function deleted($model)
      */
     public function getConfig($model)
     {
-        return config('laravel-eloquent-spreadsheets')["sync_models"][get_class($model)];
+        return config('laravel-eloquent-spreadsheets')['sync_models'][get_class($model)];
     }
-}
+}
diff --git a/src/SpreadsheetClient.php b/src/SpreadsheetClient.php
@@ -28,11 +28,11 @@ public function __construct($credentialsPath)
     }
 
     /**
-     * Init Client
+     * Init Client.
      */
     private function initClient()
     {
-        putenv('GOOGLE_APPLICATION_CREDENTIALS=' . $this->credentialsPath);
+        putenv('GOOGLE_APPLICATION_CREDENTIALS='.$this->credentialsPath);
         $this->client = new Google_Client();
         $this->client->useApplicationDefaultCredentials();
         $this->client->addScope(Google_Service_Sheets::SPREADSHEETS);
@@ -45,4 +45,4 @@ public function client()
     {
         return $this->client;
     }
-}
+}
diff --git a/src/SpreadsheetService.php b/src/SpreadsheetService.php
@@ -34,4 +34,4 @@ public function service()
     {
         return $this->service;
     }
-}
+}
diff --git a/src/helpers.php b/src/helpers.php
@@ -1,6 +1,6 @@
 <?php
 
-if (!function_exists('getLettersToIdsTable')) {
+if (! function_exists('getLettersToIdsTable')) {
     /**
      * @param $startColumn
      * @param $endColumn