View Issue Details
| ID | Project | Category | Date Submitted | Last Update | |
|---|---|---|---|---|---|
| 0021666 | AI War 2 | Bug - Gameplay | Sep 20, 2019 1:13 am | Oct 1, 2019 2:47 pm | |
| Reporter | ZeusAlmighty | Assigned To | BadgerBadger | ||
| Status | resolved | Resolution | fixed | ||
| Product Version | 0.890 Lighting and Darkness | ||||
| Fixed in Version | 0.894 Desaturation And Cross-Planet Move Orders | ||||
| Summary | 0021666: Bug Report: Technology Hack spends science to complete | ||||
| Description | Background/Problem: My flagship was unable to iniate a technology hack, given a notification essentially saying "not enough science." I was able to then do the hack when my science was sufficient, the hack proceded typically, but the hack then consumed my science as an additional cost. My tech increased, but it not required next tier spending so I could no longer to afford to mark up. Expectation--the hack costs no science and does not impact the cost to upgrade through traditional science spending Save is right before hack | ||||
| Tags | No tags attached. | ||||
|
|
|
|
|
Sigh. So the hack calls Hacker.PlanetFaction.Faction.UnlockTech(tech, false, Context); Which must decrease the science total inside it. Whoops. I need to either make a flag to pass in that says "Don't charge me science, actually" or query for the science cost before the UnlockTech call and then increase the cience afterwards. I'm not going to have any time till monday, so Chris if you're going to push between now and then, feel free to fix. Otherwise I'll get to it after the weekend. |
|
|
I can fix it, but I don't think I'll be back from work until 6 est, which means the patch will be pushed by then. |
|
|
I've made the changes to the unlock tech function by adding "bool ThisUnlockSpendsScience" and made sure to change the Game Command and tech sidebar too. That makes me think, should this hack be sending a game command or should it do things itself? |
|
|
I'm pretty sure the Hack code there is done from the main sim thread, so it should be fine. |
|
|
Weapon Master fixed this one |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| Sep 20, 2019 1:13 am | ZeusAlmighty | New Issue | |
| Sep 20, 2019 1:13 am | ZeusAlmighty | File Added: technology hack start.save | |
| Sep 20, 2019 1:13 am | ZeusAlmighty | File Added: AIWar2 9_19_2019 9_50_58 PM.png | |
| Sep 20, 2019 1:34 am | BadgerBadger | Note Added: 0053194 | |
| Sep 20, 2019 9:43 am | wm46 | Note Added: 0053198 | |
| Sep 20, 2019 9:09 pm | wm46 | Note Added: 0053208 | |
| Sep 20, 2019 11:55 pm | BadgerBadger | Note Added: 0053209 | |
| Oct 1, 2019 2:47 pm | BadgerBadger | Assigned To | => BadgerBadger |
| Oct 1, 2019 2:47 pm | BadgerBadger | Status | new => resolved |
| Oct 1, 2019 2:47 pm | BadgerBadger | Resolution | open => fixed |
| Oct 1, 2019 2:47 pm | BadgerBadger | Fixed in Version | => 0.894 Desaturation And Cross-Planet Move Orders |
| Oct 1, 2019 2:47 pm | BadgerBadger | Note Added: 0053369 |
