spiral / framework

src/Files/src/FilesInterface.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     public const RUNTIME = 0666;
 
     //Only owner can write
-    public const READONLY = 0644;
+    public const readonly = 0644;
 
     /**
      * Few size constants for better size manipulations.

src/Cookies/src/Cookie/SameSite.php

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
     public const NONE   = 'None';
 
     private const VALUES  = [self::STRICT, self::LAX, self::NONE];
-    private const DEFAULT = self::LAX;
+    private const default = self::LAX;
 
     private ?string $sameSite = null;
 
@@ -36,6 +36,6 @@ 
             return null;
         }
 
-        return ($sameSite === self::NONE && !$secure) ? self::DEFAULT : $sameSite;
+        return ($sameSite === self::NONE && !$secure) ? self::default : $sameSite;
     }
 }

Spacing +2 added lines, -2 removed lines

@@ -27,12 +27,12 @@
 
     private function defineValue(?string $sameSite, bool $secure): ?string
     {
-        if ($sameSite === null) {
+        if ($sameSite === null){
             return null;
         }
 
         $sameSite = \ucfirst(\strtolower($sameSite));
-        if (!\in_array($sameSite, self::VALUES, true)) {
+        if (!\in_array($sameSite, self::VALUES, true)){
             return null;
         }
 

Braces +4 added lines, -2 removed lines

@@ -27,12 +27,14 @@
 
     private function defineValue(?string $sameSite, bool $secure): ?string
     {
-        if ($sameSite === null) {
+        if ($sameSite === null)
+        {
             return null;
         }
 
         $sameSite = \ucfirst(\strtolower($sameSite));
-        if (!\in_array($sameSite, self::VALUES, true)) {
+        if (!\in_array($sameSite, self::VALUES, true))
+        {
             return null;
         }
 

src/Stempler/src/Traverser.php

Braces +48 added lines, -24 removed lines

@@ -29,7 +29,8 @@ 
 
     public function __construct(array $visitors = [])
     {
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor)
+        {
             $this->addVisitor($visitor);
         }
     }
@@ -44,8 +45,10 @@ 
 
     public function removeVisitor(VisitorInterface $visitor): void
     {
-        foreach ($this->visitors as $index => $added) {
-            if ($added === $visitor) {
+        foreach ($this->visitors as $index => $added)
+        {
+            if ($added === $visitor)
+            {
                 unset($this->visitors[$index]);
                 break;
             }
@@ -64,22 +67,27 @@ 
         $context ??= new VisitorContext();
 
         $ctx = clone $context;
-        foreach ($nodes as $index => $node) {
-            if ($this->stopTraversal) {
+        foreach ($nodes as $index => $node)
+        {
+            if ($this->stopTraversal)
+            {
                 break;
             }
 
             $traverseChildren = true;
             $breakVisitorID = null;
 
-            if ($node instanceof NodeInterface) {
+            if ($node instanceof NodeInterface)
+            {
                 $ctx = $context->withNode($node);
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor)
+            {
                 $result = $visitor->enterNode($node, $ctx);
 
-                switch (true) {
+                switch (true)
+                {
                     case $result === null:
                         break;
 
@@ -109,17 +117,21 @@ 
             }
 
             // sub nodes
-            if ($traverseChildren && $node instanceof NodeInterface) {
+            if ($traverseChildren && $node instanceof NodeInterface)
+            {
                 $nodes[$index] = $this->traverseNode($node, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal)
+                {
                     break;
                 }
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor)
+            {
                 $result = $visitor->leaveNode($node, $ctx);
 
-                switch (true) {
+                switch (true)
+                {
                     case $result === null:
                         break;
 
@@ -141,7 +153,8 @@ 
                         );
                 }
 
-                if ($breakVisitorID === $visitorID) {
+                if ($breakVisitorID === $visitorID)
+                {
                     break;
                 }
             }
@@ -156,18 +169,22 @@ 
     private function traverseNode(NodeInterface $node, VisitorContext $context): NodeInterface
     {
         $ctx = clone $context;
-        foreach ($node as $name => $_) {
+        foreach ($node as $name => $_)
+        {
             $child = &$node->$name;
-            if (is_array($child)) {
+            if (is_array($child))
+            {
                 $child = $this->traverse($child, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal)
+                {
                     break;
                 }
 
                 continue;
             }
 
-            if (!$child instanceof NodeInterface) {
+            if (!$child instanceof NodeInterface)
+            {
                 continue;
             }
 
@@ -176,9 +193,11 @@ 
             $traverseChildren = true;
             $breakVisitorID = null;
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor)
+            {
                 $result = $visitor->enterNode($child, $ctx);
-                switch (true) {
+                switch (true)
+                {
                     case $result === null:
                         break;
 
@@ -206,17 +225,21 @@ 
                 }
             }
 
-            if ($traverseChildren) {
+            if ($traverseChildren)
+            {
                 $child = $this->traverseNode($child, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal)
+                {
                     break;
                 }
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor)
+            {
                 $result = $visitor->leaveNode($child, $ctx);
 
-                switch (true) {
+                switch (true)
+                {
                     case $result === null:
                         break;
 
@@ -234,7 +257,8 @@ 
                         );
                 }
 
-                if ($breakVisitorID === $visitorID) {
+                if ($breakVisitorID === $visitorID)
+                {
                     break;
                 }
             }

Spacing +28 added lines, -28 removed lines

@@ -19,7 +19,7 @@ 
 
     public function __construct(array $visitors = [])
     {
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor){
             $this->addVisitor($visitor);
         }
     }
@@ -34,8 +34,8 @@ 
 
     public function removeVisitor(VisitorInterface $visitor): void
     {
-        foreach ($this->visitors as $index => $added) {
-            if ($added === $visitor) {
+        foreach ($this->visitors as $index => $added){
+            if ($added === $visitor){
                 unset($this->visitors[$index]);
                 break;
             }
@@ -56,22 +56,22 @@ 
         $context ??= new VisitorContext();
 
         $ctx = clone $context;
-        foreach ($nodes as $index => $node) {
-            if ($this->stopTraversal) {
+        foreach ($nodes as $index => $node){
+            if ($this->stopTraversal){
                 break;
             }
 
             $traverseChildren = true;
             $breakVisitorID = null;
 
-            if ($node instanceof NodeInterface) {
+            if ($node instanceof NodeInterface){
                 $ctx = $context->withNode($node);
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor){
                 $result = $visitor->enterNode($node, $ctx);
 
-                switch (true) {
+                switch (true){
                     case $result === null:
                         break;
 
@@ -95,23 +95,23 @@ 
 
                     default:
                         throw new \LogicException(
-                            'enterNode() returned invalid value of type ' . \gettype($result)
+                            'enterNode() returned invalid value of type '.\gettype($result)
                         );
                 }
             }
 
             // sub nodes
-            if ($traverseChildren && $node instanceof NodeInterface) {
+            if ($traverseChildren && $node instanceof NodeInterface){
                 $nodes[$index] = $this->traverseNode($node, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal){
                     break;
                 }
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor){
                 $result = $visitor->leaveNode($node, $ctx);
 
-                switch (true) {
+                switch (true){
                     case $result === null:
                         break;
 
@@ -129,11 +129,11 @@ 
 
                     default:
                         throw new \LogicException(
-                            'leaveNode() returned invalid value of type ' . gettype($result)
+                            'leaveNode() returned invalid value of type '.gettype($result)
                         );
                 }
 
-                if ($breakVisitorID === $visitorID) {
+                if ($breakVisitorID === $visitorID){
                     break;
                 }
             }
@@ -148,18 +148,18 @@ 
     private function traverseNode(NodeInterface $node, VisitorContext $context): NodeInterface
     {
         $ctx = clone $context;
-        foreach ($node as $name => $_) {
+        foreach ($node as $name => $_){
             $child = &$node->$name;
-            if (is_array($child)) {
+            if (is_array($child)){
                 $child = $this->traverse($child, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal){
                     break;
                 }
 
                 continue;
             }
 
-            if (!$child instanceof NodeInterface) {
+            if (!$child instanceof NodeInterface){
                 continue;
             }
 
@@ -168,9 +168,9 @@ 
             $traverseChildren = true;
             $breakVisitorID = null;
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor){
                 $result = $visitor->enterNode($child, $ctx);
-                switch (true) {
+                switch (true){
                     case $result === null:
                         break;
 
@@ -193,22 +193,22 @@ 
 
                     default:
                         throw new \LogicException(
-                            'enterNode() returned invalid value of type ' . \gettype($result)
+                            'enterNode() returned invalid value of type '.\gettype($result)
                         );
                 }
             }
 
-            if ($traverseChildren) {
+            if ($traverseChildren){
                 $child = $this->traverseNode($child, $ctx);
-                if ($this->stopTraversal) {
+                if ($this->stopTraversal){
                     break;
                 }
             }
 
-            foreach ($this->visitors as $visitorID => $visitor) {
+            foreach ($this->visitors as $visitorID => $visitor){
                 $result = $visitor->leaveNode($child, $ctx);
 
-                switch (true) {
+                switch (true){
                     case $result === null:
                         break;
 
@@ -222,11 +222,11 @@ 
 
                     default:
                         throw new \LogicException(
-                            'leaveNode() returned invalid value of type ' . \gettype($result)
+                            'leaveNode() returned invalid value of type '.\gettype($result)
                         );
                 }
 
-                if ($breakVisitorID === $visitorID) {
+                if ($breakVisitorID === $visitorID){
                     break;
                 }
             }

src/AuthHttp/src/Middleware/AuthMiddleware.php

Braces +10 added lines, -5 removed lines

@@ -70,14 +70,17 @@ 
 
     private function initContext(Request $request, AuthContextInterface $authContext): AuthContextInterface
     {
-        foreach ($this->transportRegistry->getTransports() as $name => $transport) {
+        foreach ($this->transportRegistry->getTransports() as $name => $transport)
+        {
             $tokenID = $transport->fetchToken($request);
-            if ($tokenID === null) {
+            if ($tokenID === null)
+            {
                 continue;
             }
 
             $token = $this->tokenStorage->load($tokenID);
-            if ($token === null) {
+            if ($token === null)
+            {
                 continue;
             }
 
@@ -91,13 +94,15 @@ 
 
     private function closeContext(Request $request, Response $response, AuthContextInterface $authContext): Response
     {
-        if ($authContext->getToken() === null) {
+        if ($authContext->getToken() === null)
+        {
             return $response;
         }
 
         $transport = $this->transportRegistry->getTransport($authContext->getTransport());
 
-        if ($authContext->isClosed()) {
+        if ($authContext->isClosed())
+        {
             $this->tokenStorage->delete($authContext->getToken());
 
             return $transport->removeToken(

Spacing +6 added lines, -6 removed lines

@@ -29,7 +29,7 @@ 
         private readonly TokenStorageInterface $tokenStorage,
         private readonly TransportRegistry $transportRegistry,
         private readonly ?EventDispatcherInterface $eventDispatcher = null
-    ) {
+    ){
     }
 
     /**
@@ -49,14 +49,14 @@ 
 
     private function initContext(Request $request, AuthContextInterface $authContext): AuthContextInterface
     {
-        foreach ($this->transportRegistry->getTransports() as $name => $transport) {
+        foreach ($this->transportRegistry->getTransports() as $name => $transport){
             $tokenID = $transport->fetchToken($request);
-            if ($tokenID === null) {
+            if ($tokenID === null){
                 continue;
             }
 
             $token = $this->tokenStorage->load($tokenID);
-            if ($token === null) {
+            if ($token === null){
                 continue;
             }
 
@@ -70,13 +70,13 @@ 
 
     private function closeContext(Request $request, Response $response, AuthContextInterface $authContext): Response
     {
-        if ($authContext->getToken() === null) {
+        if ($authContext->getToken() === null){
             return $response;
         }
 
         $transport = $this->transportRegistry->getTransport($authContext->getTransport());
 
-        if ($authContext->isClosed()) {
+        if ($authContext->isClosed()){
             $this->tokenStorage->delete($authContext->getToken());
 
             return $transport->removeToken(

src/Broadcasting/src/BroadcastManager.php

Braces +2 added lines, -1 removed lines

@@ -30,7 +30,8 @@
         // Replaces alias with real storage name
         $name = $this->config->getAliases()[$name] ?? $name;
 
-        if (isset($this->connections[$name])) {
+        if (isset($this->connections[$name]))
+        {
             return $this->connections[$name];
         }
 

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
     public function __construct(
         private readonly FactoryInterface $factory,
         private readonly BroadcastConfig $config
-    ) {
+    ){
     }
 
     public function connection(?string $name = null): BroadcastInterface
@@ -26,7 +26,7 @@ 
         // Replaces alias with real storage name
         $name = $this->config->getAliases()[$name] ?? $name;
 
-        if (isset($this->connections[$name])) {
+        if (isset($this->connections[$name])){
             return $this->connections[$name];
         }
 

src/Http/src/Config/HttpConfig.php

Spacing +2 added lines, -2 removed lines

@@ -58,10 +58,10 @@
      */
     public function getChunkSize(): ?int
     {
-        if (\is_null($this->config['chunkSize']) || (int) $this->config['chunkSize'] < 0) {
+        if (\is_null($this->config['chunkSize']) || (int)$this->config['chunkSize'] < 0){
             return null;
         }
 
-        return (int) $this->config['chunkSize'];
+        return (int)$this->config['chunkSize'];
     }
 }

Braces +2 added lines, -1 removed lines

@@ -58,7 +58,8 @@
      */
     public function getChunkSize(): ?int
     {
-        if (\is_null($this->config['chunkSize']) || (int) $this->config['chunkSize'] < 0) {
+        if (\is_null($this->config['chunkSize']) || (int) $this->config['chunkSize'] < 0)
+        {
             return null;
         }
 

src/Queue/src/QueueableDetector.php

Spacing +3 added lines, -3 removed lines

@@ -23,7 +23,7 @@ 
     {
         $reflection = new \ReflectionClass($object);
 
-        if ($reflection->implementsInterface(QueueableInterface::class)) {
+        if ($reflection->implementsInterface(QueueableInterface::class)){
             return true;
         }
 
@@ -38,11 +38,11 @@ 
         $reflection = new \ReflectionClass($object);
 
         $attribute = $this->reader->firstClassMetadata($reflection, Queueable::class);
-        if ($attribute !== null) {
+        if ($attribute !== null){
             return $attribute->queue;
         }
 
-        if (\is_object($object) && $reflection->hasMethod('getQueue')) {
+        if (\is_object($object) && $reflection->hasMethod('getQueue')){
             return $reflection->getMethod('getQueue')->invoke($object);
         }
 

Braces +6 added lines, -3 removed lines

@@ -23,7 +23,8 @@ 
     {
         $reflection = new \ReflectionClass($object);
 
-        if ($reflection->implementsInterface(QueueableInterface::class)) {
+        if ($reflection->implementsInterface(QueueableInterface::class))
+        {
             return true;
         }
 
@@ -38,11 +39,13 @@ 
         $reflection = new \ReflectionClass($object);
 
         $attribute = $this->reader->firstClassMetadata($reflection, Queueable::class);
-        if ($attribute !== null) {
+        if ($attribute !== null)
+        {
             return $attribute->queue;
         }
 
-        if (\is_object($object) && $reflection->hasMethod('getQueue')) {
+        if (\is_object($object) && $reflection->hasMethod('getQueue'))
+        {
             return $reflection->getMethod('getQueue')->invoke($object);
         }
 

builder/ProcessParser.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 
     public function __construct(
         private readonly ProcessRunner $processRunner
-    ) {
+    ){
     }
 
     /**

builder/ValidateVersionReleaseWorker.php

Spacing +9 added lines, -9 removed lines

@@ -13,8 +13,8 @@ 
     public function __construct(
         private readonly TagParserInterface $parser,
         private ?string $gitDirectory = null
-    ) {
-        if ($gitDirectory === null) {
+    ){
+        if ($gitDirectory === null){
             $this->gitDirectory = \dirname(__DIR__);
         }
     }
@@ -32,13 +32,13 @@ 
         $mostRecentVersion = $this->findMostRecentVersion($version);
 
         // no tag yet
-        if ($mostRecentVersion === null) {
+        if ($mostRecentVersion === null){
             return;
         }
 
         // validation
         $mostRecentVersion = new Version(\strtolower($mostRecentVersion));
-        if ($version->isGreaterThan($mostRecentVersion)) {
+        if ($version->isGreaterThan($mostRecentVersion)){
             return;
         }
 
@@ -52,23 +52,23 @@ 
     private function findMostRecentVersion(Version $version): ?string
     {
         $tags = [];
-        foreach ($this->parser->parse($this->gitDirectory) as $tag) {
+        foreach ($this->parser->parse($this->gitDirectory) as $tag){
             $tag = new Version(\strtolower($tag));
 
             // all previous major versions
-            if ($version->getMajor()->getValue() > $tag->getMajor()->getValue()) {
+            if ($version->getMajor()->getValue() > $tag->getMajor()->getValue()){
                 $tags[] = $tag;
             }
 
             // all minor versions up to the requested in the requested major version
-            if ($version->getMajor()->getValue() === $tag->getMajor()->getValue()) {
-                if ($version->getMinor()->getValue() >= $tag->getMinor()->getValue()) {
+            if ($version->getMajor()->getValue() === $tag->getMajor()->getValue()){
+                if ($version->getMinor()->getValue() >= $tag->getMinor()->getValue()){
                     $tags[] = $tag;
                 }
             }
         }
 
-        if ($tags === []) {
+        if ($tags === []){
             return null;
         }
 

Braces +16 added lines, -8 removed lines

@@ -14,7 +14,8 @@ 
         private readonly TagParserInterface $parser,
         private ?string $gitDirectory = null
     ) {
-        if ($gitDirectory === null) {
+        if ($gitDirectory === null)
+        {
             $this->gitDirectory = \dirname(__DIR__);
         }
     }
@@ -32,13 +33,15 @@ 
         $mostRecentVersion = $this->findMostRecentVersion($version);
 
         // no tag yet
-        if ($mostRecentVersion === null) {
+        if ($mostRecentVersion === null)
+        {
             return;
         }
 
         // validation
         $mostRecentVersion = new Version(\strtolower($mostRecentVersion));
-        if ($version->isGreaterThan($mostRecentVersion)) {
+        if ($version->isGreaterThan($mostRecentVersion))
+        {
             return;
         }
 
@@ -52,23 +55,28 @@ 
     private function findMostRecentVersion(Version $version): ?string
     {
         $tags = [];
-        foreach ($this->parser->parse($this->gitDirectory) as $tag) {
+        foreach ($this->parser->parse($this->gitDirectory) as $tag)
+        {
             $tag = new Version(\strtolower($tag));
 
             // all previous major versions
-            if ($version->getMajor()->getValue() > $tag->getMajor()->getValue()) {
+            if ($version->getMajor()->getValue() > $tag->getMajor()->getValue())
+            {
                 $tags[] = $tag;
             }
 
             // all minor versions up to the requested in the requested major version
-            if ($version->getMajor()->getValue() === $tag->getMajor()->getValue()) {
-                if ($version->getMinor()->getValue() >= $tag->getMinor()->getValue()) {
+            if ($version->getMajor()->getValue() === $tag->getMajor()->getValue())
+            {
+                if ($version->getMinor()->getValue() >= $tag->getMinor()->getValue())
+                {
                     $tags[] = $tag;
                 }
             }
         }
 
-        if ($tags === []) {
+        if ($tags === [])
+        {
             return null;
         }