[Form][Guesser] Symfony should guess a better field type for Doctrine 'array' type

[lmcd]

Currently defaults to a textbox which just displays 'Array'.

[Gregwar]

Symfony don't use Doctrine mapping to do the field guessing but validation constraints.

Maybe @Assert\Type(type="array") should be guessed as a collection type...

[stof]

Symfony uses both for the guessing

[Gregwar]

@stof, right, sorry. guessing for collection with doctrine array are even here, but commented

<?php
//... 
 53             //case 'array':
 54             //  return new TypeGuess('Collection', array(), Guess::HIGH_CONFIDENCE);
//...

[Gregwar]

Can't get far enough in the history of the guesser to understand why it's commented, any information ? Moreover, why there is an uppercase C on Collection ? Is is because a long time ago types began with upercases ?

[stof]

Reading the history, this is commented since the first implementation of the guesser: 3bf9f77

[stof]

@bschussek what was the reason to comment the guessing for the array fields ?

[webmozart]

@stof Simply that I didn't know what to do back then. What should we guess? "collection"?

[vicb]

Task for the BHD: guess collection for both Doctrine arrays & @Assert\Type(type="array")

[pvanliefland]

Working on it right now