This rule was deprecated as it depended on access to app Symfony container, and was error prone. If you need it, fork the rule locally and tune it to fit your needs.
Last December, we started to use PHP 8.0 and Symfony 5.2. This exact combination opens many cool tricks we could never use before.
One of those tricks is using constants for route name in #[Route]
attribute.
If you're not on PHP 8, switch with Rector today.
Ready now? Good.
We've all been there. Our memory is full of repeated strings with typos, and we have to type very slowly and carefully. IDE and CI is going the other directory - they automate our coding process, so we only type the bare minimum, so the program understands our intention. IDE autocompletes full class structures, method names, and pre-defined live templates.
Back to the strings. The only way to avoid typos is not to write at all. We're not there yet, so we go for the next minimum. Type them just once. In programming, we call it "constants".
class CompanyInfo
{
public const COMPANY_NAME = 'Edukai, s. r. o.';
public const VAT_IT = 'CZ07237626';
public const CARD_NUMBER = '0239583290850928';
}
In this way, We can avoid using an incorrect card number.
#[Route]
RebornNow back to Symfony. With Symfony 5.2, we have a new option use Routes. Instead of the old comment annotation way:
use Symfony\Component\Routing\Annotation\Route;
final class MissionController
{
/**
* @Route(path="archive", name="mission")
*/
public function __invoke()
{
// ...
}
}
We can use PHP-native attribute:
use Symfony\Component\Routing\Annotation\Route;
final class MissionController
{
#[Route(path: 'mission', name: 'mission')]
public function __invoke()
{
// ...
}
}
What's the advantage of the route attribute?
Now that we've stated the benefits of constants and attributes, we should be ready to spot the problem they bring together:
use Symfony\Component\Routing\Annotation\Route;
final class MissionController
{
#[Route(path: 'mission', name: 'mission')]
public function __invoke()
{
// ...
}
}
use Symfony\Component\Routing\Annotation\Route;
final class ContactController
{
#[Route(path: 'contact', name: 'contact')]
public function __invoke()
{
// ...
return $this->redirectToRoute('mision');
}
}
Can you see it? The route used has a "mission".
You're probably thinking, the IDE plugin would handle this, right? Well, you might be right, but on Symfony 5.2, it's broken and does not collect routes.
IDE plugin should only compensate missing PHP features,
not duplicate them with code smell approach.
When we see a string, we assume it's a unique string we can change without affecting anything else, like an error message or headline title.
So here is the suggestion - what if we constants used instead of strings for route names?
use Symfony\Component\Routing\Annotation\Route;
+use App\ValueObject\Routing\RouteName;
final class MissionController
{
- #[Route(path: 'mission', name: 'mission')]
+ #[Route(path: 'mission', name: RouteName::MISSION)]
public function __invoke()
{
// ...
}
}
use Symfony\Component\Routing\Annotation\Route;
+use App\ValueObject\Routing\RouteName;
final class ContactController
{
- #[Route(path: 'contact', name: 'contact')]
+ #[Route(path: 'contact', name: RouteName::CONTACT)]
public function __invoke()
{
// ...
- return $this->redirectToRoute('mision');
+ return $this->redirectToRoute(RouteName::MISSION);
}
}
Looks nice, right? But applying this to your 200 routes... well, not so lovely.
It is very nice to Rector because it will:
RouteName
value object with all the route constantsUpdate your rector.php
:
use Rector\SymfonyCodeQuality\Rector\Attribute\ExtractAttributeRouteNameConstantsRector;
use Rector\Config\RectorConfig;
return function (RectorConfig $rectorConfig): void {
$rectorConfig->rule(ExtractAttributeRouteNameConstantsRector::class);
$rectorConfig->importNames();
};
Run Rector:
vendor/bin/rector process
That's it!
We run this rule on our website while writing this post. How did it go? See for yourself.
With invokable controllers, we might get even to this:
use Symfony\Component\Routing\Annotation\Route;
final class ContactController
{
#[Route(path: 'contact', name: ContactController::class)]
public function __invoke()
{
return $this->redirectToRoute(MissionController::class);
}
}
Happy coding!