disabled its vacant option for vacant places

[paulklie]

fixes #6461

[westnordost]

I'd slightly prefer to have a function createVacantAnswer which does the if check, just for consistency-sake. (Or alternatively, use the takeIf also for the other one)

[westnordost]

Also, a place can be shop=vacant but also e.g. disused:shop=* (, disused:amenity=*) etc.
How is this handled? Should it be handled?

[paulklie]

I'd slightly prefer to have a function createVacantAnswer which does the if check, just for consistency-sake. (Or alternatively, use the takeIf also for the other one)

good point, done

[paulklie]

Also, a place can be shop=vacant but also e.g. disused:shop=* (, disused:amenity=*) etc. How is this handled? Should it be handled?

The button does not appear in these cases eather, since the place is still correctly identified as vacant

[westnordost]

Thank you!

[peternewman]

That's a bit of a shame as I often use this to tidy up an erroneously left name (e.g. a bank that's not just vacant still having it's brand name showing).