Opened 12 years ago

Closed 9 years ago

#773 closed defect (invalid)

delete_with_relations_and_history should actually delete with relations

Reported by: Richard Owned by: frederik@…
Priority: major Milestone:
Component: api Version:
Keywords: Cc: steve@…


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.

Change History (2)

comment:1 Changed 12 years ago by steve@…

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

comment:2 Changed 9 years ago by Tom Hughes

Resolution: invalid
Status: newclosed

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

Note: See TracTickets for help on using tickets.