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

Add ItemSpecialEffect::Indestructible #5772

Draft
wants to merge 19 commits into
base: development
Choose a base branch
from

Conversation

kphoenix137
Copy link
Collaborator

@kphoenix137 kphoenix137 commented Feb 8, 2023

Adds a new flag for Indestructible, so that 255 durability doesn't need to be used as indestructible. Indestructible items are given both the new flag, and also have their durability set to 255 for reverse compatibility. Also added code in CalcPlrItemVals() to force set the flag on any existing item if it has IPL_INDESTRUCTIBLE but doesn't already have the flag (for reverse compatibility).

Source/objects.cpp Outdated Show resolved Hide resolved
Source/items.cpp Outdated Show resolved Hide resolved
Source/items.cpp Outdated Show resolved Hide resolved
Source/pack.cpp Outdated Show resolved Hide resolved
StephenCWills
StephenCWills previously approved these changes Jul 27, 2023
@AJenbo AJenbo changed the base branch from master to development August 27, 2023 16:22
@AJenbo AJenbo dismissed StephenCWills’s stale review August 27, 2023 16:22

The base branch was changed.

@AJenbo
Copy link
Member

AJenbo commented Aug 27, 2023

Won't this break Hellfire items that have been oiled to be indestructable?

@AJenbo AJenbo added this to the 1.6.0 milestone Aug 27, 2023
@kphoenix137
Copy link
Collaborator Author

Won't this break Hellfire items that have been oiled to be indestructable?

It absolutely will. I forgot about Hellfire Oils entirely. Oil of Permanence in specific is problematic.

@StephenCWills
Copy link
Member

Won't this break Hellfire items that have been oiled to be indestructable?

The only thing I can think to do about that is to add a flag to icreateinfo_flag2 so we can differentiate between items before and after this PR. Otherwise, we can't tell the difference between oiled vanilla SP Hellfire items and DevX SP Hellfire items with 255 durability.

Well, either that, or we can simply ignore the problem. Continue to allow SP Hellfire players to abuse Hidden shrines since they have access to Oil of Permanence anyway.

@kphoenix137 kphoenix137 marked this pull request as draft September 17, 2024 22:37
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.

4 participants