From the manual: "TheStringable interface denotes a class as having a__toString() method. Unlike most interfaces,Stringable is implicitly present on any class that has the magic__toString() method defined, although it can andshould be declared explicitly."

I used the Rector ruleStringableForToStringRector

Therector.php file I used.

<?phpuseRector\Config\RectorConfig;useRector\Php80\Rector\Class_\StringableForToStringRector;return RectorConfig::configure()    ->withPaths([__DIR__ .'/src',    ])    ->withSkip([__DIR__ .'/src/*/Tests/*',    ])    ->withRules([        StringableForToStringRector::class,    ]);

Also, I want to mention that this change is allowed under theBackward Compatibility Promise.

santysisi requested review fromalexandre-daubois,chalasr,jderusse,kbond andwelcoMattic ascode owners

November 4, 2025 21:47

carsonbot added Status: Needs Review Feature labels

Nov 4, 2025

carsonbot added this to the8.0 milestone

Nov 4, 2025

carsonbot changed the title~~Refactor: addStringable when possible~~ Refactor: addStringable when possible

Nov 4, 2025

santysisi changed the title ~~Refactor: addStringable when possible~~ Refactor: implementsStringable when possible

Nov 4, 2025

Copy link

ContributorAuthor

santysisi commentedNov 4, 2025

I targeted the 8.0 branch because I think we are in feature freeze.

So I think this change could be added (if you like it) for the 8.1 🤔

Copy link

ContributorAuthor

santysisi commentedNov 4, 2025

I think the errors aren't relevant and were resolved by thiscontribution.

nicolas-grekas reviewed

Nov 5, 2025

View reviewed changes

src/Symfony/Component/Scheduler/Trigger/CronExpressionTrigger.php OutdatedShow resolvedHide resolved

nicolas-grekas reviewed

Nov 5, 2025

View reviewed changes

Copy link

Member

nicolas-grekas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Overall I'm -0 on this: it's strictly equivalent - except the explicit casts which might hide bugs.

Copy link

Member

alexandre-daubois commentedNov 5, 2025

I'm not totally convinced either by this addition. Even though I never felt the need (or lack) of it, why not if it is recommended by the docs.

Copy link

Contributor

OskarStark commentedNov 5, 2025

I agree with@nicolas-grekas and@alexandre-daubois, the only thing I like is the readability and that you see directly that it is string able when reading the class

Copy link

Member

derrabus commentedNov 5, 2025

We have rejected PRs that only change code-style without being able to enfore the new style.

Refactor: addStringable when possible

e16e36f

santysisi force-pushed therefactor/add-stringable-interface-when-possible branch from5ac5f7e toe16e36fCompare

November 5, 2025 11:56

Copy link

ContributorAuthor

santysisi commentedNov 5, 2025

We have rejected PRs that only change code-style without being able to enfore the new style.

If you and the other core members agree with this change, I can try to add this rule (probably over the weekend) tophp-cs-fixer, since Symfony uses this tool.