Opened 7 years ago

Last modified 5 years ago

#4770 closed defect

Organize treasure templates — at Version 7

Reported by: fatherbushido Owned by:
Priority: Should Have Milestone: Alpha 23
Component: UI & Simulation Keywords:
Cc: Patch:

Description (last modified by elexis)

Treasure templates inherit from template_gaia_treasure. But it seems that some are in other/ folder and some are in /gaia folder. Task:

  • check if that is intended (commit history will help)
  • move the according template in the adhoc folder
  • change the refs in the maps files
template_gaia_treasure.xml

gaia/special_ruins.xml
gaia/special_ruins_column_doric.xml
gaia/special_ruins_standing_stone.xml
gaia/special_ruins_stone_statues_egyptian.xml
gaia/special_ruins_stone_statues_roman.xml

gaia/special_treasure_food_barrel.xml
gaia/special_treasure_food_barrels_buried.xml
gaia/special_treasure_food_bin.xml
gaia/special_treasure_food_crate.xml
gaia/special_treasure_food_jars.xml
gaia/special_treasure_food_persian_big.xml
gaia/special_treasure_food_persian_small.xml
gaia/special_treasure_golden_fleece.xml
gaia/special_treasure_metal.xml
gaia/special_treasure_metal_persian_bigl.xml
gaia/special_treasure_metal_persian_small.xml
gaia/special_treasure_pegasus.xml
gaia/special_treasure_standing_stone.xml
gaia/special_treasure_stone.xml
gaia/special_treasure_wood.xml

other/special_treasure_shipwreck.xml
other/special_treasure_shipwreck_debris.xml
other/special_treasure_shipwreck_ram_bow.xml
other/special_treasure_shipwreck_sail_boat.xml
other/special_treasure_shipwreck_sail_boat_cut.xml

Change History (7)

comment:1 by fatherbushido, 7 years ago

Summary: Organize treasure templateOrganize treasure templates

comment:2 by elexis, 7 years ago

Should do a text search for other/, I suspect some things in the structree, rmgen and selection panels (and perhaps AI code) might need to be renamed (nothing serious though).

comment:3 by fatherbushido, 7 years ago

In the same spirit, gaia ruins templates are in other/ and in gaia/

(That other/ folder...)

comment:4 by elexis, 7 years ago

ruins and treasure in gaia/: Ruins work the same way as resources. On the other hand they are man-made, so not what we see as "gaia". So perhaps treasure and ruins should just be moved to other/. Then gaia looks clean enough IMO and doesn't for instance need new subdirectories.

other/ and special/: other/territory_pull.xml is weird. It's the only entity that isn't visible ingame, so maybe that should be moved to some "special" directory that isn't special/ according to the unplaceable definition. (Rename special/ to unplaceable/?) The palisade templates should be moved to templates/ and/or template/structures/ as proposed by some other ticket. Then other/ will be clean enough too I guess. (subdirectories like units and structures possibly, dunno)

comment:5 by fatherbushido, 7 years ago

(I would bend for ruins and treasure in gaia.)

comment:6 by fatherbushido, 7 years ago

Well looking a bit at history, I won't sum up that here.

r7259 is the sim2 initial commit with only 1 thing in other/

r7889 r7980 r8110 for example started to populate other/ with mostly gaia or decorative things

r8150 similar things are commited in gaia/

r10650 introduced proper treasure and ruins generic templates

r11066 some things in other/ change their parents

r8571 some things are added in other/ and some are changed in gaia/

I stopped that here. I am for moving ruins and treasures in other/ which inherits from that template_gaia_foo things to gaia/ (also I wonder if _special_ in the name of those templates is really relevant)

Last edited 7 years ago by fatherbushido (previous) (diff)

comment:7 by elexis, 7 years ago

Description: modified (diff)

(edit description: alphabetic sorting + ruins)

Note: See TracTickets for help on using tickets.