Passed
Pull Request — master (#225)
by
unknown
06:11
created
Command/MigrateCommand.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -213,9 +213,9 @@  discard block
 block discarded – undo
213 213
         $time = microtime(true) - $start;
214 214
         if ($input->getOption('separate-process')) {
215 215
             // in case of using subprocesses, we can not measure max memory used
216
-            $this->writeln("<info>Time taken: ".sprintf('%.2f', $time)." secs</info>");
216
+            $this->writeln("<info>Time taken: " . sprintf('%.2f', $time) . " secs</info>");
217 217
         } else {
218
-            $this->writeln("<info>Time taken: ".sprintf('%.2f', $time)." secs, memory: ".sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)). ' MB</info>');
218
+            $this->writeln("<info>Time taken: " . sprintf('%.2f', $time) . " secs, memory: " . sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)) . ' MB</info>');
219 219
         }
220 220
 
221 221
         return $failed;
@@ -442,7 +442,7 @@  discard block
 block discarded – undo
442 442
             $builderArgs[] = '--no-debug';
443 443
         }
444 444
         if ($input->getOption('siteaccess')) {
445
-            $builderArgs[] = '--siteaccess='.$input->getOption('siteaccess');
445
+            $builderArgs[] = '--siteaccess=' . $input->getOption('siteaccess');
446 446
         }
447 447
         switch ($this->verbosity) {
448 448
             case OutputInterface::VERBOSITY_VERBOSE:
Please login to merge, or discard this patch.
Core/Executor/TagManager.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
 
162 162
         // sort tags by depth so that there will be no errors in case we are deleting parent and child
163 163
         $tagsCollection = $tagsCollection->getArrayCopy();
164
-        uasort($tagsCollection, function ($t1, $t2) {
164
+        uasort($tagsCollection, function($t1, $t2) {
165 165
             if ($t1->depth == $t2->depth) return 0;
166 166
             return ($t1->depth > $t2->depth) ? -1 : 1;
167 167
         });
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
             case 'create':
267 267
                 // sort top to bottom
268 268
                 $tagCollection = $tagCollection->getArrayCopy();
269
-                uasort($tagCollection, function ($t1, $t2) {
269
+                uasort($tagCollection, function($t1, $t2) {
270 270
                     if ($t1->depth == $t2->depth) return 0;
271 271
                     return ($t1->depth > $t2->depth) ? 1 : -1;
272 272
                 });
@@ -274,7 +274,7 @@  discard block
 block discarded – undo
274 274
             case 'delete':
275 275
                 // sort bottom to top
276 276
                 $tagCollection = $tagCollection->getArrayCopy();
277
-                uasort($tagCollection, function ($t1, $t2) {
277
+                uasort($tagCollection, function($t1, $t2) {
278 278
                     if ($t1->depth == $t2->depth) return 0;
279 279
                     return ($t1->depth > $t2->depth) ? -1 : 1;
280 280
                 });
@@ -352,7 +352,7 @@  discard block
 block discarded – undo
352 352
     protected function getTagKeywords($tag)
353 353
     {
354 354
         $out = array();
355
-        foreach($tag->languageCodes as $languageCode) {
355
+        foreach ($tag->languageCodes as $languageCode) {
356 356
             $out[$languageCode] = $tag->getKeyword($languageCode);
357 357
         }
358 358
         return $out;
Please login to merge, or discard this patch.
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -162,7 +162,9 @@  discard block
 block discarded – undo
162 162
         // sort tags by depth so that there will be no errors in case we are deleting parent and child
163 163
         $tagsCollection = $tagsCollection->getArrayCopy();
164 164
         uasort($tagsCollection, function ($t1, $t2) {
165
-            if ($t1->depth == $t2->depth) return 0;
165
+            if ($t1->depth == $t2->depth) {
166
+                return 0;
167
+            }
166 168
             return ($t1->depth > $t2->depth) ? -1 : 1;
167 169
         });
168 170
 
@@ -267,7 +269,9 @@  discard block
 block discarded – undo
267 269
                 // sort top to bottom
268 270
                 $tagCollection = $tagCollection->getArrayCopy();
269 271
                 uasort($tagCollection, function ($t1, $t2) {
270
-                    if ($t1->depth == $t2->depth) return 0;
272
+                    if ($t1->depth == $t2->depth) {
273
+                        return 0;
274
+                    }
271 275
                     return ($t1->depth > $t2->depth) ? 1 : -1;
272 276
                 });
273 277
                 break;
@@ -275,7 +279,9 @@  discard block
 block discarded – undo
275 279
                 // sort bottom to top
276 280
                 $tagCollection = $tagCollection->getArrayCopy();
277 281
                 uasort($tagCollection, function ($t1, $t2) {
278
-                    if ($t1->depth == $t2->depth) return 0;
282
+                    if ($t1->depth == $t2->depth) {
283
+                        return 0;
284
+                    }
279 285
                     return ($t1->depth > $t2->depth) ? -1 : 1;
280 286
                 });
281 287
                 break;
Please login to merge, or discard this patch.
Core/Matcher/TagMatcher.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -156,7 +156,7 @@
 block discarded – undo
156 156
             $childTags = $this->findTagsByParentTagIds(array_keys($parentTags));
157 157
             $tags = array_merge($tags, $childTags);
158 158
             $parentTags = $childTags;
159
-        } while (count( $childTags ) > 0);
159
+        } while (count($childTags) > 0);
160 160
 
161 161
         return $tags;
162 162
     }
Please login to merge, or discard this patch.
Core/Matcher/TrashMatcher.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -56,7 +56,9 @@
 block discarded – undo
56 56
 
57 57
             $query = new Query();
58 58
             $query->limit = $this->queryLimit;
59
-            if (isset($query->performCount)) $query->performCount = false;
59
+            if (isset($query->performCount)) {
60
+                $query->performCount = false;
61
+            }
60 62
             $query->filter = $this->getQueryCriterion($key, $values);
61 63
             $results = $this->repository->getTrashService()->findTrashItems($query);
62 64
 
Please login to merge, or discard this patch.
Core/MigrationService.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -264,7 +264,7 @@  discard block
 block discarded – undo
264 264
         }
265 265
 
266 266
         if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
267
-            throw new \Exception("Can not execute " . $this->getEntityName($migrationDefinition). " '{$migrationDefinition->name}': {$migrationDefinition->parsingError}");
267
+            throw new \Exception("Can not execute " . $this->getEntityName($migrationDefinition) . " '{$migrationDefinition->name}': {$migrationDefinition->parsingError}");
268 268
         }
269 269
 
270 270
         /// @todo add support for setting in $migrationContext a userContentType, userGroupContentType ?
@@ -298,7 +298,7 @@  discard block
 block discarded – undo
298 298
 
299 299
         try {
300 300
 
301
-            $i = $stepOffset+1;
301
+            $i = $stepOffset + 1;
302 302
             $finalStatus = Migration::STATUS_DONE;
303 303
             $finalMessage = null;
304 304
 
@@ -433,12 +433,12 @@  discard block
 block discarded – undo
433 433
     public function resumeMigration(Migration $migration, $useTransaction = true)
434 434
     {
435 435
         if ($migration->status != Migration::STATUS_SUSPENDED) {
436
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': it is not in suspended status");
436
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': it is not in suspended status");
437 437
         }
438 438
 
439 439
         $migrationDefinitions = $this->getMigrationsDefinitions(array($migration->path));
440 440
         if (!count($migrationDefinitions)) {
441
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': its definition is missing");
441
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': its definition is missing");
442 442
         }
443 443
 
444 444
         $defs = $migrationDefinitions->getArrayCopy();
@@ -446,18 +446,18 @@  discard block
 block discarded – undo
446 446
 
447 447
         $migrationDefinition = $this->parseMigrationDefinition($migrationDefinition);
448 448
         if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
449
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': {$migrationDefinition->parsingError}");
449
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': {$migrationDefinition->parsingError}");
450 450
         }
451 451
 
452 452
         // restore context
453 453
         $this->contextHandler->restoreCurrentContext($migration->name);
454 454
 
455 455
         if (!isset($this->migrationContext[$migration->name])) {
456
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': the stored context is missing");
456
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': the stored context is missing");
457 457
         }
458 458
         $restoredContext = $this->migrationContext[$migration->name];
459
-        if (!is_array($restoredContext) || !isset($restoredContext['context']) || !isset($restoredContext['step'] )) {
460
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': the stored context is invalid");
459
+        if (!is_array($restoredContext) || !isset($restoredContext['context']) || !isset($restoredContext['step'])) {
460
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': the stored context is invalid");
461 461
         }
462 462
 
463 463
         // update migration status
@@ -478,7 +478,7 @@  discard block
 block discarded – undo
478 478
      * @param bool|null $forceSigchildEnabled
479 479
      * @return array
480 480
      */
481
-    protected function migrationContextFromParameters($defaultLanguageCode = null, $adminLogin = null, $forceSigchildEnabled = null )
481
+    protected function migrationContextFromParameters($defaultLanguageCode = null, $adminLogin = null, $forceSigchildEnabled = null)
482 482
     {
483 483
         $properties = array();
484 484
 
Please login to merge, or discard this patch.