return ids of the new child features on duplication

signedav · m-kuhn · commit 78eba9ed05d3 · 2017-11-16T11:13:07.000+01:00
when duplicating a feature in QgsVectorLayerUtils::duplicateFeature and this feature has over a relation the child features, we return now the ids of the new created child features in the respone object QgsDuplicateFeatureContext.
diff --git a/src/core/qgsvectorlayerutils.cpp b/src/core/qgsvectorlayerutils.cpp
@@ -367,10 +367,7 @@ QgsFeature QgsVectorLayerUtils::duplicateFeature( QgsVectorLayer *layer, const Q
           childFeature.setAttribute( fieldPair.first, newFeature.attribute( fieldPair.second ) );
         }
         //call the function for the child
-        duplicateFeature( relation.referencingLayer(), childFeature, project, depth, duplicateFeatureContext );
-
-        //add the new feature id for feedback
-        childFeatureIds.insert( childFeature.id() );
+        childFeatureIds.insert( duplicateFeature( relation.referencingLayer(), childFeature, project, depth, duplicateFeatureContext ).id() );
       }
 
       //store for feedback

Original file line number	Diff line number	Diff line change
`@@ -367,10 +367,7 @@ QgsFeature QgsVectorLayerUtils::duplicateFeature( QgsVectorLayer *layer, const Q`
`367`	`367`	`childFeature.setAttribute( fieldPair.first, newFeature.attribute( fieldPair.second ) );`
`368`	`368`	`}`
`369`	`369`	`//call the function for the child`
`370`		`- duplicateFeature( relation.referencingLayer(), childFeature, project, depth, duplicateFeatureContext );`
`371`		`-`
`372`		`- //add the new feature id for feedback`
`373`		`- childFeatureIds.insert( childFeature.id() );`
	`370`	`+ childFeatureIds.insert( duplicateFeature( relation.referencingLayer(), childFeature, project, depth, duplicateFeatureContext ).id() );`
`374`	`371`	`}`
`375`	`372`
`376`	`373`	`//store for feedback`