Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix basecamp storage zone inventory #35509

Merged
merged 1 commit into from Nov 16, 2019
Merged

Fix basecamp storage zone inventory #35509

merged 1 commit into from Nov 16, 2019

Conversation

ghost
Copy link

@ghost ghost commented Nov 13, 2019

Summary

SUMMARY: Bugfixes "Fix basecamp storage zone inventory"

Purpose of change

The new basecamp storage zone did not pick up vehicle tools or sources of water/fire etc.

Describe the solution

Used the existing inventory::form_from_map() but made a new overload to pass a set from a zone output to, and moved the vector forming to a new function to pass to the main body.

Describe alternatives you've considered

N/A

Testing

Beforehand - welding rig in storage zone didnt show up.
afterwards, loaded a safve, made a faction camp, made a storage zone, put welding rig in the zone, it is now picked up by the upgrade recipe.

Additional context

N/A

@ZhilkinSerg ZhilkinSerg added [C++] Changes (can be) made in C++. Previously named `Code` <Bugfix> This is a fix for a bug (or closes open issue) Inventory / AIM / Zones Inventory, Advanced Inventory Management or Zones labels Nov 13, 2019
@ZhilkinSerg ZhilkinSerg merged commit 3766dc6 into CleverRaven:master Nov 16, 2019
@ghost ghost deleted the camp_form_fix branch December 19, 2019 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Inventory / AIM / Zones Inventory, Advanced Inventory Management or Zones
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant