Skip to content
This repository has been archived by the owner on Jul 24, 2021. It is now read-only.

delete_with_relations_and_history should actually delete with relations #773

Closed
openstreetmap-trac opened this issue Jul 23, 2021 · 2 comments

Comments

@openstreetmap-trac
Copy link

Reporter: Richard
[Submitted to the original trac issue database at 4.22pm, Friday, 21st March 2008]

delete_with_relations_and_history in app/models/way.rb should remove the way (and its constituent nodes, if they're not used in any other ways) from all relations.

Right now it just throws a Precondition Failed if the way is a member of a relation.

@openstreetmap-trac
Copy link
Author

Author: steve[at]asklater.com
[Added to the original trac issue at 7.00pm, Friday, 21st March 2008]

This is sort of a feature - I don't like the fact that potlatch just fucks up data (eg relations) in favour of user interface. So, rather than breaking relations, it expects you to remove it from them first. Not wanting to get too heavily in to the relations code, which is almost as bad as all the other SQL, I left that battle for another day as I was intending on refactoring the relations controller and model in to something sane

@openstreetmap-trac
Copy link
Author

Author: TomH
[Added to the original trac issue at 9.34pm, Wednesday, 24th August 2011]

This routine doesn't even exist any more...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant