OXDEV-8771 Fix coding style issues

Signed-off-by: Anton Fedurtsya <[email protected]>
OXID-eSales · Oct 9, 2024 · c1f9b21 · c1f9b21
1 parent e1d7333
commit c1f9b21
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 2 deletions.
diff --git a/src/UserData/Infrastructure/GeneralTableDataSelector.php b/src/UserData/Infrastructure/GeneralTableDataSelector.php
@@ -14,6 +14,9 @@
 
 class GeneralTableDataSelector implements DataSelectorInterface
 {
+    /**
+     * @SuppressWarnings(PHPMD.BooleanArgumentFlag) Not different behaviour, just protection from explosion
+     */
     public function __construct(
         private string $collection,
         private string $selectionTable,

diff --git a/src/UserData/Infrastructure/RelatedTableDataSelector.php b/src/UserData/Infrastructure/RelatedTableDataSelector.php
@@ -14,6 +14,9 @@
 
 class RelatedTableDataSelector implements DataSelectorInterface
 {
+    /**
+     * @SuppressWarnings(PHPMD.BooleanArgumentFlag) Not different behaviour, just protection from explosion
+     */
     public function __construct(
         private string $collection,
         private string $primaryTable,

diff --git a/tests/Integration/UserData/Infrastructure/RelatedTableDataSelectorTest.php b/tests/Integration/UserData/Infrastructure/RelatedTableDataSelectorTest.php
@@ -87,7 +87,6 @@ public function testOptionalFlagDoesNotExplodeOnQueryError(): void
             filterColumn: 'oxorder.OXUSERID',
             queryBuilderFactory: $this->get(QueryBuilderFactoryInterface::class),
             optional: true
-
         );
 
         $this->assertSame([], $sut->getDataForColumnValue(self::USER_ID));

diff --git a/tests/Integration/UserData/Infrastructure/SelectorTest.php b/tests/Integration/UserData/Infrastructure/SelectorTest.php
@@ -21,4 +21,4 @@ public function testConfiguredSelectors(): void
 
         $this->addToAssertionCount(1);
     }
-}
+}
-Original file line number
+Diff line change
@@ Expand Up / @@ -21,4 +21,4 @@ public function testConfiguredSelectors(): void @@
             $this->addToAssertionCount(1);
         }
-    }
+    }