Pagination stops working after changing slot - creative #49


  • Defect
  • Accepted
Open
Assigned to _ForgeUser8106975
  • _ForgeUser7824708 created this issue Feb 7, 2012

    What steps will reproduce the problem?
    1. create a letter with multiple pages
    2. change letter's slot from i.e. slot 1 to slot 2
    3. try to paginate

    What is the expected output? What do you see instead?
    No pagination occurs and even when you left-click, those clicks get registered and destroy stuff.

    What version of Courier are you using?
    1.0.0

    What version of CraftBukkit ("Bukkit") are you using? 1.0.1-R1, dev build number etc.
    1.0.1-R1

    Do you have an error log of what happened?
    no errors on console

  • _ForgeUser7824708 added the tags New Defect Feb 7, 2012
  • _ForgeUser8106975 posted a comment Feb 11, 2012

    Thanks for the bug report, verified on Courier 1.1.0 and Bukkit 1.1-R3

  • _ForgeUser8106975 removed a tag New Feb 11, 2012
  • _ForgeUser8106975 added a tag Accepted Feb 11, 2012
  • _ForgeUser8106975 posted a comment Feb 11, 2012

    Preliminary analysis: Moving an item from one slot to another while in gamemode 1 (creative) loses the enchantments of that item. At least true for Courier Letters, which use the unsafe enchantment API. This would, if true, be a Bukkit error.

    This is somewhat obscured by Courier still rendering text on the Letter, but simply logging off/on will show that the Letter which was moved between slots is now "empty".

    (Courier uses the enchantment level of Maps for looking up the correct contents in the database. Losing the enchantment of an item results in Courier believing it's a normal map and not a Letter)

  • _ForgeUser7824708 posted a comment Feb 11, 2012

    The comment about showing "empty" letter struck me with a memory of similar experience. Sometimes when people retrieve prepared multipage letter (via a command alias having lots of text after the /letter command), the map renders empty. Right-clicking however will show map contents. Could this be related?


    Edited Feb 11, 2012

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