spiral / framework

src/Views/src/Engine/AbstractEngine.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
      */
     public function getLoader(): LoaderInterface
     {
-        if (empty($this->loader)) {
+        if (empty($this->loader)){
             throw new EngineException('No associated loader found');
         }
 

Braces +2 added lines, -1 removed lines

@@ -44,7 +44,8 @@
      */
     public function getLoader(): LoaderInterface
     {
-        if (empty($this->loader)) {
+        if (empty($this->loader))
+        {
             throw new EngineException('No associated loader found');
         }
 

src/Views/src/Engine/Native/NativeView.php

Spacing +5 added lines, -5 removed lines

@@ -49,21 +49,21 @@
         ob_start();
         $__outputLevel__ = ob_get_level();
 
-        try {
+        try{
             ContainerScope::runScope($this->container, function () use ($data): void {
                 extract($data, EXTR_OVERWRITE);
                 // render view in context and output buffer scope, context can be accessed using $this->context
                 require $this->view->getFilename();
             });
-        } catch (\Throwable $e) {
-            while (ob_get_level() >= $__outputLevel__) {
+        }catch (\Throwable $e){
+            while (ob_get_level() >= $__outputLevel__){
                 ob_end_clean();
             }
 
             throw new RenderException($e);
-        } finally {
+        }finally{
             //Closing all nested buffers
-            while (ob_get_level() > $__outputLevel__) {
+            while (ob_get_level() > $__outputLevel__){
                 ob_end_clean();
             }
         }

Braces +12 added lines, -5 removed lines

@@ -49,21 +49,28 @@
         ob_start();
         $__outputLevel__ = ob_get_level();
 
-        try {
+        try
+        {
             ContainerScope::runScope($this->container, function () use ($data): void {
                 extract($data, EXTR_OVERWRITE);
                 // render view in context and output buffer scope, context can be accessed using $this->context
                 require $this->view->getFilename();
             });
-        } catch (\Throwable $e) {
-            while (ob_get_level() >= $__outputLevel__) {
+        }
+        catch (\Throwable $e)
+        {
+            while (ob_get_level() >= $__outputLevel__)
+            {
                 ob_end_clean();
             }
 
             throw new RenderException($e);
-        } finally {
+        }
+        finally
+        {
             //Closing all nested buffers
-            while (ob_get_level() > $__outputLevel__) {
+            while (ob_get_level() > $__outputLevel__)
+            {
                 ob_end_clean();
             }
         }

src/Views/src/ViewCache.php

Spacing +3 added lines, -3 removed lines

@@ -23,7 +23,7 @@ 
      */
     public function reset(ContextInterface $context = null): void
     {
-        if (empty($context)) {
+        if (empty($context)){
             $this->cache = [];
             return;
         }
@@ -38,7 +38,7 @@ 
      */
     public function resetPath(string $path): void
     {
-        foreach ($this->cache as &$cache) {
+        foreach ($this->cache as &$cache){
             unset($cache[$path], $cache);
         }
     }
@@ -72,7 +72,7 @@ 
      */
     public function get(ContextInterface $context, string $path): ViewInterface
     {
-        if (!$this->has($context, $path)) {
+        if (!$this->has($context, $path)){
             throw new CacheException("No cache is available for {$path}.");
         }
 

Braces +6 added lines, -3 removed lines

@@ -23,7 +23,8 @@ 
      */
     public function reset(ContextInterface $context = null): void
     {
-        if (empty($context)) {
+        if (empty($context))
+        {
             $this->cache = [];
             return;
         }
@@ -38,7 +39,8 @@ 
      */
     public function resetPath(string $path): void
     {
-        foreach ($this->cache as &$cache) {
+        foreach ($this->cache as &$cache)
+        {
             unset($cache[$path], $cache);
         }
     }
@@ -72,7 +74,8 @@ 
      */
     public function get(ContextInterface $context, string $path): ViewInterface
     {
-        if (!$this->has($context, $path)) {
+        if (!$this->has($context, $path))
+        {
             throw new CacheException("No cache is available for {$path}.");
         }
 

src/Security/tests/PermissionManagerTest.php

Spacing +2 added lines, -2 removed lines

@@ -90,8 +90,8 @@
         $this->assertEquals($allowRule, $manager->getRule(static::ROLE, static::PERMISSION));
 
         // test pattern permission
-        $this->assertEquals($manager, $manager->associate(static::ROLE, static::PERMISSION . '.*', AllowRule::class));
-        $this->assertEquals($allowRule, $manager->getRule(static::ROLE, static::PERMISSION . '.' . static::PERMISSION));
+        $this->assertEquals($manager, $manager->associate(static::ROLE, static::PERMISSION.'.*', AllowRule::class));
+        $this->assertEquals($allowRule, $manager->getRule(static::ROLE, static::PERMISSION.'.'.static::PERMISSION));
 
         $this->assertEquals($manager, $manager->deassociate(static::ROLE, static::PERMISSION));
         $this->assertEquals($forbidRule, $manager->getRule(static::ROLE, static::PERMISSION));

src/Security/tests/Traits/GuardedTraitTest.php

Spacing +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
         $this->assertEquals(static::OPERATION, $guarded->resolvePermission(static::OPERATION));
 
         $guarded = new GuardedWithNamespace();
-        $resolvedPermission = GuardedWithNamespace::GUARD_NAMESPACE . '.' . static::OPERATION;
+        $resolvedPermission = GuardedWithNamespace::GUARD_NAMESPACE.'.'.static::OPERATION;
         $this->assertEquals($resolvedPermission, $guarded->resolvePermission(static::OPERATION));
     }
 }

src/Security/tests/RuleManagerTest.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
         // other rule types
         $manager->set('RuleInterface', $this->rule);
         $this->assertEquals($this->rule, $manager->get('RuleInterface'));
-        $manager->set('Closure', function () {
+        $manager->set('Closure', function (){
             return true;
         });
         $this->assertTrue($manager->get('Closure') instanceof CallableRule);

Braces +2 added lines, -1 removed lines

@@ -60,7 +60,8 @@
         // other rule types
         $manager->set('RuleInterface', $this->rule);
         $this->assertEquals($this->rule, $manager->get('RuleInterface'));
-        $manager->set('Closure', function () {
+        $manager->set('Closure', function ()
+        {
             return true;
         });
         $this->assertTrue($manager->get('Closure') instanceof CallableRule);

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