Skip to content

Fix MemberNickname class#291

Closed
EquipableMC wants to merge 5 commits intomasterfrom
equip-dev
Closed

Fix MemberNickname class#291
EquipableMC wants to merge 5 commits intomasterfrom
equip-dev

Conversation

@EquipableMC
Copy link
Collaborator

This PR fixes an issue where you are unable to reset (or clear) a user's nickname, causing you to use {_member}.modifyNickname(null).queue() to clear it.

final String name = (String) delta[0];
if (EasyElement.anyNull(this, member, name))
return;
if ((mode != Changer.ChangeMode.RESET && mode != Changer.ChangeMode.DELETE) && EasyElement.anyNull(this, name)) return;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know I initially put it on one line but the line at this point is too long to continue doing so

Suggested change
if ((mode != Changer.ChangeMode.RESET && mode != Changer.ChangeMode.DELETE) && EasyElement.anyNull(this, name)) return;
if ((mode != Changer.ChangeMode.RESET && mode != Changer.ChangeMode.DELETE) && EasyElement.anyNull(this, name))
return;

@EquipableMC
Copy link
Collaborator Author

messed up my branch, going to re-create this PR and branch

@EquipableMC EquipableMC deleted the equip-dev branch November 26, 2025 14:32
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.

2 participants