#849 Missing imports for generic type parameters on Iterable and Maps

processor/src/main/java/org/mapstruct/ap/internal/model/assignment/GetterWrapperForCollectionsAndMaps.java

@@ -75,6 +75,7 @@ public class GetterWrapperForCollectionsAndMaps extends AssignmentWrapper {
         Set<Type> imported = new HashSet<Type>();
         imported.addAll( super.getImportTypes() );
         imported.add( localVarType ); /* is a local var */
+        imported.addAll( localVarType.getTypeParameters() );
         return imported;
     }
 

processor/src/test/java/org/mapstruct/ap/test/bugs/_849/Issue849Mapper.java

@@ -0,0 +1,37 @@
+/**
+ *  Copyright 2012-2016 Gunnar Morling (http://www.gunnarmorling.de/)
+ *  and/or other contributors as indicated by the @authors tag. See the
+ *  copyright.txt file in the distribution for a full listing of all
+ *  contributors.
+ *
+ *  Licensed under the Apache License, Version 2.0 (the "License");
+ *  you may not use this file except in compliance with the License.
+ *  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ */
+package org.mapstruct.ap.test.bugs._849;
+
+import org.mapstruct.Mapper;
+import org.mapstruct.Mapping;
+import org.mapstruct.factory.Mappers;
+
+/**
+ *
+ * @author Sjaak Derksen
+ */
+@Mapper
+public interface Issue849Mapper {
+
+    Issue849Mapper INSTANCE = Mappers.getMapper( Issue849Mapper.class );
+
+    @Mapping(source = "sourceList", target = "targetList")
+    Target mapSourceToTarget(Source source);
+
+}

processor/src/test/java/org/mapstruct/ap/test/bugs/_849/Issue849Test.java

@@ -0,0 +1,50 @@
+/**
+ *  Copyright 2012-2016 Gunnar Morling (http://www.gunnarmorling.de/)
+ *  and/or other contributors as indicated by the @authors tag. See the
+ *  copyright.txt file in the distribution for a full listing of all
+ *  contributors.
+ *
+ *  Licensed under the Apache License, Version 2.0 (the "License");
+ *  you may not use this file except in compliance with the License.
+ *  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ */
+package org.mapstruct.ap.test.bugs._849;
+
+import java.io.Serializable;
+import java.util.Arrays;
+import static org.assertj.core.api.Assertions.assertThat;
+
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mapstruct.ap.testutil.IssueKey;
+import org.mapstruct.ap.testutil.WithClasses;
+import org.mapstruct.ap.testutil.runner.AnnotationProcessorTestRunner;
+
+/**
+ * @author Andreas Gudian
+ *
+ */
+@WithClasses({ Source.class, Target.class, Issue849Mapper.class })
+@RunWith(AnnotationProcessorTestRunner.class)
+public class Issue849Test {
+
+    @Test
+    @IssueKey("849")
+    public void shouldCompileWithAllImportsDeclared() {
+
+        Source source = new Source();
+        source.setSourceList( Arrays.asList( (Serializable) "test" ) );
+
+        Target target = Issue849Mapper.INSTANCE.mapSourceToTarget( source );
+        assertThat( target.getTargetList() ).containsExactly( "test" );
+
+    }
+}

processor/src/test/java/org/mapstruct/ap/test/bugs/_849/Source.java

@@ -0,0 +1,39 @@
+/**
+ *  Copyright 2012-2016 Gunnar Morling (http://www.gunnarmorling.de/)
+ *  and/or other contributors as indicated by the @authors tag. See the
+ *  copyright.txt file in the distribution for a full listing of all
+ *  contributors.
+ *
+ *  Licensed under the Apache License, Version 2.0 (the "License");
+ *  you may not use this file except in compliance with the License.
+ *  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ */
+package org.mapstruct.ap.test.bugs._849;
+
+import java.io.Serializable;
+import java.util.List;
+
+/**
+ *
+ * @author Sjaak Derksen
+ */
+public class Source {
+
+    private List<Serializable> sourceList;
+
+    public List<Serializable> getSourceList() {
+        return sourceList;
+    }
+
+    public void setSourceList(List<Serializable> sourceList) {
+        this.sourceList = sourceList;
+    }
+}

processor/src/test/java/org/mapstruct/ap/test/bugs/_849/Target.java

@@ -0,0 +1,40 @@
+/**
+ *  Copyright 2012-2016 Gunnar Morling (http://www.gunnarmorling.de/)
+ *  and/or other contributors as indicated by the @authors tag. See the
+ *  copyright.txt file in the distribution for a full listing of all
+ *  contributors.
+ *
+ *  Licensed under the Apache License, Version 2.0 (the "License");
+ *  you may not use this file except in compliance with the License.
+ *  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ */
+package org.mapstruct.ap.test.bugs._849;
+
+import java.io.Serializable;
+import java.util.ArrayList;
+import java.util.List;
+
+/**
+ *
+ * @author Sjaak Derksen
+ */
+public class Target {
+
+    private List<Serializable> targetList;
+
+    public List<Serializable> getTargetList() {
+        if ( targetList == null ) {
+            targetList = new ArrayList<Serializable>();
+        }
+        return targetList;
+    }
+
+}