primaryKey: true
sur alias et aliasedPrimaryKey
#336
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Un alias n'est plus une clé primaire implicite dès lors que la propriété originale est une clé primaire et que l'alias n'a pas de préfixe et de suffixe : un alias sera désormais toujours
primaryKey: false
par défaut, et pourra être mis àprimaryKey: true
comme les propriétés classiques et d'association.En interne, cette ancienne définition est toujours présente dans la propriété
AliasedPrimaryKey
pour garder les comportements existants sur dans les générateurs qui ne sont pas vraiment faisables autrement sans gros breaking change (en particulier le fait que les clés primaires sont générés avecisRequired: false
en JS). Elle est aussi utilisée pour continuer à pouvoir faire des mappings entre associations et compositions.Ce changement va permettre :
required: false
sur un alias de PK sans préfixe ni suffixe (c'est en théorie un "breaking change" mais en même temps c'était quelque chose qui devait marcher et qui ne marchait pas)Les DTOs n'ayant plus de PK côté topmodel impliquent aussi quelques choses qui ne vont plus marcher genre faire des associations sur des DTOs sans préciser de propriété cible, mais bon en même temps... ¯_(ツ)_/¯