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

Bugfix: Hellfire "Life to mana" and "Mana to life" #5771

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

kphoenix137
Copy link
Collaborator

@kphoenix137 kphoenix137 commented Feb 7, 2023

Fixes: #6875

Hellfire writes the Life and Mana directly to the item. This is a problem since it does not dynamically change the values, but keeps the values from when the item is originally generated. This becomes less of a problem the more frequently the item with one of these powers is regenerated, however it is a terrible way of doing it.

This PR rather sets the Life and Mana values dynamically so they stay accurate to the item description.

@AJenbo AJenbo added this to the 1.5.0 milestone Feb 19, 2023
Source/items.cpp Outdated Show resolved Hide resolved
Source/items.cpp Outdated Show resolved Hide resolved
@AJenbo AJenbo modified the milestones: 1.5.0, 1.6.0 May 31, 2023
@kphoenix137 kphoenix137 requested a review from AJenbo March 7, 2024 23:51
@kphoenix137 kphoenix137 changed the title Fix IPL_MANATOLIFE and IPL_LIFETOMANA Bugfix: Hellfire "Life to mana" and "Mana to life" Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Issue Report]: Overflow with Acolyte Amulet?
3 participants