spiral / framework

tests/bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -9,4 +9,4 @@
 ini_set('display_errors', true);
 
 //Composer
-require dirname(__DIR__) . '/vendor/autoload.php';
\ No newline at end of file
+require dirname(__DIR__).'/vendor/autoload.php';
\ No newline at end of file

src/Command/Migrate/MigrateCommand.php

Spacing +3 added lines, -3 removed lines

@@ -23,14 +23,14 @@ 
      */
     public function perform()
     {
-        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--;
 
@@ -40,7 +40,7 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found){
             $this->writeln("<fg=red>No outstanding migrations were found.</fg=red>");
         }
     }

Braces +6 added lines, -3 removed lines

@@ -23,14 +23,16 @@ 
      */
     public function perform()
     {
-        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--;
 
@@ -40,7 +42,8 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found)
+        {
             $this->writeln("<fg=red>No outstanding migrations were found.</fg=red>");
         }
     }

src/Command/Migrate/AbstractCommand.php

Spacing +3 added lines, -3 removed lines

@@ -40,7 +40,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>"
             );
@@ -58,14 +58,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

@@ -40,7 +40,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>"
             );
@@ -58,14 +59,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/Command/Migrate/StatusCommand.php

Spacing +4 added lines, -4 removed lines

@@ -27,18 +27,18 @@ 
      */
     public function perform(FilesInterface $files)
     {
-        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', 'Filename', 'Created at', 'Executed at']);
-        foreach ($this->migrator->getMigrations() as $migration) {
+        foreach ($this->migrator->getMigrations() as $migration){
             $filename = (new \ReflectionClass($migration))->getFileName();
 
             $state = $migration->getState();
@@ -51,7 +51,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>'
             ]);
         }
 

Braces +6 added lines, -3 removed lines

@@ -27,18 +27,21 @@
      */
     public function perform(FilesInterface $files)
     {
-        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', 'Filename', 'Created at', 'Executed at']);
-        foreach ($this->migrator->getMigrations() as $migration) {
+        foreach ($this->migrator->getMigrations() as $migration)
+        {
             $filename = (new \ReflectionClass($migration))->getFileName();
 
             $state = $migration->getState();

src/Command/Migrate/RollbackCommand.php

Spacing +3 added lines, -3 removed lines

@@ -23,14 +23,14 @@ 
      */
     public function perform()
     {
-        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(
@@ -39,7 +39,7 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found){
             $this->writeln("<fg=red>No executed migrations were found.</fg=red>");
         }
     }

Braces +6 added lines, -3 removed lines

@@ -23,14 +23,16 @@ 
      */
     public function perform()
     {
-        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(
@@ -39,7 +41,8 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found)
+        {
             $this->writeln("<fg=red>No executed migrations were found.</fg=red>");
         }
     }

src/Command/Database/TableCommand.php

Spacing +12 added lines, -12 removed lines

@@ -41,7 +41,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."
             );
@@ -55,11 +55,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);
         }
 
@@ -79,34 +79,34 @@ 
             'Default Value:'
         ]);
 
-        foreach ($schema->getColumns() as $column) {
+        foreach ($schema->getColumns() as $column){
             $name = $column->getName();
             $type = $column->getType();
 
             $abstractType = $column->getAbstractType();
             $defaultValue = $column->getDefaultValue();
 
-            if ($column->getSize()) {
+            if ($column->getSize()){
                 $type .= " ({$column->getSize()})";
             }
 
-            if ($abstractType == 'decimal') {
+            if ($abstractType == 'decimal'){
                 $type .= " ({$column->getPrecision()}, {$column->getScale()})";
             }
 
-            if (in_array($column->getName(), $schema->getPrimaryKeys())) {
+            if (in_array($column->getName(), $schema->getPrimaryKeys())){
                 $name = "<fg=magenta>{$name}</fg=magenta>";
             }
 
-            if (in_array($abstractType, ['primary', 'bigPrimary'])) {
+            if (in_array($abstractType, ['primary', 'bigPrimary'])){
                 $abstractType = "<fg=magenta>{$abstractType}</fg=magenta>";
             }
 
-            if ($defaultValue instanceof FragmentInterface) {
+            if ($defaultValue instanceof FragmentInterface){
                 $defaultValue = "<info>{$defaultValue}</info>";
             }
 
-            if ($defaultValue instanceof \DateTimeInterface) {
+            if ($defaultValue instanceof \DateTimeInterface){
                 $defaultValue = $defaultValue->format('c');
             }
 
@@ -135,7 +135,7 @@ 
         );
 
         $indexesTable = $this->table(['Name:', 'Type:', 'Columns:']);
-        foreach ($indexes as $index) {
+        foreach ($indexes as $index){
             $indexesTable->addRow([
                 $index->getName(),
                 $index->isUnique() ? 'UNIQUE INDEX' : 'INDEX',
@@ -166,7 +166,7 @@ 
             'On Update:'
         ]);
 
-        foreach ($foreignKeys as $reference) {
+        foreach ($foreignKeys as $reference){
             $foreignTable->addRow([
                 $reference->getName(),
                 $reference->getColumn(),

Braces +24 added lines, -12 removed lines

@@ -41,7 +41,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."
             );
@@ -55,11 +56,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);
         }
 
@@ -79,34 +82,41 @@ 
             'Default Value:'
         ]);
 
-        foreach ($schema->getColumns() as $column) {
+        foreach ($schema->getColumns() as $column)
+        {
             $name = $column->getName();
             $type = $column->getType();
 
             $abstractType = $column->getAbstractType();
             $defaultValue = $column->getDefaultValue();
 
-            if ($column->getSize()) {
+            if ($column->getSize())
+            {
                 $type .= " ({$column->getSize()})";
             }
 
-            if ($abstractType == 'decimal') {
+            if ($abstractType == 'decimal')
+            {
                 $type .= " ({$column->getPrecision()}, {$column->getScale()})";
             }
 
-            if (in_array($column->getName(), $schema->getPrimaryKeys())) {
+            if (in_array($column->getName(), $schema->getPrimaryKeys()))
+            {
                 $name = "<fg=magenta>{$name}</fg=magenta>";
             }
 
-            if (in_array($abstractType, ['primary', 'bigPrimary'])) {
+            if (in_array($abstractType, ['primary', 'bigPrimary']))
+            {
                 $abstractType = "<fg=magenta>{$abstractType}</fg=magenta>";
             }
 
-            if ($defaultValue instanceof FragmentInterface) {
+            if ($defaultValue instanceof FragmentInterface)
+            {
                 $defaultValue = "<info>{$defaultValue}</info>";
             }
 
-            if ($defaultValue instanceof \DateTimeInterface) {
+            if ($defaultValue instanceof \DateTimeInterface)
+            {
                 $defaultValue = $defaultValue->format('c');
             }
 
@@ -135,7 +145,8 @@ 
         );
 
         $indexesTable = $this->table(['Name:', 'Type:', 'Columns:']);
-        foreach ($indexes as $index) {
+        foreach ($indexes as $index)
+        {
             $indexesTable->addRow([
                 $index->getName(),
                 $index->isUnique() ? 'UNIQUE INDEX' : 'INDEX',
@@ -166,7 +177,8 @@ 
             'On Update:'
         ]);
 
-        foreach ($foreignKeys as $reference) {
+        foreach ($foreignKeys as $reference)
+        {
             $foreignTable->addRow([
                 $reference->getName(),
                 $reference->getColumn(),

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/Command/Views/ResetCommand.php

Spacing +6 added lines, -6 removed lines

@@ -26,20 +26,20 @@ 
      */
     public function perform(ViewsConfig $config, FilesInterface $files)
     {
-        if (!$files->exists($config->cacheDirectory())) {
+        if (!$files->exists($config->cacheDirectory())){
             $this->writeln("Cache directory is missing, no cache to be cleaned.");
 
             return;
         }
 
-        if ($this->isVerbose()) {
+        if ($this->isVerbose()){
             $this->writeln("<info>Cleaning view cache:</info>");
         }
 
-        foreach ($files->getFiles($config->cacheDirectory()) as $filename) {
-            try {
+        foreach ($files->getFiles($config->cacheDirectory()) as $filename){
+            try{
                 $files->delete($filename);
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 $this->sprintf("<fg=red>[errored]</fg=red> `%s`: <fg=red>%s</fg=red>\n",
                     $files->relativePath($filename, $config->cacheDirectory()),
                     $e->getMessage()
@@ -48,7 +48,7 @@ 
                 continue;
             }
 
-            if ($this->isVerbose()) {
+            if ($this->isVerbose()){
                 $this->sprintf(
                     "<fg=green>[deleted]</fg=green> `%s`\n",
                     $files->relativePath($filename, $config->cacheDirectory())

Braces +13 added lines, -6 removed lines

@@ -26,20 +26,26 @@ 
      */
     public function perform(ViewsConfig $config, FilesInterface $files)
     {
-        if (!$files->exists($config->cacheDirectory())) {
+        if (!$files->exists($config->cacheDirectory()))
+        {
             $this->writeln("Cache directory is missing, no cache to be cleaned.");
 
             return;
         }
 
-        if ($this->isVerbose()) {
+        if ($this->isVerbose())
+        {
             $this->writeln("<info>Cleaning view cache:</info>");
         }
 
-        foreach ($files->getFiles($config->cacheDirectory()) as $filename) {
-            try {
+        foreach ($files->getFiles($config->cacheDirectory()) as $filename)
+        {
+            try
+            {
                 $files->delete($filename);
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 $this->sprintf("<fg=red>[errored]</fg=red> `%s`: <fg=red>%s</fg=red>\n",
                     $files->relativePath($filename, $config->cacheDirectory()),
                     $e->getMessage()
@@ -48,7 +54,8 @@ 
                 continue;
             }
 
-            if ($this->isVerbose()) {
+            if ($this->isVerbose())
+            {
                 $this->sprintf(
                     "<fg=green>[deleted]</fg=green> `%s`\n",
                     $files->relativePath($filename, $config->cacheDirectory())

src/Module/Publisher.php

Spacing +9 added lines, -9 removed lines

@@ -43,18 +43,18 @@ 
         string $destination,
         string $mergeMode = self::FOLLOW,
         int $mode = FilesInterface::READONLY
-    ) {
-        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;
             }
         }
@@ -75,8 +75,8 @@ 
         string $destination,
         string $mergeMode = self::REPLACE,
         int $mode = FilesInterface::READONLY
-    ) {
-        if (!$this->files->isDirectory($directory)) {
+    ){
+        if (!$this->files->isDirectory($directory)){
             throw new PublishException("Given '{$directory}' is not valid directory");
         }
 
@@ -86,10 +86,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

@@ -44,17 +44,21 @@ 
         string $mergeMode = self::FOLLOW,
         int $mode = FilesInterface::READONLY
     ) {
-        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;
             }
         }
@@ -76,7 +80,8 @@ 
         string $mergeMode = self::REPLACE,
         int $mode = FilesInterface::READONLY
     ) {
-        if (!$this->files->isDirectory($directory)) {
+        if (!$this->files->isDirectory($directory))
+        {
             throw new PublishException("Given '{$directory}' is not valid directory");
         }
 
@@ -86,7 +91,8 @@ 
         /**
          * @var SplFileInfo $file
          */
-        foreach ($finder->getIterator() as $file) {
+        foreach ($finder->getIterator() as $file)
+        {
             $this->publish(
                 (string)$file,
                 $destination . '/' . $file->getRelativePathname(),

src/Http/PaginationFactory.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
      */
     public function createPaginator(string $parameter, int $limit = 25): PaginatorInterface
     {
-        if (!$this->container->has(ServerRequestInterface::class)) {
+        if (!$this->container->has(ServerRequestInterface::class)){
             throw new ScopeException("Unable to create paginator, no request scope found");
         }
         /**
@@ -53,7 +53,7 @@ 
 
         //Getting page number
         $page = 0;
-        if (!empty($query[$parameter]) && is_scalar($query[$parameter])) {
+        if (!empty($query[$parameter]) && is_scalar($query[$parameter])){
             $page = (int)$query[$parameter];
         }
 

Braces +4 added lines, -2 removed lines

@@ -43,7 +43,8 @@ 
      */
     public function createPaginator(string $parameter, int $limit = 25): PaginatorInterface
     {
-        if (!$this->container->has(ServerRequestInterface::class)) {
+        if (!$this->container->has(ServerRequestInterface::class))
+        {
             throw new ScopeException("Unable to create paginator, no request scope found");
         }
         /**
@@ -53,7 +54,8 @@ 
 
         //Getting page number
         $page = 0;
-        if (!empty($query[$parameter]) && is_scalar($query[$parameter])) {
+        if (!empty($query[$parameter]) && is_scalar($query[$parameter]))
+        {
             $page = (int)$query[$parameter];
         }