Skip to content

Commit

Permalink
[Core] Property param dans les mappers : l'alias "full" remonte est t…
Browse files Browse the repository at this point in the history
…oujours en erreur
  • Loading branch information
gideruette committed Apr 18, 2024
1 parent abbb542 commit 2ce3101
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions TopModel.Core/ModelStore.cs
Original file line number Diff line number Diff line change
Expand Up @@ -1252,7 +1252,7 @@ IEnumerable<ModelError> ResolveAliases(IEnumerable<AliasProperty> alps)
return [p.TargetPropertyReference];
}

var propRef = p.Property.GetLocation() ?? new Reference();
var propRef = new Reference();
propRef.ReferenceName = p.Property.Name;
return [propRef];
}));
Expand Down Expand Up @@ -1315,7 +1315,8 @@ IEnumerable<ModelError> ResolveAliases(IEnumerable<AliasProperty> alps)

foreach (var param in mapper.Params.Where((p, i) => p.GetRequired() && mapper.Params.Where((q, j) => !q.GetRequired() && j < i).Any()))
{
yield return new ModelError(classe, $"Le paramètre '{param.GetName()}' du mapper ne peut pas être obligatoire si l'un des paramètres précédents ne l'est pas.", param.GetLocation()) { ModelErrorType = ModelErrorType.TMD1034 };
var previousRequired = mapper.Params.Where((q, j) => !q.GetRequired() && j < mapper.Params.IndexOf(param)).Select(p => p.GetName());
yield return new ModelError(classe, $"Le paramètre '{param.GetName()}' du mapper ne peut pas être obligatoire si l'un des paramètres précédents ({string.Join(", ", previousRequired)}) ne l'est pas.", param.GetLocation()) { ModelErrorType = ModelErrorType.TMD1034 };
}
}
}
Expand Down

0 comments on commit 2ce3101

Please sign in to comment.