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

[WIP] Bugfix unload items #40000 #40004

Closed
wants to merge 1 commit into from

Conversation

Pupsi-Mupsi
Copy link
Contributor

@Pupsi-Mupsi Pupsi-Mupsi commented Apr 28, 2020

Summary

SUMMARY: None

Purpose of change

Unloading things does not pick them up anymore.
This fixes #40000.

Testing

Unloaded an mp3 player and a bunch of other things around the character and in characters inventory.


Okay. Did some more testing and came across this.
I'll take a closer look tomorrow after work.
Any [CR] is highly appreciated!

grafik

@ifreund ifreund added <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` labels Apr 28, 2020
@Pupsi-Mupsi Pupsi-Mupsi changed the title Bugfix unload items #40000 [WIP] Bugfix unload items #40000 Apr 28, 2020
@KorGgenT
Copy link
Member

KorGgenT commented Apr 29, 2020

yes, that's the message i fixed by introducing this behavior in #37816 for reference

@enaantd
Copy link
Contributor

enaantd commented Apr 29, 2020

Okay. Did some more testing and came across this.

I just got this error in my own game, so I think it is unrelated to your modifications.

@akirashirosawa
Copy link
Contributor

I just got this error in my own game, so I think it is unrelated to your modifications.

Same. See issue #39402

@Pupsi-Mupsi
Copy link
Contributor Author

Unfortunately, this only partially resolves the problem and I do not intend to aggravate the existing problem with nested containers. So ... sorry, but I prefer to close this PR.

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`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unloading items should not pick them up
5 participants