Request + Tiny Bug #31


  • Defect
  • Replied
Open
Assigned to _ForgeUser7084898
  • _ForgeUser7929042 created this issue Sep 2, 2014

    Hey, I just have a little request and also found a tiny bug I believe. I posted this in the comments: http://dev.bukkit.org/bukkit-plugins/dynamicban/#c755 then I got this reply from a random guy called Done84: http://dev.bukkit.org/bukkit-plugins/dynamicban/#c757 and just now I replied to him explaining my issue further + found a small bug - http://dev.bukkit.org/bukkit-plugins/dynamicban/#c759

    Its explained in those comments I linked you to, if you could help me that would be great. Thanks in advance!

  • _ForgeUser7929042 added the tags Defect New Sep 2, 2014
  • _ForgeUser7929042 posted a comment Sep 8, 2014

    Ignore the original ticket above. I'll update you guys on the situation here, unless you continue to contact me through the comments of this plugin.

    1. I explained the bug further in the comments of this plugin to djaddi.

    2. I fixed my issue/request about the temp ban reason not showing up, I posted the following in the comments but decided to copy and paste this here too:

    @djaddi @Done84

    Found out why the reason wasn't showing up for when doing temp bans. It was this in the config -

    tempban_message: 'You have been temporarily banned by {SENDER}. Time left: {TIME} - Reason: {REASON}'

    but the reason part wasn't there originally, so I had to manually add it and once I did the temp ban reason came up - http://i.imgur.com/AhRKD7s.png

    Developer, can you add the reason in for temp bans by default for other users of this plugin - cheers <3

    Also about the bug, posted below @djaddi


    Edited Sep 8, 2014
  • _ForgeUser7084898 unassigned issue from _ForgeUser7993968 Sep 13, 2014
  • _ForgeUser7084898 self-assigned this issue Sep 13, 2014
  • _ForgeUser7084898 posted a comment Sep 13, 2014

    @Joseei: Go

    I'll update the default messages to include the reason.

    Also, to take care of the /dp bug, could you post a stacktrace (the error log) of the issue?

  • _ForgeUser7084898 removed a tag New Sep 13, 2014
  • _ForgeUser7084898 added a tag Waiting Sep 13, 2014
  • _ForgeUser7929042 posted a comment Sep 19, 2014

    @djaddi: Go

    So you will be adding this into the config?:

    tempban_message: 'You have been temporarily banned by {SENDER}. Time left: {TIME} - Reason: {REASON}'

    Also for the /dp bug, I'll look into it now and get back to you ASAP.

  • _ForgeUser7929042 removed a tag Waiting Sep 19, 2014
  • _ForgeUser7929042 added a tag Replied Sep 19, 2014

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