kaliop-uk / ezmigrationbundle

Core/ReferenceResolver/LoopResolver.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      */
     public function getReferenceValue($identifier)
     {
-        switch(substr($identifier, 5)) {
+        switch (substr($identifier, 5)) {
             case 'iteration':
                 $current = end($this->stack);
                 return $current['step'];

Core/Executor/LoopExecutor.php

Spacing +2 added lines, -2 removed lines

@@ -80,7 +80,7 @@ 
                     $subStep = new MigrationStep($type, $stepDef, array_merge($step->context, array()));
                     try {
                         $result = $stepExecutors[$j]->execute($subStep);
-                    } catch(MigrationStepSkippedException $e) {
+                    } catch (MigrationStepSkippedException $e) {
                         // all ok, continue the loop
                     }
                 }
@@ -97,7 +97,7 @@ 
                     $subStep = new MigrationStep($type, $stepDef, array_merge($step->context, array()));
                     try {
                         $result = $stepExecutors[$j]->execute($subStep);
-                    } catch(MigrationStepSkippedException $e) {
+                    } catch (MigrationStepSkippedException $e) {
                         // all ok, continue the loop
                     }
                 }

Core/Helper/SortConverter.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     public function sortField2Hash($value)
     {
         $ref = new \ReflectionClass('eZ\Publish\API\Repository\Values\Content\Location');
-        foreach($ref->getConstants() as $key => $val) {
+        foreach ($ref->getConstants() as $key => $val) {
             if (strpos($key, 'SORT_FIELD_') === 0 && $val == $value) {
                 $out = strtolower(substr($key, 11));
                 if ($out == 'contentobject_id') {

Core/Executor/ContentVersionManager.php

Spacing +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
 
     protected function versionStatusToHash($status)
     {
-        foreach(ContentVersionMatcher::STATUS_MAP as $own => $ez) {
+        foreach (ContentVersionMatcher::STATUS_MAP as $own => $ez) {
             if ($status == $ez) {
                 return $own;
             }

Core/Matcher/ContentVersionMatcher.php

Spacing +6 added lines, -6 removed lines

@@ -45,7 +45,7 @@ 
         $versions = array();
 
         $contentCollection = $this->contentMatcher->match($contentConditions, $sort, $offset, $limit);
-        foreach($contentCollection as $content) {
+        foreach ($contentCollection as $content) {
             $versions = array_merge($versions, $this->matchContentVersions($versionConditions, $content));
         }
 
@@ -151,7 +151,7 @@ 
     {
         $versions = array();
         foreach ($this->findAllContentVersions($content) as $versionKey => $versionInfo) {
-            foreach($values as $acceptedStatus) {
+            foreach ($values as $acceptedStatus) {
                 if ($versionInfo->status == self::STATUS_MAP[$acceptedStatus]) {
                     $versions[$versionKey] = $versionInfo;
                     break;
@@ -173,15 +173,15 @@ 
         $contentVersionsCount = count($contentVersions);
         $i = 0;
         foreach ($contentVersions as $versionKey => $versionInfo) {
-            foreach($values as $acceptedVersionNo) {
-                if ($acceptedVersionNo > 0 ) {
+            foreach ($values as $acceptedVersionNo) {
+                if ($acceptedVersionNo > 0) {
                     if ($acceptedVersionNo == $versionInfo->versionNo) {
                         $versions[$versionKey] = $versionInfo;
                         break;
                     }
                 } else {
                     // negative $acceptedVersionNo means 'leave the last X versions', eg: -1 = leave the last version
-                    if ($i < $contentVersionsCount + $acceptedVersionNo)  {
+                    if ($i < $contentVersionsCount + $acceptedVersionNo) {
                         $versions[$versionKey] = $versionInfo;
                         break;
 
@@ -202,7 +202,7 @@ 
         $contentVersions = $this->repository->getContentService()->loadVersions($content->contentInfo);
         // different eZ kernels apparently sort versions in different order...
         $sortedVersions = array();
-        foreach($contentVersions as $versionInfo) {
+        foreach ($contentVersions as $versionInfo) {
             $sortedVersions[$content->contentInfo->id . '/' . $versionInfo->versionNo] = $versionInfo;
         }
         ksort($sortedVersions);

Core/Matcher/QueryBasedMatcher.php

Spacing +7 added lines, -7 removed lines

@@ -148,7 +148,7 @@ 
                 return new Query\Criterion\DateMetadata(Query\Criterion\DateMetadata::CREATED, self::$operatorsMap[$operator], $match);
 
             case self::MATCH_GROUP:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->groupMatcher->matchOneByKey($value)->id;
                     }
@@ -167,7 +167,7 @@ 
                 return new Query\Criterion\DateMetadata(Query\Criterion\DateMetadata::MODIFIED, self::$operatorsMap[$operator], $match);
 
             case self::MATCH_OBJECT_STATE:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->stateMatcher->matchOneByKey($value)->id;
                     }
@@ -175,7 +175,7 @@ 
                 return new Query\Criterion\ObjectStateId($values);
 
             case self::MATCH_OWNER:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->userMatcher->matchOneByKey($value)->id;
                     }
@@ -195,7 +195,7 @@ 
                 return new Query\Criterion\ParentLocationId($locationIds);
 
             case self::MATCH_SECTION:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->sectionMatcher->matchOneByKey($value)->id;
                     }
@@ -216,7 +216,7 @@ 
 
             case self::MATCH_AND:
                 $subCriteria = array();
-                foreach($values as $subCriterion) {
+                foreach ($values as $subCriterion) {
                     $value = reset($subCriterion);
                     $subCriteria[] = $this->getQueryCriterion(key($subCriterion), $value);
                 }
@@ -224,7 +224,7 @@ 
 
             case self::MATCH_OR:
                 $subCriteria = array();
-                foreach($values as $subCriterion) {
+                foreach ($values as $subCriterion) {
                     $value = reset($subCriterion);
                     $subCriteria[] = $this->getQueryCriterion(key($subCriterion), $value);
                 }
@@ -266,7 +266,7 @@ 
 
             $direction = $this->hash2SortOrder($sortItem['sort_order']);
 
-            switch($sortItem['sort_field']) {
+            switch ($sortItem['sort_field']) {
                 case self::SORT_CONTENT_ID:
                     $out[] = new SortClause\ContentId($direction);
                     break;

Core/FieldHandler/EzSelection.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         // allow user to pass in selection values by name
         $fieldSettings = null;
-        foreach($fieldValue as $key => $val) {
+        foreach ($fieldValue as $key => $val) {
 
             $val = $this->referenceResolver->resolveReference($val);
 
@@ -47,7 +47,7 @@ 
                     if ($fieldSettings === null) {
                         $fieldSettings = $this->loadFieldSettings($context['contentTypeIdentifier'], $context['fieldIdentifier']);
                     }
-                    foreach($fieldSettings['options'] as $pos => $name) {
+                    foreach ($fieldSettings['options'] as $pos => $name) {
                         if ($name === $val) {
                             $fieldValue[$key] = $pos;
                             break;

Core/Process/Process.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
     public static function forceSigchildEnabled($force)
     {
-        self::$forceSigchildEnabled = (bool) $force;
+        self::$forceSigchildEnabled = (bool)$force;
     }
 
     protected function isSigchildEnabled()

Core/MigrationService.php

Spacing +9 added lines, -9 removed lines

@@ -264,7 +264,7 @@ 
         }
 
         if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
-            throw new \Exception("Can not execute " . $this->getEntityName($migrationDefinition). " '{$migrationDefinition->name}': {$migrationDefinition->parsingError}");
+            throw new \Exception("Can not execute " . $this->getEntityName($migrationDefinition) . " '{$migrationDefinition->name}': {$migrationDefinition->parsingError}");
         }
 
         /// @todo add support for setting in $migrationContext a userContentType ?
@@ -298,7 +298,7 @@ 
 
         try {
 
-            $i = $stepOffset+1;
+            $i = $stepOffset + 1;
             $finalStatus = Migration::STATUS_DONE;
             $finalMessage = null;
 
@@ -433,12 +433,12 @@ 
     public function resumeMigration(Migration $migration, $useTransaction = true)
     {
         if ($migration->status != Migration::STATUS_SUSPENDED) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': it is not in suspended status");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': it is not in suspended status");
         }
 
         $migrationDefinitions = $this->getMigrationsDefinitions(array($migration->path));
         if (!count($migrationDefinitions)) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': its definition is missing");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': its definition is missing");
         }
 
         $defs = $migrationDefinitions->getArrayCopy();
@@ -446,18 +446,18 @@ 
 
         $migrationDefinition = $this->parseMigrationDefinition($migrationDefinition);
         if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': {$migrationDefinition->parsingError}");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': {$migrationDefinition->parsingError}");
         }
 
         // restore context
         $this->contextHandler->restoreCurrentContext($migration->name);
 
         if (!isset($this->migrationContext[$migration->name])) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': the stored context is missing");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': the stored context is missing");
         }
         $restoredContext = $this->migrationContext[$migration->name];
-        if (!is_array($restoredContext) || !isset($restoredContext['context']) || !isset($restoredContext['step'] )) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': the stored context is invalid");
+        if (!is_array($restoredContext) || !isset($restoredContext['context']) || !isset($restoredContext['step'])) {
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': the stored context is invalid");
         }
 
         // update migration status
@@ -478,7 +478,7 @@ 
      * @param bool|null $forceSigchildEnabled
      * @return array
      */
-    protected function migrationContextFromParameters($defaultLanguageCode = null, $adminLogin = null, $forceSigchildEnabled = null )
+    protected function migrationContextFromParameters($defaultLanguageCode = null, $adminLogin = null, $forceSigchildEnabled = null)
     {
         $properties = array();