Crazy Europe: House of Habsburg

CrazyG

@wc_sumpton
Can you explain why switch=true is a better choice for the always condition?

Thanks

Cernel

@crazyg In my opinion in this case better having a condition testing if there are any recruits on the map, not to fire a trigger for nothing. Anyways, it works either ways, and always firing it might be less demanding.

wc_sumpton

@crazyg
Just food for thought. The conditions will always equate to true. Using switch=true just reads clearer then rounds=1-+. Don't get me wrong both work and both ways can be used.

Cheers...

CrazyG

@cernel
I could change it at the end, but I really like having an always true condition while building triggers. Its useful for troubleshooting triggers and figuring out what part isn't working.

I can't say if its less demanding that checking the entire map for a unit.

CrazyG

Update:

Russia has a turn
Added WC's recommendations
Added the tech icon
Moved French units around a little bit to make its opening turns more interesting.
Updated game notes with more details, credit WC for his help

I'm testing triggers a bit more then will upload.

Hepps

@crazyg Need a new version! My Russia keeps getting its ass handed to it.

wc_sumpton

@hepps
That's all right, I changed their name to Rusties...:smiling_face_with_open_mouth_closed_eyes:

Cheers...

Hepps

@wc_sumpton I'm confused... are you just helping @CrazyG or are you taking the game and making an entirely different game?

wc_sumpton

@hepps
Just helping @CrazyG. The overlay images were just to see how that worked.

Its nice to see that I'm not the only one totally confused...

Cheers..

CrazyG

@wc_sumpton

Did you test your approach with navies? I'm getting unusual behavior with that (a navy that moves 1 space in combat move has 0 movement in NC phase, but a unit that doesn't move has 3)

Unit overlays won't be uploaded at the moment because I want the game to function on the stable engine.

redrum

@crazyg I believe the engine will just ignore the unit icons properties file and images for the current stable so shouldn't cause any harm and just benefit us cool kids using the pre-release

*** This comment is regarding including unit overlays/icons and nothing to do with the navy movement issue

wc_sumpton

@crazyg
I don't see how the unit_icons.properties file could be causing any logic change to the xml?

Will try to look more into it.

Cheers...

(What he said @redrum )

CrazyG

@wc_sumpton
not a unit properties thing, XML I have the movement swap working for everything except navies. When I tried your suggestion of movement rather than givesmovement, it seemed to stop the navies from working.

wc_sumpton

@crazyg
I have tested the movement change on all units. But I also tested the givesmovement also.
I have both working.
Using Navy movement:

<attachment name="triggerAttachmentFranceActivateNavyMoveBonus" attachTo="France" javaClass="games.strategy.triplea.attachments.TriggerAttachment" type="player">
  <option name="conditions" value="conditionAttachmentFranceDiscoveredSailing"/>
  <option name="when" value="before:FranceNonCombatMove"/>
  <option name="unitType" value="Navy"/>
  <option name="unitProperty" value="movement" count="3"/>
</attachment>

<attachment name="triggerAttachmentHabsburg_MonarchyActivateNavyMoveBonus" attachTo="Habsburg_Monarchy" javaClass="games.strategy.triplea.attachments.TriggerAttachment" type="player">
  <option name="conditions" value="conditionAttachmentHabsburg_MonarchyDiscoveredSailing"/>
  <option name="when" value="before:Habsburg_MonarchyNonCombatMove"/>
  <option name="unitType" value="Navy"/>
  <option name="unitProperty" value="movement" count="3"/>
</attachment>

<attachment name="triggerAttachmentOttoman_EmpireActivateNavyMoveBonus" attachTo="Ottoman_Empire" javaClass="games.strategy.triplea.attachments.TriggerAttachment" type="player">
  <option name="conditions" value="conditionAttachmentOttoman_EmpireDiscoveredSailing"/>
  <option name="when" value="before:Ottoman_EmpireNonCombatMove"/>
  <option name="unitType" value="Navy"/>
  <option name="unitProperty" value="movement" count="3"/>
</attachment>

<attachment name="triggerAttachmentItalian_StatesActivateNavyMoveBonus" attachTo="Italian_States" javaClass="games.strategy.triplea.attachments.TriggerAttachment" type="player">
  <option name="conditions" value="conditionAttachmentItalian_StatesDiscoveredSailing"/>
  <option name="when" value="before:Italian_StatesNonCombatMove"/>
  <option name="unitType" value="Navy"/>
  <option name="unitProperty" value="movement" count="3"/>
</attachment>

<attachment name="triggerAttachmentPoland-LithuaniaActivateNavyMoveBonus" attachTo="Poland-Lithuania" javaClass="games.strategy.triplea.attachments.TriggerAttachment" type="player">
  <option name="conditions" value="conditionAttachmentPoland-LithuaniaDiscoveredSailing"/>
  <option name="when" value="before:Poland-LithuaniaNonCombatMove"/>
  <option name="unitType" value="Navy"/>
  <option name="unitProperty" value="movement" count="3"/>
</attachment>

<attachment name="triggerAttachmentRussiaActivateNavyMoveBonus" attachTo="Russia" javaClass="games.strategy.triplea.attachments.TriggerAttachment" type="player">
  <option name="conditions" value="conditionAttachmentRussiaDiscoveredSailing"/>
  <option name="when" value="before:RussiaNonCombatMove"/>
  <option name="unitType" value="Navy"/>
  <option name="unitProperty" value="movement" count="3"/>
</attachment>

<attachment name="triggerAttachmentDenmarkActivateNavyMoveBonus" attachTo="Denmark" javaClass="games.strategy.triplea.attachments.TriggerAttachment" type="player">
  <option name="conditions" value="conditionAttachmentDenmarkDiscoveredSailing"/>
  <option name="when" value="before:DenmarkNonCombatMove"/>
  <option name="unitType" value="Navy"/>
  <option name="unitProperty" value="movement" count="3"/>
</attachment>

<attachment name="triggerAttachmentSwedenActivateNavyMoveBonus" attachTo="Sweden" javaClass="games.strategy.triplea.attachments.TriggerAttachment" type="player">
  <option name="conditions" value="conditionAttachmentSwedenDiscoveredSailing"/>
  <option name="when" value="before:SwedenNonCombatMove"/>
  <option name="unitType" value="Navy"/>
  <option name="unitProperty" value="movement" count="3"/>
</attachment>

I can post the givesmovement code also if you would like, they are very similar. The only change being the unitProperty=givesMovement=1:Navy.

Cheers...

CrazyG

@wc_sumpton
I changed it back to after:Battle and I works again. I'm scratching my head

Edit: I had NoncombatMove instead of NonCombatMove

Edit 2: No that wasn't it.
Going through Git now

CrazyG

Sorry for posting so many times.

I think I've figured this out. My boat has 2 movement. If I move my boat 1 space in combat move (but don't fight), it still loses all of its movement.

I was not aware of this behavior and kept thinking it was the triggers malfunctioning. I tested by moving a boat 0 zero spaces, it would have 3 movement in NC phase. I move it 2 spaces, it has 1 movement. Then I moved it 1 space, and it had 1 movement. I thought the trigger didn't fire.

redrum

@crazyg Correct. You can only move a non-air unit during combat move or non-combat move not both.

CrazyG

@redrum
You can move land in both

redrum

@crazyg Land and naval should work the same. I'm not sure if you trigger additional movement after combat move if they then could move again. But at least ignoring bonus movement/triggers/resets, land/sea units can only move during combat or non-combat move.

CrazyG

@redrum

Tested it on Revised. Tanks can move 1 space in both phases.

Edit: Duh blitz. I'm a fool. As long as its the engine and not my own error, I'm fine with it.