spiral / framework

src/Framework/Auth/Session/TokenStorage.php

Spacing +7 added lines, -7 removed lines

@@ -43,23 +43,23 @@ 
      */
     public function load(string $id): ?TokenInterface
     {
-        try {
+        try{
             $tokenData = $this->session->getSection(self::SESSION_SECTION)->get('token');
-            if ($tokenData === null) {
+            if ($tokenData === null){
                 return null;
             }
 
             $token = Token::unpack($tokenData);
-        } catch (Throwable $e) {
+        }catch (Throwable $e){
             throw new TokenStorageException('Unable to load session token', $e->getCode(), $e);
         }
 
-        if (!hash_equals($token->getID(), $id)) {
+        if (!hash_equals($token->getID(), $id)){
             return null;
         }
 
         $expiresAt = $token->getExpiresAt();
-        if ($expiresAt !== null && $expiresAt < new DateTimeImmutable()) {
+        if ($expiresAt !== null && $expiresAt < new DateTimeImmutable()){
             $this->delete($token);
             return null;
         }
@@ -72,12 +72,12 @@ 
      */
     public function create(array $payload, DateTimeInterface $expiresAt = null): TokenInterface
     {
-        try {
+        try{
             $token = new Token($this->randomHash(128), $payload, $expiresAt);
             $this->session->getSection(self::SESSION_SECTION)->set('token', $token->pack());
 
             return $token;
-        } catch (Throwable $e) {
+        }catch (Throwable $e){
             throw new TokenStorageException('Unable to create session token', $e->getCode(), $e);
         }
     }

Braces +16 added lines, -7 removed lines

@@ -43,23 +43,29 @@ 
      */
     public function load(string $id): ?TokenInterface
     {
-        try {
+        try
+        {
             $tokenData = $this->session->getSection(self::SESSION_SECTION)->get('token');
-            if ($tokenData === null) {
+            if ($tokenData === null)
+            {
                 return null;
             }
 
             $token = Token::unpack($tokenData);
-        } catch (Throwable $e) {
+        }
+        catch (Throwable $e)
+        {
             throw new TokenStorageException('Unable to load session token', $e->getCode(), $e);
         }
 
-        if (!hash_equals($token->getID(), $id)) {
+        if (!hash_equals($token->getID(), $id))
+        {
             return null;
         }
 
         $expiresAt = $token->getExpiresAt();
-        if ($expiresAt !== null && $expiresAt < new DateTimeImmutable()) {
+        if ($expiresAt !== null && $expiresAt < new DateTimeImmutable())
+        {
             $this->delete($token);
             return null;
         }
@@ -72,12 +78,15 @@ 
      */
     public function create(array $payload, DateTimeInterface $expiresAt = null): TokenInterface
     {
-        try {
+        try
+        {
             $token = new Token($this->randomHash(128), $payload, $expiresAt);
             $this->session->getSection(self::SESSION_SECTION)->set('token', $token->pack());
 
             return $token;
-        } catch (Throwable $e) {
+        }
+        catch (Throwable $e)
+        {
             throw new TokenStorageException('Unable to create session token', $e->getCode(), $e);
         }
     }

src/Framework/Auth/Session/Token.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
     public static function unpack(array $data): Token
     {
         $expiresAt = null;
-        if ($data['expiresAt'] !== null) {
+        if ($data['expiresAt'] !== null){
             $expiresAt = (new DateTimeImmutable())->setTimestamp($data['expiresAt']);
         }
 

Braces +2 added lines, -1 removed lines

@@ -87,7 +87,8 @@
     public static function unpack(array $data): Token
     {
         $expiresAt = null;
-        if ($data['expiresAt'] !== null) {
+        if ($data['expiresAt'] !== null)
+        {
             $expiresAt = (new DateTimeImmutable())->setTimestamp($data['expiresAt']);
         }
 

src/Framework/Command/Database/TableCommand.php

Spacing +12 added lines, -12 removed lines

@@ -44,7 +44,7 @@ 
         $database = $dbal->database($this->option('database'));
         $schema = $database->table($this->argument('table'))->getSchema();
 
-        if (!$schema->exists()) {
+        if (!$schema->exists()){
             throw new DBALException(
                 "Table {$database->getName()}.{$this->argument('table')} does not exists."
             );
@@ -58,11 +58,11 @@ 
 
         $this->describeColumns($schema);
 
-        if (!empty($indexes = $schema->getIndexes())) {
+        if (!empty($indexes = $schema->getIndexes())){
             $this->describeIndexes($database, $indexes);
         }
 
-        if (!empty($foreignKeys = $schema->getForeignKeys())) {
+        if (!empty($foreignKeys = $schema->getForeignKeys())){
             $this->describeForeignKeys($database, $foreignKeys);
         }
 
@@ -84,10 +84,10 @@ 
             ]
         );
 
-        foreach ($schema->getColumns() as $column) {
+        foreach ($schema->getColumns() as $column){
             $name = $column->getName();
 
-            if (in_array($column->getName(), $schema->getPrimaryKeys(), true)) {
+            if (in_array($column->getName(), $schema->getPrimaryKeys(), true)){
                 $name = "<fg=magenta>{$name}</fg=magenta>";
             }
 
@@ -120,7 +120,7 @@ 
         );
 
         $indexesTable = $this->table(['Name:', 'Type:', 'Columns:']);
-        foreach ($indexes as $index) {
+        foreach ($indexes as $index){
             $indexesTable->addRow(
                 [
                     $index->getName(),
@@ -155,7 +155,7 @@ 
             ]
         );
 
-        foreach ($foreignKeys as $reference) {
+        foreach ($foreignKeys as $reference){
             $foreignTable->addRow(
                 [
                     $reference->getName(),
@@ -180,13 +180,13 @@ 
     {
         $defaultValue = $column->getDefaultValue();
 
-        if ($defaultValue instanceof FragmentInterface) {
+        if ($defaultValue instanceof FragmentInterface){
             $value = $driver->getQueryCompiler()->compile(new QueryParameters(), '', $defaultValue);
 
             return "<info>{$value}</info>";
         }
 
-        if ($defaultValue instanceof \DateTimeInterface) {
+        if ($defaultValue instanceof \DateTimeInterface){
             $defaultValue = $defaultValue->format('c');
         }
 
@@ -203,11 +203,11 @@ 
 
         $abstractType = $column->getAbstractType();
 
-        if ($column->getSize()) {
+        if ($column->getSize()){
             $type .= " ({$column->getSize()})";
         }
 
-        if ($abstractType === 'decimal') {
+        if ($abstractType === 'decimal'){
             $type .= " ({$column->getPrecision()}, {$column->getScale()})";
         }
 
@@ -222,7 +222,7 @@ 
     {
         $abstractType = $column->getAbstractType();
 
-        if (in_array($abstractType, ['primary', 'bigPrimary'])) {
+        if (in_array($abstractType, ['primary', 'bigPrimary'])){
             $abstractType = "<fg=magenta>{$abstractType}</fg=magenta>";
         }
 

Braces +24 added lines, -12 removed lines

@@ -44,7 +44,8 @@ 
         $database = $dbal->database($this->option('database'));
         $schema = $database->table($this->argument('table'))->getSchema();
 
-        if (!$schema->exists()) {
+        if (!$schema->exists())
+        {
             throw new DBALException(
                 "Table {$database->getName()}.{$this->argument('table')} does not exists."
             );
@@ -58,11 +59,13 @@ 
 
         $this->describeColumns($schema);
 
-        if (!empty($indexes = $schema->getIndexes())) {
+        if (!empty($indexes = $schema->getIndexes()))
+        {
             $this->describeIndexes($database, $indexes);
         }
 
-        if (!empty($foreignKeys = $schema->getForeignKeys())) {
+        if (!empty($foreignKeys = $schema->getForeignKeys()))
+        {
             $this->describeForeignKeys($database, $foreignKeys);
         }
 
@@ -84,10 +87,12 @@ 
             ]
         );
 
-        foreach ($schema->getColumns() as $column) {
+        foreach ($schema->getColumns() as $column)
+        {
             $name = $column->getName();
 
-            if (in_array($column->getName(), $schema->getPrimaryKeys(), true)) {
+            if (in_array($column->getName(), $schema->getPrimaryKeys(), true))
+            {
                 $name = "<fg=magenta>{$name}</fg=magenta>";
             }
 
@@ -120,7 +125,8 @@ 
         );
 
         $indexesTable = $this->table(['Name:', 'Type:', 'Columns:']);
-        foreach ($indexes as $index) {
+        foreach ($indexes as $index)
+        {
             $indexesTable->addRow(
                 [
                     $index->getName(),
@@ -155,7 +161,8 @@ 
             ]
         );
 
-        foreach ($foreignKeys as $reference) {
+        foreach ($foreignKeys as $reference)
+        {
             $foreignTable->addRow(
                 [
                     $reference->getName(),
@@ -180,13 +187,15 @@ 
     {
         $defaultValue = $column->getDefaultValue();
 
-        if ($defaultValue instanceof FragmentInterface) {
+        if ($defaultValue instanceof FragmentInterface)
+        {
             $value = $driver->getQueryCompiler()->compile(new QueryParameters(), '', $defaultValue);
 
             return "<info>{$value}</info>";
         }
 
-        if ($defaultValue instanceof \DateTimeInterface) {
+        if ($defaultValue instanceof \DateTimeInterface)
+        {
             $defaultValue = $defaultValue->format('c');
         }
 
@@ -203,11 +212,13 @@ 
 
         $abstractType = $column->getAbstractType();
 
-        if ($column->getSize()) {
+        if ($column->getSize())
+        {
             $type .= " ({$column->getSize()})";
         }
 
-        if ($abstractType === 'decimal') {
+        if ($abstractType === 'decimal')
+        {
             $type .= " ({$column->getPrecision()}, {$column->getScale()})";
         }
 
@@ -222,7 +233,8 @@ 
     {
         $abstractType = $column->getAbstractType();
 
-        if (in_array($abstractType, ['primary', 'bigPrimary'])) {
+        if (in_array($abstractType, ['primary', 'bigPrimary']))
+        {
             $abstractType = "<fg=magenta>{$abstractType}</fg=magenta>";
         }
 

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

@@ -43,7 +43,7 @@
 
         if (!$schema->exists()) {
             throw new DBALException(
-                "Table {$database->getName()}.{$this->argument('table')} does not exists."
+                "table {$database->getName()}.{$this->argument('table')} does not exists."
             );
         }
 

src/Framework/Command/Migrate/StatusCommand.php

Braces +6 added lines, -3 removed lines

@@ -28,18 +28,21 @@
      */
     public function perform(FilesInterface $files): void
     {
-        if (!$this->verifyConfigured()) {
+        if (!$this->verifyConfigured())
+        {
             return;
         }
 
-        if (empty($this->migrator->getMigrations())) {
+        if (empty($this->migrator->getMigrations()))
+        {
             $this->writeln('<comment>No migrations were found.</comment>');
 
             return;
         }
 
         $table = $this->table(['Migration', 'Created at', 'Executed at']);
-        foreach ($this->migrator->getMigrations() as $migration) {
+        foreach ($this->migrator->getMigrations() as $migration)
+        {
             $state = $migration->getState();
 
             $table->addRow(

Spacing +4 added lines, -4 removed lines

@@ -28,18 +28,18 @@ 
      */
     public function perform(FilesInterface $files): void
     {
-        if (!$this->verifyConfigured()) {
+        if (!$this->verifyConfigured()){
             return;
         }
 
-        if (empty($this->migrator->getMigrations())) {
+        if (empty($this->migrator->getMigrations())){
             $this->writeln('<comment>No migrations were found.</comment>');
 
             return;
         }
 
         $table = $this->table(['Migration', 'Created at', 'Executed at']);
-        foreach ($this->migrator->getMigrations() as $migration) {
+        foreach ($this->migrator->getMigrations() as $migration){
             $state = $migration->getState();
 
             $table->addRow(
@@ -48,7 +48,7 @@ 
                     $state->getTimeCreated()->format('Y-m-d H:i:s'),
                     $state->getStatus() == State::STATUS_PENDING
                         ? self::PENDING
-                        : '<info>' . $state->getTimeExecuted()->format('Y-m-d H:i:s') . '</info>',
+                        : '<info>'.$state->getTimeExecuted()->format('Y-m-d H:i:s').'</info>',
                 ]
             );
         }

src/Framework/Command/Migrate/RollbackCommand.php

Spacing +3 added lines, -3 removed lines

@@ -24,14 +24,14 @@ 
      */
     public function perform(): void
     {
-        if (!$this->verifyConfigured() || !$this->verifyEnvironment()) {
+        if (!$this->verifyConfigured() || !$this->verifyEnvironment()){
             //Making sure we can safely migrate in this environment
             return;
         }
 
         $found = false;
         $count = !$this->option('all') ? 1 : PHP_INT_MAX;
-        while ($count > 0 && ($migration = $this->migrator->rollback())) {
+        while ($count > 0 && ($migration = $this->migrator->rollback())){
             $found = true;
             $count--;
             $this->sprintf(
@@ -40,7 +40,7 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found){
             $this->writeln('<fg=red>No executed migrations were found.</fg=red>');
         }
     }

Braces +6 added lines, -3 removed lines

@@ -24,14 +24,16 @@ 
      */
     public function perform(): void
     {
-        if (!$this->verifyConfigured() || !$this->verifyEnvironment()) {
+        if (!$this->verifyConfigured() || !$this->verifyEnvironment())
+        {
             //Making sure we can safely migrate in this environment
             return;
         }
 
         $found = false;
         $count = !$this->option('all') ? 1 : PHP_INT_MAX;
-        while ($count > 0 && ($migration = $this->migrator->rollback())) {
+        while ($count > 0 && ($migration = $this->migrator->rollback()))
+        {
             $found = true;
             $count--;
             $this->sprintf(
@@ -40,7 +42,8 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found)
+        {
             $this->writeln('<fg=red>No executed migrations were found.</fg=red>');
         }
     }

src/Framework/Command/Migrate/ReplayCommand.php

Spacing +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
      */
     public function perform(Console $console): void
     {
-        if (!$this->verifyEnvironment()) {
+        if (!$this->verifyEnvironment()){
             //Making sure we can safely migrate in this environment
             return;
         }
@@ -35,9 +35,9 @@ 
         $rollback = ['--force' => true];
         $migrate = ['--force' => true];
 
-        if ($this->option('all')) {
+        if ($this->option('all')){
             $rollback['--all'] = true;
-        } else {
+        }else{
             $migrate['--one'] = true;
         }
 

Braces +7 added lines, -3 removed lines

@@ -27,7 +27,8 @@ 
      */
     public function perform(Console $console): void
     {
-        if (!$this->verifyEnvironment()) {
+        if (!$this->verifyEnvironment())
+        {
             //Making sure we can safely migrate in this environment
             return;
         }
@@ -35,9 +36,12 @@ 
         $rollback = ['--force' => true];
         $migrate = ['--force' => true];
 
-        if ($this->option('all')) {
+        if ($this->option('all'))
+        {
             $rollback['--all'] = true;
-        } else {
+        }
+        else
+        {
             $migrate['--one'] = true;
         }
 

src/Framework/Command/Migrate/MigrateCommand.php

Spacing +3 added lines, -3 removed lines

@@ -24,14 +24,14 @@ 
      */
     public function perform(): void
     {
-        if (!$this->verifyConfigured() || !$this->verifyEnvironment()) {
+        if (!$this->verifyConfigured() || !$this->verifyEnvironment()){
             return;
         }
 
         $found = false;
         $count = $this->option('one') ? 1 : PHP_INT_MAX;
 
-        while ($count > 0 && ($migration = $this->migrator->run())) {
+        while ($count > 0 && ($migration = $this->migrator->run())){
             $found = true;
             $count--;
 
@@ -41,7 +41,7 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found){
             $this->writeln('<fg=red>No outstanding migrations were found.</fg=red>');
         }
     }

Braces +6 added lines, -3 removed lines

@@ -24,14 +24,16 @@ 
      */
     public function perform(): void
     {
-        if (!$this->verifyConfigured() || !$this->verifyEnvironment()) {
+        if (!$this->verifyConfigured() || !$this->verifyEnvironment())
+        {
             return;
         }
 
         $found = false;
         $count = $this->option('one') ? 1 : PHP_INT_MAX;
 
-        while ($count > 0 && ($migration = $this->migrator->run())) {
+        while ($count > 0 && ($migration = $this->migrator->run()))
+        {
             $found = true;
             $count--;
 
@@ -41,7 +43,8 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found)
+        {
             $this->writeln('<fg=red>No outstanding migrations were found.</fg=red>');
         }
     }

src/Framework/Command/Migrate/AbstractCommand.php

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
      */
     protected function verifyConfigured(): bool
     {
-        if (!$this->migrator->isConfigured()) {
+        if (!$this->migrator->isConfigured()){
             $this->writeln(
                 "<fg=red>Migrations are not configured yet, run '<info>migrate:init</info>' first.</fg=red>"
             );
@@ -59,14 +59,14 @@ 
      */
     protected function verifyEnvironment(): bool
     {
-        if ($this->option('force') || $this->config->isSafe()) {
+        if ($this->option('force') || $this->config->isSafe()){
             //Safe to run
             return true;
         }
 
         $this->writeln('<fg=red>Confirmation is required to run migrations!</fg=red>');
 
-        if (!$this->askConfirmation()) {
+        if (!$this->askConfirmation()){
             $this->writeln('<comment>Cancelling operation...</comment>');
 
             return false;

Braces +6 added lines, -3 removed lines

@@ -41,7 +41,8 @@ 
      */
     protected function verifyConfigured(): bool
     {
-        if (!$this->migrator->isConfigured()) {
+        if (!$this->migrator->isConfigured())
+        {
             $this->writeln(
                 "<fg=red>Migrations are not configured yet, run '<info>migrate:init</info>' first.</fg=red>"
             );
@@ -59,14 +60,16 @@ 
      */
     protected function verifyEnvironment(): bool
     {
-        if ($this->option('force') || $this->config->isSafe()) {
+        if ($this->option('force') || $this->config->isSafe())
+        {
             //Safe to run
             return true;
         }
 
         $this->writeln('<fg=red>Confirmation is required to run migrations!</fg=red>');
 
-        if (!$this->askConfirmation()) {
+        if (!$this->askConfirmation())
+        {
             $this->writeln('<comment>Cancelling operation...</comment>');
 
             return false;

src/Framework/Command/Views/CompileCommand.php

Spacing +14 added lines, -14 removed lines

@@ -34,13 +34,13 @@ 
         $generator = new ContextGenerator($views->getContext());
 
         $contexts = $generator->generate();
-        if (empty($contexts)) {
+        if (empty($contexts)){
             $contexts[] = $views->getContext();
         }
 
-        foreach ($contexts as $context) {
-            foreach ($views->getEngines() as $engine) {
-                if ($engine instanceof NativeEngine) {
+        foreach ($contexts as $context){
+            foreach ($views->getEngines() as $engine){
+                if ($engine instanceof NativeEngine){
                     // no need to compile
                     continue;
                 }
@@ -64,19 +64,19 @@ 
             $this->describeContext($context) ?? 'default'
         );
 
-        foreach ($engine->getLoader()->list() as $path) {
+        foreach ($engine->getLoader()->list() as $path){
             $start = microtime(true);
-            try {
+            try{
                 $engine->reset($path, $context);
                 $engine->compile($path, $context);
 
-                if ($this->isVerbose()) {
+                if ($this->isVerbose()){
                     $this->sprintf('<info>•</info> %s', $path);
                 }
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 $this->renderError($path, $e);
                 continue;
-            } finally {
+            }finally{
                 $this->renderElapsed($start);
             }
         }
@@ -90,7 +90,7 @@ 
      */
     protected function renderError(string $path, \Throwable $e): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose()){
             return;
         }
 
@@ -110,7 +110,7 @@ 
     {
         $values = [];
 
-        foreach ($context->getDependencies() as $dependency) {
+        foreach ($context->getDependencies() as $dependency){
             $values[] = sprintf(
                 '%s%s%s:%s%s%s',
                 Color::LIGHT_WHITE,
@@ -141,11 +141,11 @@ 
      */
     private function renderSuccess(string $lastPath = null): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose()){
             return;
         }
 
-        if ($lastPath === null) {
+        if ($lastPath === null){
             $this->writeln('• no views found');
         }
 
@@ -157,7 +157,7 @@ 
      */
     private function renderElapsed(float $start): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose()){
             return;
         }
 

Braces +30 added lines, -14 removed lines

@@ -34,13 +34,17 @@ 
         $generator = new ContextGenerator($views->getContext());
 
         $contexts = $generator->generate();
-        if (empty($contexts)) {
+        if (empty($contexts))
+        {
             $contexts[] = $views->getContext();
         }
 
-        foreach ($contexts as $context) {
-            foreach ($views->getEngines() as $engine) {
-                if ($engine instanceof NativeEngine) {
+        foreach ($contexts as $context)
+        {
+            foreach ($views->getEngines() as $engine)
+            {
+                if ($engine instanceof NativeEngine)
+                {
                     // no need to compile
                     continue;
                 }
@@ -64,19 +68,26 @@ 
             $this->describeContext($context) ?? 'default'
         );
 
-        foreach ($engine->getLoader()->list() as $path) {
+        foreach ($engine->getLoader()->list() as $path)
+        {
             $start = microtime(true);
-            try {
+            try
+            {
                 $engine->reset($path, $context);
                 $engine->compile($path, $context);
 
-                if ($this->isVerbose()) {
+                if ($this->isVerbose())
+                {
                     $this->sprintf('<info>•</info> %s', $path);
                 }
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 $this->renderError($path, $e);
                 continue;
-            } finally {
+            }
+            finally
+            {
                 $this->renderElapsed($start);
             }
         }
@@ -90,7 +101,8 @@ 
      */
     protected function renderError(string $path, \Throwable $e): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose())
+        {
             return;
         }
 
@@ -110,7 +122,8 @@ 
     {
         $values = [];
 
-        foreach ($context->getDependencies() as $dependency) {
+        foreach ($context->getDependencies() as $dependency)
+        {
             $values[] = sprintf(
                 '%s%s%s:%s%s%s',
                 Color::LIGHT_WHITE,
@@ -141,11 +154,13 @@ 
      */
     private function renderSuccess(string $lastPath = null): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose())
+        {
             return;
         }
 
-        if ($lastPath === null) {
+        if ($lastPath === null)
+        {
             $this->writeln('• no views found');
         }
 
@@ -157,7 +172,8 @@ 
      */
     private function renderElapsed(float $start): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose())
+        {
             return;
         }