[Plugin] Suggestion: New Broadcasting Plugin

Discussion in 'Suggestions' started by JamieSinn, Aug 1, 2013.

  1. 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
    Current Build Status(Travis): [​IMG]
    Minecraft Name
    :JamieSinn


    Suggestion:To change the broadcast plugin to a custom one.

    Reason: The current one lacks the customization and professional look that this plugin can provide.

    Any Other Information: This is completely custom built, over the last week I have been finding out what is done the most,
    and what can be automated in the fastest way possible, and in doing so i have also added a few features that the other plugin did not support, like configurable messages for snipe events and mineruns, which are exclusive to ecc.
    The code is right now ready for anyone to look over, on my Github: https://github.com/JamieSinn/EcoCityCraftBroadcaster
    and for integrity of the build, i have the travis automated build system managing builds here: https://travis-ci.org/JamieSinn/EcoCityCraftBroadcaster
    I also built the entire project with Spigot 1.6.2 as the dependency so that there is no issue of incompatibility with the current server version.
    I also plan to continue adding lots of features for the next while. There will be some more updates after i post this.

    Link To This Plugin: Github: https://github.com/JamieSinn/EcoCityCraftBroadcaster
    Travis(proof of building correctly): https://travis-ci.org/JamieSinn/EcoCityCraftBroadcaster
    To build the plugin, all you need to do is clone the repo, and mvn clean install to build it
    If needed, i will add a download link.
    UPDATE:
    Link:http://www.mediafire.com/download/l7f8i0ch9o7hrl0/EcoCityCraftBroadcaster.jar
     
    • Like Like x 1
    • Agree Agree x 1
    • List
    #1 JamieSinn, Aug 1, 2013
    Last edited: Aug 2, 2013
  2. Pab_Jr

    Pab_Jr Former Staff Member
    Builder ⛰️ Ex-EcoLeader ⚜️⚜️⚜️ Premium Upgrade

    Joined:
    Jan 13, 2012
    Messages:
    4,979
    Trophy Points:
    78,660
    Gender:
    Male
    Ratings:
    +1,151
    Could you post some screenshots of what the broadcasts could look like?
     
  3. 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
    Sure, :p Do you want the console or do you want in game
     
  4. Pab_Jr

    Pab_Jr Former Staff Member
    Builder ⛰️ Ex-EcoLeader ⚜️⚜️⚜️ Premium Upgrade

    Joined:
    Jan 13, 2012
    Messages:
    4,979
    Trophy Points:
    78,660
    Gender:
    Male
    Ratings:
    +1,151
    Ingame mainly but from both perspectives could help.
     
  5. DakotahFTM

    DakotahFTM Builder
    Builder ⛰️ Ex-President ⚒️⚒️

    Joined:
    Jan 6, 2013
    Messages:
    478
    Trophy Points:
    27,540
    Gender:
    Male
    Ratings:
    +132
    Not to talk about other servers, but I have seen this broadcasting alert in other places, not just ecc. It would be awesome to have our own thing ;)
     
  6. RyanJF1

    RyanJF1 Builder
    Builder ⛰️ Ex-President ⚒️⚒️

    Joined:
    Jul 1, 2012
    Messages:
    6,485
    Trophy Points:
    47,990
    Gender:
    Male
    Ratings:
    +951
    Download link?
     
  7. 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
    DL added RyanJF1
    i frequently will be updating it so the link may break, not sure
     
    #7 JamieSinn, Aug 2, 2013
    Last edited: Aug 2, 2013
  8. 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
    As the prefix, and message colour are completely configurable, ill just use the default:
    [​IMG][​IMG][​IMG][​IMG]
     
  9. Pab_Jr

    Pab_Jr Former Staff Member
    Builder ⛰️ Ex-EcoLeader ⚜️⚜️⚜️ Premium Upgrade

    Joined:
    Jan 13, 2012
    Messages:
    4,979
    Trophy Points:
    78,660
    Gender:
    Male
    Ratings:
    +1,151
    It looks great, but question, wouldnt this cause issues with the current /broadcast command incorporated in Essentials?
    I am also not 100% sure if we need a more advanced broadcasting plugin.
     
    #9 Pab_Jr, Aug 2, 2013
    Last edited: Aug 2, 2013
  10. 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
    From the essentials config, there should be no issues, as you can see, essentials already gives priority to other plugins
     
  11. Nelsin

    Nelsin Builder
    Builder ⛰️ Ex-President ⚒️⚒️

    Joined:
    Feb 24, 2012
    Messages:
    1,343
    Trophy Points:
    29,340
    Ratings:
    +175
    Ill look over the code and see if anything can be approved I see you like to put "{"
    on different lines. You a C# programmer?
     
  12. 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
    I know over 15 languages, ive been coding over 8 years now :p but yes, i do quite a bit of c#, as you can see from my bracket formatting. also, what do you mean by approved?
     
  13. Nelsin

    Nelsin Builder
    Builder ⛰️ Ex-President ⚒️⚒️

    Joined:
    Feb 24, 2012
    Messages:
    1,343
    Trophy Points:
    29,340
    Ratings:
    +175
    sorry I mean improved. Typo
     
  14. 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
    Feel free to put in a pull request :p
     
  15. Nelsin

    Nelsin Builder
    Builder ⛰️ Ex-President ⚒️⚒️

    Joined:
    Feb 24, 2012
    Messages:
    1,343
    Trophy Points:
    29,340
    Ratings:
    +175

    so feel like changing the {
     
  16. 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
    :| Not really, i find it easier to read with the brackets formatted like that sorry :(
     
  17. 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
    Fixed a ton of bugs- see github commits for more info, also thanks kukelekuuk00 for the advice and help on the code cleanup in the massive if statement
     
    #17 JamieSinn, Aug 2, 2013
    Last edited: Aug 2, 2013
  18. Fuzzydoors

    Fuzzydoors Builder
    Builder ⛰️ Ex-Resident ⚒️

    Joined:
    Jul 23, 2013
    Messages:
    11
    Trophy Points:
    25,115
    Ratings:
    +0
    Some of these features seems nice. :)

    However if you're only out to change to broadcast message that can be easily done with a messages.properties file in the essentials folder which i'm sure the staff are aware of :p
     
  19. 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
    Its not only the broadcast message, it also adds a snipe and minerun broadcast command, and lots more.