Fix fieldMapping phpdoc

[VincentLanglet]

See #8286 (comment)

[VincentLanglet]

Friendly ping @greg0ire for the review :)

[greg0ire]

Do you think this list is exhaustive now? If not please do this:

Suggested change

	* }>
	* }>&array<string,mixed>

[VincentLanglet]

I don't know at all, I just followed the phpdoc:

     * The mapping definition array has the following values:
     *
     * - <b>fieldName</b> (string)
     * The name of the field in the Entity.
     *
     * - <b>type</b> (string)
     * The type name of the mapped field. Can be one of Doctrine's mapping types
     * or a custom mapping type.
     *
     * - <b>columnName</b> (string, optional)
     * The column name. Optional. Defaults to the field name.
     *
     * - <b>length</b> (integer, optional)
     * The database length of the column. Optional. Default value taken from
     * the type.
     *
     * - <b>id</b> (boolean, optional)
     * Marks the field as the primary key of the entity. Multiple fields of an
     * entity can have the id attribute, forming a composite key.
     *
     * - <b>nullable</b> (boolean, optional)
     * Whether the column is nullable. Defaults to FALSE.
     *
     * - <b>columnDefinition</b> (string, optional, schema-only)
     * The SQL fragment that is used when generating the DDL for the column.
     *
     * - <b>precision</b> (integer, optional, schema-only)
     * The precision of a decimal column. Only valid if the column type is decimal.
     *
     * - <b>scale</b> (integer, optional, schema-only)
     * The scale of a decimal column. Only valid if the column type is decimal.
     *
     * - <b>'unique'</b> (string, optional, schema-only)
     * Whether a unique constraint should be generated for the column.

The inherited key already added wasn't in the phpdoc.
I don't know if someone can confirm there is nothing else... If nobody can, I will add your suggestion.

[orklah]

It's definitely incomplete. I think I added more in the 2.9.x branch:
https://github.com/doctrine/orm/blob/2.9.x/lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php#L440

Also this PR should supersede #8302 (which was already kinda superseded by the PR on 2.9.x)

[orklah]

#8302 (comment) this one should be considered too

[VincentLanglet]

I have added

originalClass?: class-string,
originalField?: string

from the 2.9 branch

And also

quoted?: bool
requireSQLConversion?: bool

I found nothing else in the code

[greg0ire]

@orklah please review.

[greg0ire]

Thanks @VincentLanglet !