GlobalTradingTechnologies / workflow-extensions-bundle

Schedule/ScheduledTransition.php

Doc Comments +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
     /**
      * Returns offset date interval
      *
-     * @return DateInterval
+     * @return string
      */
     public function getOffset()
     {

Tests/Functional/Kernel/BaseTestKernel.php

Doc Comments +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
     /**
      * Returns base bundle list
      *
-     * @return array
+     * @return \Symfony\Bundle\FrameworkBundle\FrameworkBundle[]
      */
     protected function getBaseBundles()
     {

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

Doc Comments +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
 
     /**
      * @param ContainerInterface $container
-     * @param array|string|false $fixturesPath
+     * @param string $fixturesPath
      * @param string $em
      * @param bool|true $append
      * @return array

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
             )

TransitionApplier.php

Doc Comments +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
      *
      * @param object $subject      subject
      * @param string $workflowName workflow name
-     * @param string $transitions  transition to be applied
+     * @param string $transition  transition to be applied
      */
     public function applyTransition($subject, $workflowName, $transition)
     {

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/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

Indentation +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
                                             ->isRequired()
                                             ->cannotBeEmpty()
                                             ->info('Holds period defines offset from time of event catching for transition scheduling. '.
-                                             'See https://en.wikipedia.org/wiki/ISO_8601#Durations for format description')
+                                                'See https://en.wikipedia.org/wiki/ISO_8601#Durations for format description')
                                             ->validate()
                                                 ->always()
                                                 ->then(function ($v) {

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']);