mapstruct / mapstruct/mapstruct-idea

Source value mapped more than once inspection for @ValueMapping

オープン
#265 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

feature hack.commit.push
主要言語
Java
スター
168
フォーク
41
PR マージ指標
30日以内にマージされた PR はありません

説明

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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

既存の inspection パターンを提供している src/main/java/org/mapstruct/intellij/inspection/TargetPropertyMappedMoreThanOnceInspection.java から始めてください。通常の重複したマッピングをどのように検出しているかを追跡し、同じアプローチを @ValueMapping のソース値に適用してください。重複したソース値が認識され、期待されるエラーメッセージで報告されれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java
領域
devtools
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
静か
明瞭さ
明確に書かれている
初心者へのやさしさ
75/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。