pmmp / PocketMine-MP

A server software for Minecraft: Bedrock Edition in PHP
https://pmmp.io
GNU Lesser General Public License v3.0
3.28k stars 1.55k forks source link

`SignUpdateEvent::getOldText()` will return a array with empty-strings #5035

Closed supercrafter333 closed 2 years ago

supercrafter333 commented 2 years ago

Issue description

Steps to reproduce the issue

  1. Place a sign and write something on it
  2. dump the old and the new text

OS and versions

Plugins

The bug still occour with and without plugins.

Crashdump, backtrace or other files

nothing

dktapps commented 2 years ago

This is expected due to a change in client behaviour in 1.18.30.

supercrafter333 commented 2 years ago

WHat should plugin-developers do now?

dktapps commented 2 years ago

If anything this change is good for plugin developers. I don't see any problem.

supercrafter333 commented 2 years ago

What's about remove getOldText() and getNewText() and replace it with anything like getText()?

dktapps commented 2 years ago

That's an API break.

supercrafter333 commented 2 years ago

PM5?

dktapps commented 2 years ago

The behaviour might still be changed again by Mojang. PM5 is a long time away. I won't make such hasty API decisions based on one version.

supercrafter333 commented 2 years ago

Okay