kaliop-uk / ezmigrationbundle

Core/Executor/TagManager.php

Spacing +4 added lines, -4 removed lines

@@ -161,7 +161,7 @@ 
 
         // sort tags by depth so that there will be no errors in case we are deleting parent and child
         $tagsCollection = $tagsCollection->getArrayCopy();
-        uasort($tagsCollection, function ($t1, $t2) {
+        uasort($tagsCollection, function($t1, $t2) {
             if ($t1->depth == $t2->depth) return 0;
             return ($t1->depth > $t2->depth) ? -1 : 1;
         });
@@ -266,7 +266,7 @@ 
             case 'create':
                 // sort top to bottom
                 $tagCollection = $tagCollection->getArrayCopy();
-                uasort($tagCollection, function ($t1, $t2) {
+                uasort($tagCollection, function($t1, $t2) {
                     if ($t1->depth == $t2->depth) return 0;
                     return ($t1->depth > $t2->depth) ? 1 : -1;
                 });
@@ -274,7 +274,7 @@ 
             case 'delete':
                 // sort bottom to top
                 $tagCollection = $tagCollection->getArrayCopy();
-                uasort($tagCollection, function ($t1, $t2) {
+                uasort($tagCollection, function($t1, $t2) {
                     if ($t1->depth == $t2->depth) return 0;
                     return ($t1->depth > $t2->depth) ? -1 : 1;
                 });
@@ -352,7 +352,7 @@ 
     protected function getTagKeywords($tag)
     {
         $out = array();
-        foreach($tag->languageCodes as $languageCode) {
+        foreach ($tag->languageCodes as $languageCode) {
             $out[$languageCode] = $tag->getKeyword($languageCode);
         }
         return $out;

Braces +9 added lines, -3 removed lines

@@ -162,7 +162,9 @@ 
         // sort tags by depth so that there will be no errors in case we are deleting parent and child
         $tagsCollection = $tagsCollection->getArrayCopy();
         uasort($tagsCollection, function ($t1, $t2) {
-            if ($t1->depth == $t2->depth) return 0;
+            if ($t1->depth == $t2->depth) {
+                return 0;
+            }
             return ($t1->depth > $t2->depth) ? -1 : 1;
         });
 
@@ -267,7 +269,9 @@ 
                 // sort top to bottom
                 $tagCollection = $tagCollection->getArrayCopy();
                 uasort($tagCollection, function ($t1, $t2) {
-                    if ($t1->depth == $t2->depth) return 0;
+                    if ($t1->depth == $t2->depth) {
+                        return 0;
+                    }
                     return ($t1->depth > $t2->depth) ? 1 : -1;
                 });
                 break;
@@ -275,7 +279,9 @@ 
                 // sort bottom to top
                 $tagCollection = $tagCollection->getArrayCopy();
                 uasort($tagCollection, function ($t1, $t2) {
-                    if ($t1->depth == $t2->depth) return 0;
+                    if ($t1->depth == $t2->depth) {
+                        return 0;
+                    }
                     return ($t1->depth > $t2->depth) ? -1 : 1;
                 });
                 break;

Core/Matcher/TagMatcher.php

Spacing +1 added lines, -1 removed lines

@@ -156,7 +156,7 @@
             $childTags = $this->findTagsByParentTagIds(array_keys($parentTags));
             $tags = array_merge($tags, $childTags);
             $parentTags = $childTags;
-        } while (count( $childTags ) > 0);
+        } while (count($childTags) > 0);
 
         return $tags;
     }

Core/Matcher/TrashMatcher.php

Braces +3 added lines, -1 removed lines

@@ -56,7 +56,9 @@
 
             $query = new Query();
             $query->limit = $this->queryLimit;
-            if (isset($query->performCount)) $query->performCount = false;
+            if (isset($query->performCount)) {
+                $query->performCount = false;
+            }
             $query->filter = $this->getQueryCriterion($key, $values);
             $results = $this->repository->getTrashService()->findTrashItems($query);
 

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();
 

Core/Matcher/ContentMatcher.php

Spacing +6 added lines, -6 removed lines

@@ -80,15 +80,15 @@ 
 
         foreach ($conditions as $key => $values) {
 
-            switch($key) {
+            switch ($key) {
 
                 case self::MATCH_RELATES_TO:
                     $contentService = $this->repository->getContentService();
                     $contents = array();
                     // allow to specify the objects to relate to using different ways
                     $relatedContents = $this->match($values);
-                    foreach($relatedContents as $relatedContent) {
-                        foreach($contentService->loadReverseRelations($relatedContent->contentInfo) as $relatingContent) {
+                    foreach ($relatedContents as $relatedContent) {
+                        foreach ($contentService->loadReverseRelations($relatedContent->contentInfo) as $relatingContent) {
                             $contents[$relatingContent->contentInfo->id] = $relatingContent;
                         }
                     }
@@ -99,8 +99,8 @@ 
                     $contents = array();
                     // allow to specify the objects we relate to using different ways
                     $relatingContents = $this->match($values);
-                    foreach($relatingContents as $relatingContent) {
-                        foreach($contentService->loadRelations($relatingContent->contentInfo) as $relatedContent) {
+                    foreach ($relatingContents as $relatingContent) {
+                        foreach ($contentService->loadRelations($relatingContent->contentInfo) as $relatedContent) {
                             $contents[$relatedContent->contentInfo->id] = $relatedContent;
                         }
                     }
@@ -349,7 +349,7 @@ 
                     $query->performCount = false;
                     $query->filter = new Query\Criterion\ContentTypeIdentifier('this_is_a_very_unlikely_content_type_identifier');
                     //$query->filter = new Query\Criterion\ContentTypeId($contentTypeIds);
-                } catch(NotImplementedException $e) {
+                } catch (NotImplementedException $e) {
                     return false;
                 }
             }

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