Build #141 Bug Report - Githubissues

deathmarine / DiabloDrops

Bukkit plugin that adjusts the mob dropping mechanics to include regularly dropped enchanted items that could be commonly found in Diablo.

http://dev.bukkit.org/bukkit-plugins/diablodrops/

GNU General Public License v3.0

17 stars 19 forks source link

Build #141 Bug Report #25

Closed UltraFaceguy closed 11 years ago

UltraFaceguy commented 11 years ago

Leather armor doesn't socket properly
Some users crash upon attacking a user with +3 confusion armor (or higher?) http://pastie.org/5440678

deathmarine commented 11 years ago

2 is odd.... actually really odd. I'm wondering if the client can handle and amplication of 3 or more...

Either way I'll look at leather armor.

deathmarine commented 11 years ago

Fixed. https://github.com/deathmarine/DiabloDrops/commit/2c22bbb1b481fc788d3382d9e05ac3553bfb4a27

2 I'm not sure we can do much about other than restrict it in the plugin.

Edit... just tested it with +/-7 Confusion out of curiosity. I can't reproduce it.Modded and unmodded. However that does look like an interesting stacktrace.

UltraFaceguy commented 11 years ago

I think it may just be this guy. Well... no that can't be. I have a +3 confusion pick I use on EVERYTHING, even him, and nothing has ever gone wrong! Only when he attacks me. I'll run some more tests, but limiting it may not be necessary, if it was just him.