Fix for issue #1848

Author: emacarron

src/main/java/org/apache/ibatis/executor/resultset/DefaultResultSetHandler.java

@@ -1048,12 +1048,7 @@ private List<ResultMapping> getResultMappingsForRowKey(ResultMap resultMap) {
 
   private void createRowKeyForMappedProperties(ResultMap resultMap, ResultSetWrapper rsw, CacheKey cacheKey, List<ResultMapping> resultMappings, String columnPrefix) throws SQLException {
     for (ResultMapping resultMapping : resultMappings) {
-      if (resultMapping.getNestedResultMapId() != null && resultMapping.getResultSet() == null) {
-        // Issue #392
-        final ResultMap nestedResultMap = configuration.getResultMap(resultMapping.getNestedResultMapId());
-        createRowKeyForMappedProperties(nestedResultMap, rsw, cacheKey, nestedResultMap.getConstructorResultMappings(),
-            prependPrefix(resultMapping.getColumnPrefix(), columnPrefix));
-      } else if (resultMapping.getNestedQueryId() == null) {
+      if (resultMapping.isSimple()) {
         final String column = prependPrefix(resultMapping.getColumn(), columnPrefix);
         final TypeHandler<?> th = resultMapping.getTypeHandler();
         List<String> mappedColumnNames = rsw.getMappedColumnNames(resultMap, columnPrefix);

src/main/java/org/apache/ibatis/mapping/ResultMapping.java

@@ -250,6 +250,10 @@ public boolean isLazy() {
   public void setLazy(boolean lazy) {
     this.lazy = lazy;
   }
+  
+  public boolean isSimple() {
+    return this.nestedResultMapId == null && this.nestedQueryId == null && this.resultSet == null;
+  }
 
   @Override
   public boolean equals(Object o) {

src/test/java/org/apache/ibatis/submitted/parent_childs/ParentChildTest.java

@@ -23,7 +23,6 @@
 import org.apache.ibatis.session.SqlSession;
 import org.apache.ibatis.session.SqlSessionFactory;
 import org.apache.ibatis.session.SqlSessionFactoryBuilder;
-import org.junit.Ignore;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
@@ -62,7 +61,6 @@ void shouldGet2Parents() {
 
   // issue #1848
   @Test
-  @Ignore
   void shouldGet2ParentsWithConstructor() {
     try (SqlSession sqlSession = sqlSessionFactory.openSession()) {
       Mapper mapper = sqlSession.getMapper(Mapper.class);