Apply suggestions from code review

Co-authored-by: Dmitri Perunov <[email protected]>
Sylius · Jun 21, 2024 · 07bac40 · 07bac40
1 parent d93f990
commit 07bac40
Show file tree

Hide file tree

Showing 6 changed files with 8 additions and 15 deletions.
diff --git a/docs/twig-extra/getting-started.md b/docs/twig-extra/getting-started.md
@@ -81,7 +81,7 @@ This is slightly the same behaviour as `sylius_test_html_attribute` Twig functio
 ```
 
 ```html
-<!-- The real results depend on your form theme. -->
+<!-- Actual html output bellow depends on your form theme -->
 <label for="book_title">Title</label>
 <input 
         type="text" 

diff --git a/src/TwigExtra/config/services.php b/src/TwigExtra/config/services.php
@@ -11,7 +11,7 @@
 return function (ContainerConfigurator $configurator): void {
     $services = $configurator->services();
 
-    $services->set('twig_extra.twig.extension.test_form_attribute', TestFormAttributeExtension::class)
+    $services->set('sylius_twig_extra.twig.extension.test_form_attribute', TestFormAttributeExtension::class)
         ->args([
             param('kernel.environment'),
             param('kernel.debug'),

diff --git a/src/TwigExtra/src/Twig/Extension/SortByExtension.php b/src/TwigExtra/src/Twig/Extension/SortByExtension.php
@@ -18,16 +18,14 @@ public function getFilters(): array
         ];
     }
 
-    /**
-     * @throws NoSuchPropertyException
-     */
+    /** @throws NoSuchPropertyException */
     public function sortBy(iterable $iterable, string $field, string $order = 'ASC'): array
     {
         $array = $this->transformIterableToArray($iterable);
 
         usort(
             $array,
-            function (array|object $firstElement, array|object $secondElement) use ($field, $order) {
+            function (array|object $firstElement, array|object $secondElement): int use ($field, $order) {
                 $accessor = PropertyAccess::createPropertyAccessor();
 
                 $firstProperty = (string) $accessor->getValue($firstElement, $field);

diff --git a/src/TwigExtra/src/Twig/Extension/TestFormAttributeExtension.php b/src/TwigExtra/src/Twig/Extension/TestFormAttributeExtension.php
@@ -12,13 +12,10 @@ final class TestFormAttributeExtension extends AbstractExtension
     public function __construct(
         private readonly string $environment,
         private readonly bool $isDebugEnabled,
-    )
-    {
+    ) {
     }
 
-    /**
-     * @return TwigFunction[]
-     */
+    /** @return TwigFunction[] */
     public function getFunctions(): array
     {
         return [

diff --git a/src/TwigExtra/src/Twig/Extension/TestHtmlAttributeExtension.php b/src/TwigExtra/src/Twig/Extension/TestHtmlAttributeExtension.php
@@ -15,9 +15,7 @@ public function __construct(
     ) {
     }
 
-    /**
-     * @return TwigFunction[]
-     */
+    /** @return TwigFunction[] */
     public function getFunctions(): array
     {
         return [

diff --git a/src/TwigExtra/tests/Unit/Twig/Extension/SortByExtensionTest.php b/src/TwigExtra/tests/Unit/Twig/Extension/SortByExtensionTest.php
@@ -159,7 +159,7 @@ public function testItReturnsInputArrayIfThereIsOnlyOneObjectInside(): void
     {
         $data = [(object) []];
 
-        $this->assertEquals($data, (new SortByExtension())->sortBy($data, 'property'), );
+        $this->assertEquals($data, (new SortByExtension())->sortBy($data, 'property'));
     }
 
     public function testItDoesNothingIfArrayIsEmpty(): void