mapstruct / mapstruct/mapstruct-idea
Source value mapped more than once inspection for @ValueMapping
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- Java
- Estrelas
- 168
- Forks
- 41
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Descrição
For value mappings a source value can only be mapped once. For example
```
import org.mapstruct.Mapper;
import org.mapstruct.ValueMapping;
@Mapper
public interface ValueMapper {
enum A {
A,B
}
enum B {
A,B
}
@ValueMapping(source = "A", target = "B")
@ValueMapping(source = "A", target = "A")
A from(B b);
}
```
is invalide as source `A` is mapped twice. MapStruct report this as
> Source value mapping: "A" cannot be mapped more than once.
Currently the plugin does not recognize this as an error. An inspection should be added for this.
For normal mappings there is already an inspection implemented and could be used as an example: https://github.com/mapstruct/mapstruct-idea/blob/main/src/main/java/org/mapstruct/intellij/inspection/TargetPropertyMappedMoreThanOnceInspection.java.
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece por src/main/java/org/mapstruct/intellij/inspection/TargetPropertyMappedMoreThanOnceInspection.java, que fornece o padrão de inspection existente. Siga como ele detecta mappings normais duplicados e aplique a mesma abordagem aos valores de origem de @ValueMapping. Está concluído quando valores de origem duplicados são reconhecidos e reportados com a mensagem de erro esperada.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- java
- Domínio
- devtools
- Tipo de issue
- Funcionalidade
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Pouca atividade
- Clareza
- Claramente especificada
- Facilidade para iniciantes
- 75/100