spiral / framework

src/Filters/src/SchemaBuilder.php

Spacing +10 added lines, -10 removed lines

@@ -70,18 +70,18 @@ 
     protected function buildMap(ReflectionEntity $filter): array
     {
         $schema = $filter->getProperty('schema', true);
-        if (empty($schema)) {
+        if (empty($schema)){
             throw new SchemaException("Filter `{$filter->getName()}` does not define any schema");
         }
 
         $result = [];
-        foreach ($schema as $field => $definition) {
+        foreach ($schema as $field => $definition){
             $optional = false;
 
             // short definition
-            if (is_string($definition)) {
+            if (is_string($definition)){
                 // simple scalar field definition
-                if (!class_exists($definition)) {
+                if (!class_exists($definition)){
                     [$source, $origin] = $this->parseDefinition($field, $definition);
                     $result[$field] = [
                         FilterProvider::SOURCE => $source,
@@ -102,25 +102,25 @@ 
                 continue;
             }
 
-            if (!is_array($definition) || count($definition) === 0) {
+            if (!is_array($definition) || count($definition) === 0){
                 throw new SchemaException(
                     "Invalid schema definition at `{$filter->getName()}`->`{$field}`"
                 );
             }
 
             // complex definition
-            if (!empty($definition[self::ORIGIN])) {
+            if (!empty($definition[self::ORIGIN])){
                 $origin = $definition[self::ORIGIN];
 
                 // [class, 'data:something.*'] vs [class, 'data:something']
                 $iterate = strpos($origin, '.*') !== false || !empty($definition[self::ITERATE]);
                 $origin = rtrim($origin, '.*');
-            } else {
+            }else{
                 $origin = $field;
                 $iterate = true;
             }
 
-            if (!empty($definition[self::OPTIONAL]) && $definition[self::OPTIONAL]) {
+            if (!empty($definition[self::OPTIONAL]) && $definition[self::OPTIONAL]){
                 $optional = true;
             }
 
@@ -133,7 +133,7 @@ 
                 FilterProvider::OPTIONAL => $optional,
             ];
 
-            if ($iterate) {
+            if ($iterate){
                 [$source, $origin] = $this->parseDefinition($field, $definition[self::ITERATE] ?? $origin);
 
                 $map[FilterProvider::ITERATE_SOURCE] = $source;
@@ -159,7 +159,7 @@ 
      */
     private function parseDefinition(string $field, string $definition): array
     {
-        if (strpos($definition, ':') === false) {
+        if (strpos($definition, ':') === false){
             return ['data', $definition ?? $field];
         }
 

Braces +21 added lines, -10 removed lines

@@ -70,18 +70,22 @@ 
     protected function buildMap(ReflectionEntity $filter): array
     {
         $schema = $filter->getProperty('schema', true);
-        if (empty($schema)) {
+        if (empty($schema))
+        {
             throw new SchemaException("Filter `{$filter->getName()}` does not define any schema");
         }
 
         $result = [];
-        foreach ($schema as $field => $definition) {
+        foreach ($schema as $field => $definition)
+        {
             $optional = false;
 
             // short definition
-            if (is_string($definition)) {
+            if (is_string($definition))
+            {
                 // simple scalar field definition
-                if (!class_exists($definition)) {
+                if (!class_exists($definition))
+                {
                     [$source, $origin] = $this->parseDefinition($field, $definition);
                     $result[$field] = [
                         FilterProvider::SOURCE => $source,
@@ -102,25 +106,30 @@ 
                 continue;
             }
 
-            if (!is_array($definition) || count($definition) === 0) {
+            if (!is_array($definition) || count($definition) === 0)
+            {
                 throw new SchemaException(
                     "Invalid schema definition at `{$filter->getName()}`->`{$field}`"
                 );
             }
 
             // complex definition
-            if (!empty($definition[self::ORIGIN])) {
+            if (!empty($definition[self::ORIGIN]))
+            {
                 $origin = $definition[self::ORIGIN];
 
                 // [class, 'data:something.*'] vs [class, 'data:something']
                 $iterate = strpos($origin, '.*') !== false || !empty($definition[self::ITERATE]);
                 $origin = rtrim($origin, '.*');
-            } else {
+            }
+            else
+            {
                 $origin = $field;
                 $iterate = true;
             }
 
-            if (!empty($definition[self::OPTIONAL]) && $definition[self::OPTIONAL]) {
+            if (!empty($definition[self::OPTIONAL]) && $definition[self::OPTIONAL])
+            {
                 $optional = true;
             }
 
@@ -133,7 +142,8 @@ 
                 FilterProvider::OPTIONAL => $optional,
             ];
 
-            if ($iterate) {
+            if ($iterate)
+            {
                 [$source, $origin] = $this->parseDefinition($field, $definition[self::ITERATE] ?? $origin);
 
                 $map[FilterProvider::ITERATE_SOURCE] = $source;
@@ -159,7 +169,8 @@ 
      */
     private function parseDefinition(string $field, string $definition): array
     {
-        if (strpos($definition, ':') === false) {
+        if (strpos($definition, ':') === false)
+        {
             return ['data', $definition ?? $field];
         }
 

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

@@ -95,7 +95,7 @@ 
                     FilterProvider::SOURCE   => null,
                     FilterProvider::ORIGIN   => $field,
                     FilterProvider::FILTER   => $definition,
-                    FilterProvider::ARRAY    => false,
+                    FilterProvider::array    => false,
                     FilterProvider::OPTIONAL => $optional,
                 ];
 
@@ -129,7 +129,7 @@ 
                 FilterProvider::FILTER   => $definition[self::NESTED],
                 FilterProvider::SOURCE   => null,
                 FilterProvider::ORIGIN   => $origin,
-                FilterProvider::ARRAY    => $iterate,
+                FilterProvider::array    => $iterate,
                 FilterProvider::OPTIONAL => $optional,
             ];
 

src/Filters/src/FilterProvider.php

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

@@ -33,7 +33,7 @@ 
     public const SOURCE         = 'source';
     public const ORIGIN         = 'origin';
     public const FILTER         = 'filter';
-    public const ARRAY          = 'array';
+    public const array          = 'array';
     public const OPTIONAL       = 'optional';
     public const ITERATE_SOURCE = 'iterate_source';
     public const ITERATE_ORIGIN = 'iterate_origin';
@@ -103,7 +103,7 @@ 
             }
 
             $nested = $map[self::FILTER];
-            if (empty($map[self::ARRAY])) {
+            if (empty($map[self::array])) {
                 // slicing down
                 $result[$field] = $this->createFilter($nested, $input->withPrefix($map[self::ORIGIN]));
                 continue;

src/Filters/src/Filter.php

Spacing +16 added lines, -16 removed lines

@@ -90,7 +90,7 @@ 
         array $schema,
         ValidatorInterface $validator,
         ErrorMapper $errorMapper
-    ) {
+    ){
         parent::__construct($data, $schema);
 
         $this->mappings = $schema[FilterProvider::MAPPING] ?? [];
@@ -151,10 +151,10 @@ 
      */
     public function getErrors(): array
     {
-        if ($this->errors === null) {
+        if ($this->errors === null){
             $this->errors = [];
-            foreach ($this->validator->withData($this)->getErrors() as $field => $error) {
-                if (is_string($error) && Translator::isMessage($error)) {
+            foreach ($this->validator->withData($this)->getErrors() as $field => $error){
+                if (is_string($error) && Translator::isMessage($error)){
                     // translate error message
                     $error = $this->say($error);
                 }
@@ -195,32 +195,32 @@ 
      */
     protected function validateNested(array $errors): array
     {
-        foreach ($this->getFields(false) as $index => $value) {
-            if (isset($errors[$index])) {
+        foreach ($this->getFields(false) as $index => $value){
+            if (isset($errors[$index])){
                 //Invalid on parent level
                 continue;
             }
 
-            if ($value instanceof FilterInterface) {
-                if ($this->isOptional($index) && !$this->hasBeenPassed($index)) {
+            if ($value instanceof FilterInterface){
+                if ($this->isOptional($index) && !$this->hasBeenPassed($index)){
                     continue;
                 }
 
-                if (!$value->isValid()) {
+                if (!$value->isValid()){
                     $errors[$index] = $value->getErrors();
                     continue;
                 }
             }
 
             //Array of nested entities for validation
-            if (is_iterable($value)) {
-                foreach ($value as $nIndex => $nValue) {
-                    if ($nValue instanceof FilterInterface) {
-                        if ($this->isOptional($nIndex) && !$this->hasBeenPassed($nIndex)) {
+            if (is_iterable($value)){
+                foreach ($value as $nIndex => $nValue){
+                    if ($nValue instanceof FilterInterface){
+                        if ($this->isOptional($nIndex) && !$this->hasBeenPassed($nIndex)){
                             continue;
                         }
 
-                        if (!$nValue->isValid()) {
+                        if (!$nValue->isValid()){
                             $errors[$index][$nIndex] = $nValue->getErrors();
                         }
                     }
@@ -254,11 +254,11 @@ 
     {
         $value = $this->getField((string)$field);
 
-        if ($value === null) {
+        if ($value === null){
             return false;
         }
 
-        if ($value instanceof FilterInterface) {
+        if ($value instanceof FilterInterface){
             return $value->getValue() !== [];
         }
 

Braces +30 added lines, -15 removed lines

@@ -151,10 +151,13 @@ 
      */
     public function getErrors(): array
     {
-        if ($this->errors === null) {
+        if ($this->errors === null)
+        {
             $this->errors = [];
-            foreach ($this->validator->withData($this)->getErrors() as $field => $error) {
-                if (is_string($error) && Translator::isMessage($error)) {
+            foreach ($this->validator->withData($this)->getErrors() as $field => $error)
+            {
+                if (is_string($error) && Translator::isMessage($error))
+                {
                     // translate error message
                     $error = $this->say($error);
                 }
@@ -195,32 +198,42 @@ 
      */
     protected function validateNested(array $errors): array
     {
-        foreach ($this->getFields(false) as $index => $value) {
-            if (isset($errors[$index])) {
+        foreach ($this->getFields(false) as $index => $value)
+        {
+            if (isset($errors[$index]))
+            {
                 //Invalid on parent level
                 continue;
             }
 
-            if ($value instanceof FilterInterface) {
-                if ($this->isOptional($index) && !$this->hasBeenPassed($index)) {
+            if ($value instanceof FilterInterface)
+            {
+                if ($this->isOptional($index) && !$this->hasBeenPassed($index))
+                {
                     continue;
                 }
 
-                if (!$value->isValid()) {
+                if (!$value->isValid())
+                {
                     $errors[$index] = $value->getErrors();
                     continue;
                 }
             }
 
             //Array of nested entities for validation
-            if (is_iterable($value)) {
-                foreach ($value as $nIndex => $nValue) {
-                    if ($nValue instanceof FilterInterface) {
-                        if ($this->isOptional($nIndex) && !$this->hasBeenPassed($nIndex)) {
+            if (is_iterable($value))
+            {
+                foreach ($value as $nIndex => $nValue)
+                {
+                    if ($nValue instanceof FilterInterface)
+                    {
+                        if ($this->isOptional($nIndex) && !$this->hasBeenPassed($nIndex))
+                        {
                             continue;
                         }
 
-                        if (!$nValue->isValid()) {
+                        if (!$nValue->isValid())
+                        {
                             $errors[$index][$nIndex] = $nValue->getErrors();
                         }
                     }
@@ -254,11 +267,13 @@ 
     {
         $value = $this->getField((string)$field);
 
-        if ($value === null) {
+        if ($value === null)
+        {
             return false;
         }
 
-        if ($value instanceof FilterInterface) {
+        if ($value instanceof FilterInterface)
+        {
             return $value->getValue() !== [];
         }
 

src/Framework/Command/Migrate/MigrateCommand.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function perform(): void
     {
-        if (!$this->verifyEnvironment()) {
+        if (!$this->verifyEnvironment()){
             return;
         }
 
@@ -35,7 +35,7 @@ 
         $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--;
 
@@ -45,7 +45,7 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found){
             $this->writeln('<fg=red>No outstanding migrations were found.</fg=red>');
         }
     }

Braces +6 added lines, -3 removed lines

@@ -26,7 +26,8 @@ 
      */
     public function perform(): void
     {
-        if (!$this->verifyEnvironment()) {
+        if (!$this->verifyEnvironment())
+        {
             return;
         }
 
@@ -35,7 +36,8 @@ 
         $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--;
 
@@ -45,7 +47,8 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found)
+        {
             $this->writeln('<fg=red>No outstanding migrations were found.</fg=red>');
         }
     }

src/Framework/Command/Migrate/StatusCommand.php

Spacing +3 added lines, -3 removed lines

@@ -30,14 +30,14 @@ 
     {
         $this->migrator->configure();
 
-        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(
@@ -46,7 +46,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 +4 added lines, -2 removed lines

@@ -30,14 +30,16 @@
     {
         $this->migrator->configure();
 
-        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(

src/Framework/Command/Migrate/RollbackCommand.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function perform(): void
     {
-        if (!$this->verifyEnvironment()) {
+        if (!$this->verifyEnvironment()){
             //Making sure we can safely migrate in this environment
             return;
         }
@@ -35,7 +35,7 @@ 
 
         $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(
@@ -44,7 +44,7 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found){
             $this->writeln('<fg=red>No executed migrations were found.</fg=red>');
         }
     }

Braces +6 added lines, -3 removed lines

@@ -26,7 +26,8 @@ 
      */
     public function perform(): void
     {
-        if (!$this->verifyEnvironment()) {
+        if (!$this->verifyEnvironment())
+        {
             //Making sure we can safely migrate in this environment
             return;
         }
@@ -35,7 +36,8 @@ 
 
         $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(
@@ -44,7 +46,8 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found)
+        {
             $this->writeln('<fg=red>No executed migrations were found.</fg=red>');
         }
     }

src/Framework/Command/Cycle/MigrateCommand.php

Spacing +4 added lines, -4 removed lines

@@ -52,8 +52,8 @@ 
     ): void {
         $migrator->configure();
 
-        foreach ($migrator->getMigrations() as $migration) {
-            if ($migration->getState()->getStatus() !== Status::STATUS_EXECUTED) {
+        foreach ($migrator->getMigrations() as $migration){
+            if ($migration->getState()->getStatus() !== Status::STATUS_EXECUTED){
                 $this->writeln('<fg=red>Outstanding migrations found, run `migrate` first.</fg=red>');
                 return;
             }
@@ -67,10 +67,10 @@ 
         );
         $schemaCompiler->toMemory($memory);
 
-        if ($show->hasChanges()) {
+        if ($show->hasChanges()){
             (new Compiler())->compile($registry, [$migrations]);
 
-            if ($this->option('run')) {
+            if ($this->option('run')){
                 $console->run('migrate', [], $this->output);
             }
         }

Braces +8 added lines, -4 removed lines

@@ -52,8 +52,10 @@ 
     ): void {
         $migrator->configure();
 
-        foreach ($migrator->getMigrations() as $migration) {
-            if ($migration->getState()->getStatus() !== Status::STATUS_EXECUTED) {
+        foreach ($migrator->getMigrations() as $migration)
+        {
+            if ($migration->getState()->getStatus() !== Status::STATUS_EXECUTED)
+            {
                 $this->writeln('<fg=red>Outstanding migrations found, run `migrate` first.</fg=red>');
                 return;
             }
@@ -67,10 +69,12 @@ 
         );
         $schemaCompiler->toMemory($memory);
 
-        if ($show->hasChanges()) {
+        if ($show->hasChanges())
+        {
             (new Compiler())->compile($registry, [$migrations]);
 
-            if ($this->option('run')) {
+            if ($this->option('run'))
+            {
                 $console->run('migrate', [], $this->output);
             }
         }

tests/Framework/ConsoleTest.php

Spacing +3 added lines, -3 removed lines

@@ -32,8 +32,8 @@ 
     private function cleanupMigrations(): void
     {
         $fs = new Files();
-        if ($fs->isDirectory(__DIR__ . '/../app/migrations')) {
-            $fs->deleteDirectory(__DIR__ . '/../app/migrations');
+        if ($fs->isDirectory(__DIR__.'/../app/migrations')){
+            $fs->deleteDirectory(__DIR__.'/../app/migrations');
         }
     }
 
@@ -46,7 +46,7 @@ 
         $fs = new Files();
         $runtime = $this->app->get(DirectoriesInterface::class)->get('runtime');
 
-        if ($fs->isDirectory($runtime)) {
+        if ($fs->isDirectory($runtime)){
             $fs->deleteDirectory($runtime);
         }
     }

Braces +4 added lines, -2 removed lines

@@ -32,7 +32,8 @@ 
     private function cleanupMigrations(): void
     {
         $fs = new Files();
-        if ($fs->isDirectory(__DIR__ . '/../app/migrations')) {
+        if ($fs->isDirectory(__DIR__ . '/../app/migrations'))
+        {
             $fs->deleteDirectory(__DIR__ . '/../app/migrations');
         }
     }
@@ -46,7 +47,8 @@ 
         $fs = new Files();
         $runtime = $this->app->get(DirectoriesInterface::class)->get('runtime');
 
-        if ($fs->isDirectory($runtime)) {
+        if ($fs->isDirectory($runtime))
+        {
             $fs->deleteDirectory($runtime);
         }
     }