View Issue Details
ID | Project | Category | Date Submitted | Last Update | |
---|---|---|---|---|---|
0020272 | AI War 2 | Bug - Gameplay | Sep 12, 2018 8:44 am | Sep 14, 2018 10:28 am | |
Reporter | HeartHunter7 | Assigned To | BadgerBadger | ||
Status | resolved | Resolution | fixed | ||
Product Version | 0.768 Golem Rebirth | ||||
Fixed in Version | 0.769-0.770 Thinking Long-Term | ||||
Summary | 0020272: AI captures himself planets with reconquest | ||||
Description | One day AI lost the planet, then he tryed return this planet to back. But he tryed it about 5 times, how you can see on screenshot, Ai has 5 command stations, and i need kill 5 command station for get this planet. Plus AI has this planet, but she is still neitral, lol, and he gets epic big army from sentinels, becouse for every command station, he makes new sentinel wormhole. | ||||
Tags | No tags attached. | ||||
|
|
|
Prima facie this is quite odd because the reconquest code specifically checks that the controlling faction is "NaturalObject" before claiming the planet. It checks this via if ( this.Planet.GetIsControlledByFactionType( FactionType.NaturalObject ) ) { //do reconquest stuff However, it occurs to me that if I have multiple Usurpers on a planet then they will all execute DoEntitySecondLogic() back to back, and if I recall correctly the Planet object doesn't realize there's a new owner until someone calls CheckForOwnershipChange(), which is only called from the World object. I suspect if we call CheckForOwnershipChange right after we claim the planet for the AI, the problem will go away. |
|
Should be fixed now |
Date Modified | Username | Field | Change |
---|---|---|---|
Sep 12, 2018 8:44 am | HeartHunter7 | New Issue | |
Sep 12, 2018 8:44 am | HeartHunter7 | File Added: bug3.save | |
Sep 12, 2018 8:44 am | HeartHunter7 | File Added: bug3.savemet | |
Sep 12, 2018 8:44 am | HeartHunter7 | File Added: 20180912154116_1.jpg | |
Sep 12, 2018 8:45 am | HeartHunter7 | Summary | AI capture himself planets with reconquest => AI captures himself planets with reconquest |
Sep 14, 2018 12:29 am | BadgerBadger | Note Added: 0049226 | |
Sep 14, 2018 12:30 am | BadgerBadger | Note Edited: 0049226 | |
Sep 14, 2018 10:28 am | BadgerBadger | Assigned To | => BadgerBadger |
Sep 14, 2018 10:28 am | BadgerBadger | Status | new => resolved |
Sep 14, 2018 10:28 am | BadgerBadger | Resolution | open => fixed |
Sep 14, 2018 10:28 am | BadgerBadger | Fixed in Version | => 0.769-0.770 Thinking Long-Term |
Sep 14, 2018 10:28 am | BadgerBadger | Note Added: 0049234 |