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

CNWSItem::SaveContainerItems no longer functions as during Bioware 1.69 #199

Closed
Stratovarius667 opened this issue Oct 18, 2020 · 28 comments
Closed
Labels
game Issues with the game code.

Comments

@Stratovarius667
Copy link

To Reproduce

Store item inside container possessed by a PC.
SetLocal on that object
Export PC
Load into another module

In 1.69, the Var is readable.
In EE, the Var has been deleted.

Specifics

  • Win10
  • Latest
  • Steam
  • PRC
  • Middle of last year (2019) was roughly when the function was broken

If needed, describe the bug

CNWSItem::SaveContainerItems previously just passed false to SaveItem() and retained local ints stored on items in containers. The Player Resource Consortium is built on this functionality, but at some point in the previous year, it was changed to wipe locals. This effectively breaks the entirety of the PRC. We also were not the originators of this code for use during development, which likely means unmaintained content is also damaged.

@Stratovarius667 Stratovarius667 added the game Issues with the game code. label Oct 18, 2020
@aldreth-hub
Copy link

Still busted as of 2021.

@AcaciaZae
Copy link

Same problem occurs with the Linux (Ubuntu) version of EE on Steam. It is presumably global for EE regardless of OS.

@altpersona
Copy link

Same problem linux (ubu) and win10 ee beam/steam/gog

@LordZin
Copy link

LordZin commented May 15, 2021

Please fix this Beamdog.

I second this.

@arnax
Copy link

arnax commented May 15, 2021

Please fix this Beamdog.

Please do fix it.

@Draygoth
Copy link

Please fix this.

@mariepascalehebert
Copy link

Please fix this

@Gustavoisaf
Copy link

Please fix this.

@acidchalk
Copy link

We need this fixed please!

@dizzitt
Copy link

dizzitt commented May 24, 2021

Could we get this fixed please.

@irisvin
Copy link

irisvin commented Jun 20, 2021

Still not fixed as of June 20, 2021. It's been 8 months. Pretty please fix it?

@Col-pas-col
Copy link

Beamdog, please fix this.

@pstemarie
Copy link

Fix this please.

@barml0t
Copy link

barml0t commented Oct 29, 2021

Here for the one year anniversary of this report. Please fix this

@pstemarie
Copy link

Considering .33 was "the last update for awhile," I think this ship has sailed.

@Fullmight
Copy link

This seriously needs a fix, it's frankly kind of ridiculous to introduce and then leave this kind of issue in production.

@Ramza1987
Copy link

How is this still not fixed?

@Thexare
Copy link

Thexare commented Oct 29, 2021

It's been a year now. This is, frankly, ridiculous.

It's also something I'll be remembering for any future Beamdog releases.

@phsmend
Copy link

phsmend commented Oct 29, 2021

Fix this please

@Heimdell
Copy link

Please, fix!

@Hackenhead
Copy link

Please fix, this needs to happen.

@dizzitt
Copy link

dizzitt commented Oct 30, 2021

Could we please get this fixed

@Red-Kai
Copy link

Red-Kai commented Oct 31, 2021

Please fix

@Gralzeim
Copy link

Gralzeim commented Feb 5, 2022

This is still a serious issue, Beamdog.

@Prince-Raymond
Copy link

Prince-Raymond commented Feb 5, 2022

In agreement with everyone here. Please fix it. So, this bug has been around a whole year, huh? Well, #155 has been around since August 30, 2006. That was when BioWare released Patch 1.68 for Neverwinter Nights. If you have not done so already, I cordially invite you all to take a look at my report. Please let me, and especially let the devs, know what you think of that issue. Have a blessed day, be safe, and stay healthy.

Thank you for reading. Happy, healthy (both mentally and physically) gaming to all.

@ebonfowl
Copy link

Please fix.

@tinygiant98
Copy link
Collaborator

Exporting object state with character export was an exploit, not a feature, and will not be reinstated. Organic sqlite implementation supplants this requirement.

@tinygiant98 tinygiant98 closed this as not planned Won't fix, can't repro, duplicate, stale Feb 24, 2023
@Thexare
Copy link

Thexare commented Feb 25, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
game Issues with the game code.
Projects
None yet
Development

No branches or pull requests