joindin / callingallpapers-cli

src/Command/ParseEventsConfsTech.php

Unused Use Statements -6 removed lines

@@ -32,15 +32,9 @@
 use Callingallpapers\CfpFilter\FilterList;
 use Callingallpapers\CfpFilter\FollowUriRedirect;
 use Callingallpapers\CfpFilter\StripParamsFromUri;
-use Callingallpapers\ErrorHandling\ErrorStore;
-use Callingallpapers\Exception\UnverifiedUriException;
 use Callingallpapers\Parser\ConfsTech\ConferenceParser;
 use Callingallpapers\Parser\ConfsTechParser;
-use Callingallpapers\Parser\JoindinCfpParser;
-use Callingallpapers\Parser\Lanyrd\LanyrdCfpParser;
-use Callingallpapers\Parser\PapercallIo\PapercallIoParserFactory;
 use Callingallpapers\ResultKeeper\ConsoleOutputResultKeeper;
-use Callingallpapers\ResultKeeper\ResultKeeper;
 use Callingallpapers\Service\ConfsTechParserFactory;
 use Callingallpapers\Service\GeolocationService;
 use Callingallpapers\Service\TimezoneService;

Indentation +6 added lines, -6 removed lines

@@ -58,11 +58,11 @@ 
     protected function configure()
     {
         $this->setName("parseCfPs:confs.tech")
-             ->setDescription("Retrieve CfPs from Confs.tech ony and parse them")
-             ->setDefinition(array(
-                 new InputOption('start', 's', InputOption::VALUE_OPTIONAL, 'What should be the first date to be taken into account?', ''),
-             ))
-             ->setHelp(<<<EOT
+                ->setDescription("Retrieve CfPs from Confs.tech ony and parse them")
+                ->setDefinition(array(
+                    new InputOption('start', 's', InputOption::VALUE_OPTIONAL, 'What should be the first date to be taken into account?', ''),
+                ))
+                ->setHelp(<<<EOT
 Get details about CfPs from https://confs.tech
 
 Usage:
@@ -72,7 +72,7 @@ 
 If you ommit the date the current date will be used instead
 <info>callingallpapers parseCfPs<env></info>
 EOT
-             );
+                );
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)

Spacing +3 added lines, -3 removed lines

@@ -83,16 +83,16 @@
 
         $start = new \DateTime($input->getOption('start'));
 
-        if (! $start instanceof \DateTime) {
+        if (!$start instanceof \DateTime) {
             throw new \InvalidArgumentException('The given date could not be parsed');
         }
 
-        $config = parse_ini_file(__DIR__ . '/../../config/callingallpapers.ini');
+        $config = parse_ini_file(__DIR__.'/../../config/callingallpapers.ini');
 
         $client = new Client([
             'headers' => [
                 'Accept' => 'application/json',
-                'Authorization' => 'token ' . $config['github.token'],
+                'Authorization' => 'token '.$config['github.token'],
             ],
         ]);
 

src/ResultKeeper/ResultKeeper.php

Unused Use Statements -2 removed lines

@@ -9,8 +9,6 @@
 
 namespace Callingallpapers\ResultKeeper;
 
-use Symfony\Component\Console\Output\OutputInterface;
-
 class ResultKeeper
 {
     private $errors = [];

src/Service/ConfsTechParserFactory.php

Unused Use Statements -1 removed lines

@@ -16,7 +16,6 @@
 use Callingallpapers\Writer\WriterInterface;
 use DateTimeImmutable;
 use GuzzleHttp\Client;
-use Symfony\Component\Console\Output\OutputInterface;
 
 class ConfsTechParserFactory
 {

src/Writer/ApiCfpWriter.php

Doc Comments +3 added lines

@@ -56,6 +56,9 @@
 
     private $keeper;
 
+    /**
+     * @param Client $client
+     */
     public function __construct($baseUri, $bearerToken, $client = null)
     {
         $this->baseUri     = $baseUri;

Spacing +5 added lines, -5 removed lines

@@ -105,7 +105,7 @@ 
 
         try {
             $this->client->request('GET', sprintf(
-                $this->baseUri . '/%1$s',
+                $this->baseUri.'/%1$s',
                 sha1($cfp->conferenceUri)
             ), []);
             $exists = true;
@@ -120,7 +120,7 @@ 
                     $this->baseUri
                 ), [
                     'headers' => [
-                        'Authorization' => 'Bearer ' . $this->bearerToken,
+                        'Authorization' => 'Bearer '.$this->bearerToken,
                     ],
                     'form_params' => $body
                 ]);
@@ -128,11 +128,11 @@ 
             } else {
                 // Exists, so update it
                 $response = $this->client->request('PUT', sprintf(
-                    $this->baseUri . '/%1$s',
+                    $this->baseUri.'/%1$s',
                     sha1($cfp->conferenceUri)
                 ), [
                     'headers' => [
-                        'Authorization' => 'Bearer ' . $this->bearerToken,
+                        'Authorization' => 'Bearer '.$this->bearerToken,
                     ],
                     'form_params' => $body
                 ]);
@@ -150,7 +150,7 @@ 
             $this->keeper->add($result);
         }
 
-        return (isset($response) && ($response->getStatusCode() === 204 || $response->getStatusCode() === 200 || $response->getStatusCode() === 201))?'Success':'Failure';
+        return (isset($response) && ($response->getStatusCode() === 204 || $response->getStatusCode() === 200 || $response->getStatusCode() === 201)) ? 'Success' : 'Failure';
     }
 
     public function setOutput(OutputInterface $output)

src/ResultKeeper/Created.php

Indentation +1 added lines, -1 removed lines

@@ -11,5 +11,5 @@
 
 class Created extends Success
 {
-   // Nothing to be done.
+    // Nothing to be done.
 }

src/Parser/ConfsTech/CategoryParser.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
             $cat = json_decode(base64_decode($cat['content']), true);
         }
         foreach ($cat as $conference) {
-            if (! isset($conference['cfpEndDate'])) {
+            if (!isset($conference['cfpEndDate'])) {
                 continue;
             }
             try {

src/Parser/ConfsTech/ConferenceParser.php

Spacing +5 added lines, -5 removed lines

@@ -35,7 +35,7 @@ 
         $cfp->location = $conference['city'];
 
         $geolocation = $this->geolocation->getLocationForAddress(
-            $conference['country'] . ', ' . $cfp->location
+            $conference['country'].', '.$cfp->location
         );
 
         $cfp->latitude = $geolocation->getLatitude();
@@ -48,19 +48,19 @@ 
 
         $cfp->conferenceName = $conference['name'];
         $cfp->eventStartDate = new DateTimeImmutable(
-            $conference['startDate'] . ' 08:00:00',
+            $conference['startDate'].' 08:00:00',
             new DateTimeZone($cfp->timezone)
         );
-        if (! isset($conference['endDate'])) {
+        if (!isset($conference['endDate'])) {
             $conference['endDate'] = $conference['startDate'];
         }
         $cfp->eventEndDate = new DateTimeImmutable(
-            $conference['endDate'] . ' 17:00:00',
+            $conference['endDate'].' 17:00:00',
             new DateTimeZone($cfp->timezone)
         );
 
         $cfp->dateEnd = new DateTimeImmutable(
-            $conference['cfpEndDate'] . ' 23:59:59',
+            $conference['cfpEndDate'].' 23:59:59',
             new DateTimeZone($cfp->timezone)
         );