mapstruct / mapstruct/mapstruct-idea
Wrong usage of mappers factory should be able to remove static initializer
Personne n'a encore pris cette issue.
- Langage dominant
- Java
- Étoiles
- 168
- Forks
- 41
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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:
@Mapper(componentModel = "spring")
abstract class WrongComponentModelMapper {
static {
Mappers.getMapper(WrongComponentModelMapper.class);
}
Target map(Target source);
}
or
@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.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par reproduire les exemples d'initialiseur statique et de champ d'interface du ticket, puis suivez la manière dont le IntelliJ plugin gère actuellement l'utilisation de Mappers.getMapper. Consultez la discussion autour de PR #179 pour le contexte. Le travail est considéré comme terminé lorsqu'il a été déterminé si ces motifs peuvent être pris en charge et, dans l'affirmative, que le comportement a été validé avec une couverture pour les deux exemples.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java
- Domaine
- tooling
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 35/100