To welcome the new forums, lets have some fun.

Discussion in 'Community News' started by Olink, Feb 21, 2015.

  1. Olink

    Olink Nyx Developer, Occasional TShock Developer
    Banned

    Joined:
    Jul 21, 2011
    Messages:
    1,397
    Hey everyone!

    Since we have these shiny new forums, how about some feedback from our wonderful community.
    This thread is going to be a request thread, where you guys can request new features, bug fixes, changes in behavior, etc etc. If the team decides your idea is worth doing, we will add it to our internal backlog of things to work on. Keep in mind this doesn't mean TShock is back in full swing. This is simply a few developers having a bit of time on their hands and wanting to potentially clean up some smelly code.

    ---------------------------------------------------------------------------------------------------------------------------------------

    Here are the rules:
    1. You are allowed exactly one suggestion at a time.
      • One post
      • One suggestion
      • You must wait until a developer has addressed your suggestion before you post again.
    2. Read through the list of suggestions and 'like' the suggestions that you feel benefit the software in order to help us find out what you guys want.
      • Repeating an idea already addressed or posted just makes us mad, don't do it.
    3. Remember that we are a server software mod, and can do nothing related to the client such as login behavior, custom items/mobs/projectiles etc.

    ------------------------------------------------------------------------------------------------------------------------------------------

    Simply post one suggestion in one post for us, and when we quote your post you are welcome to post a new idea. But only if your post is quoted does it mean we have addressed it.
    If you break the above rules, you forfeit your right to comment in this thread anymore, as well as potentially face a 24 hour ban from the forums. These rules exist to create a somewhat organized process out of the chaos that is suggestions. We feel that if we do not have some rules this will quickly get out of control. We look forward to your input.
     
    • Like Like x 7
  2. bippity

    Plugin Developer

    Joined:
    Jan 14, 2012
    Messages:
    294
    Perhaps a hook for when a user is deleted from the TShock database?
    I believe it would be pretty useful for plugins that have their own databases, such as Seconomy.

    Say that there's a user account named "Bob" who hasn't been on the server for a long time so I delete him, or I'm using a plugin that purges accounts that have been inactive for too long.
    Another player can now register as "Bob", but he'll quickly notice that his Seconomy account would have the balance of the previous "Bob" that was deleted, which means I'll have to remove the previous "Bob" from other databases and check if anything might have screwed up.

    If there's a hook that returns the deleted user's info, plugins can use that info to delete that user from their databases rather than merging the new "Bob" with previously existing data.

    I feel like this might've been suggested before already but... ¯\_(ツ)_/¯
     
    • Like Like x 1
  3. Olink

    Olink Nyx Developer, Occasional TShock Developer
    Banned

    Joined:
    Jul 21, 2011
    Messages:
    1,397
    • Like Like x 1
  4. Annonymus

    Plugin Developer

    Joined:
    Jan 8, 2015
    Messages:
    3
    I'd really welcome the return of the "sudo" command (existed as a plugin before 1.2), a command that lets you do things as if you were someone else, for example "/sudo GenericUser pvp" would make GenericUser execute /pvp and "/sudo GenericUser login tempMod pass99" would make GenericUser execute "/login tempMod pass99".
     
  5. MattTheShy

    MattTheShy Level 1

    Joined:
    Feb 4, 2015
    Messages:
    9
    The /grow command could use some touching up. All it can do is grow purity plants and blinkroot when there are so many more things to grow such as dayblooms, shiverthorn, deathweed, boreal trees, rich mahogany trees, etc.
     
    • Like Like x 1
  6. Annonymus

    Plugin Developer

    Joined:
    Jan 8, 2015
    Messages:
    3
    I hope I'm not breaking the "no multiple suggestions" rule (because I'm only commenting on another suggestion)

    This could also be expanded to "growing" living trees and maybe other snippets like pyramids.
     
    • Like Like x 1
  7. Olink

    Olink Nyx Developer, Occasional TShock Developer
    Banned

    Joined:
    Jul 21, 2011
    Messages:
    1,397
    No, someone should update the plugin that has this feature.
     
  8. Olink

    Olink Nyx Developer, Occasional TShock Developer
    Banned

    Joined:
    Jul 21, 2011
    Messages:
    1,397
    Grow should work with everything, will look over the code to see if theres an obvious reason it fails.
     
  9. Olink

    Olink Nyx Developer, Occasional TShock Developer
    Banned

    Joined:
    Jul 21, 2011
    Messages:
    1,397
    No, there is a plugin that does this, and should just be updated.
     
  10. Wight

    Wight Administrator
    TShock Admin TShock Mod Plugin Developer Zero Day Plugin Author

    Joined:
    May 20, 2012
    Messages:
    1,596
    Hi, check out the Essentials+ plugin in the Resources section.
     
  11. nicatronTg

    nicatronTg Shank / hakusaro
    TShock Founder TShock Admin

    Joined:
    Jul 21, 2011
    Messages:
    714
    ftfy
     
    • Like Like x 1
  12. Pyroxenite Rosetachi

    Joined:
    Feb 17, 2015
    Messages:
    3
    What we need to spruce this up is something simple...something easy...We need.../givecookie command! Because everyone loves cookies on servers for some strange reason
     
  13. Olink

    Olink Nyx Developer, Occasional TShock Developer
    Banned

    Joined:
    Jul 21, 2011
    Messages:
    1,397
    No.
     
    • Like Like x 1
  14. bippity

    Plugin Developer

    Joined:
    Jan 14, 2012
    Messages:
    294
    How about also displaying the BanningUser and date/expiration for the rest command: /v2/bans/list
     
    #14 bippity, Feb 23, 2015
    Last edited: Feb 23, 2015
  15. nicatronTg

    nicatronTg Shank / hakusaro
    TShock Founder TShock Admin

    Joined:
    Jul 21, 2011
    Messages:
    714
    Sounds like something to add to the new rest overhaul.
     
  16. Zaicon

    Zaicon Spam Cleaner 9000
    TShock Mod Plugin Developer

    Joined:
    Feb 15, 2014
    Messages:
    799
    Will the TShock team please fix the issue where restarting the server will cause it to crash?

    To be more specific, the newly-started server instance will throw an exception due to not being able to use ServerLog.txt, since the old instance hasn't finished using it yet. I would say this happens 40% of the time, and it's particularly annoying for me while testing plugins (and using /restart to quickly load the newest plugin build).
     
  17. nicatronTg

    nicatronTg Shank / hakusaro
    TShock Founder TShock Admin

    Joined:
    Jul 21, 2011
    Messages:
    714
    In a batch file (use
    Code:
    exit
    to restart the server):

    Code:
    @echo off
    :main
    TerrariaServer.exe [all of the command line args to start a server without interaction here]
    goto main
    
    Using rerun:
    Code:
    rerun --pattern *.dll TerrariaServer.exe [all of the args blah blah etc]
    
    See also nodemon.
     
  18. nicatronTg

    nicatronTg Shank / hakusaro
    TShock Founder TShock Admin

    Joined:
    Jul 21, 2011
    Messages:
    714
    Protip: hooks like region hooks that people suddenly find useful can be suggested here if you think you need to have an event emitted when TShock does something.
     
  19. Zaicon

    Zaicon Spam Cleaner 9000
    TShock Mod Plugin Developer

    Joined:
    Feb 15, 2014
    Messages:
    799
    Is there a reason that for RegionEntered, it doesn't include the specific region? (I realize TSPlayer.CurrentRegion takes care of that, but there may be times where that info would be useful in RegionEntered/RegionLeft.)
     
  20. Olink

    Olink Nyx Developer, Occasional TShock Developer
    Banned

    Joined:
    Jul 21, 2011
    Messages:
    1,397
    No, because of TSPlayer.CurrentRegion there is no need to send it in the hook.
     
    • Agree Agree x 1