oroinc / platform

src/Oro/Component/Layout/Tests/Unit/BlockFactoryTest.php

Spacing +23 added lines, -23 removed lines

@@ -69,7 +69,7 @@ 
             $this->expressionLanguage,
             new ExpressionEncoderRegistry([])
         );
-        $this->blockFactory       = new BlockFactory(
+        $this->blockFactory = new BlockFactory(
             $this->registry,
             $this->layoutManipulator,
             $expressionProcessor
@@ -99,13 +99,13 @@ 
         $view = $this->getLayoutView();
 
         $this->assertBlockView(
-            [ // root
+            [// root
                 'vars'     => ['id' => 'root'],
                 'children' => [
-                    [ // header
+                    [// header
                         'vars'     => ['id' => 'header'],
                         'children' => [
-                            [ // logo
+                            [// logo
                                 'vars' => ['id' => 'logo', 'title' => 'test']
                             ]
                         ]
@@ -124,7 +124,7 @@ 
         $view = $this->getLayoutView();
 
         $this->assertBlockView(
-            [ // root
+            [// root
                 'vars'     => [
                     'id'                   => 'rootId',
                     'block_type'           => 'root',
@@ -156,7 +156,7 @@ 
         $view = $this->getLayoutView();
 
         $this->assertBlockView(
-            [ // root
+            [// root
                 'vars'     => [
                     'id'                   => 'rootId',
                     'block_type'           => 'root',
@@ -172,7 +172,7 @@ 
                     'cache_key'            => '_rootId_root'
                 ],
                 'children' => [
-                    [ // header
+                    [// header
                         'vars'     => [
                             'id'                   => 'headerId',
                             'block_type'           => 'header',
@@ -188,7 +188,7 @@ 
                             'cache_key'            => '_headerId_header'
                         ],
                         'children' => [
-                            [ // logo
+                            [// logo
                                 'vars' => [
                                     'id'                   => 'logoId',
                                     'block_type'           => 'logo',
@@ -247,7 +247,7 @@ 
             ->method('updateLayout')
             ->will(
                 $this->returnCallback(
-                    function (LayoutManipulatorInterface $layoutManipulator, LayoutItemInterface $item) {
+                    function(LayoutManipulatorInterface $layoutManipulator, LayoutItemInterface $item) {
                         $layoutManipulator->add('test', 'header', 'test');
                     }
                 )
@@ -261,7 +261,7 @@ 
             ->method('configureOptions')
             ->will(
                 $this->returnCallback(
-                    function (OptionsResolver $resolver) {
+                    function(OptionsResolver $resolver) {
                         $resolver->setDefaults(
                             [
                                 'test_option_1' => '',
@@ -275,7 +275,7 @@ 
             ->method('buildBlock')
             ->will(
                 $this->returnCallback(
-                    function (BlockBuilderInterface $builder, Options $options) {
+                    function(BlockBuilderInterface $builder, Options $options) {
                         if ($options['test_option_1'] === 'move_logo_to_root') {
                             $builder->getLayoutManipulator()->move('logo', 'root');
                         }
@@ -286,7 +286,7 @@ 
             ->method('buildView')
             ->will(
                 $this->returnCallback(
-                    function (BlockView $view, BlockInterface $block, Options $options) {
+                    function(BlockView $view, BlockInterface $block, Options $options) {
                         $view->vars['attr']['block_id'] = $block->getId();
                         if ($options['test_option_1'] === 'move_logo_to_root') {
                             $view->vars['attr']['logo_moved'] = true;
@@ -299,7 +299,7 @@ 
             ->method('finishView')
             ->will(
                 $this->returnCallback(
-                    function (BlockView $view, BlockInterface $block) {
+                    function(BlockView $view, BlockInterface $block) {
                         if (isset($view['test'])) {
                             $view['test']->vars['processed_by_header_extension'] = true;
                         }
@@ -327,10 +327,10 @@ 
         $view = $this->getLayoutView();
 
         $this->assertBlockView(
-            [ // root
+            [// root
                 'vars'     => ['id' => 'root'],
                 'children' => [
-                    [ // header
+                    [// header
                         'vars'     => [
                             'id'   => 'header',
                             'attr' => [
@@ -340,7 +340,7 @@ 
                             ]
                         ],
                         'children' => [
-                            [ // test
+                            [// test
                                 'vars' => [
                                     'id'                            => 'test',
                                     'processed_by_header_extension' => true
@@ -348,7 +348,7 @@ 
                             ]
                         ]
                     ],
-                    [ // logo
+                    [// logo
                         'vars' => ['id' => 'logo', 'title' => 'test']
                     ]
                 ]
@@ -378,13 +378,13 @@ 
         $view = $this->getLayoutView();
 
         $this->assertBlockView(
-            [ // root
+            [// root
                 'vars'     => ['id' => 'root'],
                 'children' => [
-                    [ // header
+                    [// header
                         'vars'     => ['id' => 'header'],
                         'children' => [
-                            [ // logo
+                            [// logo
                                 'vars' => ['id' => 'logo', 'title' => 'test title']
                             ]
                         ]
@@ -459,13 +459,13 @@ 
         $view = $this->getLayoutView();
 
         $this->assertBlockView(
-            [ // root
+            [// root
                 'vars'     => ['id' => 'root'],
                 'children' => [
-                    [ // header
+                    [// header
                         'vars'     => ['id' => 'header'],
                         'children' => [
-                            [ // logo
+                            [// logo
                                 'vars' => ['id' => 'logo', 'title' => 'one two']
                             ]
                         ]

src/Oro/Component/Layout/BlockView.php

Spacing +1 added lines, -1 removed lines

@@ -40,6 +40,6 @@
      */
     public function isVisible()
     {
-        return array_key_exists('visible', $this->vars) ? (bool) $this->vars['visible'] : true;
+        return array_key_exists('visible', $this->vars) ? (bool)$this->vars['visible'] : true;
     }
 }

src/Oro/Component/Layout/BlockFactory.php

Spacing +3 added lines, -3 removed lines

@@ -97,7 +97,7 @@ 
             $this->typeHelper,
             $this->context
         );
-        $this->block           = new Block(
+        $this->block = new Block(
             $this->rawLayout,
             $this->typeHelper,
             $this->context,
@@ -234,7 +234,7 @@ 
         $view            = new BlockView($parentView);
 
         if (is_null($resolvedOptions)) { // Try to resolve options again to render block
-            $options  = $this->rawLayout->getProperty($id, RawLayout::OPTIONS, true);
+            $options = $this->rawLayout->getProperty($id, RawLayout::OPTIONS, true);
             $resolvedOptions = $this->setBlockResolvedOptions($id, $blockType, $options, $types);
         }
 
@@ -248,7 +248,7 @@ 
 
         array_walk_recursive(
             $view->vars,
-            function (&$var) {
+            function(&$var) {
                 if ($var instanceof Options) {
                     $var = $var->toArray();
                 }

src/Oro/Component/Testing/Unit/FormIntegrationTestCase.php

Spacing +4 added lines, -4 removed lines

@@ -60,7 +60,7 @@ 
         $loader
             ->expects($this->any())
             ->method('loadClassMetadata')
-            ->will($this->returnCallback(function (ClassMetadata $meta) {
+            ->will($this->returnCallback(function(ClassMetadata $meta) {
                 $this->loadMetadata($meta);
             }));
 
@@ -155,7 +155,7 @@ 
 
         $factory->expects($this->any())
             ->method('getInstance')
-            ->will($this->returnCallback(function (Constraint $constraint) {
+            ->will($this->returnCallback(function(Constraint $constraint) {
                 $className = $constraint->validatedBy();
 
                 if (!isset($this->validators[$className])
@@ -181,13 +181,13 @@ 
 
         $translator->expects($this->any())
             ->method('trans')
-            ->will($this->returnCallback(function ($id) {
+            ->will($this->returnCallback(function($id) {
                 return $id;
             }))
         ;
         $translator->expects($this->any())
             ->method('transChoice')
-            ->will($this->returnCallback(function ($id) {
+            ->will($this->returnCallback(function($id) {
                 return $id;
             }))
         ;

src/Oro/Component/MessageQueue/Job/JobStorage.php

Spacing +2 added lines, -2 removed lines

@@ -147,7 +147,7 @@ 
             }
 
             $em = $this->getEntityManager(true);
-            $em->getConnection()->transactional(function (Connection $connection) use ($em, $job, $lockCallback) {
+            $em->getConnection()->transactional(function(Connection $connection) use ($em, $job, $lockCallback) {
                 /** @var Job $job */
                 $job = $em->find($this->entityClass, $job->getId(), LockMode::PESSIMISTIC_WRITE);
 
@@ -167,7 +167,7 @@ 
                 // exception occurs but UniqueConstraintViolationException is expected here
                 // and we should keep EntityManager in open state.
                 $em = $this->getEntityManager(true);
-                $em->getConnection()->transactional(function (Connection $connection) use ($job, $em) {
+                $em->getConnection()->transactional(function(Connection $connection) use ($job, $em) {
                     try {
                         $connection->insert($this->uniqueTableName, ['name' => $job->getOwnerId()]);
                         if ($job->isUnique()) {

src/Oro/Component/MessageQueue/Job/CalculateRootJobProgressProcessor.php

Spacing +3 added lines, -3 removed lines

@@ -50,14 +50,14 @@ 
     {
         $data = JSON::decode($message->getBody());
 
-        if (! isset($data['jobId'])) {
+        if (!isset($data['jobId'])) {
             $this->logger->critical('Got invalid message');
 
             return self::REJECT;
         }
 
         $job = $this->jobStorage->findJobById($data['jobId']);
-        if (! $job) {
+        if (!$job) {
             $this->logger->critical(
                 sprintf('Job was not found. id: "%s"', $data['jobId'])
             );
@@ -91,7 +91,7 @@ 
         }
 
         $progress = round($processed / $numberOfChildren, 4);
-        $this->jobStorage->saveJob($rootJob, function (Job $rootJob) use ($progress) {
+        $this->jobStorage->saveJob($rootJob, function(Job $rootJob) use ($progress) {
             if ($progress !== $rootJob->getJobProgress()) {
                 $rootJob->setJobProgress($progress);
             }

src/Oro/Component/MessageQueue/Job/JobProcessor.php

Spacing +10 added lines, -10 removed lines

@@ -90,11 +90,11 @@ 
      */
     public function findOrCreateRootJob($ownerId, $jobName, $unique = false)
     {
-        if (! $ownerId) {
+        if (!$ownerId) {
             throw new \LogicException('OwnerId must not be empty');
         }
 
-        if (! $jobName) {
+        if (!$jobName) {
             throw new \LogicException('Job name must not be empty');
         }
 
@@ -111,7 +111,7 @@ 
         $job->setLastActiveAt(new \DateTime());
         $job->setStartedAt(new \DateTime());
         $job->setJobProgress(0);
-        $job->setUnique((bool) $unique);
+        $job->setUnique((bool)$unique);
 
         return $this->saveJobAndStaleDuplicateIfQualifies($job);
     }
@@ -151,7 +151,7 @@ 
 
         $timeBeforeStale = $this->getJobConfigurationProvider()->getTimeBeforeStaleForJobName($job->getName());
         if ($timeBeforeStale !== null && $timeBeforeStale != -1) {
-            return $job->getLastActiveAt() <= new \DateTime('- ' . $timeBeforeStale. ' seconds');
+            return $job->getLastActiveAt() <= new \DateTime('- ' . $timeBeforeStale . ' seconds');
         }
 
         return false;
@@ -181,7 +181,7 @@ 
      */
     public function findOrCreateChildJob($jobName, Job $rootJob)
     {
-        if (! $jobName) {
+        if (!$jobName) {
             throw new \LogicException('Job name must not be empty');
         }
 
@@ -217,7 +217,7 @@ 
 
         $job = $this->jobStorage->findJobById($job->getId());
 
-        if (! in_array($job->getStatus(), $this->getNotStartedJobStatuses(), true)) {
+        if (!in_array($job->getStatus(), $this->getNotStartedJobStatuses(), true)) {
             throw new \LogicException(sprintf(
                 'Can start only new jobs: id: "%s", status: "%s"',
                 $job->getId(),
@@ -280,7 +280,7 @@ 
             throw new \LogicException(sprintf('Can\'t stale child jobs. id: "%s"', $rootJob->getId()));
         }
 
-        $this->jobStorage->saveJob($rootJob, function (Job $rootJob) {
+        $this->jobStorage->saveJob($rootJob, function(Job $rootJob) {
             $rootJob->setStatus(Job::STATUS_STALE);
             $rootJob->setStoppedAt(new \DateTime());
 
@@ -359,7 +359,7 @@ 
 
         $job = $this->jobStorage->findJobById($job->getId());
 
-        if (! in_array($job->getStatus(), $this->getActiveJobStatuses(), true)) {
+        if (!in_array($job->getStatus(), $this->getActiveJobStatuses(), true)) {
             throw new \LogicException(sprintf(
                 'Can cancel only new or running jobs. id: "%s", status: "%s"',
                 $job->getId(),
@@ -370,7 +370,7 @@ 
         $job->setStatus(Job::STATUS_CANCELLED);
         $job->setStoppedAt($stoppedAt = new \DateTime());
 
-        if (! $job->getStartedAt()) {
+        if (!$job->getStartedAt()) {
             $job->setStartedAt($stoppedAt);
         }
 
@@ -428,7 +428,7 @@ 
             return;
         }
 
-        $this->jobStorage->saveJob($job, function (Job $job) use ($force) {
+        $this->jobStorage->saveJob($job, function(Job $job) use ($force) {
             if ($job->isInterrupted()) {
                 return;
             }

src/Oro/Component/MessageQueue/Job/CalculateRootJobStatusProcessor.php

Spacing +2 added lines, -2 removed lines

@@ -52,14 +52,14 @@
     {
         $data = JSON::decode($message->getBody());
 
-        if (! isset($data['jobId'])) {
+        if (!isset($data['jobId'])) {
             $this->logger->critical('Got invalid message');
 
             return self::REJECT;
         }
 
         $job = $this->jobStorage->findJobById($data['jobId']);
-        if (! $job) {
+        if (!$job) {
             $this->logger->critical(sprintf('Job was not found. id: "%s"', $data['jobId']));
 
             return self::REJECT;

src/Oro/Component/MessageQueue/Tests/Functional/Transport/Dbal/DbalMessageConsumerTest.php

Spacing +11 added lines, -11 removed lines

@@ -40,7 +40,7 @@ 
             'queue' => 'queue',
             'priority' => 1,
         ]);
-        $id = (int) $dbal->lastInsertId('message_queue_id_seq');
+        $id = (int)$dbal->lastInsertId('message_queue_id_seq');
 
         //guard
         $this->assertGreaterThan(0, $id);
@@ -68,7 +68,7 @@ 
             'queue' => 'queue',
             'priority' => 1,
         ]);
-        $id = (int) $dbal->lastInsertId('message_queue_id_seq');
+        $id = (int)$dbal->lastInsertId('message_queue_id_seq');
 
         //guard
         $this->assertGreaterThan(0, $id);
@@ -97,7 +97,7 @@ 
             'priority' => 1,
             'consumer_id' => 123,
         ]);
-        $id = (int) $dbal->lastInsertId('message_queue_id_seq');
+        $id = (int)$dbal->lastInsertId('message_queue_id_seq');
 
         //guard
         $this->assertGreaterThan(0, $id);
@@ -121,7 +121,7 @@ 
         $this->assertCount(1, $messages);
         $this->assertNotEquals($id, $messages[0]['id']);
         $this->assertNull($messages[0]['consumer_id']);
-        $this->assertTrue((bool) $messages[0]['redelivered']);
+        $this->assertTrue((bool)$messages[0]['redelivered']);
     }
 
     public function testShouldReceiveMessage()
@@ -134,7 +134,7 @@ 
             'body' => 'message',
             'queue' => 'default',
         ]);
-        $id = (int) $dbal->lastInsertId('message_queue_id_seq');
+        $id = (int)$dbal->lastInsertId('message_queue_id_seq');
 
         //guard
         $this->assertGreaterThan(0, $id);
@@ -159,13 +159,13 @@ 
             'priority' => 5,
             'queue' => 'default',
         ]);
-        $id1 = (int) $dbal->lastInsertId('message_queue_id_seq');
+        $id1 = (int)$dbal->lastInsertId('message_queue_id_seq');
 
         $dbal->insert('message_queue', [
             'priority' => 10,
             'queue' => 'default',
         ]);
-        $id2 = (int) $dbal->lastInsertId('message_queue_id_seq');
+        $id2 = (int)$dbal->lastInsertId('message_queue_id_seq');
 
         //guard
         $this->assertGreaterThan(0, $id1);
@@ -196,13 +196,13 @@ 
             'priority' => 5,
             'queue' => 'default',
         ]);
-        $id1 = (int) $dbal->lastInsertId('message_queue_id_seq');
+        $id1 = (int)$dbal->lastInsertId('message_queue_id_seq');
 
         $dbal->insert('message_queue', [
             'priority' => 5,
             'queue' => 'default',
         ]);
-        $id2 = (int) $dbal->lastInsertId('message_queue_id_seq');
+        $id2 = (int)$dbal->lastInsertId('message_queue_id_seq');
 
         //guard
         $this->assertGreaterThan(0, $id1);
@@ -234,7 +234,7 @@ 
             'queue' => 'default',
             'priority' => 1,
         ]);
-        $id = (int) $dbal->lastInsertId('message_queue_id_seq');
+        $id = (int)$dbal->lastInsertId('message_queue_id_seq');
 
         //guard
         $this->assertGreaterThan(0, $id);
@@ -258,7 +258,7 @@ 
             'queue' => 'default',
             'priority' => 1,
         ]);
-        $id = (int) $dbal->lastInsertId('message_queue_id_seq');
+        $id = (int)$dbal->lastInsertId('message_queue_id_seq');
 
         //guard
         $this->assertGreaterThan(0, $id);