Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Armor durability never goes down #2032

Closed
sn3akrr opened this issue Feb 18, 2018 · 4 comments
Closed

Armor durability never goes down #2032

sn3akrr opened this issue Feb 18, 2018 · 4 comments
Labels
Category: Gameplay Related to Minecraft gameplay experience Status: Not Implemented

Comments

@sn3akrr
Copy link

sn3akrr commented Feb 18, 2018

Issue description

Armor durability won't go down. Damage a player and it stays full always.

  • Expected result: What were you expecting to happen?
  • Actual result: What actually happened?

Steps to reproduce the issue

  1. ...
  2. ...

OS and versions

  • PocketMine-MP: eeaf75a
  • PHP: 7.2.2
  • Server OS: Ubuntu
  • 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

  • Do not paste crashdumps into an issue - please use our Crash Archive at https://crash.pmmp.io for submitting crash reports to not spam the issue tracker. Add links to your reports in the Crash Archive here.
  • Please use gist or anything else to add other files and add links here
  • ...
@dktapps
Copy link
Member

dktapps commented Feb 18, 2018

This is not implemented yet.

@dktapps dktapps closed this as completed Feb 18, 2018
@dktapps dktapps added Category: Gameplay Related to Minecraft gameplay experience Status: Not Implemented labels Feb 18, 2018
@sn3akrr
Copy link
Author

sn3akrr commented Feb 19, 2018

Oh, I thought it has done that before.

@95CivicSi
Copy link
Contributor

It did until this commit
efca9f0#diff-be0be6071c2c2f0abecaa7dd4e16e2f1

@dktapps
Copy link
Member

dktapps commented Feb 19, 2018

No, it has never been implemented. @95CivicSi that commit has no bearing on armor whatsoever, unless you used a plugin to make it work. It's still yet to be implemented into the core code, as seen here (look at the documentation) and in subclasses overriding that method.

(P.S. Until recently armor did not even work for anything except PvP. The implementation is incomplete.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Gameplay Related to Minecraft gameplay experience Status: Not Implemented
Projects
None yet
Development

No branches or pull requests

3 participants