scriptotek / colligator-backend

app/Jobs/OaiPmhHarvest.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 use Phpoaipmh\Endpoint;
 use Phpoaipmh\Exception\OaipmhException;
 use Phpoaipmh\Granularity;
-use Storage;
 
 class OaiPmhHarvest extends Job
 {

Spacing +4 added lines, -4 removed lines

@@ -85,7 +85,7 @@ 
     public function fromNetwork()
     {
         $client = new Client($this->url);
-        $endpoint = new Endpoint($client,Granularity::DATE);
+        $endpoint = new Endpoint($client, Granularity::DATE);
 
         $recordsHarvested = 0;
 
@@ -109,7 +109,7 @@ 
      */
     public function handle()
     {
-        Log::info('[OaiPmhHarvest] Starting job. Requesting records from ' . ($this->start ?: '(no limit)') . ' until ' . ($this->until ?: '(no limit)') . '.');
+        Log::info('[OaiPmhHarvest] Starting job. Requesting records from '.($this->start ?: '(no limit)').' until '.($this->until ?: '(no limit)').'.');
 
         // For timing
         $this->startTime = $this->batchTime = microtime(true) - 1;
@@ -123,9 +123,9 @@ 
 
         try {
             $recordsHarvested = $this->fromNetwork();
-            Log::info('[OaiPmhHarvest] Harvest complete, got ' . $recordsHarvested . ' records.');
+            Log::info('[OaiPmhHarvest] Harvest complete, got '.$recordsHarvested.' records.');
         } catch (OaipmhException $e) {
-            Log::warning('[OaiPmhHarvest] Harvest stopped with error ' . $e->getCode() . ': ' . $e->getMessage());
+            Log::warning('[OaiPmhHarvest] Harvest stopped with error '.$e->getCode().': '.$e->getMessage());
         }
     }
 

app/Console/Commands/OaiPmhHarvest.php

Spacing +8 added lines, -8 removed lines

@@ -47,7 +47,7 @@ 
         $this->comment('Available configurations:');
         $config = \Config::get('oaipmh.harvests', null);
         foreach (array_keys($config) as $key) {
-            $this->comment(' - ' . $key);
+            $this->comment(' - '.$key);
         }
     }
 
@@ -58,7 +58,7 @@ 
 
             return false;
         }
-        $harvestConfig = \Config::get('oaipmh.harvests.' . $this->argument('name'), null);
+        $harvestConfig = \Config::get('oaipmh.harvests.'.$this->argument('name'), null);
         if (is_null($harvestConfig)) {
             $this->error('Unknown configuration specified.');
             $this->listConfigurations();
@@ -100,13 +100,13 @@ 
         }
 
         $harvestName = $this->argument('name');
-        $harvestConfig = \Config::get('oaipmh.harvests.' . $harvestName, null);
+        $harvestConfig = \Config::get('oaipmh.harvests.'.$harvestName, null);
 
         $this->comment(sprintf('Dispatching new harvest job'));
 
-        $this->comment(' - Repo: ' . $harvestConfig['url']);
-        $this->comment(' - Schema: ' . $harvestConfig['schema']);
-        $this->comment(' - Set: ' . $harvestConfig['set']);
+        $this->comment(' - Repo: '.$harvestConfig['url']);
+        $this->comment(' - Schema: '.$harvestConfig['schema']);
+        $this->comment(' - Set: '.$harvestConfig['set']);
 
         foreach (['from', 'until', 'resume', 'daily'] as $key) {
             if (!is_null($this->option($key))) {
@@ -117,8 +117,8 @@ 
         $from = Carbon::parse($this->option('from'));
         $until = Carbon::parse($this->option('until'));
         if ($this->option('daily')) {
-            $from = Carbon::now()->setTime(0,0,0)->subDay();
-            $until = Carbon::now()->setTime(0,0,0);
+            $from = Carbon::now()->setTime(0, 0, 0)->subDay();
+            $until = Carbon::now()->setTime(0, 0, 0);
         }
 
         $this->dispatch(