Fix CS v3.7.0

atk4 · Mar 12, 2022 · bd6b5aa · bd6b5aa
1 parent 8c9be7b
commit bd6b5aa
Show file tree

Hide file tree

Showing 11 changed files with 21 additions and 23 deletions.
diff --git a/src/Model/Join.php b/src/Model/Join.php
@@ -510,7 +510,7 @@ public function beforeInsert(Model $entity, array &$data): void
         $foreignModel = $this->getForeignModel();
         $foreignEntity = $foreignModel->createEntity()
             ->setMulti($this->getAndUnsetSaveBuffer($entity))
-            /*->set($this->foreign_field, null)*/;
+            /* ->set($this->foreign_field, null) */;
         $foreignEntity->save();
 
         $this->setId($entity, $foreignEntity->getId());

diff --git a/src/Model/ReferencesTrait.php b/src/Model/ReferencesTrait.php
@@ -63,7 +63,7 @@ public function addRef(string $link, array $defaults): Reference
      *
      * @return Reference\HasOne|Reference\HasOneSql
      */
-    public function hasOne(string $link, array $defaults = []) //: Reference
+    public function hasOne(string $link, array $defaults = []) // : Reference
     {
         return $this->_addRef($this->_default_seed_hasOne, $link, $defaults); // @phpstan-ignore-line
     }
@@ -73,7 +73,7 @@ public function hasOne(string $link, array $defaults = []) //: Reference
      *
      * @return Reference\HasMany
      */
-    public function hasMany(string $link, array $defaults = []) //: Reference
+    public function hasMany(string $link, array $defaults = []) // : Reference
     {
         return $this->_addRef($this->_default_seed_hasMany, $link, $defaults); // @phpstan-ignore-line
     }
@@ -83,7 +83,7 @@ public function hasMany(string $link, array $defaults = []) //: Reference
      *
      * @return Reference\ContainsOne
      */
-    public function containsOne(string $link, array $defaults = []) //: Reference
+    public function containsOne(string $link, array $defaults = []) // : Reference
     {
         return $this->_addRef($this->_default_seed_containsOne, $link, $defaults); // @phpstan-ignore-line
     }
@@ -93,7 +93,7 @@ public function containsOne(string $link, array $defaults = []) //: Reference
      *
      * @return Reference\ContainsMany
      */
-    public function containsMany(string $link, array $defaults = []) //: Reference
+    public function containsMany(string $link, array $defaults = []) // : Reference
     {
         return $this->_addRef($this->_default_seed_containsMany, $link, $defaults); // @phpstan-ignore-line
     }

diff --git a/src/Model/UserActionsTrait.php b/src/Model/UserActionsTrait.php
@@ -145,7 +145,7 @@ protected function initUserActions(): void
         ]);
 
         $this->addUserAction('validate', [
-            //'appliesTo' => any!
+            // 'appliesTo' => any!
             'description' => 'Provided with modified values will validate them but will not save',
             'modifier' => UserAction::MODIFIER_READ,
             'fields' => true,

diff --git a/src/Persistence/Sql/Query.php b/src/Persistence/Sql/Query.php
@@ -254,7 +254,7 @@ protected function _render_from(): ?string
         return empty($this->args['table']) ? '' : 'from';
     }
 
-    /// }}}
+    // }}}
 
     // {{{ with()
 
@@ -328,7 +328,7 @@ protected function _render_with(): ?string
         return 'with ' . ($isRecursive ? 'recursive ' : '') . implode(',' . "\n", $ret) . "\n";
     }
 
-    /// }}}
+    // }}}
 
     // {{{ join()
 
@@ -820,7 +820,7 @@ protected function _render_set_fields(): ?string
         $ret = [];
 
         if ($this->args['set']) {
-            foreach ($this->args['set'] as [$field/*, $value*/ ]) {
+            foreach ($this->args['set'] as [$field/* , $value */ ]) {
                 $field = $this->consume($field, self::ESCAPE_IDENTIFIER);
 
                 $ret[] = $field;
@@ -836,7 +836,7 @@ protected function _render_set_values(): ?string
         $ret = [];
 
         if ($this->args['set']) {
-            foreach ($this->args['set'] as [/*$field*/ , $value]) {
+            foreach ($this->args['set'] as [/* $field */, $value]) {
                 $value = $this->consume($value, self::ESCAPE_PARAM);
 
                 $ret[] = $value;
@@ -1004,11 +1004,11 @@ public function exists()
     public function __debugInfo(): array
     {
         $arr = [
-            //'mode' => $this->mode,
+            // 'mode' => $this->mode,
             'R' => 'n/a',
             'R_params' => 'n/a',
-            //'template' => $this->template,
-            //'templateArgs' => $this->args,
+            // 'template' => $this->template,
+            // 'templateArgs' => $this->args,
         ];
 
         try {
@@ -1269,5 +1269,5 @@ protected function _set_args($what, $alias, $value): void
         }
     }
 
-    /// }}}
+    // }}}
 }
diff --git a/tests/ContainsOneTest.php b/tests/ContainsOneTest.php
@@ -159,8 +159,6 @@ public function testContainsOne(): void
         $i->addr->delete();
         $this->assertNull($i->get($i->fieldName()->addr));
         $this->assertFalse($i->addr->isLoaded());
-
-        //var_dump($i->export(), $i->export(null, null, false));
     }
 
     /**

diff --git a/tests/JoinSqlTest.php b/tests/JoinSqlTest.php
@@ -574,7 +574,7 @@ public function testJoinReverseOneOnOne(): void
         $m_user = new Model($this->db, ['table' => 'user']);
         $m_user->addField('name');
         $j = $m_user->join('detail.my_user_id', [
-            //'reverse' => true, // this will be reverse join by default
+            // 'reverse' => true, // this will be reverse join by default
             // also no need to set these (will be done automatically), but still let's do that for test sake
             'master_field' => 'id',
             'foreign_field' => 'my_user_id',

diff --git a/tests/LookupSqlTest.php b/tests/LookupSqlTest.php
@@ -299,7 +299,7 @@ public function testImportByLookup(): void
         $u->import([
             ['name' => 'Alain', 'country_code' => 'CA'],
             ['name' => 'Imants', 'country_code' => 'LV'],
-            //'name' => 'Romans', 'country_code' => 'UK'],  // does not exist
+            // 'name' => 'Romans', 'country_code' => 'UK'],  // does not exist
         ]);
 
         $this->assertSameExportUnordered([

diff --git a/tests/Persistence/Sql/QueryTest.php b/tests/Persistence/Sql/QueryTest.php
@@ -197,7 +197,7 @@ public function testFieldException1(): void
      */
     public function testTableException3(): void
     {
-        //$this->expectException(Exception::class); // no more
+        // $this->expectException(Exception::class); // no more
         $this->q()->table($this->q()->expr('test'));
     }
 
@@ -1548,7 +1548,7 @@ public function testCaseExprShortForm(): void
      */
     public function testCaseExprException1(): void
     {
-        //$this->expectException(Exception::class);
+        // $this->expectException(Exception::class);
         $this->q()->caseExpr()
             ->caseWhen(['status'], 't2.expose_new')
             ->render();

diff --git a/tests/Persistence/StaticTest.php b/tests/Persistence/StaticTest.php
@@ -20,7 +20,7 @@ public function testBasicStatic(): void
         $this->assertSame('world', $m->get('name'));
 
         // custom title field and try loading from same static twice
-        $m = new Model($p); //, ['title_field' => 'foo']);
+        $m = new Model($p); // , ['title_field' => 'foo']);
         $m = $m->load(2);
         $this->assertSame('world', $m->get('name')); // still 'name' here not 'foo'
     }

diff --git a/tests/SmboTransferTest.php b/tests/SmboTransferTest.php
@@ -35,7 +35,7 @@ protected function setUp(): void
             ->field('document_id', ['type' => 'integer'])
             ->field('account_id', ['type' => 'integer'])
             ->field('cheque_no')
-            //->field('misc_payment', ['type' => 'enum(\'N\',\'Y\')'])
+            // ->field('misc_payment', ['type' => 'enum(\'N\',\'Y\')'])
             ->field('misc_payment')
             ->field('transfer_document_id')
             ->create();

diff --git a/tests/SubTypesTest.php b/tests/SubTypesTest.php
@@ -129,7 +129,7 @@ protected function init(): void
         parent::init();
         $this->hasOne('link_id', ['model' => [StTransaction_TransferIn::class]]);
 
-        //$this->join('transaction', 'linked_transaction');
+        // $this->join('transaction', 'linked_transaction');
     }
 }