Currently fish is a child of template_unit_fauna.xml; however, fish does not move, has no armour, attack, or health, and disables a few other things as well. In many ways fish is much more similar to gaia objects such as metal mines or fruit trees.
This proposal moves fish to template_gaia_fish.xml and adjust all fish templates accordingly.
Details
Check if nothing is overlooked.
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- /ps/trunk
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 6892 Build 11303: Vulcan Build Jenkins Build 11302: arc lint + arc unit
Event Timeline
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/differential/1041/
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/differential/1042/
Idea seems good.
Opening a map with fish on it throws a bunch of errors (invalid template)
binaries/data/mods/public/simulation/templates/gaia/fauna_fish_tilapia.xml | ||
---|---|---|
4–6 | This indeed seems better, the identity description is false either way | |
binaries/data/mods/public/simulation/templates/template_gaia_fish.xml | ||
9 | SeaCreature Animal Organic -ConquestCritical | |
20 | k docks |
binaries/data/mods/public/simulation/templates/template_gaia_fish.xml | ||
---|---|---|
9 | Its new parent (gaia) does not have ConquestCritical, therefore it doesn't need to be removed. |
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/differential/1093/display/redirect
Game works, Atlas works changes make sense => accept
binaries/data/mods/public/simulation/templates/template_gaia_fish.xml | ||
---|---|---|
9 | The conclusion seems right, trees don't have organic either, so that excludes that one. Animal seems counterpart of Human, so leaving them out looks ok on second sight. (You were already right about conqestcritical) | |
14 | kindof sad that this duplicates these values, nothing to do about that I suppose (anytime someone could create a gaia food harvest place in need of these values) |