Too much fuel consumed if there's more than one stack of it in the inventory. #7


  • New
  • Defect
Open
Assigned to j_selby
  • _ForgeUser12965205 created this issue Aug 30, 2014

    What steps will reproduce the problem?
    1. Obviously: Craft a Jetpack.
    2. Take more than 1 stack of coal with you.
    3. Press sneak.

    What is the expected output?
    1 piece of coal is taken from my inventory.

    What do you see instead?
    At least 1 stack of coal plus one piece are taken from my inventory.

    What version of the product are you using?
    EasyJetpack 1.3

    Do you have an error log of what happened?
    Nope, it happens silently.

    Please provide any additional information below.
    No additional plugins; the purest bukkit setup you can imagine (1.7.2; happens with any build)
    Just looked quickly at the source code: I think the problem is mainly line 174 in Utils.java. I believe the line removes every stack of coal from the inventory.

    I think the complete fuel thing can be improved, so coal isn't jumping from every slot in your inventory to every other slot randomly. It should take a piece of coal from a stack of coal, and not take the whole stack and give it back later. Also if the Burning coal reaches 0%, it should get replaced in the same slot and not switched with the stack of new coal or something like that.

  • _ForgeUser12965205 added the tags Defect New Aug 30, 2014

To post a comment, please login or register a new account.