BuildAdapter().AdaptToType<T>() does not include all expected properties when source is using inheritance
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Comienza reproduciendo la diferencia entre Adapt() y BuildAdapter().AdaptToType() usando el ejemplo de consola, especialmente AdaptToTypeWithInterface con IMyInterface y MyImplementation. Sigue el punto de entrada BuildAdapter().AdaptToType() y compara su tratamiento del tipo de origen con Adapt(); se considera terminado cuando las propiedades de implementación heredadas se copian tanto en las rutas síncrona como asíncrona.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
While rewriting some code to use the async implementation of adapt using package Mapster.Async v2.0.1, I stubled upon an issue where not all expected source properties were copied to the result class.
However, the code was working as expected when I was using .Adapt<T>().
I've tracked it down to the source class being inherited from a base class and using a method where the parameter is of this base class type.
The following code can be used (e.g. in a console application) to reproduce the issue and show the difference in behavior between .Adapt<T>() and AdaptToType<T>():
using Mapster;
using Newtonsoft.Json;
namespace MapsterBug
{
internal class Program
{
static void Main(string[] args)
{
// sample source object
var myImplementation = new MyImplementation
{
InterfaceProperty = 123,
FirstImplementationProperty = 789,
SecondImplementationProperty = "test"
};
// adapt using .Adapt<T>(): the result is as expected
var resultAdapt = myImplementation.Adapt<MyDto>();
Console.WriteLine("Result of myImplementation.Adapt<MyDto>():");
Console.WriteLine(JsonConvert.SerializeObject(resultAdapt, Formatting.Indented));
Console.WriteLine("\r\n-----------------------------------------\r\n");
// adapt using .AdaptToType<T>(): the result is as expected
var resultAdaptToType = myImplementation.BuildAdapter().AdaptToType<MyDto>();
Console.WriteLine("Result of myImplementation.BuildAdapter().AdaptToType<MyDto>():");
Console.WriteLine(JsonConvert.SerializeObject(resultAdaptToType, Formatting.Indented));
Console.WriteLine("\r\n-----------------------------------------\r\n");
// adapt using .Adapt<T>() inside a method where the parameter is of the interface type: the result is as expected
var resultAdaptWithInterface = AdaptWithInterface(myImplementation);
Console.WriteLine("Result of myImplementation.Adapt<MyDto>() in method with interface:");
Console.WriteLine(JsonConvert.SerializeObject(resultAdaptWithInterface, Formatting.Indented));
Console.WriteLine("\r\n-----------------------------------------\r\n");
// adapt using .AdaptToType<T>() inside a method where the parameter is of the interface type:
// the result only includes the interface property, but it does not include the properties of the implementation
var resultAdaptToTypeWithInterface = AdaptToTypeWithInterface(myImplementation);
Console.WriteLine("Result of myInterface.BuildAdapter().AdaptToType<MyDto>() in method with interface:");
Console.WriteLine(JsonConvert.SerializeObject(resultAdaptToTypeWithInterface, Formatting.Indented));
}
static MyDto AdaptWithInterface(IMyInterface myInterface) => myInterface.Adapt<MyDto>();
static MyDto AdaptToTypeWithInterface(IMyInterface myInterface) => myInterface.BuildAdapter().AdaptToType<MyDto>();
}
public interface IMyInterface
{
public int InterfaceProperty { get; set; }
}
public class MyImplementation : IMyInterface
{
public int InterfaceProperty { get; set; }
public int FirstImplementationProperty { get; set; }
public string SecondImplementationProperty { get; set; }
}
public class MyDto
{
public int InterfaceProperty { get; set; }
public int FirstImplementationProperty { get; set; }
public string SecondImplementationProperty { get; set; }
}
}
This code is using AdaptToType<T>() to not complicate things by using an additional package (Mapster.Async), but the AdaptToTypeAsync<T>() method has the same behavior.
When running the application, the output is as follows:
Result of myImplementation.Adapt<MyDto>():
{
"InterfaceProperty": 123,
"FirstImplementationProperty": 789,
"SecondImplementationProperty": "test"
}
-----------------------------------------
Result of myImplementation.BuildAdapter().AdaptToType<MyDto>():
{
"InterfaceProperty": 123,
"FirstImplementationProperty": 789,
"SecondImplementationProperty": "test"
}
-----------------------------------------
Result of myImplementation.Adapt<MyDto>() in method with interface:
{
"InterfaceProperty": 123,
"FirstImplementationProperty": 789,
"SecondImplementationProperty": "test"
}
-----------------------------------------
Result of myInterface.BuildAdapter().AdaptToType<MyDto>() in method with interface:
{
"InterfaceProperty": 123,
**"FirstImplementationProperty": 0,**
**"SecondImplementationProperty": null**
}
Results 1, 2 and 3 are as expected, but the last result only includes the IMyInterface property (InterfaceProperty) and not the additional MyImplementation properties (FirstImplementationProperty and SecondImplementationProperty) marked in bold.
Versions used in code sample:
- .NET 8
- Mapster 7.4.0
- Newtonsoft.Json 13.0.3
- Lenguaje dominante
- C#
- Estrellas
- 5.2k
- Forks
- 411
- Merge medio
- 1 d 8 h
- PR fusionados (30 d)
- 4
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de MapsterMapper/Mapster
-
.NET MatrixAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
MapsterMapper/Mapster#1003 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
MapsterMapper/Mapster#1024 · 1 comentario ·
-
[Mapster Tool] Add support for global:: to disambiguate type names in generated mappersPosiblemente ocupada @DocSvartz la tomó hace 3 días. Abiertoimprovement
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
MapsterMapper/Mapster#1020 · 5 comentarios ·
-
Mapster.Tool - PropertySettingBuilder.Map(member, mapFunc, targetPropertyName) method is not working as expected!Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
MapsterMapper/Mapster#1017 · 6 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 57/100
MapsterMapper/Mapster#1007 · 5 comentarios ·
Todos los issues de MapsterMapper/Mapster
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
-
[Bug]: Revit selector reads process ids 2000-2099 as years (flaky AgentClientTests.AYearOrAProcessIdSelectsTheHost)Posiblemente ocupada @Wurschdhaud la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
pyrevitlabs/pyRevit#3817 ·
Los mantenedores suelen responder en 1 día
-
bug effort:S P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
nightscout/nocturne#2449 ·
Los mantenedores suelen responder en 1 día
-
audit:1 bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
NethermindEth/dotnet-libp2p#395 ·
Los mantenedores suelen responder en 1 día
-
area: client bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día