Skip to content

Add Adventure playSound(Sound, Location)#14100

Open
GliczDev wants to merge 1 commit into
PaperMC:mainfrom
GliczDev:playSound
Open

Add Adventure playSound(Sound, Location)#14100
GliczDev wants to merge 1 commit into
PaperMC:mainfrom
GliczDev:playSound

Conversation

@GliczDev

Copy link
Copy Markdown
Contributor

No description provided.

@GliczDev
GliczDev requested a review from a team as a code owner July 24, 2026 22:00
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Jul 24, 2026
Comment on lines +853 to +854
default void playSound(net.kyori.adventure.sound.Sound sound, Location location) {
playSound(sound, location.getX(), location.getY(), location.getZ());

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
default void playSound(net.kyori.adventure.sound.Sound sound, Location location) {
playSound(sound, location.getX(), location.getY(), location.getZ());
default void playSound(net.kyori.adventure.sound.Sound sound, io.papermc.paper.math.FinePosition pos) {
playSound(sound, pos.x(), pos.y(), pos.z());

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

why not just Position then? Also, wouldn't it make using that method show a warning regarding Position being experimental?

Comment on lines +3781 to +3782
default void playSound(net.kyori.adventure.sound.Sound sound, Location location) {
playSound(sound, location.getX(), location.getY(), location.getZ());

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
default void playSound(net.kyori.adventure.sound.Sound sound, Location location) {
playSound(sound, location.getX(), location.getY(), location.getZ());
default void playSound(net.kyori.adventure.sound.Sound sound, io.papermc.paper.math.FinePosition pos) {
playSound(sound, pos.x(), pos.y(), pos.z());

@github-project-automation github-project-automation Bot moved this from Awaiting review to Changes required in Paper PR Queue Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Changes required

Development

Successfully merging this pull request may close these issues.

2 participants