Coding Help

Discussion in 'Plugin Development' started by zakarls, May 27, 2014.

Thread Status:
Not open for further replies.
  1. Offline

    zakarls

    I'm not sure what i'm doing wrong but my plugin does not work.

    Code:
    Code:java
    1. @EventHandler
    2. public void onCommandSend(PlayerCommandPreprocessEvent event){
    3. Player player = event.getPlayer();
    4. String msg = event.getMessage();
    5. //overall blocked commands
    6. List<String> blockedcmd = plugin.getConfig().getStringList("BlockedCommands");
    7. for (String block : blockedcmd){
    8. List<String> newThing = plugin.getConfig().getStringList(player.getName());
    9. String cmdcmd = ("/" + block).replace("_", "");
    10. if(msg.contains(cmdcmd) && !newThing.contains("override")){
    11. event.setCancelled(true);
    12. player.sendMessage(ChatColor.RED + "You May Not Use This Command.");
    13. }
    14. }
    15. //worldedit section
    16. if(msg.contains("//") || msg.contains("/we") || (msg.contains("/worldedit"))){
    17. if(!((plugin.getConfig().getString(player.getName() + ".worldedit")) == "true")
    18. && !((plugin.getConfig().getString(player.getName() + ".override")) == "true")){
    19. event.setCancelled(true);
    20. player.sendMessage(ChatColor.RED + "You May Not Use This Command. " + ChatColor.AQUA + "Donate at store.purepvp.ca to Gain Access");
    21. }
    22. }
    23. //helper section
    24. List<String> helper = plugin.getConfig().getStringList("Helper");
    25. for(String block : helper){
    26. if(msg.contains(block)){
    27. if(!(plugin.getConfig().getString(player.getName() + ".helper") == "true")
    28. && !(plugin.getConfig().getString(player.getName() + ".override") == "true")){
    29. event.setCancelled(true);
    30. player.sendMessage(ChatColor.RED + "You May Not Use This Command.");
    31. }
    32. }
    33. }
    34. //mod section
    35. List<String> mod = plugin.getConfig().getStringList("Mod");
    36. for(String block : mod){
    37. if(msg.contains(block)){
    38. if(!(plugin.getConfig().getString(player.getName() + ".mod") == "true")
    39. && !(plugin.getConfig().getString(player.getName() + ".override") == "true")){
    40. event.setCancelled(true);
    41. player.sendMessage(ChatColor.RED + "You May Not Use This Command.");
    42. }
    43. }
    44. }
    45. //admin section
    46. List<String> admin = plugin.getConfig().getStringList("Admin");
    47. for(String block : admin){
    48. if(msg.contains(block)){
    49. if(!(plugin.getConfig().getString(player.getName() + ".admin") == "true")
    50. && !(plugin.getConfig().getString(player.getName() + ".override") == "true")){
    51. event.setCancelled(true);
    52. player.sendMessage(ChatColor.RED + "You May Not Use This Command.");
    53. }
    54. }
    55. }
    56. //owner section
    57. List<String> owner = plugin.getConfig().getStringList("Owner");
    58. for(String block : owner){
    59. if(msg.contains(block)){
    60. if(!(plugin.getConfig().getString(player.getName() + ".owner") == "true")
    61. && !(plugin.getConfig().getString(player.getName() + ".override") == "true")){
    62. event.setCancelled(true);
    63. player.sendMessage(ChatColor.RED + "You May Not Use This Command.");
    64. }
    65. }
    66. }
    67. //donor section
    68. List<String> donor = plugin.getConfig().getStringList("Donor");
    69. for(String block : donor){
    70. if(msg.contains(block)){
    71. if(!(plugin.getConfig().getString(player.getName() + ".donor") == "true")
    72. && !(plugin.getConfig().getString(player.getName() + ".override") == "true")){
    73. event.setCancelled(true);
    74. player.sendMessage(ChatColor.RED + "You May Not Use This Command. " + ChatColor.AQUA + "Donate at store.purepvp.ca to Gain Access");
    75. }
    76. }
    77. }
    78.  
    79. }


    The problem is, even I have the things in the config set to "true" the plugin still prevents me from doing the command and it cancels the event. Thanks for any help.
     
  2. zakarls Why are you using Strings in your config to get boolean values? You should be using
    Code:
    getBoolean()
    instead of getString == true. Also, either way, you use .equals() to check for Strings, not ==
     
  3. Offline

    zakarls

    DJSkepter I will try getBoolean(). I've tried .equals() but when that didn't work I went to ==
     
Thread Status:
Not open for further replies.

Share This Page