kaliop-uk / ezmigrationbundle

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, userGroupContentType ?
@@ -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();
 

Braces +3 added lines, -2 removed lines

@@ -599,8 +599,9 @@
      */
     public function getCurrentContext($migrationName)
     {
-        if (!isset($this->migrationContext[$migrationName]))
-            return null;
+        if (!isset($this->migrationContext[$migrationName])) {
+                    return null;
+        }
         $context = $this->migrationContext[$migrationName];
         // avoid attempting to store the current outputInterface when saving the context
         if (isset($context['output'])) {

Core/Matcher/LocationMatcher.php

Braces +6 added lines, -2 removed lines

@@ -87,7 +87,9 @@ 
             if (!empty($sort)) {
                 $query->sortClauses = $this->getSortClauses($sort);
             }
-            if (isset($query->performCount)) $query->performCount = false;
+            if (isset($query->performCount)) {
+                $query->performCount = false;
+            }
             $query->filter = $this->getQueryCriterion($key, $values);
             $results = $this->getSearchService()->findLocations($query);
 
@@ -234,7 +236,9 @@ 
     {
         $query = new LocationQuery();
         $query->limit = self::INT_MAX_16BIT;
-        if (isset($query->performCount)) $query->performCount = false;
+        if (isset($query->performCount)) {
+            $query->performCount = false;
+        }
         $query->filter = new Query\Criterion\ParentLocationId($parentLocationIds);
 
         $results = $this->getSearchService()->findLocations($query);

Core/Matcher/ContentMatcher.php

Braces +12 added lines, -4 removed lines

@@ -120,7 +120,9 @@ 
                     $query = new Query();
                     $query->limit = $limit != 0 ? $limit : $this->queryLimit;
                     $query->offset = $offset;
-                    if (isset($query->performCount)) $query->performCount = false;
+                    if (isset($query->performCount)) {
+                        $query->performCount = false;
+                    }
                     $query->filter = $this->getQueryCriterion($key, $values);
                     if (!empty($sort)) {
                         $query->sortClauses = $this->getSortClauses($sort);
@@ -246,7 +248,9 @@ 
     {
         $query = new Query();
         $query->limit = $this->queryLimit;
-        if (isset($query->performCount)) $query->performCount = false;
+        if (isset($query->performCount)) {
+            $query->performCount = false;
+        }
         $query->filter = new Query\Criterion\ParentLocationId($parentLocationIds);
         $results = $this->getSearchService()->findContent($query);
 
@@ -286,7 +290,9 @@ 
     {
         $query = new Query();
         $query->limit = $this->queryLimit;
-        if (isset($query->performCount)) $query->performCount = false;
+        if (isset($query->performCount)) {
+            $query->performCount = false;
+        }
         $query->filter = new Query\Criterion\ContentTypeIdentifier($contentTypeIdentifiers);
         // sort objects by depth, lower to higher, so that deleting them has less chances of failure
         // NB: we only do this in eZP versions that allow depth sorting on content queries
@@ -314,7 +320,9 @@ 
     {
         $query = new Query();
         $query->limit = $this->queryLimit;
-        if (isset($query->performCount)) $query->performCount = false;
+        if (isset($query->performCount)) {
+            $query->performCount = false;
+        }
         $query->filter = new Query\Criterion\ContentTypeId($contentTypeIds);
         // sort objects by depth, lower to higher, so that deleting them has less chances of failure
         // NB: we only do this in eZP versions that allow depth sorting on content queries

Core/FieldHandler/EzCountry.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
             $fieldValue = array($fieldValue);
         }
         if (!$refsResolved) {
-            foreach($fieldValue as $key => $countryValue) {
+            foreach ($fieldValue as $key => $countryValue) {
                 $fieldValue[$key] = $this->referenceResolver->resolveReference($countryValue);
             }
         }

Core/Matcher/RoleMatcher.php

Spacing +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
                 $role = $this->repository->getRoleService()->loadRole($roleId);
                 $roles[$role->id] = $role;
             /// @todo should we survive as well UnauthorizedException ? It seems to be a different kind of error than non-existing roles...
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }
@@ -130,7 +130,7 @@ 
                 $role = $this->repository->getRoleService()->loadRoleByIdentifier($roleIdentifier);
                 $roles[$role->id] = $role;
             /// @todo should we survive as well UnauthorizedException ? It seems to be a different kind of error than non-existing roles...
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }

Core/Matcher/TagMatcher.php

Spacing +4 added lines, -4 removed lines

@@ -131,7 +131,7 @@ 
                 // return unique contents
                 $tag = $this->tagService->loadTag($tagId);
                 $tags[$tag->id] = $tag;
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }
@@ -160,7 +160,7 @@ 
                 foreach ($this->tagService->loadTagChildren($tag) as $childTag) {
                     $tags[$childTag->id] = $childTag;
                 }
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }
@@ -188,7 +188,7 @@ 
             $childTags = $this->findTagsByParentTagIds(array_keys($parentTags));
             $tags = array_merge($tags, $childTags);
             $parentTags = $childTags;
-        } while (count( $childTags ) > 0);
+        } while (count($childTags) > 0);
 
         return $tags;
     }
@@ -221,7 +221,7 @@ 
                 // return unique contents
                 $tag = $this->tagService->loadTagByRemoteId($tagRemoteId);
                 $tags[$tag->id] = $tag;
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }

Core/Matcher/UserMatcher.php

Spacing +4 added lines, -4 removed lines

@@ -108,12 +108,12 @@ 
         $users = [];
 
         foreach ($userIds as $userId) {
-            try{
+            try {
                 // return unique contents
                 $user = $this->repository->getUserService()->loadUser($userId);
 
                 $users[$user->id] = $user;
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }
@@ -139,7 +139,7 @@ 
                 $user = $this->repository->getUserService()->loadUserByLogin($login);
 
                 $users[$user->id] = $user;
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }
@@ -188,7 +188,7 @@ 
 
             try {
                 $group = $this->repository->getUserService()->loadUserGroup($groupId);
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if ($tolerateMisses) {
                     continue;
                 } else {

Core/Matcher/ObjectStateGroupMatcher.php

Spacing +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
                 // return unique contents
                 $objectStateGroup = $this->repository->getObjectStateService()->loadObjectStateGroup($objectStateGroupId);
                 $objectStateGroups[$objectStateGroup->id] = $objectStateGroup;
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }

Core/Matcher/LocationMatcherDirectLoad.php

Spacing +4 added lines, -4 removed lines

@@ -36,11 +36,11 @@ 
             $locations = array();
             switch ($key) {
                 case self::MATCH_LOCATION_ID:
-                    foreach($match as $locationId) {
+                    foreach ($match as $locationId) {
                         try {
                             $location = $this->repository->getLocationService()->loadLocation($locationId);
                             $locations[$location->id] = $location;
-                        } catch(NotFoundException $e) {
+                        } catch (NotFoundException $e) {
                             if (!$tolerateMisses) {
                                 throw $e;
                             }
@@ -48,11 +48,11 @@ 
                     }
                     break;
                 case self::MATCH_LOCATION_REMOTE_ID:
-                    foreach($match as $locationRemoteId) {
+                    foreach ($match as $locationRemoteId) {
                         try {
                             $location = $this->repository->getLocationService()->loadLocationByRemoteId($locationRemoteId);
                             $locations[$location->id] = $location;
-                        } catch(NotFoundException $e) {
+                        } catch (NotFoundException $e) {
                             if (!$tolerateMisses) {
                                 throw $e;
                             }