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.

Sign up for GitHub

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

Jump to bottom

Consumables with Destroy on Empty chat bug. #363

Open
rpganalyst opened this issue Jan 28, 2022 · 1 comment
Open

Consumables with Destroy on Empty chat bug. #363

rpganalyst opened this issue Jan 28, 2022 · 1 comment

Comments

@rpganalyst
Copy link

When you use the last Consumable with Destroy on Empty flagged, it does not show in chat.
E.g. I have 2 Antitoxins, I use the first and a message appears in chat.
Then I use the last Antitoxins and nothing appears in chat.
@cs96and
Copy link
Contributor

cs96and commented Feb 1, 2022

#365

CarlosFdez pushed a commit that referenced this issue Feb 3, 2022
@cs96and
Various v9 bug fixes (#365)
256c5e0 
* Fixed deletion of empty consumables (Fixes #358, #363)

* Fixed settings of flags in ensureFlags() (Fixes #354)

* Revert ensureFlags() to previous version.  Call ensureFlags() in addButtonsToItemLi() instead (Fixes #354)

Calling item.data.update() causes item.data.data.prof to become undefined for some reason, which means that tool rolls lose their proficiency bonus.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cs96and @rpganalyst