Apply fabbot rules

Author: nicolas-grekas

README.md

@@ -1,9 +1,9 @@
 # Twig Components
 
 Twig components give you the power to bind an object to a template, making
-it easier to render and re-use small template "units" - like an "alert",
+it easier to render and reuse small template "units" - like an "alert",
 markup for a modal, or a category sidebar. A very simple example
-would be a re-usable alert component:
+would be a reusable alert component:
 
 ![Example of the AlertComponent](https://github.com/symfony/ux/blob/2.x/src/TwigComponent/alert-example.png?raw=true)
 

doc/index.rst

@@ -2,7 +2,7 @@ Twig Components
 ===============
 
 Twig components give you the power to bind an object to a template,
-making it easier to render and re-use small template "units" - like an
+making it easier to render and reuse small template "units" - like an
 "alert", markup for a modal, or a category sidebar:
 
 Every component consists of (1) a class::
@@ -921,7 +921,7 @@ For example, imagine we want to create a ``SuccessAlert`` component:
         We will successfully <em>forward</em> this block content!
     <twig:SuccessAlert>
 
-We already have a generic ``Alert`` component, so let's re-use it:
+We already have a generic ``Alert`` component, so let's reuse it:
 
 .. code-block:: html+twig
 

src/ComponentFactory.php

@@ -167,7 +167,7 @@ private function mount(object $component, array &$data, ComponentMetadata $compo
             } elseif ($refParameter->isDefaultValueAvailable()) {
                 $parameters[] = $refParameter->getDefaultValue();
             } else {
-                throw new \LogicException(\sprintf('%s has a required $%s parameter. Make sure to pass it or give it a default value.', $component::class.'::mount()', $name));
+                throw new \LogicException(\sprintf('"%s" has a required $%s parameter. Make sure to pass it or give it a default value.', $component::class.'::mount()', $name));
             }
         }
 

tests/Integration/Command/TwigComponentDebugCommandTest.php

@@ -17,7 +17,7 @@
 
 class TwigComponentDebugCommandTest extends KernelTestCase
 {
-    public function testWithNoComponent(): void
+    public function testWithNoComponent()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->execute([]);
@@ -32,7 +32,7 @@ public function testWithNoComponent(): void
         $this->assertStringContainsString('Type', $display);
     }
 
-    public function testWithNoMatchComponent(): void
+    public function testWithNoMatchComponent()
     {
         $commandTester = $this->createCommandTester();
         $result = $commandTester->execute(['name' => 'NoMatchComponent']);
@@ -41,7 +41,7 @@ public function testWithNoMatchComponent(): void
         $this->assertStringContainsString('Unknown component "NoMatchComponent".', $commandTester->getDisplay());
     }
 
-    public function testNotComponentsIsNotListed(): void
+    public function testNotComponentsIsNotListed()
     {
         $commandTester = $this->createCommandTester();
         $result = $commandTester->execute(['name' => 'NotAComponent']);
@@ -50,7 +50,7 @@ public function testNotComponentsIsNotListed(): void
         $this->assertStringContainsString('Unknown component "NotAComponent".', $commandTester->getDisplay());
     }
 
-    public function testWithOnePartialMatchComponent(): void
+    public function testWithOnePartialMatchComponent()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->setInputs([]);
@@ -64,7 +64,7 @@ public function testWithOnePartialMatchComponent(): void
         $this->assertStringContainsString('Component\\DivComponentNoPass', $commandTester->getDisplay());
     }
 
-    public function testWithMultiplePartialMatchComponent(): void
+    public function testWithMultiplePartialMatchComponent()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->setInputs(['DivComponent5']);
@@ -82,7 +82,7 @@ public function testWithMultiplePartialMatchComponent(): void
         $this->assertStringNotContainsString('Component\\DivComponent6', $commandTester->getDisplay());
     }
 
-    public function testComponentWithClass(): void
+    public function testComponentWithClass()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->execute(['name' => 'BasicComponent']);
@@ -97,7 +97,7 @@ public function testComponentWithClass(): void
         $this->assertStringContainsString('components/BasicComponent.html.twig', $display);
     }
 
-    public function testComponentWithClassPropertiesAndCustomName(): void
+    public function testComponentWithClassPropertiesAndCustomName()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->execute(['name' => 'component_c']);
@@ -116,7 +116,7 @@ public function testComponentWithClassPropertiesAndCustomName(): void
         $this->assertStringContainsString('Mount', $display);
     }
 
-    public function testComponentWithClassPropertiesCustomNameAndCustomTemplate(): void
+    public function testComponentWithClassPropertiesCustomNameAndCustomTemplate()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->execute(['name' => 'component_b']);
@@ -133,7 +133,7 @@ public function testComponentWithClassPropertiesCustomNameAndCustomTemplate(): v
         $this->assertStringContainsString('string $postValue', $display);
     }
 
-    public function testWithAnonymousComponent(): void
+    public function testWithAnonymousComponent()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->execute(['name' => 'Button']);
@@ -150,7 +150,7 @@ public function testWithAnonymousComponent(): void
         $this->assertStringContainsString('primary = true', $display);
     }
 
-    public function testWithBundleAnonymousComponent(): void
+    public function testWithBundleAnonymousComponent()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->execute(['name' => 'Acme:Button']);
@@ -165,7 +165,7 @@ public function testWithBundleAnonymousComponent(): void
         $this->assertStringContainsString('Anonymous', $display);
     }
 
-    public function testWithBundleAnonymousComponentSubDir(): void
+    public function testWithBundleAnonymousComponentSubDir()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->execute(['name' => 'Acme:Table:Header']);
@@ -180,7 +180,7 @@ public function testWithBundleAnonymousComponentSubDir(): void
         $this->assertStringContainsString('Anonymous', $display);
     }
 
-    public function testWithoutPublicProps(): void
+    public function testWithoutPublicProps()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->execute(['name' => 'no_public_props']);
@@ -194,7 +194,7 @@ public function testWithoutPublicProps(): void
         $this->assertStringNotContainsString('prop1', $display);
     }
 
-    public function testWithExposedVariables(): void
+    public function testWithExposedVariables()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->execute(['name' => 'with_exposed_variables']);
@@ -212,7 +212,7 @@ public function testWithExposedVariables(): void
         $this->assertStringNotContainsString('prop3', $display);
     }
 
-    public function testWithUnionTypeProps(): void
+    public function testWithUnionTypeProps()
     {
         $commandTester = $this->createCommandTester();
         $commandTester->execute(['name' => 'union_type_props']);

tests/Integration/ComponentEventTest.php

@@ -25,7 +25,7 @@ final class ComponentEventTest extends KernelTestCase
     /**
      * @dataProvider provideFooBarSyntaxes
      */
-    public function testTemplateIsUpdatedByEventListener(string $syntax): void
+    public function testTemplateIsUpdatedByEventListener(string $syntax)
     {
         /** @var Environment $environment */
         $environment = self::getContainer()->get(Environment::class);