search

fire-eggs / CivOne

An open source implementation of Sid Meier's Civilization 1 using C# and .NET.
Creative Commons Zero v1.0 Universal
21 stars 4 forks source link

Mines + Forest ??? #94

Open fire-eggs opened 5 years ago

fire-eggs commented 5 years ago

image

[Earth, 7 civs, chinese, rseed=7595]

fire-eggs commented 5 years ago

Need to examine the Settlers code - Build Mines - to see how this might happen.

axx0 commented 5 years ago

There is no option to build mines when on top of forest with settlers. Can it be an issue with loading the game? The resources (irrigation, mines, roads...) can sometimes shift some tiles when loading the game (at least I know this was an issue).

iegik commented 3 years ago

@axx0 But there is an option to build forest using M key. I'll fix this.

fire-eggs commented 3 years ago

@axx0 But there is an option to build forest using M key. I'll fix this.

Not sure what you're saying, that logic already exists.

If you look at Settlers.cs, NewTurn: when BuildingMine finishes, either a Mine is built or the terrain changes to forest, depending on what the "old" terrain was.

iegik commented 3 years ago

Oh, I'm sorry. My bug. (see https://github.com/fire-eggs/CivOne/pull/141/commits/3a02662b05d74426ba8e716608aa4ae55f7574b8)

iegik commented 2 years ago

Cannot reproduce.