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

[Bug Fix] Preserve Permanent pet buffs across zone transitions #2035

Merged
merged 1 commit into from
Mar 7, 2022

Conversation

catapultam-habeo
Copy link
Contributor

No description provided.

…buffs to maintain them through zone transitions
@mackal
Copy link
Member

mackal commented Mar 6, 2022

Can you provide a list of spells that this is needed for?

@catapultam-habeo
Copy link
Contributor Author

Can you provide a list of spells that this is needed for?

There aren't any in PEQ or live-like content. Honestly I don't know of anything in PEQ or live-like content which uses buffdurationformula 50, pet or no.

That said, -1 is a valid buff duration, while 0 is not. The methods that account for saving and loading pet buffs cannot account for -1 as a valid buff duration without this change.

@Akkadius Akkadius requested a review from KayenEQ March 7, 2022 00:39
@Akkadius
Copy link
Member

Akkadius commented Mar 7, 2022

This is less of a bug and more of an optional feature if server admins want it

@Akkadius Akkadius merged commit 751db6f into EQEmu:master Mar 7, 2022
@catapultam-habeo catapultam-habeo deleted the permpetbuffs branch March 7, 2022 05:23
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.

3 participants