scriptotek / colligator-backend

app/Jobs/OaiPmhHarvest.php

Spacing +10 added lines, -10 removed lines

@@ -102,7 +102,7 @@ 
      */
     public function fromDump()
     {
-        $files = Storage::disk('local')->files('harvests/' . $this->name);
+        $files = Storage::disk('local')->files('harvests/'.$this->name);
         $recordsHarvested = 0;
         foreach ($files as $filename) {
             if (!preg_match('/.xml$/', $filename)) {
@@ -124,11 +124,11 @@ 
 
     public function fromNetwork()
     {
-        $dest_path = 'harvests/' . $this->name . '/';
+        $dest_path = 'harvests/'.$this->name.'/';
 
         Storage::disk('local')->deleteDir($dest_path);
 
-        $latest = $dest_path . 'latest.xml';
+        $latest = $dest_path.'latest.xml';
 
         $client = new OaiPmhClient($this->url, [
             'schema'              => $this->schema,
@@ -137,12 +137,12 @@ 
             'sleep-time-on-error' => $this->sleepTimeOnError,
         ]);
 
-        $client->on('request.error', function ($msg) {
+        $client->on('request.error', function($msg) {
             $this->error($msg);
         });
 
         // Store each response to disk just in case
-        $client->on('request.complete', function ($verb, $args, $body) use ($latest) {
+        $client->on('request.complete', function($verb, $args, $body) use ($latest) {
             Storage::disk('local')->put($latest, $body);
         });
 
@@ -166,9 +166,9 @@ 
             if ($this->resume != $records->getResumptionToken()) {
                 $this->resume = $records->getResumptionToken();
                 if (is_null($this->resume)) {
-                    Storage::disk('local')->delete($dest_path . '/resumption_token');
+                    Storage::disk('local')->delete($dest_path.'/resumption_token');
                 } else {
-                    Storage::disk('local')->put($dest_path . '/resumption_token', $this->resume);
+                    Storage::disk('local')->put($dest_path.'/resumption_token', $this->resume);
                 }
             }
 
@@ -201,7 +201,7 @@ 
                 } catch (BadRequestError $e) {
                     $this->error($e->getMessage());
                     $this->error($e->getCode());
-                    $this->error('Bad request. Attempt ' . $attempt . ' of 100. Sleeping 10 secs.');
+                    $this->error('Bad request. Attempt '.$attempt.' of 100. Sleeping 10 secs.');
                     if ($attempt > 100) {
                         throw $e;
                     }
@@ -225,7 +225,7 @@ 
      */
     public function handle(DocumentsIndex $docIndex, Marc21Importer $importer)
     {
-        Log::info('[OaiPmhHarvestJob] Starting job. Requesting records from ' . ($this->start ?: '(no limit)') . ' until ' . ($this->until ?: '(no limit)') . '.');
+        Log::info('[OaiPmhHarvestJob] Starting job. Requesting records from '.($this->start ?: '(no limit)').' until '.($this->until ?: '(no limit)').'.');
         $this->docIndex = $docIndex;
         $this->importer = $importer;
 
@@ -245,7 +245,7 @@ 
         }
 
         // $this->postProcess($this->importedDocuments);
-        Log::info('[OaiPmhHarvestJob] Complete, got ' . $recordsHarvested . ' records.');
+        Log::info('[OaiPmhHarvestJob] Complete, got '.$recordsHarvested.' records.');
 
         Event::fire(new OaiPmhHarvestComplete($recordsHarvested));
     }

app/Http/Requests/SearchDocumentsRequest.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
                 $this->warnings[] = 'Offset cannot be negative.';
             } elseif ($input['offset'] > $maxPaginationDepth) {
                 $input['offset'] = $maxPaginationDepth;
-                $this->warnings[] = 'Pagination depth is limited to ' . $maxPaginationDepth . ' results.';
+                $this->warnings[] = 'Pagination depth is limited to '.$maxPaginationDepth.' results.';
             }
         }
 
@@ -36,7 +36,7 @@ 
                 $this->warnings[] = 'Limit cannot be negative.';
             } elseif ($input['limit'] > $maxResultsPerRequest) {
                 $input['limit'] = $maxResultsPerRequest;
-                $this->warnings[] = 'Limiting to max ' . $maxResultsPerRequest . ' results per request.';
+                $this->warnings[] = 'Limiting to max '.$maxResultsPerRequest.' results per request.';
             }
         }
 

app/Document.php

Doc Comments +1 added lines

@@ -60,6 +60,7 @@
 
     /**
      * Get enrichments associated with this document.
+     * @param string $serviceName
      */
     public function enrichmentsByService($serviceName)
     {

app/GoogleBooksService.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Colligator;
 
 use Scriptotek\GoogleBooks\GoogleBooks;
-use Scriptotek\GoogleBooks\Exceptions\UsageRateExceeded;
 
 class GoogleBooksService implements EnrichmentService
 {

Spacing +4 added lines, -4 removed lines

@@ -19,7 +19,7 @@ 
     {
         foreach ($isbns as $isbn) {
             $volume = $this->books->volumes->byIsbn($isbn);
-            \Log::debug('[GoogleBooksService] - ' . $isbn . ': ' . (is_null($volume) ? 'not found' : 'found'));
+            \Log::debug('[GoogleBooksService] - '.$isbn.': '.(is_null($volume) ? 'not found' : 'found'));
             if (!is_null($volume)) {
                 return $volume;
             }
@@ -29,7 +29,7 @@ 
 
     public function enrich(Document $doc)
     {
-        \Log::debug('[GoogleBooksService] Checking document ' . $doc->id);
+        \Log::debug('[GoogleBooksService] Checking document '.$doc->id);
 
         $volume = $this->get($doc->bibliographic['isbns']);
 
@@ -47,11 +47,11 @@ 
         if ($volume) {
             $cover_url = $volume->getCover();
             if (!is_null($cover_url) && is_null($doc->cover)) {
-                \Log::debug('[GoogleBooksService] Setting new cover to document #' . $doc->id);
+                \Log::debug('[GoogleBooksService] Setting new cover to document #'.$doc->id);
                 try {
                     $doc->storeCover($cover_url);
                 } catch (\ErrorException $e) {
-                    \Log::error('[GoogleBooksService] Failed to store cover: ' . $cover_url . "\n" . $e->getMessage());
+                    \Log::error('[GoogleBooksService] Failed to store cover: '.$cover_url."\n".$e->getMessage());
                 }
             }
         }

app/Console/Commands/EnrichDocuments.php

Spacing +6 added lines, -6 removed lines

@@ -71,7 +71,7 @@ 
 
     public function handleDocuments(EnrichmentService $service, $docs)
     {
-        \Log::info('[EnrichDocuments] Starting job. ' . count($docs) . ' documents to be checked using the "' . $service::$serviceName . '" service.');
+        \Log::info('[EnrichDocuments] Starting job. '.count($docs).' documents to be checked using the "'.$service::$serviceName.'" service.');
 
         // $this->output->progressStart(count($docs));
         foreach ($docs as $doc) {
@@ -107,22 +107,22 @@ 
         $collectionId = intval($this->option('collection'));
 
         if (!isset($this->services[$serviceName])) {
-            $this->error('Service "' . $serviceName . '" is not defined. Available servies: "' . implode('", "', array_keys($this->services)) . '".');
+            $this->error('Service "'.$serviceName.'" is not defined. Available servies: "'.implode('", "', array_keys($this->services)).'".');
             return;
         }
 
         if ($verbose) {
-            \DB::listen(function ($query, $bindings) {
-                $this->comment('Query: ' . $query . '. Bindings: ' . implode(', ', $bindings));
+            \DB::listen(function($query, $bindings) {
+                $this->comment('Query: '.$query.'. Bindings: '.implode(', ', $bindings));
             });
         }
 
-        $collectionHelp = ($collectionId > 0) ? ' in collection ' . $collectionId : '';
+        $collectionHelp = ($collectionId > 0) ? ' in collection '.$collectionId : '';
 
         $docs = $this->getDocumentsToBeChecked($serviceName, $force, $collectionId);
 
         if (!count($docs)) {
-            $this->info('No new documents' . $collectionHelp . '. Exiting.');
+            $this->info('No new documents'.$collectionHelp.'. Exiting.');
             \Log::info('[EnrichDocuments] No new documents to be checked.');
             return;
         }

app/Providers/BroadcastServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
         /*
          * Authenticate the user's personal channel...
          */
-        Broadcast::channel('Colligator.User.{userId}', function ($user, $userId) {
+        Broadcast::channel('Colligator.User.{userId}', function($user, $userId) {
             return (int) $user->id === (int) $userId;
         });
     }

app/Providers/RouteServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
         Route::group([
             'middleware' => 'web',
             'namespace' => $this->namespace,
-        ], function () {
+        ], function() {
             require base_path('routes/web.php');
         });
     }
@@ -58,7 +58,7 @@ 
             'middleware' => 'api',
             'namespace' => $this->namespace,
             // 'prefix' => 'api',
-        ], function () {
+        ], function() {
             require base_path('routes/api.php');
         });
     }