oroinc / platform

src/Oro/Bundle/ImportExportBundle/Async/Export/PostExportMessageProcessor.php

Spacing +3 added lines, -3 removed lines

@@ -81,11 +81,11 @@ 
     {
         $body = JSON::decode($message->getBody());
 
-        if (! isset($body['jobId'], $body['jobName'], $body['exportType'], $body['outputFormat'], $body['email'])) {
+        if (!isset($body['jobId'], $body['jobName'], $body['exportType'], $body['outputFormat'], $body['email'])) {
             $this->logger->critical('Invalid message');
         }
 
-        if (! ($job = $this->jobStorage->findJobById($body['jobId']))) {
+        if (!($job = $this->jobStorage->findJobById($body['jobId']))) {
             $this->logger->critical('Job not found');
 
             return self::REJECT;
@@ -95,7 +95,7 @@ 
         $files = [];
 
         foreach ($job->getChildJobs() as $childJob) {
-            if (! empty($childJob->getData()) && ($file = $childJob->getData()['file'])) {
+            if (!empty($childJob->getData()) && ($file = $childJob->getData()['file'])) {
                 $files[] = $file;
             }
         }

src/Oro/Bundle/ReminderBundle/DependencyInjection/OroReminderExtension.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
         $configuration = new Configuration();
         $config = $this->processConfiguration($configuration, $configs);
 
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
         $loader->load('form.yml');
         $loader->load('services.yml');
     }

src/Oro/Bundle/TestFrameworkBundle/Tests/Unit/Behat/Listener/JsLogSubscriberTest.php

Spacing +12 added lines, -12 removed lines

@@ -31,7 +31,7 @@ 
         $jsLogSubscriber->method('getUrl')->willReturn('example.com');
         $jsLogSubscriber->log($this->getEventMock());
 
-        $expectedLogFile = sys_get_temp_dir().DIRECTORY_SEPARATOR.'behat_browser.log';
+        $expectedLogFile = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'behat_browser.log';
         $this->assertFileExists($expectedLogFile);
         $this->assertStringEqualsFile($expectedLogFile, $expectedContent);
     }
@@ -47,7 +47,7 @@ 
         $jsLogSubscriber->method('getLogs')->willReturn([]);
         $jsLogSubscriber->log($this->getEventMock());
 
-        $expectedLogFile = sys_get_temp_dir().DIRECTORY_SEPARATOR.'behat_browser.log';
+        $expectedLogFile = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'behat_browser.log';
         $this->assertFileNotExists($expectedLogFile);
     }
 
@@ -68,12 +68,12 @@ 
                     ],
                 ],
                 'expected result' =>
-                    '[ERROR - 2017-07-10T19:04:21+00:00] [URL: "example.com"] '.
-                        '[Feature: "Feature Example", On line: 0, Step: "Test JsLogger Mock Step"] '.
-                        'Something went wrong'.PHP_EOL.
-                    '[WARNING - 2017-07-10T19:04:21+00:00] [URL: "example.com"] '.
-                        '[Feature: "Feature Example", On line: 0, Step: "Test JsLogger Mock Step"] '.
-                        'Check your environment'.PHP_EOL
+                    '[ERROR - 2017-07-10T19:04:21+00:00] [URL: "example.com"] ' .
+                        '[Feature: "Feature Example", On line: 0, Step: "Test JsLogger Mock Step"] ' .
+                        'Something went wrong' . PHP_EOL .
+                    '[WARNING - 2017-07-10T19:04:21+00:00] [URL: "example.com"] ' .
+                        '[Feature: "Feature Example", On line: 0, Step: "Test JsLogger Mock Step"] ' .
+                        'Check your environment' . PHP_EOL
             ],
             'irregular log' => [
                 'logs' => [
@@ -82,9 +82,9 @@ 
                     ],
                 ],
                 'expected result' =>
-                    '[UNKNOWN_LEVEL - 2017-07-10T19:04:21+00:00] [URL: "example.com"] '.
-                        '[Feature: "Feature Example", On line: 0, Step: "Test JsLogger Mock Step"] '.
-                        'UNKNOWN_MESSAGE'.PHP_EOL
+                    '[UNKNOWN_LEVEL - 2017-07-10T19:04:21+00:00] [URL: "example.com"] ' .
+                        '[Feature: "Feature Example", On line: 0, Step: "Test JsLogger Mock Step"] ' .
+                        'UNKNOWN_MESSAGE' . PHP_EOL
             ],
         ];
     }
@@ -94,7 +94,7 @@ 
      */
     public function removeLog()
     {
-        $log = sys_get_temp_dir().DIRECTORY_SEPARATOR.'behat_browser.log';
+        $log = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'behat_browser.log';
 
         if (file_exists($log)) {
             unlink($log);

src/Oro/Bundle/TestFrameworkBundle/Tests/Behat/Context/OroMainContext.php

Spacing +12 added lines, -12 removed lines

@@ -49,10 +49,10 @@ 
     SessionAliasProviderAwareInterface,
     MessageQueueIsolatorAwareInterface
 {
-    const SKIP_WAIT_PATTERN = '/'.
-        '^(?:|I )should see ".+" flash message$|'.
-        '^(?:|I )should see ".+" error message$|'.
-        '^(?:|I )should see Schema updated flash message$'.
+    const SKIP_WAIT_PATTERN = '/' .
+        '^(?:|I )should see ".+" flash message$|' .
+        '^(?:|I )should see ".+" error message$|' .
+        '^(?:|I )should see Schema updated flash message$' .
     '/';
 
     use AssertTrait, KernelDictionary, PageObjectDictionary, SessionAliasProviderAwareTrait;
@@ -232,7 +232,7 @@ 
         $actualFlashMessages = [];
         /** @var Element|null $flashMessage */
         $flashMessage = $this->spin(
-            function (OroMainContext $context) use ($title, &$actualFlashMessages, $flashMessageElement) {
+            function(OroMainContext $context) use ($title, &$actualFlashMessages, $flashMessageElement) {
                 $flashMessages = $context->findAllElements($flashMessageElement);
 
                 foreach ($flashMessages as $flashMessage) {
@@ -288,7 +288,7 @@ 
     public function iShouldSeeFollowingFlashMessages(TableNode $table, $strict = false)
     {
         $expectedMessages = array_map(
-            function ($item) {
+            function($item) {
                 return $item[0];
             },
             $table->getRows()
@@ -338,7 +338,7 @@ 
      */
     public function assertPageContainsText($text)
     {
-        $result = $this->spin(function (OroMainContext $context) use ($text) {
+        $result = $this->spin(function(OroMainContext $context) use ($text) {
             $context->assertSession()->pageTextContains($this->fixStepArgument($text));
 
             return true;
@@ -380,7 +380,7 @@ 
      */
     public function iShouldSeeErrorMessage($title)
     {
-        $errorElement = $this->spin(function (MinkContext $context) {
+        $errorElement = $this->spin(function(MinkContext $context) {
             return $context->getSession()->getPage()->find('css', '.alert-error');
         });
 
@@ -435,7 +435,7 @@ 
             return;
         }
 
-        self::fail('Expect to see no alert but alert with "'.$alertMessage.'" message is present');
+        self::fail('Expect to see no alert but alert with "' . $alertMessage . '" message is present');
     }
 
     /**
@@ -1099,7 +1099,7 @@ 
      */
     public function assertElementOnPage($element)
     {
-        $isVisible = $this->spin(function (OroMainContext $context) use ($element) {
+        $isVisible = $this->spin(function(OroMainContext $context) use ($element) {
             return $context->createElement($element)->isVisible();
         }, 3);
 
@@ -1115,7 +1115,7 @@ 
     public function assertElementNotOnPage($element)
     {
         $elementOnPage = $this->createElement($element);
-        $result = $this->spin(function (OroMainContext $context) use ($elementOnPage, $element) {
+        $result = $this->spin(function(OroMainContext $context) use ($elementOnPage, $element) {
             try {
                 return !$elementOnPage->isVisible();
             } catch (NoSuchElement $e) {
@@ -1789,7 +1789,7 @@ 
      */
     public function assertPageAddress($page)
     {
-        $this->spin(function () use ($page) {
+        $this->spin(function() use ($page) {
             parent::assertPageAddress($page);
         }, 10);
 

src/Oro/Bundle/TestFrameworkBundle/Test/WebTestCase.php

Spacing +6 added lines, -6 removed lines

@@ -118,7 +118,7 @@ 
     {
         if (!self::$resetCallback) {
             $self = $this;
-            self::$resetCallback = function () use ($self) {
+            self::$resetCallback = function() use ($self) {
                 $self->client = null;
                 $self->soapClient = null;
             };
@@ -355,7 +355,7 @@ 
         }
 
         if (is_array($data)) {
-            array_walk_recursive($data, function (&$item) use ($aliceLoader) {
+            array_walk_recursive($data, function(&$item) use ($aliceLoader) {
                 $item = $aliceLoader->getProcessor()->process($item, [], null);
             });
         } elseif (is_int($data) || is_string($data)) {
@@ -580,7 +580,7 @@ 
         }
 
         if (!$force) {
-            $removeLoadedFixturesCallback = function ($fixtureId) {
+            $removeLoadedFixturesCallback = function($fixtureId) {
                 return !in_array($fixtureId, self::$loadedFixtures, true);
             };
 
@@ -691,7 +691,7 @@ 
     {
         $oroDefaultPrefix = $this->getUrl('oro_default');
 
-        $replaceOroDefaultPrefixCallback = function (&$value) use ($oroDefaultPrefix, $urlParameterKey) {
+        $replaceOroDefaultPrefixCallback = function(&$value) use ($oroDefaultPrefix, $urlParameterKey) {
             if (!is_null($value[$urlParameterKey])) {
                 $value[$urlParameterKey] = str_replace(
                     '%oro_default_prefix%',
@@ -732,7 +732,7 @@ 
             }
         }
 
-        $replaceCallback = function (&$value) use ($randomString) {
+        $replaceCallback = function(&$value) use ($randomString) {
             if (!is_null($value)) {
                 $value = str_replace('%str%', $randomString, $value);
             }
@@ -1091,7 +1091,7 @@ 
             return $this->getCurrentDir() . DIRECTORY_SEPARATOR . $fileName;
         }
 
-        return $this->getCurrentDir() . DIRECTORY_SEPARATOR .  $folderName . DIRECTORY_SEPARATOR . $fileName;
+        return $this->getCurrentDir() . DIRECTORY_SEPARATOR . $folderName . DIRECTORY_SEPARATOR . $fileName;
     }
 
     /**

src/Oro/Bundle/TestFrameworkBundle/BehatStatisticExtension/Repository/StatisticRepository.php

Spacing +3 added lines, -3 removed lines

@@ -65,7 +65,7 @@ 
         }
         $ids = $buildIdsQueryBuilder->execute()->fetchAll();
 
-        $ids = array_map(function ($data) {
+        $ids = array_map(function($data) {
             return $data['build_id'];
         }, $ids);
         $ids = array_filter($ids);
@@ -155,7 +155,7 @@ 
             ->fetchAll()
         ;
 
-        $models = array_map(function (array $data) {
+        $models = array_map(function(array $data) {
             $model = $this->className::fromArray($data);
             $this->collection[$model->getId()] = $model;
 
@@ -177,7 +177,7 @@ 
         $result = $this->connection->createQueryBuilder()
             ->select('*')
             ->from($this->className::getName())
-            ->where($this->className::getIdField().' = ?')
+            ->where($this->className::getIdField() . ' = ?')
             ->setParameter(0, $id)
             ->orderBy('id', 'DESC')
             ->execute()

src/Oro/Bundle/TestFrameworkBundle/BehatStatisticExtension/Tests/Suite/SuiteConfigurationRegistryTest.php

Spacing +12 added lines, -12 removed lines

@@ -22,16 +22,16 @@ 
                 'type' => null,
                 'settings' => [
                     'extra_key' => 'extra_value',
-                    'paths' => array_map(function ($n) {
-                        return '/tmp/'.$n;
+                    'paths' => array_map(function($n) {
+                        return '/tmp/' . $n;
                     }, range(1, 3))
                 ],
             ],
             'AcmeSuite5' => [
                 'type' => null,
                 'settings' => [
-                    'paths' => array_map(function ($n) {
-                        return '/tmp/'.$n;
+                    'paths' => array_map(function($n) {
+                        return '/tmp/' . $n;
                     }, range(1, 5))
                 ],
             ],
@@ -62,7 +62,7 @@ 
             $this->assertArrayHasKey($setName, $expectedSets, sprintf('"%s" is not expected set name', $setName));
             $this->assertSame(
                 $expectedSets[$setName],
-                array_map(function (Suite $configuration) {
+                array_map(function(Suite $configuration) {
                     return $configuration->getName();
                 }, $configs)
             );
@@ -75,7 +75,7 @@ 
             [
                 'divider' => 10,
                 'expectedSets' => [
-                    SuiteConfigurationRegistry::PREFIX_SUITE_SET.'_0' => [
+                    SuiteConfigurationRegistry::PREFIX_SUITE_SET . '_0' => [
                         'AcmeSuite3_0',
                         'AcmeSuite3_1',
                         'AcmeSuite3_2',
@@ -90,13 +90,13 @@ 
             [
                 'divider' => 5,
                 'expectedSets' => [
-                    SuiteConfigurationRegistry::PREFIX_SUITE_SET.'_0' => [
+                    SuiteConfigurationRegistry::PREFIX_SUITE_SET . '_0' => [
                         'AcmeSuite5_2',
                         'AcmeSuite5_3',
                         'AcmeSuite5_4',
                         'AcmeSuite3_0',
                     ],
-                    SuiteConfigurationRegistry::PREFIX_SUITE_SET.'_1' => [
+                    SuiteConfigurationRegistry::PREFIX_SUITE_SET . '_1' => [
                         'AcmeSuite3_1',
                         'AcmeSuite3_2',
                         'AcmeSuite5_0',
@@ -107,17 +107,17 @@ 
             [
                 'divider' => 3,
                 'expectedSets' => [
-                    SuiteConfigurationRegistry::PREFIX_SUITE_SET.'_0' => [
+                    SuiteConfigurationRegistry::PREFIX_SUITE_SET . '_0' => [
                         'AcmeSuite3_0',
                         'AcmeSuite3_1',
                         'AcmeSuite3_2',
                     ],
-                    SuiteConfigurationRegistry::PREFIX_SUITE_SET.'_1' => [
+                    SuiteConfigurationRegistry::PREFIX_SUITE_SET . '_1' => [
                         'AcmeSuite5_3',
                         'AcmeSuite5_4',
                         'AcmeSuite5_0',
                     ],
-                    SuiteConfigurationRegistry::PREFIX_SUITE_SET.'_2' => [
+                    SuiteConfigurationRegistry::PREFIX_SUITE_SET . '_2' => [
                         'AcmeSuite5_1',
                         'AcmeSuite5_2',
                     ],
@@ -195,7 +195,7 @@ 
 
         $this->assertInternalType(
             'array',
-            $suiteConfigRegistry->getSet(SuiteConfigurationRegistry::PREFIX_SUITE_SET.'_0')
+            $suiteConfigRegistry->getSet(SuiteConfigurationRegistry::PREFIX_SUITE_SET . '_0')
         );
     }
 

src/Oro/Bundle/TestFrameworkBundle/BehatStatisticExtension/Tests/Specification/SpecificationDividerTest.php

Spacing +11 added lines, -11 removed lines

@@ -32,35 +32,35 @@
                 'array' => [1, 2, 3, 4, 5, 6, 7, 8, 9, 10],
                 'Suite divider' => 3,
                 'Expected result' => [
-                    self::BASE_NAME.'_0' => [1, 2, 3],
-                    self::BASE_NAME.'_1' => [4, 5, 6],
-                    self::BASE_NAME.'_2' => [10, 7],
-                    self::BASE_NAME.'_3' => [8, 9]
+                    self::BASE_NAME . '_0' => [1, 2, 3],
+                    self::BASE_NAME . '_1' => [4, 5, 6],
+                    self::BASE_NAME . '_2' => [10, 7],
+                    self::BASE_NAME . '_3' => [8, 9]
                 ],
             ],
             [
                 'array' => [1, 2, 3, 4],
                 'Suite divider' => 3,
                 'Expected result' => [
-                    self::BASE_NAME.'_0' => [4, 1],
-                    self::BASE_NAME.'_1' => [2, 3],
+                    self::BASE_NAME . '_0' => [4, 1],
+                    self::BASE_NAME . '_1' => [2, 3],
                 ],
             ],
             [
                 'array' => [1, 2, 3, 4],
                 'Suite divider' => 1,
                 'Expected result' => [
-                    self::BASE_NAME.'_0' => [1],
-                    self::BASE_NAME.'_1' => [2],
-                    self::BASE_NAME.'_2' => [3],
-                    self::BASE_NAME.'_3' => [4],
+                    self::BASE_NAME . '_0' => [1],
+                    self::BASE_NAME . '_1' => [2],
+                    self::BASE_NAME . '_2' => [3],
+                    self::BASE_NAME . '_3' => [4],
                 ]
             ],
             [
                 'array' => [1, 2, 3, 4, 5],
                 'Suite divider' => 7,
                 'Expected result' => [
-                    self::BASE_NAME.'_0' => [1, 2, 3, 4, 5]
+                    self::BASE_NAME . '_0' => [1, 2, 3, 4, 5]
                 ]
             ],
         ];

src/Oro/Bundle/TestFrameworkBundle/BehatStatisticExtension/Features/Context/StatisticContext.php

Spacing +15 added lines, -15 removed lines

@@ -42,7 +42,7 @@ 
      */
     public function prepareProcess()
     {
-        $this->testAppPath = realpath(__DIR__.'/../../TestApp');
+        $this->testAppPath = realpath(__DIR__ . '/../../TestApp');
         $phpFinder = new PhpExecutableFinder();
         if (false === $php = $phpFinder->find()) {
             throw new \RuntimeException('Unable to find the PHP executable.');
@@ -54,13 +54,13 @@ 
     /** @BeforeScenario */
     public function prepareBehatYaml()
     {
-        $behatYaml = $this->testAppPath.'/behat.yml';
+        $behatYaml = $this->testAppPath . '/behat.yml';
 
         if (file_exists($behatYaml)) {
             unlink($behatYaml);
         }
 
-        copy($this->testAppPath.'/behat.yml.dist', $behatYaml);
+        copy($this->testAppPath . '/behat.yml.dist', $behatYaml);
     }
 
     /**
@@ -82,7 +82,7 @@ 
                 escapeshellarg(BEHAT_BIN_PATH),
                 $argumentsString,
                 strtr('--format-settings=\'{"timer": false}\'', ['\'' => '"', '"' => '\"']),
-                $this->testAppPath.'/behat.yml'
+                $this->testAppPath . '/behat.yml'
             )
         );
         $this->process->start();
@@ -94,12 +94,12 @@ 
      */
     public function enabledExtensionInBehatYml(PyStringNode $extensionConfig)
     {
-        $config = Yaml::parse(file_get_contents($this->testAppPath.'/behat.yml'));
+        $config = Yaml::parse(file_get_contents($this->testAppPath . '/behat.yml'));
         $extensionConfig = Yaml::parse($extensionConfig->getRaw());
         $config['default']['extensions'] = $extensionConfig;
         $this->dbConfig = array_shift($extensionConfig)['connection'];
 
-        file_put_contents($this->testAppPath.'/behat.yml', Yaml::dump($config, 6));
+        file_put_contents($this->testAppPath . '/behat.yml', Yaml::dump($config, 6));
     }
 
     /**
@@ -107,11 +107,11 @@ 
      */
     public function enabledSuitesInBehatYml(PyStringNode $suitesConfig)
     {
-        $config = Yaml::parse(file_get_contents($this->testAppPath.'/behat.yml'));
+        $config = Yaml::parse(file_get_contents($this->testAppPath . '/behat.yml'));
         $suitesConfig = Yaml::parse($suitesConfig->getRaw());
         $config['default']['suites'] = $suitesConfig;
 
-        file_put_contents($this->testAppPath.'/behat.yml', Yaml::dump($config, 6));
+        file_put_contents($this->testAppPath . '/behat.yml', Yaml::dump($config, 6));
     }
 
     /**
@@ -206,7 +206,7 @@ 
      */
     public function sqliteDatabaseExists($dbname)
     {
-        $dbFile = $this->testAppPath.'/'.$dbname;
+        $dbFile = $this->testAppPath . '/' . $dbname;
 
         if (file_exists($dbFile)) {
             unlink($dbFile);
@@ -225,7 +225,7 @@ 
             throw new RuntimeException('No db name in configuration');
         }
 
-        $dbName =  $this->dbConfig['dbname'];
+        $dbName = $this->dbConfig['dbname'];
         $dbConfig = $this->dbConfig;
         $dbConfig['dbname'] = null;
 
@@ -290,21 +290,21 @@ 
         if ('/' !== DIRECTORY_SEPARATOR) {
             $text = preg_replace_callback(
                 '/ features\/[^\n ]+/',
-                function ($matches) {
+                function($matches) {
                     return str_replace('/', DIRECTORY_SEPARATOR, $matches[0]);
                 },
                 $text
             );
             $text = preg_replace_callback(
                 '/\<span class\="path"\>features\/[^\<]+/',
-                function ($matches) {
+                function($matches) {
                     return str_replace('/', DIRECTORY_SEPARATOR, $matches[0]);
                 },
                 $text
             );
             $text = preg_replace_callback(
                 '/\+[fd] [^ ]+/',
-                function ($matches) {
+                function($matches) {
                     return str_replace('/', DIRECTORY_SEPARATOR, $matches[0]);
                 },
                 $text
@@ -333,10 +333,10 @@ 
      */
     public function iReconfigureStatisticExtension(PyStringNode $extensionConfig)
     {
-        $config = Yaml::parse(file_get_contents($this->testAppPath.'/behat.yml'));
+        $config = Yaml::parse(file_get_contents($this->testAppPath . '/behat.yml'));
         $extensionConfig = Yaml::parse($extensionConfig->getRaw());
         $config['default']['extensions'][BehatStatisticExtension::class] = $extensionConfig;
 
-        file_put_contents($this->testAppPath.'/behat.yml', Yaml::dump($config, 6));
+        file_put_contents($this->testAppPath . '/behat.yml', Yaml::dump($config, 6));
     }
 }