We could even remove the <territoryAttachment> node altogether, eg:
<territory name="Quebec">
<owner value="Canadians"/>
<unitProduction value="3" />
<kamikazeZone value="true" />
<placements>
:
:
Thoughts?
For canals, they attach to multiple territories, nesting underneath one territory would potentially not be as helpful as it could be. Here is an example of what we have today:
<attachment name="canalAttachmentCentral America - Colombia" attachTo="25 Sea Zone" javaClass="games.s
trategy.triplea.attachments.CanalAttachment" type="territory">
<option name="canalName" value="the Central America - Colombia channel"/>
<option name="landTerritories" value="Central America:Colombia"/>
</attachment>
<attachment name="canalAttachmentCentral America - Colombia" attachTo="100 Sea Zone" javaClass="games.strategy.triplea.attachments.CanalAttachment" type="territory">
<option name="canalName" value="the Central America - Colombia channel"/>
<option name="landTerritories" value="Central America:Colombia"/>
</attachment>
I think that might be better as one 'canal' tag. It's a really good call-out - though if it's going to be a different tag then perhaps topic for another thread (thread created: https://forums.triplea-game.org/topic/2269/proposal-simpler-canal-node)