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

Soul Linker Class returns null bl in skills #3145

Closed
bWolfie opened this issue Jun 15, 2022 · 4 comments
Closed

Soul Linker Class returns null bl in skills #3145

bWolfie opened this issue Jun 15, 2022 · 4 comments

Comments

@bWolfie
Copy link
Contributor

bWolfie commented Jun 15, 2022

Something I noticed when testing with CG_TAROTCARD.

When (eff == 2), which is High Priestess card (remove all buffs), was used on a Soul Linker, buffs would not be removed. I added some ShowDebug clauses to the top of status_change_clear_buffs and it wouldn't even register.

					case 2: // all buffs removed
						status->change_clear_buffs(bl,1);
						break;

Then when I changed it to:

					case 2: // all buffs removed
						if (sd != NULL)
							status->change_clear_buffs(&sd->bl,1);
						else
							status->change_clear_buffs(bl, 1);
						break;

It did work and removed all buffs. I don't know what is causing this interaction to happen like this.

@MrKeiKun
Copy link
Contributor

What buffs are you trying to clear? I tested with SC_INCAGI and SC_BLESSING buffed on the Soul Linker, works fine.

@bWolfie
Copy link
Contributor Author

bWolfie commented Jun 17, 2022

I used a Blessing Scroll.

@MrKeiKun
Copy link
Contributor

Okay so instead of using @useskill, I used Blessing_10_Scroll and this is Renewal btw.

On Buff:
image

On Tarot Cast:
image

@bWolfie
Copy link
Contributor Author

bWolfie commented Jun 21, 2022

Sorry must be custom code. Closing.

@bWolfie bWolfie closed this as completed Jun 21, 2022
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

No branches or pull requests

2 participants