spiral / framework

src/Security/src/PermissionManager.php

Spacing +9 added lines, -9 removed lines

@@ -72,7 +72,7 @@ 
      */
     public function addRole(string $role): PermissionManager
     {
-        if ($this->hasRole($role)) {
+        if ($this->hasRole($role)){
             throw new RoleException("Role '{$role}' already exists");
         }
 
@@ -90,7 +90,7 @@ 
      */
     public function removeRole(string $role): PermissionManager
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role)){
             throw new RoleException("Undefined role '{$role}'");
         }
 
@@ -112,7 +112,7 @@ 
      */
     public function getPermissions(string $role): array
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role)){
             throw new RoleException("Undefined role '{$role}'");
         }
 
@@ -124,7 +124,7 @@ 
      */
     public function getRule(string $role, string $permission): RuleInterface
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role)){
             throw new RoleException("Undefined role '{$role}'");
         }
 
@@ -139,11 +139,11 @@ 
      */
     public function associate(string $role, string $permission, string $rule = AllowRule::class): PermissionManager
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role)){
             throw new RoleException("Undefined role '{$role}'");
         }
 
-        if (!$this->rules->has($rule)) {
+        if (!$this->rules->has($rule)){
             throw new PermissionException("Undefined rule '{$rule}'");
         }
 
@@ -176,14 +176,14 @@ 
      */
     private function findRule(string $role, string $permission): string
     {
-        if (isset($this->permissions[$role][$permission])) {
+        if (isset($this->permissions[$role][$permission])){
             //O(1) check
             return $this->permissions[$role][$permission];
         }
 
         //Matching using star syntax
-        foreach ($this->permissions[$role] as $pattern => $rule) {
-            if ($this->matcher->matches($permission, $pattern)) {
+        foreach ($this->permissions[$role] as $pattern => $rule){
+            if ($this->matcher->matches($permission, $pattern)){
                 return $rule;
             }
         }

Braces +18 added lines, -9 removed lines

@@ -72,7 +72,8 @@ 
      */
     public function addRole(string $role): PermissionManager
     {
-        if ($this->hasRole($role)) {
+        if ($this->hasRole($role))
+        {
             throw new RoleException("Role '{$role}' already exists");
         }
 
@@ -90,7 +91,8 @@ 
      */
     public function removeRole(string $role): PermissionManager
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role))
+        {
             throw new RoleException("Undefined role '{$role}'");
         }
 
@@ -112,7 +114,8 @@ 
      */
     public function getPermissions(string $role): array
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role))
+        {
             throw new RoleException("Undefined role '{$role}'");
         }
 
@@ -124,7 +127,8 @@ 
      */
     public function getRule(string $role, string $permission): RuleInterface
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role))
+        {
             throw new RoleException("Undefined role '{$role}'");
         }
 
@@ -139,11 +143,13 @@ 
      */
     public function associate(string $role, string $permission, string $rule = AllowRule::class): PermissionManager
     {
-        if (!$this->hasRole($role)) {
+        if (!$this->hasRole($role))
+        {
             throw new RoleException("Undefined role '{$role}'");
         }
 
-        if (!$this->rules->has($rule)) {
+        if (!$this->rules->has($rule))
+        {
             throw new PermissionException("Undefined rule '{$rule}'");
         }
 
@@ -176,14 +182,17 @@ 
      */
     private function findRule(string $role, string $permission): string
     {
-        if (isset($this->permissions[$role][$permission])) {
+        if (isset($this->permissions[$role][$permission]))
+        {
             //O(1) check
             return $this->permissions[$role][$permission];
         }
 
         //Matching using star syntax
-        foreach ($this->permissions[$role] as $pattern => $rule) {
-            if ($this->matcher->matches($permission, $pattern)) {
+        foreach ($this->permissions[$role] as $pattern => $rule)
+        {
+            if ($this->matcher->matches($permission, $pattern))
+            {
                 return $rule;
             }
         }

src/Security/src/Guard.php

Spacing +4 added lines, -4 removed lines

@@ -36,7 +36,7 @@ 
         PermissionsInterface $permissions,
         ActorInterface $actor = null,
         array $roles = []
-    ) {
+    ){
         $this->roles = $roles;
         $this->actor = $actor;
         $this->permissions = $permissions;
@@ -48,8 +48,8 @@ 
     public function allows(string $permission, array $context = []): bool
     {
         $allows = false;
-        foreach ($this->getRoles() as $role) {
-            if (!$this->permissions->hasRole($role)) {
+        foreach ($this->getRoles() as $role){
+            if (!$this->permissions->hasRole($role)){
                 continue;
             }
 
@@ -95,7 +95,7 @@ 
      */
     public function getActor(): ActorInterface
     {
-        if (empty($this->actor)) {
+        if (empty($this->actor)){
             throw new GuardException('Unable to get Guard Actor, no value set');
         }
 

Braces +6 added lines, -3 removed lines

@@ -48,8 +48,10 @@ 
     public function allows(string $permission, array $context = []): bool
     {
         $allows = false;
-        foreach ($this->getRoles() as $role) {
-            if (!$this->permissions->hasRole($role)) {
+        foreach ($this->getRoles() as $role)
+        {
+            if (!$this->permissions->hasRole($role))
+            {
                 continue;
             }
 
@@ -95,7 +97,8 @@ 
      */
     public function getActor(): ActorInterface
     {
-        if (empty($this->actor)) {
+        if (empty($this->actor))
+        {
             throw new GuardException('Unable to get Guard Actor, no value set');
         }
 

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

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";
         }
 

Spacing +10 added lines, -10 removed lines

@@ -80,7 +80,7 @@ 
     public function addElement(DeclarationInterface $element): FileDeclaration
     {
         $this->elements->add($element);
-        if ($element instanceof DependedInterface) {
+        if ($element instanceof DependedInterface){
             $this->addUses($element->getDependencies());
         }
 
@@ -105,28 +105,28 @@ 
     {
         $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);
 
-        return $result . "\n";
+        return $result."\n";
     }
 
     /**

src/Reactor/src/Traits/CommentTrait.php

Spacing +4 added lines, -4 removed lines

@@ -42,10 +42,10 @@ 
      */
     public function setComment($comment): self
     {
-        if (!empty($comment)) {
-            if (is_array($comment)) {
+        if (!empty($comment)){
+            if (is_array($comment)){
                 $this->docComment->setLines($comment);
-            } elseif (is_string($comment)) {
+            } elseif (is_string($comment)){
                 $this->docComment->setString($comment);
             }
         }
@@ -60,7 +60,7 @@ 
      */
     private function initComment($comment): void
     {
-        if (empty($this->docComment)) {
+        if (empty($this->docComment)){
             $this->docComment = new Comment();
         }
 

Braces +9 added lines, -4 removed lines

@@ -42,10 +42,14 @@ 
      */
     public function setComment($comment): self
     {
-        if (!empty($comment)) {
-            if (is_array($comment)) {
+        if (!empty($comment))
+        {
+            if (is_array($comment))
+            {
                 $this->docComment->setLines($comment);
-            } elseif (is_string($comment)) {
+            }
+            elseif (is_string($comment))
+            {
                 $this->docComment->setString($comment);
             }
         }
@@ -60,7 +64,8 @@ 
      */
     private function initComment($comment): void
     {
-        if (empty($this->docComment)) {
+        if (empty($this->docComment))
+        {
             $this->docComment = new Comment();
         }
 

src/Reactor/src/Traits/UsesTrait.php

Spacing +4 added lines, -4 removed lines

@@ -56,7 +56,7 @@ 
      */
     public function addUses(array $uses): self
     {
-        foreach ($uses as $class => $alias) {
+        foreach ($uses as $class => $alias){
             $this->addUse($class, $alias);
         }
 
@@ -112,12 +112,12 @@ 
     private function renderUses(int $indentLevel = 0): string
     {
         $lines = [];
-        foreach ($this->getUses() as $class => $alias) {
+        foreach ($this->getUses() as $class => $alias){
             $line = "use {$class}";
 
-            if (!empty($alias)) {
+            if (!empty($alias)){
                 $line .= " as {$alias};";
-            } else {
+            }else{
                 $line .= ';';
             }
 

Braces +9 added lines, -4 removed lines

@@ -56,7 +56,8 @@ 
      */
     public function addUses(array $uses): self
     {
-        foreach ($uses as $class => $alias) {
+        foreach ($uses as $class => $alias)
+        {
             $this->addUse($class, $alias);
         }
 
@@ -112,12 +113,16 @@ 
     private function renderUses(int $indentLevel = 0): string
     {
         $lines = [];
-        foreach ($this->getUses() as $class => $alias) {
+        foreach ($this->getUses() as $class => $alias)
+        {
             $line = "use {$class}";
 
-            if (!empty($alias)) {
+            if (!empty($alias))
+            {
                 $line .= " as {$alias};";
-            } else {
+            }
+            else
+            {
                 $line .= ';';
             }
 

src/Reactor/src/Partial/Comment.php

Braces +6 added lines, -3 removed lines

@@ -38,12 +38,14 @@ 
      */
     public function render(int $indentLevel = 0): string
     {
-        if ($this->isEmpty()) {
+        if ($this->isEmpty())
+        {
             return '';
         }
 
         $result = $this->addIndent("/**\n", $indentLevel);
-        foreach ($this->getLines() as $line) {
+        foreach ($this->getLines() as $line)
+        {
             $result .= $this->addIndent(" * {$line}\n", $indentLevel);
         }
 
@@ -58,7 +60,8 @@ 
     protected function prepareLine(string $line): ?string
     {
         $line = trim($line);
-        if (in_array($line, ['/*', '/**', '*/'], true)) {
+        if (in_array($line, ['/*', '/**', '*/'], true))
+        {
             return null;
         }
 

Spacing +4 added lines, -4 removed lines

@@ -37,16 +37,16 @@ 
      */
     public function render(int $indentLevel = 0): string
     {
-        if ($this->isEmpty()) {
+        if ($this->isEmpty()){
             return '';
         }
 
         $result = $this->addIndent("/**\n", $indentLevel);
-        foreach ($this->getLines() as $line) {
+        foreach ($this->getLines() as $line){
             $result .= $this->addIndent(" * {$line}\n", $indentLevel);
         }
 
-        return $result . $this->addIndent(' */', $indentLevel);
+        return $result.$this->addIndent(' */', $indentLevel);
     }
 
     /**
@@ -55,7 +55,7 @@ 
     protected function prepareLine(string $line): ?string
     {
         $line = trim($line);
-        if (in_array($line, ['/*', '/**', '*/'], true)) {
+        if (in_array($line, ['/*', '/**', '*/'], true)){
             return null;
         }