This fixes #247 because I realized how painfully easy this was likely to fix. This may likely need to be updated again if Mojang ever decides to add more buckets of fish, but I would guess it would be unlikely given how long fishing has been in.
*Note: my Java is very rusty so I'm not sure of best practices. I assigned the materials to a variable before the if because I assumed every get() would see a 1ns performance hit. I'm just used to doing it like that in C#.
This fixes #247 because I realized how painfully easy this was likely to fix. This may likely need to be updated again if Mojang ever decides to add more buckets of fish, but I would guess it would be unlikely given how long fishing has been in.
*Note: my Java is very rusty so I'm not sure of best practices. I assigned the materials to a variable before the
if
because I assumed everyget()
would see a 1ns performance hit. I'm just used to doing it like that in C#.