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

Examining UPS modded tool causes error #31633

Closed
ghost opened this issue Jun 19, 2019 · 5 comments · Fixed by #31800
Closed

Examining UPS modded tool causes error #31633

ghost opened this issue Jun 19, 2019 · 5 comments · Fixed by #31800
Labels
<Bug> This needs to be fixed Items: Gunmod / Toolmod Weapon and tool attachments, and add-ons Items / Item Actions / Item Qualities Items and how they work and interact

Comments

@ghost
Copy link

ghost commented Jun 19, 2019

Describe the bug

It has been quite a long time since UPS toolmod was working correctly, but recent ammo changes furthermore made any UPS modded tool cause an error on examining in inventory and same error, but cyclic, softlocking your game, when inspected lying on the ground.

Steps To Reproduce

Steps to reproduce the behavior:

  1. Choose any tool using plain batteries.
  2. Apply UPS conversion mod.
  3. Examine modded tool in inventory or drop it and try to pick it up.

Expected behavior

No error occurs.
Probably a minor problem though, because UPS toolmod doesn't work as intended anyway #31377.

Versions and configuration

  • OS: Reproduced on Windows 7, Windows 10, Android
  • Game Version: 0.D-4674-g2e5c67a
  • Graphics version: Tiles
  • Mods loaded: [dda], [alt_map_key], [no_npc_food]

Additional context

-MinGW Runtime Assertion-

Assertion failed!

Program: D:\CDDA\cdda\cataclysm-tiles.exe
File: src/optional.h, Line 45

Expression: full

@TechyBen
Copy link
Contributor

I also got this. Picking it up sometimes caused the error I think.

@Night-Pryanik Night-Pryanik added <Bug> This needs to be fixed Items: Gunmod / Toolmod Weapon and tool attachments, and add-ons Items / Item Actions / Item Qualities Items and how they work and interact labels Jun 20, 2019
@Inglonias
Copy link
Contributor

The crash notwithstanding, my research shows that some UPS modded tools do work as expected if you're carrying the UPS with you, which is the intended behavior going forwards.

@ghost
Copy link
Author

ghost commented Jun 20, 2019

Well if I happen to not crash, when ignoring the error, I found out that some of the ups'd tools actually work without using any power whatsoever after verifying your claim. So that's another bug I'd have to file separately, which seems like a wasted effort at this point.

UPS was in a sorry state even before the ammo wells, now it's just pitiful and it probably won't get any love until this battery revolution gets resolved first. I'm not salty about it, but I think all I can do now is just leave my UPS tools somewhere in a locker to avoid any crashes and just use up my stash of disposable batteries I left in last town.

@eilaattwood
Copy link
Contributor

I'm not salty about it, but I think all I can do now is just leave my UPS tools somewhere in a locker

It's time to use sealed crate for the first time. I guess ;)

@eilaattwood
Copy link
Contributor

Well, Looks like it`s somehow related to wrong ammo type, because I have the same crash when inspecting a gun (S&W 610) that is using realguns ammo that is not currently defined correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Items: Gunmod / Toolmod Weapon and tool attachments, and add-ons Items / Item Actions / Item Qualities Items and how they work and interact
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants