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/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

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."
             );
         }
 

Spacing +12 added lines, -12 removed lines

@@ -45,7 +45,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."
             );
@@ -59,11 +59,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);
         }
 
@@ -83,10 +83,10 @@ 
             'Default Value:'
         ]);
 
-        foreach ($schema->getColumns() as $column) {
+        foreach ($schema->getColumns() as $column){
             $name = $column->getName();
 
-            if (in_array($column->getName(), $schema->getPrimaryKeys())) {
+            if (in_array($column->getName(), $schema->getPrimaryKeys())){
                 $name = "<fg=magenta>{$name}</fg=magenta>";
             }
 
@@ -115,7 +115,7 @@ 
         );
 
         $indexesTable = $this->table(['Name:', 'Type:', 'Columns:']);
-        foreach ($indexes as $index) {
+        foreach ($indexes as $index){
             $indexesTable->addRow([
                 $index->getName(),
                 $index->isUnique() ? 'UNIQUE INDEX' : 'INDEX',
@@ -146,7 +146,7 @@ 
             'On Update:'
         ]);
 
-        foreach ($foreignKeys as $reference) {
+        foreach ($foreignKeys as $reference){
             $foreignTable->addRow([
                 $reference->getName(),
                 join(', ', $reference->getColumns()),
@@ -170,11 +170,11 @@ 
 
         $abstractType = $column->getAbstractType();
 
-        if ($column->getSize()) {
+        if ($column->getSize()){
             $type .= " ({$column->getSize()})";
         }
 
-        if ($abstractType == 'decimal') {
+        if ($abstractType == 'decimal'){
             $type .= " ({$column->getPrecision()}, {$column->getScale()})";
         }
 
@@ -189,7 +189,7 @@ 
     {
         $abstractType = $column->getAbstractType();
 
-        if (in_array($abstractType, ['primary', 'bigPrimary'])) {
+        if (in_array($abstractType, ['primary', 'bigPrimary'])){
             $abstractType = "<fg=magenta>{$abstractType}</fg=magenta>";
         }
 
@@ -204,12 +204,12 @@ 
     {
         $defaultValue = $column->getDefaultValue();
 
-        if ($defaultValue instanceof FragmentInterface) {
+        if ($defaultValue instanceof FragmentInterface){
             $value = $defaultValue->compile(new QueryBindings(), $driver->getCompiler());
             return "<info>{$value}</info>";
         }
 
-        if ($defaultValue instanceof \DateTimeInterface) {
+        if ($defaultValue instanceof \DateTimeInterface){
             $defaultValue = $defaultValue->format('c');
         }
 

Braces +24 added lines, -12 removed lines

@@ -45,7 +45,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."
             );
@@ -59,11 +60,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);
         }
 
@@ -83,10 +86,12 @@ 
             'Default Value:'
         ]);
 
-        foreach ($schema->getColumns() as $column) {
+        foreach ($schema->getColumns() as $column)
+        {
             $name = $column->getName();
 
-            if (in_array($column->getName(), $schema->getPrimaryKeys())) {
+            if (in_array($column->getName(), $schema->getPrimaryKeys()))
+            {
                 $name = "<fg=magenta>{$name}</fg=magenta>";
             }
 
@@ -115,7 +120,8 @@ 
         );
 
         $indexesTable = $this->table(['Name:', 'Type:', 'Columns:']);
-        foreach ($indexes as $index) {
+        foreach ($indexes as $index)
+        {
             $indexesTable->addRow([
                 $index->getName(),
                 $index->isUnique() ? 'UNIQUE INDEX' : 'INDEX',
@@ -146,7 +152,8 @@ 
             'On Update:'
         ]);
 
-        foreach ($foreignKeys as $reference) {
+        foreach ($foreignKeys as $reference)
+        {
             $foreignTable->addRow([
                 $reference->getName(),
                 join(', ', $reference->getColumns()),
@@ -170,11 +177,13 @@ 
 
         $abstractType = $column->getAbstractType();
 
-        if ($column->getSize()) {
+        if ($column->getSize())
+        {
             $type .= " ({$column->getSize()})";
         }
 
-        if ($abstractType == 'decimal') {
+        if ($abstractType == 'decimal')
+        {
             $type .= " ({$column->getPrecision()}, {$column->getScale()})";
         }
 
@@ -189,7 +198,8 @@ 
     {
         $abstractType = $column->getAbstractType();
 
-        if (in_array($abstractType, ['primary', 'bigPrimary'])) {
+        if (in_array($abstractType, ['primary', 'bigPrimary']))
+        {
             $abstractType = "<fg=magenta>{$abstractType}</fg=magenta>";
         }
 
@@ -204,12 +214,14 @@ 
     {
         $defaultValue = $column->getDefaultValue();
 
-        if ($defaultValue instanceof FragmentInterface) {
+        if ($defaultValue instanceof FragmentInterface)
+        {
             $value = $defaultValue->compile(new QueryBindings(), $driver->getCompiler());
             return "<info>{$value}</info>";
         }
 
-        if ($defaultValue instanceof \DateTimeInterface) {
+        if ($defaultValue instanceof \DateTimeInterface)
+        {
             $defaultValue = $defaultValue->format('c');
         }
 

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];
         }
 

tests/Framework/HttpTest.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
     protected function fetchCookies(array $header)
     {
         $result = [];
-        foreach ($header as $line) {
+        foreach ($header as $line){
             $cookie = explode('=', $line);
             $result[$cookie[0]] = rawurldecode(substr($cookie[1], 0, strpos($cookie[1], ';')));
         }

Braces +2 added lines, -1 removed lines

@@ -68,7 +68,8 @@
     protected function fetchCookies(array $header)
     {
         $result = [];
-        foreach ($header as $line) {
+        foreach ($header as $line)
+        {
             $cookie = explode('=', $line);
             $result[$cookie[0]] = rawurldecode(substr($cookie[1], 0, strpos($cookie[1], ';')));
         }

tests/Framework/BaseTest.php

Spacing +4 added lines, -4 removed lines

@@ -17,10 +17,10 @@
     public function makeApp(array $env = []): TestApp
     {
         return TestApp::init([
-            'root'    => __DIR__ . '/../..',
-            'app'     => __DIR__ . '/../app',
-            'runtime' => sys_get_temp_dir() . '/spiral',
-            'cache'   => sys_get_temp_dir() . '/spiral',
+            'root'    => __DIR__.'/../..',
+            'app'     => __DIR__.'/../app',
+            'runtime' => sys_get_temp_dir().'/spiral',
+            'cache'   => sys_get_temp_dir().'/spiral',
         ], new Environment($env), false);
     }
 }
\ No newline at end of file