GlobalTradingTechnologies / workflow-extensions-bundle

Tests/Guard/ExpressionGuardTest.php

Spacing +2 added lines, -2 removed lines

@@ -76,7 +76,7 @@ 
         $event->expects(self::never())->method("setBlocked");
 
         $guard->registerGuardExpression("eventName", "workflow", "expression");
-        $guard->guardTransition($event ,"eventName");
+        $guard->guardTransition($event, "eventName");
     }
 
     /**
@@ -113,7 +113,7 @@ 
         $logger->expects($this->exactly($loggerInvocationCount))->method("debug");
 
         $guard->registerGuardExpression("eventName", "workflow", $expression);
-        $guard->guardTransition($event ,"eventName");
+        $guard->guardTransition($event, "eventName");
     }
 
     public function expressionProvider()

Tests/Functional/Kernel/BaseTestKernel.php

Spacing +3 added lines, -3 removed lines

@@ -65,14 +65,14 @@ 
         $this->setRootDir($rootDir);
 
         $this->configDir = realpath($configDir);
-        if (!is_dir($this->configDir . '/' . $testCase)) {
+        if (!is_dir($this->configDir.'/'.$testCase)) {
             throw new \InvalidArgumentException(sprintf('The test case "%s" does not exist.', $testCase));
         }
         $this->testCase = $testCase;
 
         $fs = new Filesystem();
         if (!$fs->isAbsolutePath($rootConfig) &&
-            !file_exists($rootConfig = $this->configDir . '/' . $testCase . '/' . $rootConfig)) {
+            !file_exists($rootConfig = $this->configDir.'/'.$testCase.'/'.$rootConfig)) {
             throw new \InvalidArgumentException(sprintf('The root config "%s" does not exist.', $rootConfig));
         }
         $this->rootConfig = $rootConfig;
@@ -127,7 +127,7 @@ 
     {
         $bundles     = array();
         $baseBundles = $this->getBaseBundles();
-        if (file_exists($filename = $this->configDir . '/' . $this->testCase . '/bundles.php')) {
+        if (file_exists($filename = $this->configDir.'/'.$this->testCase.'/bundles.php')) {
             $bundles = include $filename;
         }
         return array_merge($baseBundles, $bundles);

Tests/Functional/ScheduleCaseTest.php

Spacing +3 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
 
         // load fixtures
         $fixturesBundle = new ClientBundle();
-        $fixturesPath = $fixturesBundle->getPath() . '/DataFixtures/ORM';
+        $fixturesPath = $fixturesBundle->getPath().'/DataFixtures/ORM';
         $this->loadFixtures($this->client->getContainer(), $fixturesPath);
     }
 
@@ -67,8 +67,8 @@ 
                 "app_name"    => "ScheduleCaseTest",
                 "test_case"   => "ScheduleCase",
                 "root_config" => "config.yml",
-                "config_dir"  => __DIR__ . "/Configuration",
-                "root_dir"    => __DIR__ . "/Configuration/ScheduleCase",
+                "config_dir"  => __DIR__."/Configuration",
+                "root_dir"    => __DIR__."/Configuration/ScheduleCase",
                 "environment" => "test",
                 "debug"       => false
             )

Tests/Functional/EventTriggerWithGuardsCaseTest.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
                 "app_name" => "EventTriggerWithGuardsCaseTest",
                 "test_case" => "EventTriggerWithGuardsCase",
                 "root_config" => "config.yml",
-                "config_dir" => __DIR__ . "/Configuration",
+                "config_dir" => __DIR__."/Configuration",
                 "environment" => "test",
                 "debug" => false
             )

DependencyInjection/Compiler/CheckSubjectManipulatorConfigPass.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
     /**
      * Workflow registry definition id
      */
-    const WORKFLOW_REGISTRY_ID= 'workflow.registry';
+    const WORKFLOW_REGISTRY_ID = 'workflow.registry';
 
     /**
      * Workflow id prefix used in main workflow bundle
@@ -71,7 +71,7 @@ 
                 $workflowReference      = $callArgs[0];
                 $workflowSupportedClass = $callArgs[1];
 
-                $workflowIdWithPrefix =  (string) $workflowReference;
+                $workflowIdWithPrefix = (string) $workflowReference;
                 if (substr($workflowIdWithPrefix, 0, strlen(self::WORKFLOW_ID_PREFIX)) != self::WORKFLOW_ID_PREFIX) {
                     throw new RuntimeException(
                         sprintf(

DependencyInjection/WorkflowExtensionsExtension.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     {
         $configuration = $this->getConfiguration($config, $container);
         $config = $this->processConfiguration($configuration, $config);
-        $loader = new XmlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
+        $loader = new XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('applier.xml');
 
         $this->registerSubjectManipulatorConfiguration($loader, $container, $config['subject_manipulator']);

Schedule/ActionScheduler.php

Spacing +5 added lines, -5 removed lines

@@ -66,11 +66,11 @@
 
         $jobToSchedule = new Job(ExecuteActionCommand::COMMAND_NAME,
             [
-                '--action='       . $scheduledAction->getName(),
-                '--arguments='    . json_encode($scheduledAction->getArguments()),
-                '--workflow='     . $workflowContext->getWorkflow()->getName(),
-                '--subjectClass=' . get_class($workflowContext->getSubject()),
-                '--subjectId='    . $workflowContext->getSubjectId(),
+                '--action='.$scheduledAction->getName(),
+                '--arguments='.json_encode($scheduledAction->getArguments()),
+                '--workflow='.$workflowContext->getWorkflow()->getName(),
+                '--subjectClass='.get_class($workflowContext->getSubject()),
+                '--subjectId='.$workflowContext->getSubjectId(),
             ]
         );
 

DependencyInjection/Configuration.php

Spacing +13 added lines, -13 removed lines

@@ -94,7 +94,7 @@ 
             ->end()
             ->validate()
                 ->always()
-                ->then(function ($v) {
+                ->then(function($v) {
                     foreach ($v as $actionName => $actionConfig) {
                         if (!preg_match('/^[a-z0-9_]+$/i', $actionName)) {
                             throw new InvalidConfigurationException(
@@ -192,7 +192,7 @@ 
                                 ->defaultValue([])
                                 ->beforeNormalization()
                                     ->ifString()
-                                    ->then(function ($v) { return array($v); })
+                                    ->then(function($v) { return array($v); })
                                 ->end()
                                 ->useAttributeAsKey('name')
                                 ->prototype('array')
@@ -202,16 +202,16 @@ 
                                         ->end()
                                         ->beforeNormalization()
                                             ->ifString()
-                                            ->then(function ($v) {
+                                            ->then(function($v) {
                                                 return ['arguments' => [$v]];
                                             })
                                         ->end()
                                         ->beforeNormalization()
-                                            ->ifTrue(function ($v) {
+                                            ->ifTrue(function($v) {
                                                 // place arguments under 'arguments' key in order to validate it in common way
                                                 return is_array($v) && !(count($v) == 1 && array_key_exists('arguments', $v));
                                             })
-                                            ->then(function ($v) {
+                                            ->then(function($v) {
                                                 return ['arguments' => $v];
                                             })
                                         ->end()
@@ -290,10 +290,10 @@ 
                     ->end()
                 ->end()
                 ->beforeNormalization()
-                    ->ifTrue(function ($v) {
+                    ->ifTrue(function($v) {
                         return is_scalar($v) || is_array($v) && !ArrayUtils::isArrayAssoc($v);
                     })
-                    ->then(function ($v) {
+                    ->then(function($v) {
                         $type = is_scalar($v) ? ActionArgumentTypes::TYPE_SCALAR : ActionArgumentTypes::TYPE_ARRAY;
                         return [
                             'type'  => $type,
@@ -302,10 +302,10 @@ 
                     })
                 ->end()
                 ->validate()
-                    ->ifTrue(function ($v) {
+                    ->ifTrue(function($v) {
                         return is_array($v) && $v['type'] == ActionArgumentTypes::TYPE_ARRAY;
                     })
-                    ->then(function ($v) {
+                    ->then(function($v) {
                         // recursive processing of array values
                         $treeBuilder = new TreeBuilder();
                         $node = $treeBuilder->root('value');
@@ -319,7 +319,7 @@ 
                 ->end()
             ->end()
             ->validate()
-                ->ifTrue(function ($v) {
+                ->ifTrue(function($v) {
                     return ArrayUtils::isArrayAssoc($v, false);
                 })
                 // To provide cross-platform arguments handling we are not supporting here assoc arrays
@@ -366,7 +366,7 @@ 
              'See https://en.wikipedia.org/wiki/ISO_8601#Durations for format description')
             ->validate()
                 ->always()
-                ->then(function ($v) {
+                ->then(function($v) {
                     try {
                         new DateInterval($v);
 
@@ -407,7 +407,7 @@ 
                     ->prototype('array')
                         ->beforeNormalization()
                             ->ifString()
-                            ->then(function ($v) { return ['expression' => $v]; })
+                            ->then(function($v) { return ['expression' => $v]; })
                         ->end()
                         ->children()
                             ->scalarNode('expression')
@@ -438,7 +438,7 @@ 
             ->beforeNormalization()
                 ->always()
                 ->then(
-                    function ($v) {
+                    function($v) {
                         if (isset($v) && is_array($v)) {
                             foreach ($v as $key => $value) {
                                 unset($v[$key]);

Trigger/Event/SchedulerListener.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
 
         foreach ($actions as $scheduledAction) {
             $this->executeSafely(
-                function () use ($workflowContext, $scheduledAction) {
+                function() use ($workflowContext, $scheduledAction) {
                     $this->actionScheduler->scheduleAction($workflowContext, $scheduledAction);
                 },
                 $eventName,