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

delete ways before nodes? #27

Closed
polarbearing opened this issue Feb 1, 2016 · 3 comments
Closed

delete ways before nodes? #27

polarbearing opened this issue Feb 1, 2016 · 3 comments

Comments

@polarbearing
Copy link

Trying to delete a few simple ways today by setting minus signs in front of the 'way' and 'node' statements, the upload failed with error 412 (Precondition Failed), complaining that a node was still used by a way, which I however tried to delete in the same changeset. In L0 statement, each way deletion appeared before its nodes.

Looking at the OsmChange contents I found all nodes being sorted numerically, followed by the numerically sorted ways. Maybe the ways should be listed first?

(I worked around for that edit by removing only the ways in the first changeset, then loading only the nodes via the temporary OsmChange file, and deleting them afterwords.)

@Zverik
Copy link
Owner

Zverik commented Feb 1, 2016

Thanks for reminding about this, I've encountered the issue, but forgot to add it here.
I'll fix this a bit later.

@polarbearing
Copy link
Author

Tnx. For some reason, the error-412-attempts do not show up in my changeset history. Isn't there a CS opened before the upload attempt? Or do the erratic CSs just not show up? (probably the latter, because my edit count went up a bit more than the 2 successful CSs)

Zverik added a commit that referenced this issue Feb 8, 2016
@Zverik
Copy link
Owner

Zverik commented Feb 8, 2016

Fixed.

@Zverik Zverik closed this as completed Feb 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants