#243: Do not warn for unmapped source properties on maps

Author: thunderhook

src/main/java/org/mapstruct/intellij/codeinsight/references/BaseReference.java

@@ -43,7 +43,7 @@ public abstract class BaseReference extends PsiReferenceBase<PsiElement> impleme
      * @return The mapping method that this reference belongs to
      */
     @Nullable
-    PsiMethod getMappingMethod() {
+    public PsiMethod getMappingMethod() {
         PsiElement element = getElement();
         UExpression expression = UastContextKt.toUElement( element, UExpression.class );
         if ( expression != null ) {

src/main/java/org/mapstruct/intellij/codeinsight/references/BaseValueMappingReference.java

@@ -47,7 +47,7 @@ public final PsiElement resolve() {
 
     @Override
     @Nullable
-    PsiMethod getMappingMethod() {
+    public PsiMethod getMappingMethod() {
         PsiMethod mappingMethod = super.getMappingMethod();
         if ( isNotValueMapping( mappingMethod ) ) {
             return null;

src/main/java/org/mapstruct/intellij/inspection/MapstructReferenceInspection.java

@@ -8,14 +8,20 @@
 import com.intellij.codeInspection.ProblemHighlightType;
 import com.intellij.codeInspection.ProblemsHolder;
 import com.intellij.openapi.util.TextRange;
+import com.intellij.psi.CommonClassNames;
 import com.intellij.psi.ContributedReferenceHost;
 import com.intellij.psi.PsiClass;
 import com.intellij.psi.PsiElement;
 import com.intellij.psi.PsiElementVisitor;
 import com.intellij.psi.PsiLanguageInjectionHost;
+import com.intellij.psi.PsiMethod;
+import com.intellij.psi.PsiParameter;
 import com.intellij.psi.PsiReference;
+import com.intellij.psi.PsiType;
 import com.intellij.psi.util.PsiTreeUtil;
+import com.intellij.psi.util.PsiUtil;
 import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 import org.mapstruct.intellij.codeinsight.references.BaseReference;
 import org.mapstruct.intellij.codeinsight.references.BaseValueMappingReference;
 
@@ -67,9 +73,35 @@ private boolean shouldRegisterProblem(BaseReference reference) {
                 return valueMappingReference.getEnumClass() != null;
             }
 
+            if ( singleSourceParameterIsOfTypeMap( reference.getMappingMethod() ) ) {
+                return false;
+            }
+
             return !containingClassIsAnnotationType( reference.getElement() );
         }
 
+        private boolean singleSourceParameterIsOfTypeMap(@Nullable PsiMethod mappingMethod) {
+
+            if (mappingMethod != null) {
+                PsiParameter[] parameters = mappingMethod.getParameterList().getParameters();
+                if (parameters.length > 0) {
+                    PsiType parameterType = parameters[0].getType();
+                    return isMapType( parameterType );
+                }
+            }
+
+            return false;
+        }
+
+        private boolean isMapType(PsiType type) {
+            PsiClass psiClass = PsiUtil.resolveClassInType(type);
+            if (psiClass == null) {
+                return false;
+            }
+            return CommonClassNames.JAVA_UTIL_MAP.equals(psiClass.getQualifiedName());
+        }
+
+
         private boolean containingClassIsAnnotationType(PsiElement element) {
 
             PsiClass containingClass = PsiTreeUtil.getParentOfType( element, PsiClass.class );

src/test/java/org/mapstruct/intellij/bugs/_243/DisableSourcePropertyInspectionOnMapTest.java

@@ -0,0 +1,31 @@
+/*
+ * Copyright MapStruct Authors.
+ *
+ * Licensed under the Apache License version 2.0, available at https://www.apache.org/licenses/LICENSE-2.0
+ */
+package org.mapstruct.intellij.bugs._243;
+
+import org.jetbrains.annotations.NotNull;
+import org.mapstruct.intellij.inspection.BaseInspectionTest;
+import org.mapstruct.intellij.inspection.MapstructReferenceInspection;
+
+/**
+ * @author Oliver Erhart
+ */
+public class DisableSourcePropertyInspectionOnMapTest extends BaseInspectionTest {
+
+    @Override
+    protected String getTestDataPath() {
+        return "testData/bugs/_243";
+    }
+
+    @NotNull
+    @Override
+    protected Class<MapstructReferenceInspection> getInspection() {
+        return MapstructReferenceInspection.class;
+    }
+
+    public void testDisableSourcePropertyInspectionOnMapTest() {
+        doTest();
+    }
+}

testData/bugs/_243/DisableSourcePropertyInspectionOnMapTest.java

@@ -0,0 +1,28 @@
+/*
+ * Copyright MapStruct Authors.
+ *
+ * Licensed under the Apache License version 2.0, available at https://www.apache.org/licenses/LICENSE-2.0
+ */
+
+import java.time.LocalDate;
+import java.util.Map;
+
+import org.mapstruct.Mapper;
+import org.mapstruct.Mapping;
+
+@Mapper
+abstract class Issue243Mapper {
+
+    @Mapping(source = "exDate", target = "exDate", dateFormat = "yyyy-MM-dd")
+    @Mapping(source = "payDate", target = "payDate", dateFormat = "yyyy-MM-dd")
+    @Mapping(source = "recordDate", target = "recordDate", dateFormat = "yyyy-MM-dd")
+    @Mapping(source = "annDate", target = "annDate", dateFormat = "yyyy-MM-dd")
+    public abstract CorporateAction toCorporateAction(Map<String, String> rowValues);
+}
+
+class CorporateAction {
+    public LocalDate exDate;
+    public LocalDate payDate;
+    public LocalDate recordDate;
+    public LocalDate annDate;
+}