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.

Sign up for GitHub

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

Jump to bottom

invalid body part id "" #79226

Open
osuphobia opened this issue Jan 18, 2025 · 0 comments · May be fixed by #79248
Open

invalid body part id "" #79226

osuphobia opened this issue Jan 18, 2025 · 0 comments · May be fixed by #79248
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility

Comments

@osuphobia
Copy link
Contributor

osuphobia commented Jan 18, 2025 •
edited
Loading

Describe the bug

 DEBUG    : invalid body part id ""

 FUNCTION : convert
 FILE     : D:\a\Cataclysm-DDA\Cataclysm-DDA\src\generic_factory.h
 LINE     : 442
 VERSION  : 36ff082

Seems that after #79190 the line below is causing error on bodypart_id.

Cataclysm-DDA/src/mattack_actors.cpp

Line 955 in ac53e7f

const bodypart_id affected_bp = eff.affect_hit_bp ? bp : eff.bp.id();

It's happening when monster_attack applying effects to monsters.

Attach save file

McBride-trimmed.tar.gz

Steps to reproduce

Load the save, wait until error pops.

Expected behavior

No error msg

Screenshots

No response

Versions and configuration

  • OS: Windows
    • OS Version: 10.0.19045.5371 (22H2)
  • Game Version: 36ff082 [64-bit]
  • Graphics Version: Tiles
  • Game Language: English [en]
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Portal Storms Ignore NPCs [personal_portal_storms],
    Slowdown Fungal Growth [no_fungal_growth]
    ]

Additional context

debug.log
@osuphobia osuphobia added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Jan 18, 2025
@osuphobia osuphobia linked a pull request Jan 19, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

set default_value bodypart_str_id::NULL_ID() correctly osuphobia/Cataclysm-DDA
1 participant
@osuphobia