Skip to content

Fix banner patterns not working in kits#5723

Merged
JRoy merged 2 commits intoEssentialsX:2.xfrom
cati-xf:2.x
Mar 4, 2024
Merged

Fix banner patterns not working in kits#5723
JRoy merged 2 commits intoEssentialsX:2.xfrom
cati-xf:2.x

Conversation

@cati-xf
Copy link
Contributor

@cati-xf cati-xf commented Mar 4, 2024

Information

This PR fixes #4737.

Details

Proposed fix:

Function PatternType.valueOf was not the appropriate function for finding the PatternType by the identifier. I replaced with the the appropriate function "getByIdentifier".

Environments tested:

OS: Windows 11

Java version: 1.8.0_51

  • Most recent Paper version (1.XX.Y, git-Paper-BUILD)
  • CraftBukkit/Spigot/Paper 1.12.2
  • CraftBukkit 1.8.8

Demonstration:

Before:
image

After:
image

Function PatternType.valueOf was not the appropriate function for
finding the PatternType by the identifier. I replaced with the the
appropriate function getByIdentifier.
@JRoy JRoy added the type: bugfix PRs that fix bugs in EssentialsX. label Mar 4, 2024
@JRoy JRoy changed the title Fix custom Banners in kits not returning patterns (EssentialsX#4737) Fix banner patterns not working in kits Mar 4, 2024
@JRoy JRoy merged commit c19b20e into EssentialsX:2.x Mar 4, 2024
HarvelsX pushed a commit to HarvelsX/Essentials that referenced this pull request Jun 2, 2024
Co-authored-by: Josh Roy <10731363+JRoy@users.noreply.github.com>
Starmism pushed a commit to valence-smp/Essentials that referenced this pull request Jan 8, 2025
Co-authored-by: Josh Roy <10731363+JRoy@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bugfix PRs that fix bugs in EssentialsX.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Custom Banners in kits not returning patterns

2 participants

Comments