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.

@kphoenix137 kphoenix137 force-pushed the bugfix-lifetomana-manatolife branch from 5ea5118 to 6a5d6f9 Compare February 10, 2023 00:59
@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 force-pushed the bugfix-lifetomana-manatolife branch from 6a5d6f9 to d0f750a Compare March 7, 2024 23:47
@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
@kphoenix137 kphoenix137 force-pushed the bugfix-lifetomana-manatolife branch from 59db9df to d78f723 Compare March 24, 2024 07:56
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