use 'allowCities' from terrain data

C7-Game / Prototype

An early-stage, open-source 4X strategy game

https://c7-game.github.io/

MIT License

34 stars 9 forks source link

use 'allowCities' from terrain data #300

Closed Kright closed 2 years ago

Kright commented 2 years ago

I add "allowCity" to C7 TerrainType. This lead to some fixes:

update terrain types in .json map
update settlerAI
add methods to Tile and Unit
add check to "buildCity" action

Closes: #120

QuintillusCFC commented 2 years ago

Ran the code, confirmed it's all working right other than that players can spawn on mountains/volcanoes.

(As a side note, the generateDummyGameMap method that was updated isn't currently used; it was used for our early random map generation before we realized it was a lot quicker to get maps working via importing them. It might be used again someday, so it hasn't been deleted, just marked obsolete)

Kright commented 2 years ago

There was conflict, so I rebased on current Development branch.

QuintillusCFC commented 2 years ago

Looks good! Approved and merging!