Pocketmine-mp: Broken enchants?

Created on 14 Jun 2017 · 7Comments · Source: pmmp/PocketMine-MP

Issue description

I did /enchant (myname) 9 ( 9 is sharpness ) 3 ( 3 is the level so im supposed to get sharpness 3 )

I did that and it said "There is no such enchantment with ID 9"

The Enchant.php says Sharpness' id is 9

and then I did /enchant (myname) sharpness 9 and it gave me Protection IX.

Steps to reproduce the issue

... Wrong enchant ID
...

OS and versions

PocketMine-MP: PocketMine-MP 1.6.2dev-182 「Unleashed」 implementing API version 3.0.0-ALPHA5 for Minecraft: PE v1.1.0.55 (protocol version 113)
PHP: 7.0.3
Server OS: Linux
Game version: PE/Win10 (delete as appropriate)

Plugins

Test on a clean server without plugins: is the issue reproducible without any plugins loaded?

If the issue is not reproducible without plugins:

Have you asked for help on our forums before creating an issue?
Can you provide sample, minimal reproducing code for the issue? If so, paste it in the bottom section
Paste your list of plugins here (use the 'plugins' command in PocketMine-MP)

Crashdump, backtrace or other files

Gameplay Duplicate

Source

XDqrkneZ

Most helpful comment

Read it, you can close this issue..

asyncthreading on 14 Jun 2017

😄3 👍3 ❤2 😕2 🎉2 👎2

All 7 comments

Read it, you can close this issue..

asyncthreading on 14 Jun 2017

😄3 👍3 ❤2 😕2 🎉2 👎2

I believe (but could be wrong) that better enchantment handling was implemented on the api3/blocks branch.

TheDiamondYT1 on 14 Jun 2017

@TheDiamondYT1 You are right.

asyncthreading on 14 Jun 2017

is that api3/blocks different from this? @TheDiamondYT1

XDqrkneZ on 14 Jun 2017

@XDqrkneZ dm me @glavanmadalin2 and i'll explain you

asyncthreading on 14 Jun 2017

Yeah, this was fixed in api3/blocks branch along with new enchantment functions.

Muqsit on 14 Jun 2017

Enchanment is not implemented yet in master branch, right?

On Jun 14, 2017 3:28 PM, "Dylan K. Taylor" notifications@github.com wrote:

Closed #1082 https://github.com/pmmp/PocketMine-MP/issues/1082.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pmmp/PocketMine-MP/issues/1082#event-1122737664, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ARW33QUKsbeRtxFE2Dh6qc-EpnUmaT60ks5sD4uvgaJpZM4N5LSO
.