mapstruct / mapstruct/mapstruct-idea

Wrong usage of mappers factory should be able to remove static initializer

Abierto
#183 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
Java
Estrellas
168
Forks
41
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Why reviewing PR #179 @hduelme pointed out that the Wrong usage of mappers factory does not work (i.e. it does not remove it) with the following code:

```java
@Mapper(componentModel = "spring")
abstract class WrongComponentModelMapper {

static {
Mappers.getMapper(WrongComponentModelMapper.class);
}

Target map(Target source);
}
```

or
```java
@Mapper(config = MyConfig.class, componentModel = "spring")
interface WrongComponentModelMapper {

WrongComponentModelMapper INSTANCE = () -> {
return Mappers.getMapper(WrongComponentModelMapper.class);
};

Target map(Target source);
}
```

We need to see if we can support this.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza reproduciendo los ejemplos de inicializador estático y campo de interfaz del issue y rastreando cómo gestiona actualmente el IntelliJ plugin el uso de Mappers.getMapper. Consulta la discusión en torno a PR #179 para obtener contexto. Se considerará hecho cuando se determine si estos patrones pueden admitirse y, en caso afirmativo, se valide el comportamiento con cobertura para ambos ejemplos.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
tooling
Tipo de issue
Nueva funcionalidad
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.