Freeciv

I guess I should comment something, though I don't have time to investigate in depth at the moment.

Some random considerations:
- That it's already possible by removing the wonder first means also that it's a workaround that people could use even if we didn't reopen semi-frozen S3_1 d3f
- What's the status of other methods related to this? Would it still be easy to grant the Wonder only if it's not already built?
- As create_building() is new in 3.1, and having the change in the features initial release, would save us from handling it as a change between two stable releases of the API
- Patch submitted in time for me to handle before my summer break (-> to be in testing over the summer) would increase the chances that this gets accepted to S3_1

Reply To cazfi

Some random considerations:
- That it's already possible by removing the wonder first means also that it's a workaround that people could use even if we didn't reopen semi-frozen S3_1 d3f

Well, behaviour with wonders was never documented behind "creates a building to a city".

- What's the status of other methods related to this? Would it still be easy to grant the Wonder only if it's not already built?

- Patch submitted in time for me to handle before my summer break (-> to be in testing over the summer) would increase the chances that this gets accepted to S3_1

Ok, I'll try to make the patch soon.

Centralized the code with editor. A bug or two got fixed in process. I'll see tomorrow how much more I have caused.

I have caused a lot... Well, reworking this. What really should be fixed in process:

• Lua does not refresh the cities before sending or rearrange the workers if the building changes the radius.
• Lua may put a coinage into the city that works buggy.
• Editor (client and server part) considers any Special genus building unplaceable. While the reciprocal is more or less guaranteed by ruleset loading, the straight seems to be not true. Making Special genus building obligatory either a mint or a space part is a format change (let alone the fact that wether something is a space part is calculated dependent to the current state of the city it's being built in). I will replace the check to reflect the current logic of city_build_building() but maybe we should change something, at least in 3.2, and this change should go also to 3.0.

What is not to be handled in this patch:

• Building obsoletion
• Trade route effects for third party cities (actually, I didn't check, maybe client calculates them?)
• Both editor and Lua don't call Lua "building_built/lost" callbacks, shouldn't they?