spiral / framework

src/Security/src/Matcher.php

Spacing +2 added lines, -2 removed lines

@@ -40,10 +40,10 @@
      */
     public function matches(string $string, string $pattern): bool
     {
-        if ($string === $pattern) {
+        if ($string === $pattern){
             return true;
         }
-        if (!$this->isPattern($pattern)) {
+        if (!$this->isPattern($pattern)){
             return false;
         }
         return (bool)preg_match($this->getRegex($pattern), $string);

Braces +4 added lines, -2 removed lines

@@ -40,10 +40,12 @@
      */
     public function matches(string $string, string $pattern): bool
     {
-        if ($string === $pattern) {
+        if ($string === $pattern)
+        {
             return true;
         }
-        if (!$this->isPattern($pattern)) {
+        if (!$this->isPattern($pattern))
+        {
             return false;
         }
         return (bool)preg_match($this->getRegex($pattern), $string);

src/Security/src/Rule.php

Spacing +5 added lines, -5 removed lines

@@ -63,20 +63,20 @@
         $parameters = compact('actor', 'permission', 'context') + $context;
 
         //Mounting aliases
-        foreach (static::ALIASES as $target => $alias) {
+        foreach (static::ALIASES as $target => $alias){
             $parameters[$target] = $parameters[$alias];
         }
 
-        try {
+        try{
             $method = new \ReflectionMethod($this, static::CHECK_METHOD);
             $method->setAccessible(true);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             throw new RuleException($e->getMessage(), $e->getCode(), $e);
         }
 
-        try {
+        try{
             return $method->invokeArgs($this, $this->resolver->resolveArguments($method, $parameters));
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             throw new RuleException(sprintf('[%s] %s', get_class($this), $e->getMessage()), $e->getCode(), $e);
         }
     }

Braces +12 added lines, -5 removed lines

@@ -63,20 +63,27 @@
         $parameters = compact('actor', 'permission', 'context') + $context;
 
         //Mounting aliases
-        foreach (static::ALIASES as $target => $alias) {
+        foreach (static::ALIASES as $target => $alias)
+        {
             $parameters[$target] = $parameters[$alias];
         }
 
-        try {
+        try
+        {
             $method = new \ReflectionMethod($this, static::CHECK_METHOD);
             $method->setAccessible(true);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             throw new RuleException($e->getMessage(), $e->getCode(), $e);
         }
 
-        try {
+        try
+        {
             return $method->invokeArgs($this, $this->resolver->resolveArguments($method, $parameters));
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             throw new RuleException(sprintf('[%s] %s', get_class($this), $e->getMessage()), $e->getCode(), $e);
         }
     }

src/Security/src/RuleManager.php

Spacing +16 added lines, -16 removed lines

@@ -42,11 +42,11 @@ 
      */
     public function set(string $name, $rule = null): RuleManager
     {
-        if (empty($rule)) {
+        if (empty($rule)){
             $rule = $name;
         }
 
-        if (!$this->validateRule($rule)) {
+        if (!$this->validateRule($rule)){
             throw new RuleException("Unable to set rule '{$name}', invalid rule body");
         }
 
@@ -62,7 +62,7 @@ 
      */
     public function remove(string $name): RuleManager
     {
-        if (!$this->has($name)) {
+        if (!$this->has($name)){
             throw new RuleException("Undefined rule '{$name}'");
         }
 
@@ -76,11 +76,11 @@ 
      */
     public function has(string $name): bool
     {
-        if (isset($this->rules[$name])) {
+        if (isset($this->rules[$name])){
             return true;
         }
 
-        if (class_exists($name)) {
+        if (class_exists($name)){
             //We are allowing to use class names without direct registration
             return true;
         }
@@ -94,26 +94,26 @@ 
      */
     public function get(string $name): RuleInterface
     {
-        if (!$this->has($name)) {
+        if (!$this->has($name)){
             throw new RuleException("Undefined rule '{$name}'");
         }
 
-        if (!isset($this->rules[$name])) {
+        if (!isset($this->rules[$name])){
             //Rule represented as class name
             $rule = $name;
-        } else {
+        }else{
             $rule = $this->rules[$name];
         }
 
-        if ($rule instanceof RuleInterface) {
+        if ($rule instanceof RuleInterface){
             return $rule;
         }
 
-        if (is_string($rule)) {
+        if (is_string($rule)){
             //We are expecting that rule points to
             $rule = $this->container->get($rule);
 
-            if (!$rule instanceof RuleInterface) {
+            if (!$rule instanceof RuleInterface){
                 throw new RuleException(sprintf(
                     "Rule '%s' must point to RuleInterface, '%s' given",
                     $name,
@@ -136,18 +136,18 @@ 
      */
     private function validateRule($rule): bool
     {
-        if ($rule instanceof \Closure || $rule instanceof RuleInterface) {
+        if ($rule instanceof \Closure || $rule instanceof RuleInterface){
             return true;
         }
 
-        if (is_array($rule)) {
+        if (is_array($rule)){
             return is_callable($rule, true);
         }
 
-        if (is_string($rule) && class_exists($rule)) {
-            try {
+        if (is_string($rule) && class_exists($rule)){
+            try{
                 $reflection = new \ReflectionClass($rule);
-            } catch (\ReflectionException $e) {
+            }catch (\ReflectionException $e){
                 return false;
             }
 

Braces +34 added lines, -16 removed lines

@@ -42,11 +42,13 @@ 
      */
     public function set(string $name, $rule = null): RuleManager
     {
-        if (empty($rule)) {
+        if (empty($rule))
+        {
             $rule = $name;
         }
 
-        if (!$this->validateRule($rule)) {
+        if (!$this->validateRule($rule))
+        {
             throw new RuleException("Unable to set rule '{$name}', invalid rule body");
         }
 
@@ -62,7 +64,8 @@ 
      */
     public function remove(string $name): RuleManager
     {
-        if (!$this->has($name)) {
+        if (!$this->has($name))
+        {
             throw new RuleException("Undefined rule '{$name}'");
         }
 
@@ -76,11 +79,13 @@ 
      */
     public function has(string $name): bool
     {
-        if (isset($this->rules[$name])) {
+        if (isset($this->rules[$name]))
+        {
             return true;
         }
 
-        if (class_exists($name)) {
+        if (class_exists($name))
+        {
             //We are allowing to use class names without direct registration
             return true;
         }
@@ -94,26 +99,33 @@ 
      */
     public function get(string $name): RuleInterface
     {
-        if (!$this->has($name)) {
+        if (!$this->has($name))
+        {
             throw new RuleException("Undefined rule '{$name}'");
         }
 
-        if (!isset($this->rules[$name])) {
+        if (!isset($this->rules[$name]))
+        {
             //Rule represented as class name
             $rule = $name;
-        } else {
+        }
+        else
+        {
             $rule = $this->rules[$name];
         }
 
-        if ($rule instanceof RuleInterface) {
+        if ($rule instanceof RuleInterface)
+        {
             return $rule;
         }
 
-        if (is_string($rule)) {
+        if (is_string($rule))
+        {
             //We are expecting that rule points to
             $rule = $this->container->get($rule);
 
-            if (!$rule instanceof RuleInterface) {
+            if (!$rule instanceof RuleInterface)
+            {
                 throw new RuleException(sprintf(
                     "Rule '%s' must point to RuleInterface, '%s' given",
                     $name,
@@ -136,18 +148,24 @@ 
      */
     private function validateRule($rule): bool
     {
-        if ($rule instanceof \Closure || $rule instanceof RuleInterface) {
+        if ($rule instanceof \Closure || $rule instanceof RuleInterface)
+        {
             return true;
         }
 
-        if (is_array($rule)) {
+        if (is_array($rule))
+        {
             return is_callable($rule, true);
         }
 
-        if (is_string($rule) && class_exists($rule)) {
-            try {
+        if (is_string($rule) && class_exists($rule))
+        {
+            try
+            {
                 $reflection = new \ReflectionClass($rule);
-            } catch (\ReflectionException $e) {
+            }
+            catch (\ReflectionException $e)
+            {
                 return false;
             }
 

src/Security/src/Traits/GuardedTrait.php

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
     public function getGuard(): GuardInterface
     {
         $container = ContainerScope::getContainer();
-        if (empty($container) || !$container->has(GuardInterface::class)) {
+        if (empty($container) || !$container->has(GuardInterface::class)){
             throw new ScopeException(
                 'Unable to get `GuardInterface`, binding is missing or container scope is not set'
             );
@@ -65,9 +65,9 @@ 
      */
     protected function resolvePermission(string $permission): string
     {
-        if (defined('static::GUARD_NAMESPACE')) {
+        if (defined('static::GUARD_NAMESPACE')){
             // Yay! Isolation
-            $permission = constant(get_called_class() . '::' . 'GUARD_NAMESPACE') . '.' . $permission;
+            $permission = constant(get_called_class().'::'.'GUARD_NAMESPACE').'.'.$permission;
         }
 
         return $permission;

Braces +4 added lines, -2 removed lines

@@ -28,7 +28,8 @@ 
     public function getGuard(): GuardInterface
     {
         $container = ContainerScope::getContainer();
-        if (empty($container) || !$container->has(GuardInterface::class)) {
+        if (empty($container) || !$container->has(GuardInterface::class))
+        {
             throw new ScopeException(
                 'Unable to get `GuardInterface`, binding is missing or container scope is not set'
             );
@@ -65,7 +66,8 @@ 
      */
     protected function resolvePermission(string $permission): string
     {
-        if (defined('static::GUARD_NAMESPACE')) {
+        if (defined('static::GUARD_NAMESPACE'))
+        {
             // Yay! Isolation
             $permission = constant(get_called_class() . '::' . 'GUARD_NAMESPACE') . '.' . $permission;
         }

src/Security/src/Rule/CompositeRule.php

Spacing +4 added lines, -4 removed lines

@@ -54,16 +54,16 @@
     public function allows(ActorInterface $actor, string $permission, array $context): bool
     {
         $allowed = 0;
-        foreach (static::RULES as $rule) {
+        foreach (static::RULES as $rule){
             $rule = $this->repository->get($rule);
 
-            if ($rule->allows($actor, $permission, $context)) {
-                if (static::BEHAVIOUR == self::AT_LEAST_ONE) {
+            if ($rule->allows($actor, $permission, $context)){
+                if (static::BEHAVIOUR == self::AT_LEAST_ONE){
                     return true;
                 }
 
                 $allowed++;
-            } elseif (static::BEHAVIOUR == self::ALL) {
+            } elseif (static::BEHAVIOUR == self::ALL){
                 return false;
             }
         }

Braces +9 added lines, -4 removed lines

@@ -54,16 +54,21 @@
     public function allows(ActorInterface $actor, string $permission, array $context): bool
     {
         $allowed = 0;
-        foreach (static::RULES as $rule) {
+        foreach (static::RULES as $rule)
+        {
             $rule = $this->repository->get($rule);
 
-            if ($rule->allows($actor, $permission, $context)) {
-                if (static::BEHAVIOUR == self::AT_LEAST_ONE) {
+            if ($rule->allows($actor, $permission, $context))
+            {
+                if (static::BEHAVIOUR == self::AT_LEAST_ONE)
+                {
                     return true;
                 }
 
                 $allowed++;
-            } elseif (static::BEHAVIOUR == self::ALL) {
+            }
+            elseif (static::BEHAVIOUR == self::ALL)
+            {
                 return false;
             }
         }

src/Pagination/src/Paginator.php

Spacing +8 added lines, -8 removed lines

@@ -103,11 +103,11 @@ 
      */
     public function getPage(): int
     {
-        if ($this->pageNumber < 1) {
+        if ($this->pageNumber < 1){
             return 1;
         }
 
-        if ($this->pageNumber > $this->countPages) {
+        if ($this->pageNumber > $this->countPages){
             return $this->countPages;
         }
 
@@ -128,7 +128,7 @@ 
     public function paginate(PaginableInterface $target): PaginatorInterface
     {
         $paginator = clone $this;
-        if ($target instanceof \Countable && $paginator->count === 0) {
+        if ($target instanceof \Countable && $paginator->count === 0){
             $paginator->setCount($target->count());
         }
 
@@ -159,7 +159,7 @@ 
      */
     public function countDisplayed(): int
     {
-        if ($this->getPage() == $this->countPages) {
+        if ($this->getPage() == $this->countPages){
             return $this->count - $this->getOffset();
         }
 
@@ -179,7 +179,7 @@ 
      */
     public function nextPage()
     {
-        if ($this->getPage() != $this->countPages) {
+        if ($this->getPage() != $this->countPages){
             return $this->getPage() + 1;
         }
 
@@ -191,7 +191,7 @@ 
      */
     public function previousPage()
     {
-        if ($this->getPage() > 1) {
+        if ($this->getPage() > 1){
             return $this->getPage() - 1;
         }
 
@@ -208,9 +208,9 @@ 
     private function setCount(int $count): self
     {
         $this->count = max($count, 0);
-        if ($this->count > 0) {
+        if ($this->count > 0){
             $this->countPages = (int)ceil($this->count / $this->limit);
-        } else {
+        }else{
             $this->countPages = 1;
         }
 

Braces +17 added lines, -8 removed lines

@@ -103,11 +103,13 @@ 
      */
     public function getPage(): int
     {
-        if ($this->pageNumber < 1) {
+        if ($this->pageNumber < 1)
+        {
             return 1;
         }
 
-        if ($this->pageNumber > $this->countPages) {
+        if ($this->pageNumber > $this->countPages)
+        {
             return $this->countPages;
         }
 
@@ -128,7 +130,8 @@ 
     public function paginate(PaginableInterface $target): PaginatorInterface
     {
         $paginator = clone $this;
-        if ($target instanceof \Countable && $paginator->count === 0) {
+        if ($target instanceof \Countable && $paginator->count === 0)
+        {
             $paginator->setCount($target->count());
         }
 
@@ -159,7 +162,8 @@ 
      */
     public function countDisplayed(): int
     {
-        if ($this->getPage() == $this->countPages) {
+        if ($this->getPage() == $this->countPages)
+        {
             return $this->count - $this->getOffset();
         }
 
@@ -179,7 +183,8 @@ 
      */
     public function nextPage()
     {
-        if ($this->getPage() != $this->countPages) {
+        if ($this->getPage() != $this->countPages)
+        {
             return $this->getPage() + 1;
         }
 
@@ -191,7 +196,8 @@ 
      */
     public function previousPage()
     {
-        if ($this->getPage() > 1) {
+        if ($this->getPage() > 1)
+        {
             return $this->getPage() - 1;
         }
 
@@ -208,9 +214,12 @@ 
     private function setCount(int $count): self
     {
         $this->count = max($count, 0);
-        if ($this->count > 0) {
+        if ($this->count > 0)
+        {
             $this->countPages = (int)ceil($this->count / $this->limit);
-        } else {
+        }
+        else
+        {
             $this->countPages = 1;
         }
 

src/Reactor/src/AbstractDeclaration.php

Spacing +1 added lines, -1 removed lines

@@ -30,6 +30,6 @@
      */
     protected function addIndent(string $string, int $indent = 0): string
     {
-        return str_repeat(self::INDENT, max($indent, 0)) . $string;
+        return str_repeat(self::INDENT, max($indent, 0)).$string;
     }
 }

src/Reactor/src/NamespaceDeclaration.php

Spacing +9 added lines, -9 removed lines

@@ -58,7 +58,7 @@ 
     public function addElement(DeclarationInterface $element): NamespaceDeclaration
     {
         $this->elements->add($element);
-        if ($element instanceof DependedInterface) {
+        if ($element instanceof DependedInterface){
             $this->addUses($element->getDependencies());
         }
 
@@ -85,23 +85,23 @@ 
         $result = '';
         $indentShift = 0;
 
-        if (!$this->docComment->isEmpty()) {
-            $result .= $this->docComment->render($indentLevel) . "\n";
+        if (!$this->docComment->isEmpty()){
+            $result .= $this->docComment->render($indentLevel)."\n";
         }
 
-        if (!empty($this->getName())) {
-            $result .= $this->addIndent("namespace {$this->getName()} {", $indentLevel) . "\n";
+        if (!empty($this->getName())){
+            $result .= $this->addIndent("namespace {$this->getName()} {", $indentLevel)."\n";
             $indentShift = 1;
         }
 
-        if (!empty($this->uses)) {
-            $result .= $this->renderUses($indentLevel + $indentShift) . "\n\n";
+        if (!empty($this->uses)){
+            $result .= $this->renderUses($indentLevel + $indentShift)."\n\n";
         }
 
         $result .= $this->elements->render($indentLevel + $indentShift);
 
-        if (!empty($this->getName())) {
-            $result .= "\n" . $this->addIndent('}', $indentLevel);
+        if (!empty($this->getName())){
+            $result .= "\n".$this->addIndent('}', $indentLevel);
         }
 
         return $result;

Braces +10 added lines, -5 removed lines

@@ -58,7 +58,8 @@ 
     public function addElement(DeclarationInterface $element): NamespaceDeclaration
     {
         $this->elements->add($element);
-        if ($element instanceof DependedInterface) {
+        if ($element instanceof DependedInterface)
+        {
             $this->addUses($element->getDependencies());
         }
 
@@ -85,22 +86,26 @@ 
         $result = '';
         $indentShift = 0;
 
-        if (!$this->docComment->isEmpty()) {
+        if (!$this->docComment->isEmpty())
+        {
             $result .= $this->docComment->render($indentLevel) . "\n";
         }
 
-        if (!empty($this->getName())) {
+        if (!empty($this->getName()))
+        {
             $result .= $this->addIndent("namespace {$this->getName()} {", $indentLevel) . "\n";
             $indentShift = 1;
         }
 
-        if (!empty($this->uses)) {
+        if (!empty($this->uses))
+        {
             $result .= $this->renderUses($indentLevel + $indentShift) . "\n\n";
         }
 
         $result .= $this->elements->render($indentLevel + $indentShift);
 
-        if (!empty($this->getName())) {
+        if (!empty($this->getName()))
+        {
             $result .= "\n" . $this->addIndent('}', $indentLevel);
         }
 

src/Reactor/src/FileDeclaration.php

Spacing +9 added lines, -9 removed lines

@@ -106,7 +106,7 @@ 
     public function addElement(DeclarationInterface $element): FileDeclaration
     {
         $this->elements->add($element);
-        if ($element instanceof DependedInterface) {
+        if ($element instanceof DependedInterface){
             $this->addUses($element->getDependencies());
         }
 
@@ -132,23 +132,23 @@ 
     {
         $result = "<?php\n";
 
-        if (!$this->docComment->isEmpty()) {
-            $result .= $this->docComment->render($indentLevel) . "\n";
+        if (!$this->docComment->isEmpty()){
+            $result .= $this->docComment->render($indentLevel)."\n";
         }
 
-        if ($this->directives !== null && !empty($this->directives->render())) {
-            $result .= $this->directives->render() . "\n\n";
+        if ($this->directives !== null && !empty($this->directives->render())){
+            $result .= $this->directives->render()."\n\n";
         }
 
-        if (!empty($this->namespace)) {
-            if ($this->docComment->isEmpty()) {
+        if (!empty($this->namespace)){
+            if ($this->docComment->isEmpty()){
                 $result .= "\n";
             }
             $result .= "namespace {$this->namespace};\n\n";
         }
 
-        if (!empty($this->uses)) {
-            $result .= $this->renderUses($indentLevel) . "\n\n";
+        if (!empty($this->uses)){
+            $result .= $this->renderUses($indentLevel)."\n\n";
         }
 
         $result .= $this->elements->render($indentLevel);

Braces +12 added lines, -6 removed lines

@@ -106,7 +106,8 @@ 
     public function addElement(DeclarationInterface $element): FileDeclaration
     {
         $this->elements->add($element);
-        if ($element instanceof DependedInterface) {
+        if ($element instanceof DependedInterface)
+        {
             $this->addUses($element->getDependencies());
         }
 
@@ -132,22 +133,27 @@ 
     {
         $result = "<?php\n";
 
-        if (!$this->docComment->isEmpty()) {
+        if (!$this->docComment->isEmpty())
+        {
             $result .= $this->docComment->render($indentLevel) . "\n";
         }
 
-        if ($this->directives !== null && !empty($this->directives->render())) {
+        if ($this->directives !== null && !empty($this->directives->render()))
+        {
             $result .= $this->directives->render() . "\n\n";
         }
 
-        if (!empty($this->namespace)) {
-            if ($this->docComment->isEmpty()) {
+        if (!empty($this->namespace))
+        {
+            if ($this->docComment->isEmpty())
+            {
                 $result .= "\n";
             }
             $result .= "namespace {$this->namespace};\n\n";
         }
 
-        if (!empty($this->uses)) {
+        if (!empty($this->uses))
+        {
             $result .= $this->renderUses($indentLevel) . "\n\n";
         }