Fix ownership changing assertion with a garrison holder with multiple…
rP25034
Actions

Description

Fix ownership changing assertion with a garrison holder with multiple garrisoned entities.

In lieu of a better solution.

Reverts: rP25031 / r25031

Details

Committed

Freagarach

Mar 11 2021, 7:14 AM

Parents

rP25033: Fix typos in log and warn messages

Branches

Unknown

Tags

Unknown

Build Status

Buildable 16106
Build 35978: Post-Commit Build	Jenkins
Build 35977: Post-Commit Build (macOS)	Jenkins

Event Timeline

Freagarach committed rP25034: Fix ownership changing assertion with a garrison holder with multiple….Mar 11 2021, 7:14 AM

Harbormaster completed building B16106: rP25034: Fix ownership changing assertion with a garrison holder with multiple….Mar 11 2021, 7:23 AM

Freagarach mentioned this in rP25031: Handle ownership changes of a garrisoned entity in cmpGarrisonable..Mar 14 2021, 8:50 AM

Freagarach mentioned this in D3658: Fix failed assertion when converting structure that has multiple garrisoned entities..

Changes (3)

Path

Size

ps/

trunk/

binaries/

data/

mods/

public/

simulation/

components/

GarrisonHolder.js

Garrisonable.js

tests/

test_GarrisonHolder.js

rP25034

View Options

ps/trunk/binaries/data/mods/public/simulation/components/GarrisonHolder.js

View Options

ps/trunk/binaries/data/mods/public/simulation/components/Garrisonable.js

View Options

ps/trunk/binaries/data/mods/public/simulation/components/tests/test_GarrisonHolder.js

Fix ownership changing assertion with a garrison holder with multiple…rP25034Actions

Description

Details

Event Timeline

Changes (3)

rP25034

ps/trunk/binaries/data/mods/public/simulation/components/GarrisonHolder.js

ps/trunk/binaries/data/mods/public/simulation/components/Garrisonable.js

ps/trunk/binaries/data/mods/public/simulation/components/tests/test_GarrisonHolder.js

Fix ownership changing assertion with a garrison holder with multiple…
rP25034
Actions