[Bug on Skyblock with Spawnertrade command]

Discussion in 'Help & Support / Errors & Bugs' started by twistedjonny, Jun 2, 2018.

  1. twistedjonny

    twistedjonny ECC Investor
    ECC Investor President ⛰️⛰️ Ex-Tycoon ⚜️⚜️⚜️ Prestige ⭐ II ⭐ Premium Upgrade

    Joined:
    Jun 27, 2013
    Messages:
    781
    Trophy Points:
    58,010
    Gender:
    Male
    Ratings:
    +184
    Minecraft Username:
    @twistedjonny

    Brief Description:
    I was able to, with 2 spawners in my inventory, just type /spawnertrade confirm and it automatically gave me the key. I did not have to first type /spawnertrade. So basically it cut out the confirmation step and traded the 2 spawners in for 1 key with one command.


    How many times did you recreate this?:
    I only did this one time.

    Result:
    I got the key and it took the spawners. I'm okay with this because they were the 2 I wanted to trade anyways.

    Expected Result:
    I don't believe it's intended to just be able to use one command for the trade of 2 spawners to get the 1 key. I'm sure most would like the ability to really make sure those are the 2 spawners they wanted to trade in.

    Evidence:
    Didn't take a screenshot.
     
  2. 314

    314 Irrational SuperMod, former ServerAdmin
    SuperMod EcoLegend ⛰️⛰️⛰️⛰️ Ex-President ⚒️⚒️ Prestige ⭐ VI ⭐ Premium Upgrade

    Joined:
    Apr 1, 2014
    Messages:
    7,056
    Trophy Points:
    97,160
    EcoDollars:
    $2,400,000
    Ratings:
    +4,922
    Here's the way I see it:

    If you use /spawnertrade, you are saying "I would like to exchange the spawners in my inventory. Please tell me which ones I am going to give away".
    If you use /spawnertrade confirm, you are saying "Yes, I am absolutely sure that those are the correct spawners and confirm that they will be removed immediately" because you are explicitly adding a "confirm" statement to your command.

    That's exactly what /spawnertrade without the "confirm" parameter is for.



    In my opinion that's intentional.
     
    #2 314, Jun 2, 2018
    Last edited: Jun 2, 2018
  3. twistedjonny

    twistedjonny ECC Investor
    ECC Investor President ⛰️⛰️ Ex-Tycoon ⚜️⚜️⚜️ Prestige ⭐ II ⭐ Premium Upgrade

    Joined:
    Jun 27, 2013
    Messages:
    781
    Trophy Points:
    58,010
    Gender:
    Male
    Ratings:
    +184
    I wasn’t sure, that’s all.
     
  4. JamieSinn

    JamieSinn Retired Lead Administrator/Developer
    Builder ⛰️ Ex-Tycoon ⚜️⚜️⚜️ Premium Upgrade

    Joined:
    Jun 4, 2011
    Messages:
    5,517
    Trophy Points:
    78,090
    Gender:
    Male
    Ratings:
    +4,588
    314 is correct, this is intended.
     
  5. andrewkm

    Founder Premium Upgrade

    Joined:
    Apr 5, 2011
    Messages:
    20,683
    Trophy Points:
    102,160
    Ratings:
    +15,144
    This is intentional, especially for those who have a large amount of the same spawners and want to quickly /spawnertrade confirm them out quickly.