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/Configuration.php

Spacing +4 added lines, -4 removed lines

@@ -77,7 +77,7 @@ 
                                 ->defaultValue([])
                                 ->beforeNormalization()
                                     ->ifString()
-                                    ->then(function ($v) { return array($v); })
+                                    ->then(function($v) { return array($v); })
                                 ->end()
                                 ->useAttributeAsKey('name')
                                 ->prototype('scalar')
@@ -98,7 +98,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);
 
@@ -156,7 +156,7 @@ 
                     ->prototype('array')
                         ->beforeNormalization()
                             ->ifString()
-                            ->then(function ($v) { return ['expression' => $v]; })
+                            ->then(function($v) { return ['expression' => $v]; })
                         ->end()
                         ->children()
                             ->scalarNode('expression')
@@ -187,7 +187,7 @@ 
             ->beforeNormalization()
                 ->always()
                 ->then(
-                    function ($v) {
+                    function($v) {
                         if (isset($v) && is_array($v)) {
                             foreach ($v as $key => $value) {
                                 unset($v[$key]);

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/TransitionScheduler.php

Spacing +4 added lines, -4 removed lines

@@ -84,10 +84,10 @@
 
             $transitionTriggerJobToSchedule = new Job('workflow:transition:trigger',
                 [
-                    '--transition='   . $transitionName,
-                    '--workflow='     . $workflowName,
-                    '--subjectClass=' . $subjectClass,
-                    '--subjectId='    . $subjectId,
+                    '--transition='.$transitionName,
+                    '--workflow='.$workflowName,
+                    '--subjectClass='.$subjectClass,
+                    '--subjectId='.$subjectId,
                 ]
             );
 

Tests/Schedule/TransitionSchedulerTest.php

Spacing +7 added lines, -7 removed lines

@@ -66,10 +66,10 @@ 
         $transition = new ScheduledTransition('t1', 'PT1S');
 
         $transitionTriggerJobArgs = [
-            '--transition='   . $transition->getTransitionName(),
-            '--workflow='     . $workflowName,
-            '--subjectClass=' . $subjectClass,
-            '--subjectId='    . $subjectId,
+            '--transition='.$transition->getTransitionName(),
+            '--workflow='.$workflowName,
+            '--subjectClass='.$subjectClass,
+            '--subjectId='.$subjectId,
         ];
 
         Carbon::setTestNow(Carbon::now());
@@ -79,14 +79,14 @@ 
         /** @var $em \PHPUnit_Framework_MockObject_MockObject|ObjectManager */
         $em->expects(self::exactly(2))->method('persist')->withConsecutive(
             [self::callback(
-                function (Job $job) use ($transitionTriggerJobArgs, $executeJobAfter) {
+                function(Job $job) use ($transitionTriggerJobArgs, $executeJobAfter) {
                     return
                         $job->getArgs() == $transitionTriggerJobArgs &&
                         $job->getExecuteAfter() == $executeJobAfter;
                 }
             )],
             [self::callback(
-                function (ScheduledJob $job) {
+                function(ScheduledJob $job) {
                     return
                         $job->getReschedulable() == true && $job->getJob() instanceof Job;
                 }
@@ -120,7 +120,7 @@ 
         /** @var $em \PHPUnit_Framework_MockObject_MockObject|ObjectManager */
         $em->expects(self::once())->method('persist')->withConsecutive(
             [self::callback(
-                function (Job $job) use ($newExecuteJobAfter) {
+                function(Job $job) use ($newExecuteJobAfter) {
                     return $job->getExecuteAfter() == $newExecuteJobAfter;
                 }
             )]