Merge pull request #7 from rapidwebltd/analysis-qoOAvK

Apply fixes from StyleCI
rapidwebltd · Mar 28, 2018 · 4c2d26b · 4c2d26b
2 parents 48559e3 + 336da5d
commit 4c2d26b
Show file tree

Hide file tree

Showing 7 changed files with 10 additions and 6 deletions.
diff --git a/src/Interfaces/SourceInterface.php b/src/Interfaces/SourceInterface.php
@@ -5,6 +5,8 @@
 interface SourceInterface
 {
     public function getDataRows($page = 1, $fieldsToRetrieve = []);
+
     public function countDataRows();
+
     public function getFields();
 }
diff --git a/src/Objects/Migrator.php b/src/Objects/Migrator.php
@@ -3,12 +3,12 @@
 namespace RapidWeb\uxdm\Objects;
 
 use Psr\Cache\CacheItemPoolInterface;
+use RapidWeb\CliProgressBar\ProgressBar;
 use RapidWeb\uxdm\Interfaces\DestinationInterface;
 use RapidWeb\uxdm\Interfaces\SourceInterface;
 use RapidWeb\uxdm\Objects\Exceptions\MissingFieldToMigrateException;
 use RapidWeb\uxdm\Objects\Exceptions\NoDestinationException;
 use RapidWeb\uxdm\Objects\Exceptions\NoSourceException;
-use RapidWeb\CliProgressBar\ProgressBar;
 
 class Migrator
 {
@@ -102,6 +102,7 @@ public function setSourceCache(CacheItemPoolInterface $sourceCachePool, $sourceC
     public function withProgressBar()
     {
         $this->showProgressBar = true;
+
         return $this;
     }
 
@@ -163,7 +164,6 @@ public function migrate()
         }
 
         for ($page = 1; $page < PHP_INT_MAX; $page++) {
-
             $dataRows = $this->getSourceDataRows($page);
 
             if (!$dataRows) {
@@ -224,7 +224,7 @@ public function migrate()
     private function advanceProgressBar($amount)
     {
         if ($this->showProgressBar) {
-            for ($i=0; $i < $amount; $i++) { 
+            for ($i = 0; $i < $amount; $i++) {
                 $this->progressBar->advance()->display();
             }
         }

diff --git a/src/Objects/Sources/CSVSource.php b/src/Objects/Sources/CSVSource.php
@@ -74,6 +74,7 @@ public function countDataRows()
     {
         $file = new \SplFileObject($this->file, 'r');
         $file->seek(PHP_INT_MAX);
+
         return $file->key();
     }
 }
diff --git a/src/Objects/Sources/PDOSource.php b/src/Objects/Sources/PDOSource.php
@@ -133,7 +133,7 @@ public function countDataRows()
         $sql = $this->getSQL([]);
         $fromPos = strpos($sql, 'from');
         $limitPos = strrpos($sql, 'limit');
-        $sqlSuffix = substr($sql, $fromPos, $limitPos-$fromPos);
+        $sqlSuffix = substr($sql, $fromPos, $limitPos - $fromPos);
 
         $sql = 'select count(*) as countDataRows '.$sqlSuffix;
 

diff --git a/src/Objects/Sources/WordPressPostSource.php b/src/Objects/Sources/WordPressPostSource.php
@@ -148,7 +148,7 @@ public function countDataRows()
         $sql = $this->getPostSQL([]);
         $fromPos = strpos($sql, 'from');
         $limitPos = strrpos($sql, 'limit');
-        $sqlSuffix = substr($sql, $fromPos, $limitPos-$fromPos);
+        $sqlSuffix = substr($sql, $fromPos, $limitPos - $fromPos);
 
         $sql = 'count (*) '.$sqlSuffix;
     }

diff --git a/src/Objects/Sources/WordPressUserSource.php b/src/Objects/Sources/WordPressUserSource.php
@@ -146,7 +146,7 @@ public function countDataRows()
         $sql = $this->getUserSQL([]);
         $fromPos = strpos($sql, 'from');
         $limitPos = strrpos($sql, 'limit');
-        $sqlSuffix = substr($sql, $fromPos, $limitPos-$fromPos);
+        $sqlSuffix = substr($sql, $fromPos, $limitPos - $fromPos);
 
         $sql = 'count (*) '.$sqlSuffix;
     }

diff --git a/src/Objects/Sources/XMLSource.php b/src/Objects/Sources/XMLSource.php
@@ -93,6 +93,7 @@ public function getFields()
     public function countDataRows()
     {
         $domNodeList = $this->xpath->query($this->xpathQuery);
+
         return $domNodeList->length;
     }
 }