aliens

Author Topic: [DONE] Storestate grand totals.  (Read 1187 times)

Offline R1dO

  • Colonel
  • ****
  • Posts: 436
    • View Profile
[DONE] Storestate grand totals.
« on: August 07, 2022, 10:19:27 pm »
As a player i would probably never noticed it, but found this one while browsing through the code.

In StoreState::InitList() when tallying items for the grand total this method goes to great length in order to include every item that can theoretically return to a base.

It feels like it is missing two options though
1) A scenario 6. "Items in manufacture if they are refundable"
    To have the same behavior as scenario 4, which includes items that are destroyed after research.
2) Armors worn by soldiers while in transfer to a base.

Was this an oversight or design choice?
« Last Edit: August 08, 2022, 05:06:23 pm by Meridian »

Online Meridian

  • Global Moderator
  • Commander
  • *****
  • Posts: 8596
    • View Profile
Re: Storestate grand totals.
« Reply #1 on: August 07, 2022, 10:31:07 pm »
1. sounds like a bug... I don't know/remember why I would include items destroyed by research
2. sounds like my laziness

I'll check both...

Offline R1dO

  • Colonel
  • ****
  • Posts: 436
    • View Profile
Re: Storestate grand totals.
« Reply #2 on: August 07, 2022, 10:43:33 pm »
If i understand the code correct it is because items destroyed by research are refundable when a player cancels the appropriate research.
That would mean such an item could theoretically return to base stores.

Then again .. i am not your memory, so it is all assumptions on my part.  ;)

Online Meridian

  • Global Moderator
  • Commander
  • *****
  • Posts: 8596
    • View Profile
Re: [DONE] Storestate grand totals.
« Reply #3 on: August 08, 2022, 05:12:09 pm »
2. was fixed, armor worn by soldiers in transfer is now included

1. I'd prefer removing research items over adding manufacture items... but since the research items are there for a long time and it is also described here on the forum, I'll just leave it be as it is

I've probably (not 100% sure) added the research items into the total because they are visible in the Alien Containment Management GUI and block space there; but manufacture items are not visible anywhere and don't block any storage space (they can block manufacturing space). Also, refund option didn't exist back then :)

Offline R1dO

  • Colonel
  • ****
  • Posts: 436
    • View Profile
Re: [DONE] Storestate grand totals.
« Reply #4 on: August 08, 2022, 06:25:13 pm »
That was swift .. thank you (also for taking the time to explain rationale).