The `ldi_enchantments` table is useless #1


  • New
  • Enhancment
Open
  • rautamiekka created this issue Apr 18, 2014

    What is the enhancement in mind? How should it look and feel?
    The table saves too little info to be of any use to give back lost stuff. For it to be useful, it should log event ID. With this there's no need to expand the ldi_items table with list of enchants.

    Please provide any additional information below.
    I might be missing something in this ticket due to being half way to midnight, so I'll update later if necessary.

  • rautamiekka added the tags New Enhancment Apr 18, 2014
  • yarkhs posted a comment Jun 15, 2014

    how would you feel if your bow with infinity, flame, power and unbreaking come without these enchants?

  • yarkhs removed their assignment Jun 15, 2014
  • rautamiekka posted a comment Jun 15, 2014

    <<reply 2228828>>

    Are you saying the table is used to rollback something or to dictate what someone has ?


    Edited Jun 15, 2014
  • yarkhs posted a comment Jun 15, 2014

    @rautamiekka: Go

    Not just the table, but all the plugin. The plugin saves inventory from all deaths.

    If the admin realize that death was against server rules, he can use the restore command to give back the lost items and the info command to see the killer.

  • rautamiekka posted a comment Jun 16, 2014

    <<reply 2229259>>

    But how the plugin knows when there's not enough info in the table to know what was in what ?

  • yarkhs posted a comment Jun 16, 2014

    @rautamiekka: Go

    I dont know how to explain, but you dont need to worry about. It is a simple logic commonly used for developers in huge projects.

  • rautamiekka posted a comment Jun 17, 2014

    <<reply 2230351>>

    If the table has a real use, then I'll let it be, otherwise I'll turn it into BLACKHOLE cuz I won't keep useless tables and their useless data.

    I'd love an explanation cuz it teaches me about some mysteries.


    Edited Jun 17, 2014

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