WeakAuras / WeakAuras2

World of Warcraft addon that provides a powerful framework to display customizable graphics on your screen.
https://weakauras.wtf
GNU General Public License v2.0
1.31k stars 318 forks source link

aura_env.saved is wiped on aura reimport #5520

Closed lootwant closed 2 weeks ago

lootwant commented 2 weeks ago

Is there an existing issue for this?

Description

On aura reimport it's aura_env.saved is apparently wiped. There is no mention of such behaviour in the docs, and it kinda defeats the purpose of persistent local storage.

If it's by design, it would be great to mention it in the docs. If a hard wipe is not actually required, would be great to make it optional and disabled by default. A checkbox on import along with the others, "Erase saved data"

WeakAuras Version

5.17.4

World of Warcraft Flavor

Retail (Default)

World of Warcraft Region

EU

Tested with only WeakAuras

I got this issue with only WeakAuras enabled

Lua Error

No response

Reproduction Steps

Reimport an aura that has aura_env.saved populated. Observe the nil after reimport instead of saved data.

Last Good Version

No response

Screenshots

No response

Export String

No response

Bisector Report

No response

InfusOnWoW commented 2 weeks ago

The intended behaviour is that saved is not overwritten on update, that was a thus a bug.

lootwant commented 1 week ago

Tested, confirming the fix. Ty ty!