PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Undo deletes all currently equipped Jewels
Nobody has claimed this yet.
- Dominant language
- Lua
- Stars
- 5.4k
- Forks
- 2.4k
- Avg merge
- 1d 12h
- Merged PRs (30d)
- 26
Description
Check version
- I'm running the latest version of Path of Building and I've verified this by checking the changelog
Check for duplicates
- I've checked for duplicate issues by using the search function of the issue tracker
What is the expected behaviour/value?
Using the undo feature (CTRL+Z) should not wipe the equipped jewel list
What is the actual behaviour/value?
When there is only one change in the undo stack and CTRL+Z is used, all Jewels are deleted from the current build.
It seems to affect all Jewel types (Cluster, Abyssal, Crimson/Veridian etc.)
How to reproduce the issue
- Load up a saved build with at least 1 socketed Jewel
- Make a single change. e.g. set any single equipment slot to
Noneor equip an item. - Undo (CTRL+Z) the change
- All Jewel sockets become unequipped
Build code
https://pastebin.com/n964bZ4w
Screenshots
No response
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Load the supplied build code with at least one socketed Jewel, make one equipment change, and reproduce the issue with CTRL+Z. Trace the undo path and the equipped-jewel state, checking that undo restores the prior equipment without unequipping Cluster, Abyssal, or Crimson/Veridian Jewels. Done means the listed reproduction steps no longer delete any equipped Jewels.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- lua
- Domain
- desktop
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 42/100