spiral / framework

src/Framework/Module/Publisher.php

Spacing +7 added lines, -7 removed lines

@@ -45,17 +45,17 @@ 
         string $mergeMode = self::FOLLOW,
         int $mode = FilesInterface::READONLY
     ): void {
-        if (!$this->files->isFile($filename)) {
+        if (!$this->files->isFile($filename)){
             throw new PublishException("Given '{$filename}' is not valid file");
         }
 
-        if ($this->files->exists($destination)) {
-            if ($this->files->md5($destination) == $this->files->md5($filename)) {
+        if ($this->files->exists($destination)){
+            if ($this->files->md5($destination) == $this->files->md5($filename)){
                 //Nothing to do
                 return;
             }
 
-            if ($mergeMode == self::FOLLOW) {
+            if ($mergeMode == self::FOLLOW){
                 return;
             }
         }
@@ -77,7 +77,7 @@ 
         string $mergeMode = self::REPLACE,
         int $mode = FilesInterface::READONLY
     ): void {
-        if (!$this->files->isDirectory($directory)) {
+        if (!$this->files->isDirectory($directory)){
             throw new PublishException("Given '{$directory}' is not valid directory");
         }
 
@@ -87,10 +87,10 @@ 
         /**
          * @var SplFileInfo $file
          */
-        foreach ($finder->getIterator() as $file) {
+        foreach ($finder->getIterator() as $file){
             $this->publish(
                 (string)$file,
-                $destination . '/' . $file->getRelativePathname(),
+                $destination.'/'.$file->getRelativePathname(),
                 $mergeMode,
                 $mode
             );

Braces +12 added lines, -6 removed lines

@@ -45,17 +45,21 @@ 
         string $mergeMode = self::FOLLOW,
         int $mode = FilesInterface::READONLY
     ): void {
-        if (!$this->files->isFile($filename)) {
+        if (!$this->files->isFile($filename))
+        {
             throw new PublishException("Given '{$filename}' is not valid file");
         }
 
-        if ($this->files->exists($destination)) {
-            if ($this->files->md5($destination) == $this->files->md5($filename)) {
+        if ($this->files->exists($destination))
+        {
+            if ($this->files->md5($destination) == $this->files->md5($filename))
+            {
                 //Nothing to do
                 return;
             }
 
-            if ($mergeMode == self::FOLLOW) {
+            if ($mergeMode == self::FOLLOW)
+            {
                 return;
             }
         }
@@ -77,7 +81,8 @@ 
         string $mergeMode = self::REPLACE,
         int $mode = FilesInterface::READONLY
     ): void {
-        if (!$this->files->isDirectory($directory)) {
+        if (!$this->files->isDirectory($directory))
+        {
             throw new PublishException("Given '{$directory}' is not valid directory");
         }
 
@@ -87,7 +92,8 @@ 
         /**
          * @var SplFileInfo $file
          */
-        foreach ($finder->getIterator() as $file) {
+        foreach ($finder->getIterator() as $file)
+        {
             $this->publish(
                 (string)$file,
                 $destination . '/' . $file->getRelativePathname(),

src/Framework/Validation/Checker/EntityChecker.php

Spacing +7 added lines, -7 removed lines

@@ -38,7 +38,7 @@ 
     public function exists($value, string $role, ?string $field = null): bool
     {
         $repository = $this->orm->getRepository($role);
-        if ($field === null) {
+        if ($field === null){
             return $repository->findByPK($value) !== null;
         }
 
@@ -57,7 +57,7 @@ 
         $values = $this->withValues($withFields);
         $values[$field] = $value;
 
-        if ($this->isProvidedByContext($role, $values)) {
+        if ($this->isProvidedByContext($role, $values)){
             return true;
         }
 
@@ -71,8 +71,8 @@ 
     private function withValues(array $fields): array
     {
         $values = [];
-        foreach ($fields as $field) {
-            if ($this->getValidator()->hasValue($field)) {
+        foreach ($fields as $field){
+            if ($this->getValidator()->hasValue($field)){
                 $values[$field] = $this->getValidator()->getValue($field);
             }
         }
@@ -88,13 +88,13 @@ 
     private function isProvidedByContext(string $role, array $values): bool
     {
         $entity = $this->getValidator()->getContext();
-        if (!is_object($entity) || !$this->orm->getHeap()->has($entity)) {
+        if (!is_object($entity) || !$this->orm->getHeap()->has($entity)){
             return false;
         }
 
         $extract = $this->orm->getMapper($role)->extract($entity);
-        foreach ($values as $field => $value) {
-            if (!isset($extract[$field]) || $extract[$field] !== $value) {
+        foreach ($values as $field => $value){
+            if (!isset($extract[$field]) || $extract[$field] !== $value){
                 return false;
             }
         }

Braces +14 added lines, -7 removed lines

@@ -38,7 +38,8 @@ 
     public function exists($value, string $role, ?string $field = null): bool
     {
         $repository = $this->orm->getRepository($role);
-        if ($field === null) {
+        if ($field === null)
+        {
             return $repository->findByPK($value) !== null;
         }
 
@@ -57,7 +58,8 @@ 
         $values = $this->withValues($withFields);
         $values[$field] = $value;
 
-        if ($this->isProvidedByContext($role, $values)) {
+        if ($this->isProvidedByContext($role, $values))
+        {
             return true;
         }
 
@@ -71,8 +73,10 @@ 
     private function withValues(array $fields): array
     {
         $values = [];
-        foreach ($fields as $field) {
-            if ($this->getValidator()->hasValue($field)) {
+        foreach ($fields as $field)
+        {
+            if ($this->getValidator()->hasValue($field))
+            {
                 $values[$field] = $this->getValidator()->getValue($field);
             }
         }
@@ -88,13 +92,16 @@ 
     private function isProvidedByContext(string $role, array $values): bool
     {
         $entity = $this->getValidator()->getContext();
-        if (!is_object($entity) || !$this->orm->getHeap()->has($entity)) {
+        if (!is_object($entity) || !$this->orm->getHeap()->has($entity))
+        {
             return false;
         }
 
         $extract = $this->orm->getMapper($role)->extract($entity);
-        foreach ($values as $field => $value) {
-            if (!isset($extract[$field]) || $extract[$field] !== $value) {
+        foreach ($values as $field => $value)
+        {
+            if (!isset($extract[$field]) || $extract[$field] !== $value)
+            {
                 return false;
             }
         }

src/Framework/Snapshots/FileSnapshooter.php

Spacing +7 added lines, -7 removed lines

@@ -53,7 +53,7 @@ 
         HandlerInterface $handler,
         FilesInterface $files,
         LoggerInterface $logger = null
-    ) {
+    ){
         $this->directory = $directory;
         $this->maxFiles = $maxFiles;
         $this->verbosity = $verbosity;
@@ -69,7 +69,7 @@ 
     {
         $snapshot = new Snapshot($this->getID($e), $e);
 
-        if ($this->logger !== null) {
+        if ($this->logger !== null){
             $this->logger->error($snapshot->getMessage());
         }
 
@@ -102,18 +102,18 @@ 
     {
         $finder = new Finder();
         $finder->in($this->directory)->sort(
-            static function (SplFileInfo $a, SplFileInfo $b) {
+            static function (SplFileInfo $a, SplFileInfo $b){
                 return $b->getMTime() - $a->getMTime();
             }
         );
 
         $count = 0;
-        foreach ($finder as $file) {
+        foreach ($finder as $file){
             $count++;
-            if ($count > $this->maxFiles) {
-                try {
+            if ($count > $this->maxFiles){
+                try{
                     $this->files->delete($file->getRealPath());
-                } catch (FilesException $e) {
+                }catch (FilesException $e){
                     // ignore
                 }
             }

Braces +13 added lines, -6 removed lines

@@ -69,7 +69,8 @@ 
     {
         $snapshot = new Snapshot($this->getID($e), $e);
 
-        if ($this->logger !== null) {
+        if ($this->logger !== null)
+        {
             $this->logger->error($snapshot->getMessage());
         }
 
@@ -102,18 +103,24 @@ 
     {
         $finder = new Finder();
         $finder->in($this->directory)->sort(
-            static function (SplFileInfo $a, SplFileInfo $b) {
+            static function (SplFileInfo $a, SplFileInfo $b)
+            {
                 return $b->getMTime() - $a->getMTime();
             }
         );
 
         $count = 0;
-        foreach ($finder as $file) {
+        foreach ($finder as $file)
+        {
             $count++;
-            if ($count > $this->maxFiles) {
-                try {
+            if ($count > $this->maxFiles)
+            {
+                try
+                {
                     $this->files->delete($file->getRealPath());
-                } catch (FilesException $e) {
+                }
+                catch (FilesException $e)
+                {
                     // ignore
                 }
             }

src/Framework/Debug/StateCollector/EnvironmentCollector.php

Spacing +2 added lines, -2 removed lines

@@ -43,8 +43,8 @@
     {
         $state->setTag('php', phpversion());
 
-        if ($this->container->get(DispatcherInterface::class)) {
-            switch (get_class($this->container->get(DispatcherInterface::class))) {
+        if ($this->container->get(DispatcherInterface::class)){
+            switch (get_class($this->container->get(DispatcherInterface::class))){
                 case RrDispatcher::class:
                     $state->setTag('dispatcher', 'roadrunner');
                     break;

Braces +4 added lines, -2 removed lines

@@ -43,8 +43,10 @@
     {
         $state->setTag('php', phpversion());
 
-        if ($this->container->get(DispatcherInterface::class)) {
-            switch (get_class($this->container->get(DispatcherInterface::class))) {
+        if ($this->container->get(DispatcherInterface::class))
+        {
+            switch (get_class($this->container->get(DispatcherInterface::class)))
+            {
                 case RrDispatcher::class:
                     $state->setTag('dispatcher', 'roadrunner');
                     break;

src/Framework/Debug/StateCollector/HttpCollector.php

Spacing +3 added lines, -3 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function populate(StateInterface $state): void
     {
-        if ($this->request === null) {
+        if ($this->request === null){
             return;
         }
 
@@ -48,11 +48,11 @@ 
 
         $state->setVariable('headers', $this->request->getHeaders());
 
-        if ($this->request->getQueryParams() !== []) {
+        if ($this->request->getQueryParams() !== []){
             $state->setVariable('query', $this->request->getQueryParams());
         }
 
-        if ($this->request->getParsedBody() !== null) {
+        if ($this->request->getParsedBody() !== null){
             $state->setVariable('data', $this->request->getParsedBody());
         }
     }

Braces +6 added lines, -3 removed lines

@@ -39,7 +39,8 @@ 
      */
     public function populate(StateInterface $state): void
     {
-        if ($this->request === null) {
+        if ($this->request === null)
+        {
             return;
         }
 
@@ -48,11 +49,13 @@ 
 
         $state->setVariable('headers', $this->request->getHeaders());
 
-        if ($this->request->getQueryParams() !== []) {
+        if ($this->request->getQueryParams() !== [])
+        {
             $state->setVariable('query', $this->request->getQueryParams());
         }
 
-        if ($this->request->getParsedBody() !== null) {
+        if ($this->request->getParsedBody() !== null)
+        {
             $state->setVariable('data', $this->request->getParsedBody());
         }
     }

src/Framework/Framework/Kernel.php

Spacing +9 added lines, -9 removed lines

@@ -43,29 +43,29 @@
      */
     protected function mapDirectories(array $directories): array
     {
-        if (!isset($directories['root'])) {
+        if (!isset($directories['root'])){
             throw new BootException('Missing required directory `root`');
         }
 
-        if (!isset($directories['app'])) {
-            $directories['app'] = $directories['root'] . '/app/';
+        if (!isset($directories['app'])){
+            $directories['app'] = $directories['root'].'/app/';
         }
 
         return array_merge(
             [
                 // public root
-                'public'    => $directories['root'] . '/public/',
+                'public'    => $directories['root'].'/public/',
 
                 // vendor libraries
-                'vendor'    => $directories['root'] . '/vendor/',
+                'vendor'    => $directories['root'].'/vendor/',
 
                 // data directories
-                'runtime'   => $directories['root'] . '/runtime/',
-                'cache'     => $directories['root'] . '/runtime/cache/',
+                'runtime'   => $directories['root'].'/runtime/',
+                'cache'     => $directories['root'].'/runtime/cache/',
 
                 // application directories
-                'config'    => $directories['app'] . '/config/',
-                'resources' => $directories['app'] . '/resources/',
+                'config'    => $directories['app'].'/config/',
+                'resources' => $directories['app'].'/resources/',
             ],
             $directories
         );

Braces +4 added lines, -2 removed lines

@@ -43,11 +43,13 @@
      */
     protected function mapDirectories(array $directories): array
     {
-        if (!isset($directories['root'])) {
+        if (!isset($directories['root']))
+        {
             throw new BootException('Missing required directory `root`');
         }
 
-        if (!isset($directories['app'])) {
+        if (!isset($directories['app']))
+        {
             $directories['app'] = $directories['root'] . '/app/';
         }
 

src/Framework/Broadcast/Middleware/WebsocketsMiddleware.php

Spacing +10 added lines, -10 removed lines

@@ -50,7 +50,7 @@ 
         ScopeInterface $scope,
         ResolverInterface $resolver,
         ResponseFactoryInterface $responseFactory
-    ) {
+    ){
         $this->config = $config;
         $this->scope = $scope;
         $this->resolver = $resolver;
@@ -67,13 +67,13 @@ 
      */
     public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
     {
-        if ($request->getUri()->getPath() !== $this->config->getPath()) {
+        if ($request->getUri()->getPath() !== $this->config->getPath()){
             return $handler->handle($request);
         }
 
         // server authorization
-        if ($request->getAttribute('ws:joinServer', null) !== null) {
-            if (!$this->authorizeServer($request)) {
+        if ($request->getAttribute('ws:joinServer', null) !== null){
+            if (!$this->authorizeServer($request)){
                 return $this->responseFactory->createResponse(403);
             }
 
@@ -81,10 +81,10 @@ 
         }
 
         // topic authorization
-        if (is_string($request->getAttribute('ws:joinTopics', null))) {
+        if (is_string($request->getAttribute('ws:joinTopics', null))){
             $topics = explode(',', $request->getAttribute('ws:joinTopics'));
-            foreach ($topics as $topic) {
-                if (!$this->authorizeTopic($request, $topic)) {
+            foreach ($topics as $topic){
+                if (!$this->authorizeTopic($request, $topic)){
                     return $this->responseFactory->createResponse(403);
                 }
             }
@@ -104,7 +104,7 @@ 
     private function authorizeServer(ServerRequestInterface $request): bool
     {
         $callback = $this->config->getServerCallback();
-        if ($callback === null) {
+        if ($callback === null){
             return true;
         }
 
@@ -122,7 +122,7 @@ 
     {
         $parameters = [];
         $callback = $this->config->findTopicCallback($topic, $parameters);
-        if ($callback === null) {
+        if ($callback === null){
             return false;
         }
 
@@ -141,7 +141,7 @@ 
     {
         /** @var \ReflectionFunctionAbstract $reflection */
         $reflection = null;
-        switch (true) {
+        switch (true){
             case $callback instanceof \Closure || is_string($callback):
                 $reflection = new \ReflectionFunction($callback);
                 break;

Braces +20 added lines, -10 removed lines

@@ -67,13 +67,16 @@ 
      */
     public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
     {
-        if ($request->getUri()->getPath() !== $this->config->getPath()) {
+        if ($request->getUri()->getPath() !== $this->config->getPath())
+        {
             return $handler->handle($request);
         }
 
         // server authorization
-        if ($request->getAttribute('ws:joinServer', null) !== null) {
-            if (!$this->authorizeServer($request)) {
+        if ($request->getAttribute('ws:joinServer', null) !== null)
+        {
+            if (!$this->authorizeServer($request))
+            {
                 return $this->responseFactory->createResponse(403);
             }
 
@@ -81,10 +84,13 @@ 
         }
 
         // topic authorization
-        if (is_string($request->getAttribute('ws:joinTopics', null))) {
+        if (is_string($request->getAttribute('ws:joinTopics', null)))
+        {
             $topics = explode(',', $request->getAttribute('ws:joinTopics'));
-            foreach ($topics as $topic) {
-                if (!$this->authorizeTopic($request, $topic)) {
+            foreach ($topics as $topic)
+            {
+                if (!$this->authorizeTopic($request, $topic))
+                {
                     return $this->responseFactory->createResponse(403);
                 }
             }
@@ -104,7 +110,8 @@ 
     private function authorizeServer(ServerRequestInterface $request): bool
     {
         $callback = $this->config->getServerCallback();
-        if ($callback === null) {
+        if ($callback === null)
+        {
             return true;
         }
 
@@ -122,7 +129,8 @@ 
     {
         $parameters = [];
         $callback = $this->config->findTopicCallback($topic, $parameters);
-        if ($callback === null) {
+        if ($callback === null)
+        {
             return false;
         }
 
@@ -141,7 +149,8 @@ 
     {
         /** @var \ReflectionFunctionAbstract $reflection */
         $reflection = null;
-        switch (true) {
+        switch (true)
+        {
             case $callback instanceof \Closure || is_string($callback):
                 $reflection = new \ReflectionFunction($callback);
                 break;
@@ -159,7 +168,8 @@ 
             [
                 ServerRequestInterface::class => $request
             ],
-            function () use ($reflection, $parameters, $callback) {
+            function () use ($reflection, $parameters, $callback)
+            {
                 return call_user_func_array(
                     $callback,
                     $this->resolver->resolveArguments($reflection, $parameters)

src/Framework/Broadcast/Config/WebsocketsConfig.php

Spacing +7 added lines, -7 removed lines

@@ -34,7 +34,7 @@ 
     {
         parent::__construct($config);
 
-        foreach ($config['authorizeTopics'] as $topic => $callback) {
+        foreach ($config['authorizeTopics'] as $topic => $callback){
             $this->patterns[$this->compilePattern($topic)] = $callback;
         }
     }
@@ -62,8 +62,8 @@ 
      */
     public function findTopicCallback(string $topic, array &$matches): ?callable
     {
-        foreach ($this->patterns as $pattern => $callback) {
-            if (preg_match($pattern, $topic, $matches)) {
+        foreach ($this->patterns as $pattern => $callback){
+            if (preg_match($pattern, $topic, $matches)){
                 return $callback;
             }
         }
@@ -78,13 +78,13 @@ 
     private function compilePattern(string $topic): string
     {
         $replaces = [];
-        if (preg_match_all('/\{(\w+):?(.*?)?\}/', $topic, $matches)) {
+        if (preg_match_all('/\{(\w+):?(.*?)?\}/', $topic, $matches)){
             $variables = array_combine($matches[1], $matches[2]);
-            foreach ($variables as $key => $segment) {
-                $replaces['{' . $key . '}'] = '(?P<' . $key . '>[^\/\.]+)';
+            foreach ($variables as $key => $segment){
+                $replaces['{'.$key.'}'] = '(?P<'.$key.'>[^\/\.]+)';
             }
         }
 
-        return '/^' . strtr($topic, $replaces + ['/' => '\\/', '[' => '(?:', ']' => ')?', '.' => '\.']) . '$/iu';
+        return '/^'.strtr($topic, $replaces + ['/' => '\\/', '[' => '(?:', ']' => ')?', '.' => '\.']).'$/iu';
     }
 }

Braces +10 added lines, -5 removed lines

@@ -34,7 +34,8 @@ 
     {
         parent::__construct($config);
 
-        foreach ($config['authorizeTopics'] as $topic => $callback) {
+        foreach ($config['authorizeTopics'] as $topic => $callback)
+        {
             $this->patterns[$this->compilePattern($topic)] = $callback;
         }
     }
@@ -62,8 +63,10 @@ 
      */
     public function findTopicCallback(string $topic, array &$matches): ?callable
     {
-        foreach ($this->patterns as $pattern => $callback) {
-            if (preg_match($pattern, $topic, $matches)) {
+        foreach ($this->patterns as $pattern => $callback)
+        {
+            if (preg_match($pattern, $topic, $matches))
+            {
                 return $callback;
             }
         }
@@ -78,9 +81,11 @@ 
     private function compilePattern(string $topic): string
     {
         $replaces = [];
-        if (preg_match_all('/\{(\w+):?(.*?)?\}/', $topic, $matches)) {
+        if (preg_match_all('/\{(\w+):?(.*?)?\}/', $topic, $matches))
+        {
             $variables = array_combine($matches[1], $matches[2]);
-            foreach ($variables as $key => $segment) {
+            foreach ($variables as $key => $segment)
+            {
                 $replaces['{' . $key . '}'] = '(?P<' . $key . '>[^\/\.]+)';
             }
         }

src/Framework/Cookies/CookieManager.php

Spacing +4 added lines, -4 removed lines

@@ -174,9 +174,9 @@ 
      */
     private function getRequest(): ServerRequestInterface
     {
-        try {
+        try{
             return $this->container->get(ServerRequestInterface::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }catch (NotFoundExceptionInterface $e){
             throw new ScopeException('Unable to receive active request', $e->getCode(), $e);
         }
     }
@@ -188,9 +188,9 @@ 
      */
     private function getCookieQueue(): CookieQueue
     {
-        try {
+        try{
             return $this->container->get(CookieQueue::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }catch (NotFoundExceptionInterface $e){
             throw new ScopeException('Unable to receive cookie queue, invalid request scope', $e->getCode(), $e);
         }
     }

Braces +10 added lines, -4 removed lines

@@ -169,9 +169,12 @@ 
      */
     private function getRequest(): ServerRequestInterface
     {
-        try {
+        try
+        {
             return $this->container->get(ServerRequestInterface::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }
+        catch (NotFoundExceptionInterface $e)
+        {
             throw new ScopeException('Unable to receive active request', $e->getCode(), $e);
         }
     }
@@ -183,9 +186,12 @@ 
      */
     private function getCookieQueue(): CookieQueue
     {
-        try {
+        try
+        {
             return $this->container->get(CookieQueue::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }
+        catch (NotFoundExceptionInterface $e)
+        {
             throw new ScopeException('Unable to receive cookie queue, invalid request scope', $e->getCode(), $e);
         }
     }