Remove NormalizeNamespaceByPSR4ComposerAutoloadRector as only moves files to namespaces, does not update any class referenes; better use IDE here #4057
Conversation
TomasVotruba
force-pushed
the
tv-less-psr4-namespace
branch
3 times, most recently
from
fa4738e to
7a983ee
Compare
…iles to namespaces, does not update any class referenes; better use IDE here
|
/cc @jtojnar |
|
Thanks for letting me know. The removal is a bit disappointing but I guess it’s expected since we were unable to make the rector work properly. I had hopes for Rector becoming a one-stop shop for transformations of PHP code but I guess that was unrealistic. Especially for one-off global transformations like this. Anyway, I am not aware of any IDE offering such functionality (maybe we could combine macros with single class/directory renaming?) nor do I work with PHP enough to justify adjusting to PHPStorm but https://github.com/lesphp/psr4-converter looks promising. |
|
@jtojnar It was not easy descission, but the rule was adding more work than saving. Changing just some class names was rather annoying. As for IDE, we use "move class" or "move namespace" refactoring, that handles the same operation much better. |
No description provided.