libgraviton / import-export

src/Command/ImportCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -344,7 +344,7 @@
      *
      * @param Document $doc        Data source for import data
      * @param string   $originFile Original full filename used toimport
-     * @return bool|mixed
+     * @return false|string
      */
     private function validateUploadFile(Document $doc, $originFile)
     {

Spacing +12 added lines, -12 removed lines

@@ -231,7 +231,7 @@ 
             $output->writeln("<info>Loading data from ${file}</info>");
 
             if (!array_key_exists('target', $doc->getData())) {
-                throw new MissingTargetException('Missing target in \'' . $file . '\'');
+                throw new MissingTargetException('Missing target in \''.$file.'\'');
             }
 
             $targetUrl = sprintf('%s%s', $host, $doc->getData()['target']);
@@ -277,16 +277,16 @@ 
         $content = str_replace($rewriteHost, $rewriteTo, $doc->getContent());
         $uploadFile = $this->validateUploadFile($doc, $file);
 
-        $successFunc = function (ResponseInterface $response) use ($output) {
+        $successFunc = function(ResponseInterface $response) use ($output) {
             $output->writeln(
-                '<comment>Wrote ' . $response->getHeader('Link')[0] . '</comment>'
+                '<comment>Wrote '.$response->getHeader('Link')[0].'</comment>'
             );
         };
 
-        $errFunc = function (RequestException $e) use ($output, $file) {
+        $errFunc = function(RequestException $e) use ($output, $file) {
             $this->errors[$file] = $e->getMessage();
             $output->writeln(
-                '<error>' . str_pad(
+                '<error>'.str_pad(
                     sprintf(
                         'Failed to write <%s> from \'%s\' with message \'%s\'',
                         $e->getRequest()->getUri(),
@@ -295,17 +295,17 @@ 
                     ),
                     140,
                     ' '
-                ) . '</error>'
+                ).'</error>'
             );
             if ($output->getVerbosity() >= OutputInterface::VERBOSITY_VERBOSE) {
                 $this->dumper->dump(
                     $this->cloner->cloneVar(
                         $this->parser->parse($e->getResponse()->getBody(), false, false, true)
                     ),
-                    function ($line, $depth) use ($output) {
+                    function($line, $depth) use ($output) {
                         if ($depth > 0) {
                             $output->writeln(
-                                '<error>' . str_pad(str_repeat('  ', $depth) . $line, 140, ' ') . '</error>'
+                                '<error>'.str_pad(str_repeat('  ', $depth).$line, 140, ' ').'</error>'
                             );
                         }
                     }
@@ -321,7 +321,7 @@ 
 
         // Authentication or custom headers.
         if ($this->headerBasicAuth) {
-            $data['headers']['Authorization'] = 'Basic '. base64_encode($this->headerBasicAuth);
+            $data['headers']['Authorization'] = 'Basic '.base64_encode($this->headerBasicAuth);
         }
         if ($this->customHeaders) {
             foreach ($this->customHeaders as $headers) {
@@ -343,11 +343,11 @@ 
         // TODO This part, $uploadFile, promise should be removed once Graviton/File service is resolved in new Story.
         $fileRepeatFunc = false;
         if ($uploadFile) {
-            $fileRepeatFunc = function () use ($targetUrl, $successFunc, $errFunc, $output, $file, $data) {
+            $fileRepeatFunc = function() use ($targetUrl, $successFunc, $errFunc, $output, $file, $data) {
                 unset($this->errors[$file]);
                 $output->writeln('<info>File deleting: '.$targetUrl.'</info>');
                 $deleteRequest = $this->client->requestAsync('DELETE', $targetUrl);
-                $insert = function () use ($targetUrl, $successFunc, $errFunc, $output, $data) {
+                $insert = function() use ($targetUrl, $successFunc, $errFunc, $output, $data) {
                     $output->writeln('<info>File inserting: '.$targetUrl.'</info>');
                     $promiseInsert = $this->client->requestAsync('PUT', $targetUrl, $data);
                     $promiseInsert->then($successFunc, $errFunc);
@@ -417,7 +417,7 @@ 
         }
 
         // Find file
-        $fileName = dirname($originFile) . DIRECTORY_SEPARATOR . $documentData['file'];
+        $fileName = dirname($originFile).DIRECTORY_SEPARATOR.$documentData['file'];
         $fileName = str_replace('//', '/', $fileName);
         if (!file_exists($fileName)) {
             return false;

src/Command/ValidationCommand.php

Unused Use Statements -2 removed lines

@@ -6,12 +6,10 @@
 
 namespace Graviton\ImportExport\Command;
 
-use Graviton\ImportExport\Exception\MissingTargetException;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Helper\ProgressBar;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Finder\Finder;
 use Symfony\Component\Finder\SplFileInfo;

Spacing +3 added lines, -3 removed lines

@@ -84,15 +84,15 @@
     protected function execute(InputInterface $input, OutputInterface $output)
     {
         $files = $input->getArgument('file');
-        $finder =  new Finder();
+        $finder = new Finder();
         $finder = $finder->files();
 
         /**
          * @param SplFileInfo $file
          * @return bool
          */
-        $filter = function (SplFileInfo $file) {
-            if (!in_array($file->getExtension(), ['yml','json']) || $file->isDir()) {
+        $filter = function(SplFileInfo $file) {
+            if (!in_array($file->getExtension(), ['yml', 'json']) || $file->isDir()) {
                 return false;
             }
             return true;

src/Command/ImportCommandAbstract.php

Doc Comments +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
      *
      * @param InputInterface $input User input on console
      *
-     * @return Finder|SplFileInfo[] finder
+     * @return Finder finder
      */
     protected function getFinder($input)
     {

Spacing +4 added lines, -4 removed lines

@@ -49,7 +49,7 @@ 
         try {
             $exitCode = $this->doImport($this->getFinder($input), $input, $output);
         } catch (MissingTargetException $e) {
-            $output->writeln('<error>' . $e->getMessage() . '</error>');
+            $output->writeln('<error>'.$e->getMessage().'</error>');
         }
         return $exitCode;
     }
@@ -76,8 +76,8 @@ 
          * @param SplFileInfo $file
          * @return bool
          */
-        $filter = function (SplFileInfo $file) {
-            if (!in_array($file->getExtension(), ['yml','json']) || $file->isDir()) {
+        $filter = function(SplFileInfo $file) {
+            if (!in_array($file->getExtension(), ['yml', 'json']) || $file->isDir()) {
                 return false;
             }
             return true;
@@ -102,7 +102,7 @@ 
 
             $fileList = array_filter(
                 $fileList,
-                function ($val) {
+                function($val) {
                     if (!empty($val)) {
                         return true;
                     }