[Symfony 7.3] Add RegexToSlugConstraintRector #723

config/sets/symfony/symfony7/symfony73.php

@@ -9,4 +9,5 @@
 return static function (RectorConfig $rectorConfig): void {
     $rectorConfig->import(__DIR__ . '/symfony73/symfony73-console.php');
     $rectorConfig->import(__DIR__ . '/symfony73/symfony73-twig-bundle.php');
+    $rectorConfig->import(__DIR__ . '/symfony73/symfony73-validator.php');
 };

config/sets/symfony/symfony7/symfony73/symfony73-validator.php

@@ -0,0 +1,11 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\Symfony\Symfony73\Rector\New_\RegexToSlugConstraintRector;
+
+return static function (RectorConfig $rectorConfig): void {
+    // @see https://symfony.com/blog/new-in-symfony-7-3-slug-and-twig-constraints#slug-constraint
+    $rectorConfig->rule(RegexToSlugConstraintRector::class);
+};

rules-tests/Symfony73/Rector/New_/RegexToSlugConstraintRector/Fixture/replace.php.inc

@@ -0,0 +1,31 @@
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints\Regex;
+
+class Replace
+{
+    public function createConstraint()
+    {
+        return new Regex('/[a-z0-9]+(?:-[a-z0-9]+)*/');
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints\Regex;
+
+class Replace
+{
+    public function createConstraint()
+    {
+        return new Symfony\Component\Validator\Constraints\Slug();
+    }
+}
+
+?>

rules-tests/Symfony73/Rector/New_/RegexToSlugConstraintRector/Fixture/replace_all_params.php.inc

@@ -0,0 +1,31 @@
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints\Regex;
+
+class ReplaceAllParams
+{
+    public function createConstraint()
+    {
+        return new Regex(pattern: '/[a-z0-9]+(?:-[a-z0-9]+)*/', message: 'some message', htmlPattern: null, match: null, normalizer: null, groups: ['Default'], payload: null, options: []);
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints\Regex;
+
+class ReplaceAllParams
+{
+    public function createConstraint()
+    {
+        return new Symfony\Component\Validator\Constraints\Slug(message: 'some message', groups: ['Default'], payload: null, options: []);
+    }
+}
+
+?>

rules-tests/Symfony73/Rector/New_/RegexToSlugConstraintRector/Fixture/replace_param_array_syntax.php.inc

@@ -0,0 +1,31 @@
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints\Regex;
+
+class ReplaceParamArraySyntax
+{
+    public function createConstraint()
+    {
+        return new Regex(['pattern' => '/^[a-z0-9]+(?:-[a-z0-9]+)*$/']);
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints\Regex;
+
+class ReplaceParamArraySyntax
+{
+    public function createConstraint()
+    {
+        return new Symfony\Component\Validator\Constraints\Slug();
+    }
+}
+
+?>

rules-tests/Symfony73/Rector/New_/RegexToSlugConstraintRector/Fixture/replace_start_and_end_tags.php.inc

@@ -0,0 +1,31 @@
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints\Regex;
+
+class ReplaceStartAndEndTags
+{
+    public function createConstraint()
+    {
+        return new Regex('/^[a-z0-9]+(?:-[a-z0-9]+)*$/');
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints\Regex;
+
+class ReplaceStartAndEndTags
+{
+    public function createConstraint()
+    {
+        return new Symfony\Component\Validator\Constraints\Slug();
+    }
+}
+
+?>

rules-tests/Symfony73/Rector/New_/RegexToSlugConstraintRector/Fixture/replace_with_alias.php.inc

@@ -0,0 +1,31 @@
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints as Assert;
+
+class ReplaceWithAlias
+{
+    public function createConstraint()
+    {
+        return new Assert\Regex('/^[a-z0-9]+(?:-[a-z0-9]+)*$/');
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints as Assert;
+
+class ReplaceWithAlias
+{
+    public function createConstraint()
+    {
+        return new Symfony\Component\Validator\Constraints\Slug();
+    }
+}
+
+?>

rules-tests/Symfony73/Rector/New_/RegexToSlugConstraintRector/Fixture/skip_diffrent_regex.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints\Regex;
+
+class SkipDifferentRegex
+{
+    public function createConstraint()
+    {
+        return new Regex('/^[a-z0-9]$/');
+    }
+}
+
+?>

rules-tests/Symfony73/Rector/New_/RegexToSlugConstraintRector/Fixture/skip_variable_regex.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector\Fixture;
+
+use Symfony\Component\Validator\Constraints\Regex;
+
+class SkipDifferentRegex
+{
+    public function createConstraint(string $regex = '/^[a-z0-9]+(?:-[a-z0-9]+)*$/')
+    {
+        return new Regex($regex);
+    }
+}
+
+?>

rules-tests/Symfony73/Rector/New_/RegexToSlugConstraintRector/RegexToSlugConstraintRectorTest.php

@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\New_\RegexToSlugConstraintRector;
+
+use Iterator;
+use PHPUnit\Framework\Attributes\DataProvider;
+use Rector\Testing\PHPUnit\AbstractRectorTestCase;
+
+final class RegexToSlugConstraintRectorTest extends AbstractRectorTestCase
+{
+    #[DataProvider('provideData')]
+    public function test(string $filePath): void
+    {
+        $this->doTestFile($filePath);
+    }
+
+    public static function provideData(): Iterator
+    {
+        return self::yieldFilesFromDirectory(__DIR__ . '/Fixture');
+    }
+
+    public function provideConfigFilePath(): string
+    {
+        return __DIR__ . '/config/configured_rule.php';
+    }
+}

rules-tests/Symfony73/Rector/New_/RegexToSlugConstraintRector/config/configured_rule.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\Symfony\Symfony73\Rector\New_\RegexToSlugConstraintRector;
+
+return static function (RectorConfig $rectorConfig): void {
+    $rectorConfig->rule(RegexToSlugConstraintRector::class);
+};