Inneair / transaction-bundle

src/Aop/TransactionalInterceptor.php

Indentation +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
             // have been invoked.
             $this->logger->warning(
                 'Transactional interceptor was invoked, but no annotation was found for method \'' .
-                 $methodDefinition->getDeclaringClass()->getName() . '::' . $methodDefinition->getName() . '\''
+                    $methodDefinition->getDeclaringClass()->getName() . '::' . $methodDefinition->getName() . '\''
             );
         }
 

Spacing +10 added lines, -10 removed lines

@@ -88,7 +88,7 @@ 
             $policy = Transactional::NOT_REQUIRED;
         } elseif ($annotation->getPolicy() === null) {
             $policy = $this->container->getParameter(
-                Configuration::ROOT_NODE_NAME . '.' . Configuration::DEFAULT_POLICY);
+                Configuration::ROOT_NODE_NAME.'.'.Configuration::DEFAULT_POLICY);
         } else {
             $policy = $annotation->getPolicy();
         }
@@ -97,8 +97,8 @@ 
             // No annotation found: there is probably a bug in the pointcut class, because the interceptor should not
             // have been invoked.
             $this->logger->warning(
-                'Transactional interceptor was invoked, but no annotation was found for method \'' .
-                 $methodDefinition->getDeclaringClass()->getName() . '::' . $methodDefinition->getName() . '\''
+                'Transactional interceptor was invoked, but no annotation was found for method \''.
+                 $methodDefinition->getDeclaringClass()->getName().'::'.$methodDefinition->getName().'\''
             );
         }
 
@@ -115,7 +115,7 @@ 
         try {
             // Invokes the method.
             $this->logger->debug(
-                $methodDefinition->getDeclaringClass()->getName() . '::' . $methodDefinition->getName()
+                $methodDefinition->getDeclaringClass()->getName().'::'.$methodDefinition->getName()
             );
             $result = $method->proceed();
             $this->afterMethodInvocationSuccess($transactionRequired);
@@ -127,7 +127,7 @@ 
             } elseif ($annotation->getNoRollbackExceptions() === null) {
                 // No exceptions set in the annotation (even if the parameter was found), use the default configuration.
                 $noRollbackExceptions = $this->container->getParameter(
-                    Configuration::ROOT_NODE_NAME . '.' . Configuration::NO_ROLLBACK_EXCEPTIONS
+                    Configuration::ROOT_NODE_NAME.'.'.Configuration::NO_ROLLBACK_EXCEPTIONS
                 );
             } else {
                 // Use the annotation parameter.
@@ -166,11 +166,11 @@ 
         if ($transactionRequired) {
             if ($this->isRollbackEnabled($e, $noRollbackExceptions)) {
                 // Rollbacks the transaction.
-                $this->logger->debug('Exception ' . get_class($e) . ' causes rollback');
+                $this->logger->debug('Exception '.get_class($e).' causes rollback');
                 $this->rollback();
             } else {
                 // Commits the transaction.
-                $this->logger->debug('No rollback for exception ' . get_class($e));
+                $this->logger->debug('No rollback for exception '.get_class($e));
                 $this->commit();
             }
         }
@@ -209,7 +209,7 @@ 
      */
     protected function beginTransaction()
     {
-        $this->logger->debug(static::class . '::beginTransaction');
+        $this->logger->debug(static::class.'::beginTransaction');
         $this->getEntityManager()->beginTransaction();
     }
 
@@ -218,7 +218,7 @@ 
      */
     protected function commit()
     {
-        $this->logger->debug(static::class . '::commit');
+        $this->logger->debug(static::class.'::commit');
         $this->getEntityManager()->flush();
         $this->getEntityManager()->commit();
     }
@@ -230,7 +230,7 @@ 
      */
     protected function rollback()
     {
-        $this->logger->debug(static::class . '::rollback');
+        $this->logger->debug(static::class.'::rollback');
         $this->getEntityManager()->rollback();
 
         // Close the manager if there is no transaction started.

src/Aop/TransactionalPointcut.php

Switch Indentation +11 added lines, -11 removed lines

@@ -96,17 +96,17 @@
 
             if ($transactionalEnabled) {
                 switch ($annotation->getPolicy()) {
-                    case Transactional::NOT_REQUIRED:
-                        $policyName = 'not required';
-                        break;
-                    case Transactional::REQUIRED:
-                        $policyName = 'required';
-                        break;
-                    case Transactional::NESTED:
-                        $policyName = 'nested';
-                        break;
-                    default:
-                        $policyName = 'default';
+                case Transactional::NOT_REQUIRED:
+                    $policyName = 'not required';
+                    break;
+                case Transactional::REQUIRED:
+                    $policyName = 'required';
+                    break;
+                case Transactional::NESTED:
+                    $policyName = 'nested';
+                    break;
+                default:
+                    $policyName = 'default';
                 }
                 $methodString = $method->getDeclaringClass()->name . '::' . $method->name;
                 $this->logger->debug('TX policy for \'' . $methodString . '\': ' . $policyName);

Spacing +3 added lines, -3 removed lines

@@ -108,8 +108,8 @@ 
                     default:
                         $policyName = 'default';
                 }
-                $methodString = $method->getDeclaringClass()->name . '::' . $method->name;
-                $this->logger->debug('TX policy for \'' . $methodString . '\': ' . $policyName);
+                $methodString = $method->getDeclaringClass()->name.'::'.$method->name;
+                $this->logger->debug('TX policy for \''.$methodString.'\': '.$policyName);
                 $noRollbackExceptionsStr = implode(
                     ', ',
                     ($annotation->getNoRollbackExceptions() === null)
@@ -117,7 +117,7 @@ 
                         : $annotation->getNoRollbackExceptions()
                 );
                 $this->logger->debug(
-                    'TX no-rollback exceptions for \'' . $methodString . '\': ' . $noRollbackExceptionsStr
+                    'TX no-rollback exceptions for \''.$methodString.'\': '.$noRollbackExceptionsStr
                 );
             }
         }

src/Annotation/Transactional.php

Spacing +4 added lines, -4 removed lines

@@ -138,8 +138,8 @@ 
             $policy = $annotationPolicy;
         } else {
             throw new AnnotationException(
-                'Invalid policy: "' . $annotationPolicy . '", must be one of the constants [' .
-                implode(', ', $policies) . ']'
+                'Invalid policy: "'.$annotationPolicy.'", must be one of the constants ['.
+                implode(', ', $policies).']'
             );
         }
         return $policy;
@@ -159,11 +159,11 @@ 
             try {
                 $exceptionClass = new ReflectionClass($exceptionClassName);
             } catch (ReflectionException $e) {
-                throw new AnnotationException('Class not found: \'' . $exceptionClassName . '\'', null, $e);
+                throw new AnnotationException('Class not found: \''.$exceptionClassName.'\'', null, $e);
             }
 
             if (($exceptionClassName !== Exception::class) && !$exceptionClass->isSubclassOf(Exception::class)) {
-                throw new AnnotationException('Not an exception: \'' . $exceptionClassName . '\'');
+                throw new AnnotationException('Not an exception: \''.$exceptionClassName.'\'');
             }
 
             $noRollbackExceptions[] = $exceptionClassName;

src/DependencyInjection/InneairTransactionExtension.php

Switch Indentation +13 added lines, -13 removed lines

@@ -54,19 +54,19 @@
         );
 
         switch ($config[Configuration::DEFAULT_POLICY]) {
-            case Configuration::POLICY_NOT_REQUIRED:
-                $policy = Transactional::NOT_REQUIRED;
-                break;
-            case Configuration::POLICY_REQUIRED:
-                $policy = Transactional::REQUIRED;
-                break;
-            case Configuration::POLICY_NESTED:
-                $policy = Transactional::NESTED;
-                break;
-            default:
-                throw new InvalidArgumentException(
-                    'Unsupported default policy "' . $config[Configuration::DEFAULT_POLICY] . '"'
-                );
+        case Configuration::POLICY_NOT_REQUIRED:
+            $policy = Transactional::NOT_REQUIRED;
+            break;
+        case Configuration::POLICY_REQUIRED:
+            $policy = Transactional::REQUIRED;
+            break;
+        case Configuration::POLICY_NESTED:
+            $policy = Transactional::NESTED;
+            break;
+        default:
+            throw new InvalidArgumentException(
+                'Unsupported default policy "' . $config[Configuration::DEFAULT_POLICY] . '"'
+            );
         }
         $container->setParameter(Configuration::ROOT_NODE_NAME . '.' . Configuration::DEFAULT_POLICY, $policy);
 

Spacing +8 added lines, -8 removed lines

@@ -43,13 +43,13 @@ 
      */
     public function load(array $configs, ContainerBuilder $container)
     {
-        $loader = new YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
+        $loader = new YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('services.yml');
 
         $config = $this->processConfiguration($this->getConfiguration($configs, $container), $configs);
 
         $container->setParameter(
-            Configuration::ROOT_NODE_NAME . '.' . Configuration::STRICT_MODE,
+            Configuration::ROOT_NODE_NAME.'.'.Configuration::STRICT_MODE,
             $config[Configuration::STRICT_MODE]
         );
 
@@ -65,25 +65,25 @@ 
                 break;
             default:
                 throw new InvalidArgumentException(
-                    'Unsupported default policy "' . $config[Configuration::DEFAULT_POLICY] . '"'
+                    'Unsupported default policy "'.$config[Configuration::DEFAULT_POLICY].'"'
                 );
         }
-        $container->setParameter(Configuration::ROOT_NODE_NAME . '.' . Configuration::DEFAULT_POLICY, $policy);
+        $container->setParameter(Configuration::ROOT_NODE_NAME.'.'.Configuration::DEFAULT_POLICY, $policy);
 
         $noRollbackExceptions = array_unique($config[Configuration::NO_ROLLBACK_EXCEPTIONS]);
         foreach ($noRollbackExceptions as $exceptionClassName) {
             try {
                 $exceptionClass = new ReflectionClass($exceptionClassName);
             } catch (ReflectionException $e) {
-                throw new InvalidArgumentException('Class not found: \'' . $exceptionClassName . '\'', null, $e);
+                throw new InvalidArgumentException('Class not found: \''.$exceptionClassName.'\'', null, $e);
             }
 
             if (($exceptionClassName !== Exception::class) && !$exceptionClass->isSubclassOf(Exception::class)) {
-                throw new InvalidArgumentException('Not an exception: \'' . $exceptionClassName . '\'');
+                throw new InvalidArgumentException('Not an exception: \''.$exceptionClassName.'\'');
             }
         }
         $container->setParameter(
-            Configuration::ROOT_NODE_NAME . '.' . Configuration::NO_ROLLBACK_EXCEPTIONS,
+            Configuration::ROOT_NODE_NAME.'.'.Configuration::NO_ROLLBACK_EXCEPTIONS,
             $noRollbackExceptions
         );
     }
@@ -93,6 +93,6 @@ 
      */
     public function getXsdValidationBasePath()
     {
-        return __DIR__ . '/../Resources/config/schema';
+        return __DIR__.'/../Resources/config/schema';
     }
 }