Bug report #17853

Duplicate Feature and Duplicate Feature (redigizited) operate on multiple features

Added by Nyall Dawson almost 7 years ago. Updated about 6 years ago.

Status:Closed
Priority:Normal
Assignee:David Signer
Category:Actions
Affected QGIS version:master Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:
Crashes QGIS or corrupts data:No Copied to github as #:25748

Description

If I use the new Duplicate Feature or Duplicate Feature (redigitized) actions, and click on overlapping features, all are duplicated. I think either the actions should be renamed to "Duplicate Features", or just the first encountered feature should be duplicated (even better, prioritize selected features if there's any selection on the layer).

Associated revisions

Revision 8839e2fc
Added by David Signer about 6 years ago

all features option in menu

means this can be controlled by the user

fixes #17853

History

#1 Updated by Denis Rouzaud almost 7 years ago

  • Category set to Actions
  • Assignee set to David Signer

#2 Updated by David Signer almost 7 years ago

This is concerning not only the duplicate actions but any action listed in the feature action menu of the canvas.
So I guess for all actions just the first feature should be handled. Right?

#3 Updated by David Signer about 6 years ago

David Signer wrote:

This is concerning not only the duplicate actions but any action listed in the feature action menu of the canvas.
So I guess for all actions just the first feature should be handled. Right?

I just checked out this bug again. I understand that it's not expected to duplicate the whole stack of features at the same time. But because it's for all actions I'm not sure how to handle it. If it should only select the feature on the top, should this be done for all actions or only the duplicate feature action? I think it would make sense to have it general. But aren't there actions where we expect to do it on all the features? And if we make it individually, how should we handle it for manually created actions - having a configuration on that?

#4 Updated by Nyall Dawson about 6 years ago

In that case, maybe go the easy approach of renaming to "Duplicate Features"?

#5 Updated by David Signer about 6 years ago

Thanks for your feedback.

I'm still thinking about. Because on redigitized duplication it only takes one feature. But what seems to be correct here, is actually the bug. Because the action is started for every feature on the "stack". But every action overwrites the MapTool of the action before, and so it digitizes and duplicates only the one on top of the stack. I would not avoid that, because I'm sure we don't want to redigitize a duplication of the whole stack, because we don't know which one is the currently digitized.

So I see some options:
1. Rename "Duplicate feature" to "Duplicate features" and keep the "Duplicate feature and digitize" in singular. This would be a quick fix. Not realy nice, but at least less confusing for the user.
2. I make sure that it only takes the one on the top for the duplication-actions. So they are different to all the other actions. But probably that's not that bad.
3. I make sure that it only takes the one on the top for all the actions. But like I mentioned before, the user possibly expects to have the action on all the features, so I wouldn't do that.

#6 Updated by David Signer about 6 years ago

  • % Done changed from 0 to 100
  • Status changed from Open to Closed

Also available in: Atom PDF