Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Edit features in place - fix geometries not working #28631

Closed
qgib opened this issue Dec 14, 2018 · 13 comments
Closed

Edit features in place - fix geometries not working #28631

qgib opened this issue Dec 14, 2018 · 13 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Processing Relating to QGIS Processing framework or individual Processing algorithms

Comments

@qgib
Copy link
Contributor

qgib commented Dec 14, 2018

Author Name: Andy Whyte (Andy Whyte)
Original Redmine Issue: 20812
Affected QGIS version: 3.5(master)
Redmine category:processing/core
Assignee: Alessandro Pasotti


When using the edit features in place function with fix geometries it does not seem to work.

I tested on the same data set with and without the "in place" function and it worked without.

Please see Gif


@qgib
Copy link
Contributor Author

qgib commented Dec 15, 2018

Author Name: Alessandro Pasotti (@elpaso)


  • assigned_to_id was configured as Alessandro Pasotti

@qgib
Copy link
Contributor Author

qgib commented Dec 15, 2018

Author Name: Alessandro Pasotti (@elpaso)


Would you please add a test project and data? That would speed up the bug fixing process a lot.

@qgib
Copy link
Contributor Author

qgib commented Dec 17, 2018

Author Name: Andy Whyte (Andy Whyte)


  • 13982 was configured as NHS_CCG_Boundaries.shp
  • 13983 was configured as Test project.qgz

@qgib
Copy link
Contributor Author

qgib commented Dec 18, 2018

Author Name: Alessandro Pasotti (@elpaso)


Thanks for attaching data: I'll have a look ASAP.

@qgib
Copy link
Contributor Author

qgib commented Dec 21, 2018

Author Name: Alessandro Pasotti (@elpaso)


.SHP is not enough for shapefile data, you need to attach all the other files.

@qgib
Copy link
Contributor Author

qgib commented Dec 21, 2018

Author Name: Andy Whyte (Andy Whyte)


Apologies


  • 14013 was configured as NHS_CCG_Boundaries.dbf
  • 14012 was configured as NHS_CCG_Boundaries.prj

@qgib
Copy link
Contributor Author

qgib commented Dec 21, 2018

Author Name: Andy Whyte (Andy Whyte)


  • 14014 was configured as NHS_CCG_Boundaries.shx

@qgib
Copy link
Contributor Author

qgib commented Dec 21, 2018

Author Name: Alessandro Pasotti (@elpaso)


Your problem is probably caused by a processing setting, please check under

Settings -> Processing -> General -> Invalid features filtering => Do not filter (better performances)

Obviously that is the only value that makes sense for this algorithm or the invalid features will not even get passed in.


  • status_id was changed from Open to Feedback

@qgib
Copy link
Contributor Author

qgib commented Dec 21, 2018

Author Name: Alessandro Pasotti (@elpaso)


PR: #8730

@qgib
Copy link
Contributor Author

qgib commented Dec 21, 2018

Author Name: Alessandro Pasotti (@elpaso)


  • version was changed from 3.4.2 to 3.5(master)
  • operating_system was changed from Windows 10 to

@qgib
Copy link
Contributor Author

qgib commented Dec 21, 2018

Author Name: Andy Whyte (Andy Whyte)


Thanks for your help

@qgib
Copy link
Contributor Author

qgib commented Dec 21, 2018

Author Name: Alessandro Pasotti (@elpaso)


Thank you for filing this issue, it was indeed a bug: the setting should be ignored for those algorithms that fix broken geometries.


  • status_id was changed from Feedback to In Progress

@qgib
Copy link
Contributor Author

qgib commented Dec 22, 2018

Author Name: Alessandro Pasotti (@elpaso)


Applied in changeset f4d7506.


  • status_id was changed from In Progress to Closed
  • done_ratio was changed from 0 to 100

@qgib qgib closed this as completed Dec 22, 2018
@qgib qgib added Bug Either a bug report, or a bug fix. Let's hope for the latter! Processing Relating to QGIS Processing framework or individual Processing algorithms labels May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Processing Relating to QGIS Processing framework or individual Processing algorithms
Projects
None yet
Development

No branches or pull requests

1 participant