fabot

Author: smnandre

src/TwigComponent/src/Attribute/AsTwigComponent.php

@@ -82,8 +82,6 @@ public function serviceConfig(): array
      * @return \ReflectionMethod[]
      *
      * @internal
-     *
-     * @deprecated since 2.22 - to be removed in 3.0
      */
     protected static function attributeMethodsByPriorityFor(object|string $component, string $attributeClass): array
     {
@@ -100,8 +98,6 @@ protected static function attributeMethodsByPriorityFor(object|string $component
      * @return \Traversable<\ReflectionMethod>
      *
      * @internal
-     *
-     * @deprecated since 2.22 - to be removed in 3.0
      */
     protected static function attributeMethodsFor(string $attribute, object|string $component): \Traversable
     {

src/TwigComponent/src/ComponentFactory.php

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

src/TwigComponent/src/ComponentMetadata.php

@@ -69,7 +69,7 @@ public function getAttributesVar(): string
 
     /**
      * @return list<string>
-     *     
+     *
      * @internal
      */
     public function getPreMounts(): array
@@ -79,7 +79,7 @@ public function getPreMounts(): array
 
     /**
      * @return list<string>
-     *     
+     *
      * @internal
      */
     public function getMounts(): array
@@ -89,7 +89,7 @@ public function getMounts(): array
 
     /**
      * @return list<string>
-     *     
+     *
      * @internal
      */
     public function getPostMounts(): array

src/TwigComponent/src/DependencyInjection/Compiler/TwigComponentPass.php

@@ -114,6 +114,7 @@ private function calculateTemplate(string $componentName, ?array $defaults): str
 
     /**
      * @param class-string $component
+     *
      * @return array{preMount: string[], mount: string[], postMount: string[]}
      */
     private function getMountMethods(string $component): array
@@ -131,9 +132,9 @@ private function getMountMethods(string $component): array
             }
         }
 
-        arsort($preMount, SORT_NUMERIC);
-        arsort($mount, SORT_NUMERIC);
-        arsort($postMount, SORT_NUMERIC);
+        arsort($preMount, \SORT_NUMERIC);
+        arsort($mount, \SORT_NUMERIC);
+        arsort($postMount, \SORT_NUMERIC);
 
         return [
             'pre_mount' => array_keys($preMount),

src/TwigComponent/tests/Integration/ComponentFactoryTest.php

@@ -94,7 +94,7 @@ public function testExceptionThrownIfRequiredMountParameterIsMissingFromPassedDa
 
     public function testStringableObjectCanBePassedToComponent(): void
     {
-        $attributes = $this->factory()->create('component_a', ['propB' => 'B', 'data-item-id-param' => new class () {
+        $attributes = $this->factory()->create('component_a', ['propB' => 'B', 'data-item-id-param' => new class {
             public function __toString(): string
             {
                 return 'test';

src/TwigComponent/tests/Unit/ComponentAttributesTest.php

@@ -27,7 +27,7 @@ public function testCanConvertToString(): void
     {
         $attributes = new ComponentAttributes([
             'class' => 'foo',
-            'style' => new class () {
+            'style' => new class {
                 public function __toString(): string
                 {
                     return 'color:black;';
@@ -216,7 +216,7 @@ public function testRenderSingleAttribute(): void
     public function testRenderingSingleAttributeExcludesFromString(): void
     {
         $attributes = new ComponentAttributes([
-            'attr1' => new class () {
+            'attr1' => new class {
                 public function __toString(): string
                 {
                     return 'value1';

src/TwigComponent/tests/Unit/ComputedPropertiesProxyTest.php

@@ -21,7 +21,7 @@ final class ComputedPropertiesProxyTest extends TestCase
 {
     public function testProxyCachesGetMethodReturns(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class () {
+        $proxy = new ComputedPropertiesProxy(new class {
             private int $count = 0;
 
             public function getCount(): int
@@ -37,7 +37,7 @@ public function getCount(): int
 
     public function testProxyCachesIsMethodReturns(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class () {
+        $proxy = new ComputedPropertiesProxy(new class {
             private int $count = 0;
 
             public function isCount(): int
@@ -53,7 +53,7 @@ public function isCount(): int
 
     public function testProxyCachesHasMethodReturns(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class () {
+        $proxy = new ComputedPropertiesProxy(new class {
             private int $count = 0;
 
             public function hasCount(): int
@@ -69,7 +69,7 @@ public function hasCount(): int
 
     public function testCanProxyPublicProperties(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class () {
+        $proxy = new ComputedPropertiesProxy(new class {
             public $foo = 'bar';
         });
 
@@ -78,7 +78,7 @@ public function testCanProxyPublicProperties(): void
 
     public function testCanProxyArrayAccess(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class () implements \ArrayAccess {
+        $proxy = new ComputedPropertiesProxy(new class implements \ArrayAccess {
             private $array = ['foo' => 'bar'];
 
             public function offsetExists(mixed $offset): bool
@@ -105,7 +105,7 @@ public function offsetUnset(mixed $offset): void
 
     public function testCannotProxyMethodsThatDoNotExist(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class () {});
+        $proxy = new ComputedPropertiesProxy(new class {});
 
         $this->expectException(\InvalidArgumentException::class);
 
@@ -114,7 +114,7 @@ public function testCannotProxyMethodsThatDoNotExist(): void
 
     public function testCannotPassArgumentsToProxiedMethods(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class () {});
+        $proxy = new ComputedPropertiesProxy(new class {});
 
         $this->expectException(\InvalidArgumentException::class);
 
@@ -123,7 +123,7 @@ public function testCannotPassArgumentsToProxiedMethods(): void
 
     public function testCannotProxyMethodsWithRequiredArguments(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class () {
+        $proxy = new ComputedPropertiesProxy(new class {
             public function getValue(int $value): int
             {
                 return $value;

src/TwigComponent/tests/Unit/DependencyInjection/TwigComponentExtensionTest.php

@@ -98,7 +98,7 @@ private function createContainer()
             'kernel.debug' => true,
             'kernel.project_dir' => __DIR__,
             'kernel.bundles' => [
-                'TwigBundle' => new class () {},
+                'TwigBundle' => new class {},
                 'TwigComponentBundle' => TwigComponentBundle::class,
             ],
         ]));