bug #469 Fixed make:auth when using a custom repository method as UserProvider (romaricdrigon)

This PR was merged into the 1.0-dev branch.

Discussion
----------

Fixed make:auth when using a custom repository method as UserProvider

Hello,

It is possible in vanilla Symfony to change how the User is retrieved by the `entity` user provider, so you use your own repository - [https://symfony.com/doc/current/security/user_provider.html#using-a-custom-query-to-load-the-user](https://symfony.com/doc/current/security/user_provider.html#using-a-custom-query-to-load-the-user). Then you remove `providers.xxxx.entity.property` key.

Unfortunately, it causes `make:auth` command to crash:
![image](https://user-images.githubusercontent.com/919405/64244498-6fde3680-cf09-11e9-8edb-79db50b3936e.png)

This PR patches `InteractiveSecurityHelper`, so it checks `property` key exists before reading from it. No BC break.
I added a test for that very specific use case.

Commits
-------

7014d93 Fixed make:auth when using a custom repository method as UserProvider

Author: weaverryan

src/Security/InteractiveSecurityHelper.php

@@ -114,7 +114,7 @@ private function guessUserClassDefault(): string
 
     public function guessUserNameField(SymfonyStyle $io, string $userClass, array $providers): string
     {
-        if (1 === \count($providers) && isset(current($providers)['entity'])) {
+        if (1 === \count($providers) && isset(current($providers)['entity']) && isset(current($providers)['entity']['property'])) {
             $entityProvider = current($providers);
 
             return $entityProvider['entity']['property'];

tests/Maker/FunctionalTest.php

@@ -438,6 +438,8 @@ function (string $output, string $directory) {
                     'AppCustomAuthenticator',
                     // controller name
                     'SecurityController',
+                    // field name
+                    'userEmail',
                     "no"
                 ]
             )

tests/Security/InteractiveSecurityHelperTest.php

@@ -187,6 +187,13 @@ public function getUsernameFieldsTest()
             true
         ];
 
+        yield 'guess_with_providers_and_custom_repository_method' => [
+            'providers' => ['app_provider' => ['entity' => null]],
+            'expectedUsernameField' => 'email',
+            true,
+            FixtureClass::class
+        ];
+
         yield 'guess_fixture_class' => [
             'providers' => [],
             'expectedUsernameField' => 'email',