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

Error moving item due to invalid container #59185

Open
Merikona opened this issue Jul 11, 2022 · 4 comments
Open

Error moving item due to invalid container #59185

Merikona opened this issue Jul 11, 2022 · 4 comments
Labels
(S2 - Confirmed) Bug that's been confirmed to exist

Comments

@Merikona
Copy link
Contributor

Describe the bug

DEBUG : tried to put an item (steel_lump, amount 1) in a container (soft_3a_vest) that cannot contain it

FUNCTION : int item::fill_with(const item&, int, bool, bool, bool)
FILE : src/item.cpp
LINE : 11888
VERSION : fa6f25b

Error code above.

Steps to reproduce

Tried to move 40 steel lumps to inventory (I believe with the 'g' key) and got this error. See screenshots below for my inventory after moving the steel lumps and the error message itself. I presume one of the containers is at fault here, possibly the security ballistic vest, which I set that items won't be inserted unless I manually do so.

Expected behavior

There were no averse effects that I could see, but the error shouldn't happen.

Screenshots

image
image

Versions and configuration

  • OS: Windows
    • OS Version: 10.0.19044.1766 (21H2)
  • Game Version: fa6f25b [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    No Fungal Growth [no_fungal_growth],
    Bionic Professions [package_bionic_professions]
    ]

Additional context

No response

@Merikona Merikona added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Jul 11, 2022
@hexagonrecursion
Copy link
Contributor

@Merikona, please upload your save file to help us reproduce the issue.

@Merikona
Copy link
Contributor Author

Merikona commented Jul 12, 2022

@hexagonrecursion I've made a new issue and attached a zip file of my save there. See #59189.

@hexagonrecursion
Copy link
Contributor

/confirmed

Thank you for uploading your save file. For some reason the security ballistic vest in your save file is confused about whether it has pockets or not. Note the lack of the "Total capacity" line.

image

I was unable to replicate the debug error message or the lack of total capacity in a fresh save:
image

@github-actions github-actions bot added (S2 - Confirmed) Bug that's been confirmed to exist and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Jul 13, 2022
@Merikona
Copy link
Contributor Author

Huh, well that's strange. Thanks for confirming.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

No branches or pull requests

2 participants