Thread Status:
Not open for further replies.
  1. Kozz Builder
    Builder ⛰️ Ex-President ⚒️⚒️ Gameplay Architect

    Joined:
    Jan 11, 2017
    Messages:
    272
    Trophy Points:
    28,460
    EcoDollars:
    $0
    Ratings:
    +430
    In Game Name:
    Kozz
    What part of EcoCityCraft is this suggestion for:
    Main Server
    Short title for your suggestion:
    Disable '/pt clear', or make it '/pt ci'
    What are you suggesting:
    Either remove the ability to add 'clear' to a powertool, or configure it so you must type '/pt ci' to actually clear your inventory.
    Why is this a good addition for EcoCityCraft?:
    Powertool noob here. Intuitively typed '/pt clear' on my enchanted pickaxe under the impression that it would clear the commands on my pickaxe.

    Low and behold, next time I went mining all my items disappeared. 'What a bug this is! Ahh well, let's try again!'

    I enchanted another pickaxe, and went to mine again.
    I hit a block. There goes all my items again! RIP.

    Not until I inquired about it had I realized that typing '/pt clear' binds 'clear inventory' to your powertool. Usually this requires confirmation, but as I binded it to my pickaxe that confirmation was given instantaneously when I held it down.
    Other information:
    I know it's my fault for being silly, and looking back it's quite hilarious honestly. I should have read up more about it.

    That being said, this is a dangerous command that is not necessarily noob friendly. I feel I'm definitely not the only person who has made this mistake.
    Plugin or custom addition:
    Powertool
    One suggestion per form:
    I Understand.
     
    • Winner x 1
    • Optimistic x 1
    • List
  2. Ladyvamptress Bloodlust Ex-EcoLegend
    ECC Sponsor Builder ⛰️ Ex-EcoLegend ⚜️⚜️⚜️⚜️ Prestige ⭐ III ⭐ Premium Upgrade

    Joined:
    Jul 26, 2014
    Messages:
    2,253
    Trophy Points:
    81,160
    Gender:
    Female
    Ratings:
    +3,184
    +1
    You’re not alone, I think Mission did something like this a time or two while he was building stuff and lost all his tools and building materials. I think it would be a good idea to change it to “ci” instead of just clear.
     
  3. samiwarbler Sheep Queen
    Mayor ⛰️⛰️ Ex-EcoLegend ⚜️⚜️⚜️⚜️ Prestige ⭐ I ⭐ Premium Upgrade

    Joined:
    Oct 25, 2014
    Messages:
    1,023
    Trophy Points:
    49,410
    Gender:
    Female
    Ratings:
    +552
    #3 samiwarbler, Feb 21, 2020
    Last edited: Feb 21, 2020
  4. Kozz Builder
    Builder ⛰️ Ex-President ⚒️⚒️ Gameplay Architect

    Joined:
    Jan 11, 2017
    Messages:
    272
    Trophy Points:
    28,460
    EcoDollars:
    $0
    Ratings:
    +430
    Ahh shucks.
    I don't necessarily think you have to be "an idiot" to attempt '/pt clear'.
    It's pretty intuitive and very easy to assume it does what it sounds unless explicitly thinking about it.
    IMO, the powertool commands could be better defined.

    Edit:
    I was actually reading up on the Powertool wiki.
    https://dev.bukkit.org/projects/powertool

    I'm not sure if this is actually what the server uses, but from their documentation:
    • /powertool clear — Clears all bound commands from the current item.
    • /powertool clear <item-id-or-name> — Clears all bound commands from the specified item.
    • /powertool clear -a — Clears all bound commands from all items.

    So either another plugin is used, or this command has actually been overrided to produce this behavior.

    Nonetheless, I was unaware this suggestion has already been denied. I will consider this one as such.
     
    #4 Kozz, Feb 22, 2020
    Last edited: Feb 22, 2020
  5. Nicit6 N6
    Mayor ⛰️⛰️ Ex-EcoLegend ⚜️⚜️⚜️⚜️ Prestige ⭐ II ⭐ Gameplay Architect Premium Upgrade

    Joined:
    Jul 13, 2013
    Messages:
    9,916
    Trophy Points:
    102,160
    Ratings:
    +8,061
    Powertool is part of essentials
     
  6. samiwarbler Sheep Queen
    Mayor ⛰️⛰️ Ex-EcoLegend ⚜️⚜️⚜️⚜️ Prestige ⭐ I ⭐ Premium Upgrade

    Joined:
    Oct 25, 2014
    Messages:
    1,023
    Trophy Points:
    49,410
    Gender:
    Female
    Ratings:
    +552
    Also the server wiki, does talk about the command to clear a pt.

    “Note: To clear a current items powertool simply run "/powertool" while holding it in your hand.”

    Not that everyone reads it beforehand but it is there.
     
  7. cida true blue full grown fair dinkum fkn aussie
    Builder ⛰️ Ex-President ⚒️⚒️ Premium Upgrade

    Joined:
    Apr 22, 2012
    Messages:
    119
    Trophy Points:
    42,810
    Gender:
    Male
    Ratings:
    +46
  8. Founder Premium Upgrade

    Joined:
    Apr 5, 2011
    Messages:
    20,552
    Trophy Points:
    102,160
    Ratings:
    +15,113
    We had a lot of garbage aliases for our clearinventory command.
    This has now been changed to:



    Meaning only the following will work now:
    • /clearinventory
    • /ci
    This will prevent the mistake you made as clear will no longer be a valid command.
    Changes will be live after next maintenance period.

    @ClarinetPhoenix please ensure the wiki states that /clearinventory and /ci are the only available commands under the specific feature package description.
     
    #8 andrewkm, Mar 5, 2020
    Last edited: Mar 5, 2020
Thread Status:
Not open for further replies.