[Symfony] Set few default common service names for Symfony App Analyzer

[stloyd]

Allow to set custom ones via configuration:

# rector.yaml
services:
  Rector\Symfony\Bridge\DefaultAnalyzedSymfonyApplicationContainer:
    autowire: true
    arguments:
      $commonNamesToTypes:
        'doctrine.orm.other_entity_manager': Doctrine\ORM\EntityManagerInterface
        'pseudo.buggy.service_name': App\Some\ServiceClass

[TomasVotruba]

There should be no duplicated for PHP code info.
The sniff is disabled for this, because it was doing to much

[TomasVotruba]

What is this needed for? Default names should not depend on custom user input, but on default Symfony naming.

The point is 0-configuration. So services are loaded from the App container itself and we don't bother user to basically copy-paste it to rector.yaml config :)

[stloyd]

Problem is for example that by default container says that: doctrine.orm.entity_manager is instance of Doctrine\Common\Persistence\ObjectManager, which in fact is not really true, cause it's instance of Doctrine\ORM\EntityManagerInterface (which is also used for autowire functionality).

This is just one example, if there is more edge cases (in big legacy like we have, we can occur them most likely), user is not able to have automatic fix for such cases and he/she will be forced to fix it manually.

With this you can have most common known cases for replacement, but also user is able to not wait for new release of Rector if he/she finds another edge case.

[TomasVotruba]

This is just one example, if there is more edge cases (in big legacy like we have, we can occur them most likely), user is not able to have automatic fix for such cases and he/she will be forced to fix it manually.

I see. It makes sense now :) 👍

I miss this description on PR. Could you add it next time, so I can read the code with context of your intention? It would speed up the review