Bukkit

What steps will reproduce the problem?
1. Equip armor
2. /ss to hide armor
3. Equip more armor or a pumpkin, mobhead, elytra, etc
4. Take damage, causing the hidden armor to briefly reappear.
5. The items equipped in step 3 are gone.

What is the expected output? What do you see instead?
   At step 3, I would expect to be stopped from equipping the conflicting items, or at step 5 I would expect the over equipped items to reappear.

What version of the product are you using?
   name: InvisibleArmor
   main: cl.netgamer.showskin.SS
   version: 3

Do you have an error log of what happened?
   Server log has the user issuing the /ss command and the client log has the chat message, "armor hidden", No other related messages in logs.

Please provide any additional information below.
   Really like the concept of this mod.

thanks for your report and your support

i remember i put code to avoid this, and if you try to over equip an armor piece it denies and re equip the hidden armor

i need some extra info ...

- what spigot/craftbukkit version are you using? - what way did you try to over equip your armor? .. dragging armor piece over inventory slots or "using" it with right click? .. with a dispenser? .. all? (those are different events)

please check these events if some matches with your > https://github.com/atesin/showskin/blob/master/ShowSkin/src/cl/netgamer/showskin/Events.java

I am using Spigot 1.10.2. Also, I restested and saw correct behavior for most equipped items like pumpkins and wither skulls, with the armor instantly being re-equipped, effectively stopping the over-equip. The actual issue seems to be new items in 1.10. The elytra (wings) and dragon mob head don't trigger the re-equip. Doesn't seem to matter how I equip them, trying drap/drop and "use" both fail, but only for the new equippable items.

BTW, I was looking at the code on github at https://github.com/atesin/showskin and it seems out of date, though I could be wrong. Tried to figure this out and it looks like, at least in that code, it picks up the wearable attribute, which seems logical.

Thanks for looking into this!

indeed, i have no code to manage elytra nor dragon heads .. those features (amongst others maybe) were not present when i wrote the plugin

sadly now i have no time to fix it, i just found a very time consuming job ... you could issue some patch if you can/know so i would merge it

what do you say?, could you?

I'll give it a shot.

I pulled the code from https://github.com/atesin/showskin and did a test build. It seems the source code in github is for v1, while the jar is v3. Do you have current source somewhere else?

there is ... sorry, i remembered i had a sync problem on my repos , already fixed (hopefully)

take my email / hangouts / skype for better contact > atesin#gmail,com