kaliop-uk / ezmigrationbundle

Command/MigrateCommand.php

Spacing +3 added lines, -3 removed lines

@@ -124,7 +124,7 @@ 
                 $builderArgs[] = '--no-transactions';
             }
             if ($input->getOption('siteaccess')) {
-                $builderArgs[]='--siteaccess='.$input->getOption('siteaccess');
+                $builderArgs[] = '--siteaccess=' . $input->getOption('siteaccess');
             }
         }
 
@@ -233,9 +233,9 @@ 
         $this->writeln("Executed $executed migrations, failed $failed, skipped $skipped");
         if ($input->getOption('separate-process')) {
             // in case of using subprocesses, we can not measure max memory used
-            $this->writeln("Time taken: ".sprintf('%.2f', $time)." secs");
+            $this->writeln("Time taken: " . sprintf('%.2f', $time) . " secs");
         } else {
-            $this->writeln("Time taken: ".sprintf('%.2f', $time)." secs, memory: ".sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)). ' MB');
+            $this->writeln("Time taken: " . sprintf('%.2f', $time) . " secs, memory: " . sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)) . ' MB');
         }
     }
 

Core/Executor/ServiceExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
         } else {
             $args = array();
         }
-        foreach($args as &$val) {
+        foreach ($args as &$val) {
             $val = $this->resolveReferencesRecursively($val);
         }
 

Core/Executor/FileExecutor.php

Doc Comments +4 added lines, -4 removed lines

@@ -68,7 +68,7 @@ 
     /**
      * @param array $dsl
      * @param array $context
-     * @return string
+     * @return boolean
      * @throws \Exception
      */
     protected function exists($dsl, $context)
@@ -196,7 +196,7 @@ 
     /**
      * @param array $dsl
      * @param array $context
-     * @return true
+     * @return boolean
      * @throws \Exception
      */
     protected function copy($dsl, $context)
@@ -228,7 +228,7 @@ 
     /**
      * @param array $dsl
      * @param array $context
-     * @return true
+     * @return boolean
      * @throws \Exception
      */
     protected function move($dsl, $context)
@@ -260,7 +260,7 @@ 
     /**
      * @param array $dsl
      * @param array $context
-     * @return true
+     * @return boolean
      * @throws \Exception
      */
     protected function delete($dsl, $context)

Core/ReferenceResolver/EmbeddedRegexpReferenceResolverTrait.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
     public function hasEmbeddedReferences($string)
     {
         $regexp = $this->getEmbeddedRegexp();
-        return (bool) preg_match_all($regexp, $string, $matches);
+        return (bool)preg_match_all($regexp, $string, $matches);
     }
     
     /**
@@ -52,7 +52,7 @@ 
     {
         // we need to alter the regexp we usr for std ref resolving, as it will be used to match parts of text, not the whole string
         $regexp = substr($this->getRegexp(), 1, -1);
-        return '/' . preg_quote($this->beginToken). preg_replace(array('/^\^/'), array(''), $regexp) . '[^' . $this->endToken . ']+' . preg_quote($this->endToken) . '/';
+        return '/' . preg_quote($this->beginToken) . preg_replace(array('/^\^/'), array(''), $regexp) . '[^' . $this->endToken . ']+' . preg_quote($this->endToken) . '/';
     }
 
     /**

Core/FieldHandler/EzRichText.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 {
     public function setReferenceResolver(ReferenceResolverInterface $referenceResolver)
     {
-        if (! $referenceResolver instanceof EmbeddedReferenceResolverInterface) {
+        if (!$referenceResolver instanceof EmbeddedReferenceResolverInterface) {
             throw new \Exception("Reference resolver injected into EzRichText field handler should implement EmbeddedReferenceResolverInterface");
         }
         parent::setReferenceResolver($referenceResolver);

Core/FieldHandler/EzXmlText.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 {
     public function setReferenceResolver(ReferenceResolverInterface $referenceResolver)
     {
-        if (! $referenceResolver instanceof EmbeddedReferenceResolverInterface) {
+        if (!$referenceResolver instanceof EmbeddedReferenceResolverInterface) {
             throw new \Exception("Reference resolver injected into EzXmlText field handler should implement EmbeddedReferenceResolverInterface");
         }
         parent::setReferenceResolver($referenceResolver);

Command/TestBenchCommand.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use Kaliop\eZMigrationBundle\Core\MigrationService;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Input\InputOption;
 use Kaliop\eZMigrationBundle\API\Value\Migration;
 use Kaliop\eZMigrationBundle\API\Value\MigrationDefinition;
 

Spacing +3 added lines, -3 removed lines

@@ -49,7 +49,7 @@ 
 echo 'B';*/
 
         $ms->executeMigration(new MigrationDefinition(
-            'test_'.time(),
+            'test_' . time(),
             '/hello',
             '123',
             MigrationDefinition::STATUS_PARSED,
@@ -62,8 +62,8 @@ 
 
         $sh = $this->getContainer()->get('ez_migration_bundle.storage_handler');
 
-        $m1 = 'hello_'.time();
-        $m2 = 'there_'.time();
+        $m1 = 'hello_' . time();
+        $m2 = 'there_' . time();
         $sh->startMigration(new MigrationDefinition(
             $m1,
             '/hello',

Core/Executor/LanguageManager.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Kaliop\eZMigrationBundle\Core\Executor;
 
 use eZ\Publish\API\Repository\Values\Content\Language;
-use Kaliop\eZMigrationBundle\API\Collection\LanguageCollection;
 
 /**
  * Handles language migrations.

Core/Executor/RepositoryExecutor.php

Spacing +2 added lines, -2 removed lines

@@ -225,7 +225,7 @@ 
             }
         }
 
-        foreach($referencesDefs as $reference) {
+        foreach ($referencesDefs as $reference) {
             $overwrite = false;
             if (isset($reference['overwrite'])) {
                 $overwrite = $reference['overwrite'];
@@ -246,7 +246,7 @@ 
     {
         // allow setting *some* refs even when we have 0 or N matches
         foreach ($referencesDefinition as $key => $reference) {
-            switch($reference['attribute']) {
+            switch ($reference['attribute']) {
 
                 case 'count':
                     $value = count($entity);

Doc Comments +5 added lines, -2 removed lines

@@ -313,7 +313,7 @@ 
     }
 
     /**
-     * @param array $step
+     * @param MigrationStep $step
      * @return string
      */
     protected function getReferencesType($step)
@@ -322,7 +322,7 @@ 
     }
 
     /**
-     * @param array $step
+     * @param MigrationStep $step
      * @return bool
      */
     protected function allowEmptyReferences($step)
@@ -343,6 +343,9 @@ 
         return preg_replace('/(?<!^)[A-Z]/', ' $0', $className);
     }
 
+    /**
+     * @param AbstractCollection $collection
+     */
     protected function getCollectionName($collection)
     {
         $className = get_class($collection);

Braces +3 added lines, -2 removed lines

@@ -329,8 +329,9 @@
     {
         if (isset($step->dsl['references_type']) && $step->dsl['references_type'] == self::REFERENCE_TYPE_ARRAY &&
             isset($step->dsl['references_allow_empty']) && $step->dsl['references_allow_empty'] == true
-        )
-            return true;
+        ) {
+                    return true;
+        }
         return false;
     }