From e637e7591e46a22b1df964229748b9496ff3dab4 Mon Sep 17 00:00:00 2001
From: Esophose <esophose@gmail.com>
Date: Sat, 6 Oct 2018 13:53:31 -0600
Subject: [PATCH] /pp data finished, change some wording, remove debug output

---
 .../playerparticles/PlayerParticles.java      |   1 -
 .../command/AddCommandModule.java             |   3 +-
 .../command/DataCommandModule.java            |  43 ++++---
 .../command/FixedCommandModule.java           |   1 -
 .../command/ParticleCommandHandler.java       | 118 +++++-------------
 .../command/ResetCommandModule.java           |  33 +----
 .../playerparticles/manager/DataManager.java  |  12 +-
 .../playerparticles/manager/LangManager.java  |  18 ++-
 .../manager/PermissionManager.java            |  10 --
 .../styles/ParticleStyleBlockBreak.java       |   4 +-
 .../styles/ParticleStyleBlockPlace.java       |   4 +-
 src/config.yml                                |   2 +-
 src/lang/default.lang                         |  24 ++--
 src/plugin.yml                                |   2 +-
 14 files changed, 91 insertions(+), 184 deletions(-)

diff --git a/src/com/esophose/playerparticles/PlayerParticles.java b/src/com/esophose/playerparticles/PlayerParticles.java
index 807d91c..93a9a32 100644
--- a/src/com/esophose/playerparticles/PlayerParticles.java
+++ b/src/com/esophose/playerparticles/PlayerParticles.java
@@ -1,6 +1,5 @@
 /*
  * TODO: v5.3
- * + Command to force set an effect/style for a player
  * + Add new style 'tornado'
  * + Add new style 'companion'
  * + Add new style 'atom'
diff --git a/src/com/esophose/playerparticles/command/AddCommandModule.java b/src/com/esophose/playerparticles/command/AddCommandModule.java
index a884742..35550be 100644
--- a/src/com/esophose/playerparticles/command/AddCommandModule.java
+++ b/src/com/esophose/playerparticles/command/AddCommandModule.java
@@ -1,7 +1,6 @@
 package com.esophose.playerparticles.command;
 
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.List;
 
 import org.bukkit.Material;
@@ -133,7 +132,7 @@ public class AddCommandModule implements CommandModule {
     public List<String> onTabComplete(PPlayer pplayer, String[] args) {
         Player p = pplayer.getPlayer();
         List<String> matches = new ArrayList<String>();
-        System.out.println("Args in AddCommandModule: " + Arrays.toString(args) + " " + args.length);
+
         if (args.length <= 1) { // Effect name
             if (args.length == 0) matches = PermissionManager.getEffectsUserHasPermissionFor(p);
             else StringUtil.copyPartialMatches(args[0], PermissionManager.getEffectsUserHasPermissionFor(p), matches);
diff --git a/src/com/esophose/playerparticles/command/DataCommandModule.java b/src/com/esophose/playerparticles/command/DataCommandModule.java
index 0bba88e..1add1bd 100644
--- a/src/com/esophose/playerparticles/command/DataCommandModule.java
+++ b/src/com/esophose/playerparticles/command/DataCommandModule.java
@@ -1,9 +1,13 @@
 package com.esophose.playerparticles.command;
 
+import java.util.ArrayList;
 import java.util.List;
 
+import org.bukkit.util.StringUtil;
+
 import com.esophose.playerparticles.manager.LangManager;
 import com.esophose.playerparticles.manager.LangManager.Lang;
+import com.esophose.playerparticles.manager.PermissionManager;
 import com.esophose.playerparticles.particles.PPlayer;
 import com.esophose.playerparticles.particles.ParticleEffect;
 import com.esophose.playerparticles.particles.ParticleEffect.ParticleProperty;
@@ -13,31 +17,38 @@ public class DataCommandModule implements CommandModule {
     public void onCommandExecute(PPlayer pplayer, String[] args) {
         if (args.length > 0) {
             ParticleEffect effect = ParticleEffect.fromName(args[0]);
+            if (effect == null) {
+                LangManager.sendMessage(pplayer, Lang.EFFECT_INVALID, args[0]);
+                return;
+            }
 
-            if ((!effect.hasProperty(ParticleProperty.REQUIRES_MATERIAL_DATA) && !effect.hasProperty(ParticleProperty.COLORABLE)) || args.length == 0) {
-                if (effect.hasProperty(ParticleProperty.COLORABLE)) {
-                    if (effect == ParticleEffect.NOTE) {
-                        LangManager.sendMessage(pplayer, Lang.DATA_USAGE_NOTE);
-                    } else {
-                        LangManager.sendMessage(pplayer, Lang.DATA_USAGE_COLOR);
-                    }
-                } else if (effect.hasProperty(ParticleProperty.REQUIRES_MATERIAL_DATA)) {
-                    if (effect == ParticleEffect.ITEM) {
-                        LangManager.sendMessage(pplayer, Lang.DATA_USAGE_ITEM);
-                    } else {
-                        LangManager.sendMessage(pplayer, Lang.DATA_USAGE_BLOCK);
-                    }
+            if (effect.hasProperty(ParticleProperty.COLORABLE)) {
+                if (effect == ParticleEffect.NOTE) {
+                    LangManager.sendMessage(pplayer, Lang.DATA_USAGE_NOTE, effect.getName());
                 } else {
-                    LangManager.sendMessage(pplayer, Lang.DATA_USAGE_NONE);
+                    LangManager.sendMessage(pplayer, Lang.DATA_USAGE_COLOR, effect.getName());
                 }
+            } else if (effect.hasProperty(ParticleProperty.REQUIRES_MATERIAL_DATA)) {
+                if (effect == ParticleEffect.ITEM) {
+                    LangManager.sendMessage(pplayer, Lang.DATA_USAGE_ITEM, effect.getName());
+                } else {
+                    LangManager.sendMessage(pplayer, Lang.DATA_USAGE_BLOCK, effect.getName());
+                }
+            } else {
+                LangManager.sendMessage(pplayer, Lang.DATA_USAGE_NONE, effect.getName());
             }
         } else {
-            LangManager.sendMessage(pplayer, Lang.EFFECT_INVALID);
+            LangManager.sendMessage(pplayer, Lang.COMMAND_DATA_NO_ARGS);
         }
     }
 
     public List<String> onTabComplete(PPlayer pplayer, String[] args) {
-        return null;
+        List<String> matches = new ArrayList<String>();
+        if (args.length <= 1) {
+            if (args.length == 0) matches = PermissionManager.getEffectsUserHasPermissionFor(pplayer.getPlayer());
+            else StringUtil.copyPartialMatches(args[0], PermissionManager.getEffectsUserHasPermissionFor(pplayer.getPlayer()), matches);
+        }
+        return matches;
     }
 
     public String getName() {
diff --git a/src/com/esophose/playerparticles/command/FixedCommandModule.java b/src/com/esophose/playerparticles/command/FixedCommandModule.java
index fdd51f3..cf3bb2c 100644
--- a/src/com/esophose/playerparticles/command/FixedCommandModule.java
+++ b/src/com/esophose/playerparticles/command/FixedCommandModule.java
@@ -36,7 +36,6 @@ public class FixedCommandModule implements CommandModule {
         }
 
         if (args.length == 0) { // General information on command
-            LangManager.sendMessage(p, Lang.FIXED_INVALID_COMMAND);
             LangManager.sendMessage(p, Lang.COMMAND_DESCRIPTION_FIXED_CREATE);
             LangManager.sendMessage(p, Lang.COMMAND_DESCRIPTION_FIXED_REMOVE);
             LangManager.sendMessage(p, Lang.COMMAND_DESCRIPTION_FIXED_LIST);
diff --git a/src/com/esophose/playerparticles/command/ParticleCommandHandler.java b/src/com/esophose/playerparticles/command/ParticleCommandHandler.java
index 2384b53..6cc0590 100644
--- a/src/com/esophose/playerparticles/command/ParticleCommandHandler.java
+++ b/src/com/esophose/playerparticles/command/ParticleCommandHandler.java
@@ -16,15 +16,15 @@ import com.esophose.playerparticles.manager.LangManager.Lang;
 import com.esophose.playerparticles.manager.PermissionManager;
 
 public class ParticleCommandHandler implements CommandExecutor, TabCompleter {
-    
+
     /**
      * A list of all commands
      */
     private static List<CommandModule> commands;
-    
+
     static {
         commands = new ArrayList<CommandModule>();
-        
+
         commands.add(new AddCommandModule());
         commands.add(new DataCommandModule());
         commands.add(new DefaultCommandModule());
@@ -44,7 +44,7 @@ public class ParticleCommandHandler implements CommandExecutor, TabCompleter {
         commands.add(new VersionCommandModule());
         commands.add(new WorldsCommandModule());
     }
-    
+
     /**
      * Finds a matching CommandModule by its name
      * 
@@ -52,12 +52,12 @@ public class ParticleCommandHandler implements CommandExecutor, TabCompleter {
      * @return The found CommandModule, otherwise null
      */
     public static CommandModule findMatchingCommand(String commandName) {
-    	for (CommandModule commandModule : commands)
-    		if (commandModule.getName().equalsIgnoreCase(commandName))
-    			return commandModule;
-    	return null;
+        for (CommandModule commandModule : commands)
+            if (commandModule.getName().equalsIgnoreCase(commandName)) 
+                return commandModule;
+        return null;
     }
-    
+
     /**
      * Get a list of all available commands
      * 
@@ -66,7 +66,7 @@ public class ParticleCommandHandler implements CommandExecutor, TabCompleter {
     public static List<CommandModule> getCommands() {
         return commands;
     }
-    
+
     /**
      * Get all available command names
      * 
@@ -74,7 +74,7 @@ public class ParticleCommandHandler implements CommandExecutor, TabCompleter {
      */
     public static List<String> getCommandNames() {
         List<String> commandNames = new ArrayList<String>();
-        for (CommandModule cmd : commands) 
+        for (CommandModule cmd : commands)
             commandNames.add(cmd.getName());
         return commandNames;
     }
@@ -92,31 +92,26 @@ public class ParticleCommandHandler implements CommandExecutor, TabCompleter {
     public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
         if (!(sender instanceof Player)) return true;
         Player p = (Player) sender;
-        
+
         DataManager.getPPlayer(p.getUniqueId(), (pplayer) -> {
-        	String commandName = args.length > 0 ? args[0] : "";
-        	CommandModule commandModule = findMatchingCommand(commandName);
-        	
-        	if (commandModule != null) {
-        		if (commandModule.requiresEffects() && PermissionManager.getEffectsUserHasPermissionFor(p).size() == 1) {
-        			LangManager.sendMessage(p, Lang.COMMAND_ERROR_NO_EFFECTS);
-        		} else {
-        			String[] cmdArgs = new String[0];
+            String commandName = args.length > 0 ? args[0] : "";
+            CommandModule commandModule = findMatchingCommand(commandName);
+
+            if (commandModule != null) {
+                if (commandModule.requiresEffects() && PermissionManager.getEffectsUserHasPermissionFor(p).size() == 1) {
+                    LangManager.sendMessage(p, Lang.COMMAND_ERROR_NO_EFFECTS);
+                } else {
+                    String[] cmdArgs = new String[0];
                     if (args.length > 1) cmdArgs = Arrays.copyOfRange(args, 1, args.length);
-            		commandModule.onCommandExecute(pplayer, cmdArgs);
-        		}
-        	} else {
-        		LangManager.sendMessage(p, Lang.COMMAND_ERROR_UNKNOWN);
-        	}
+                    commandModule.onCommandExecute(pplayer, cmdArgs);
+                }
+            } else {
+                LangManager.sendMessage(p, Lang.COMMAND_ERROR_UNKNOWN);
+            }
         });
-        
+
         return true;
     }
-    
-//    private static final String[] COMMANDS = { "help", "gui", "effect", "effects", "style", "styles", "data", "fixed", "reset", "worlds", "version" };
-//    private static final String[] FIXED_COMMANDS = { "create", "remove", "list", "info" };
-//    private static final List<String> BLOCK_MATERIALS = ParticleUtils.getAllBlockMaterials();
-//    private static final List<String> ITEM_MATERIALS = ParticleUtils.getAllItemMaterials();
 
     /**
      * Activated when a user pushes tab in chat prefixed with /pp
@@ -131,67 +126,18 @@ public class ParticleCommandHandler implements CommandExecutor, TabCompleter {
         if (!(sender instanceof Player)) return new ArrayList<String>();
 
         if (cmd.getName().equalsIgnoreCase("pp")) {
-            System.out.println("Args in onTabComplete: " + Arrays.toString(args) + " " + args.length);
             if (args.length <= 1) {
                 CommandModule commandModule = findMatchingCommand(""); // Get the default command module
                 return commandModule.onTabComplete(DataManager.getPPlayer(((Player) sender).getUniqueId()), args);
             } else if (args.length >= 2) {
-            	CommandModule commandModule = findMatchingCommand(args[0]);
-            	String[] cmdArgs = Arrays.copyOfRange(args, 1, args.length);
-            	if (commandModule != null) {
-            		return commandModule.onTabComplete(DataManager.getPPlayer(((Player) sender).getUniqueId()), cmdArgs);
-            	}
-            	
-            	// TODO: Move to correct CommandModules
-//                if (args[0].equalsIgnoreCase("effect") && args.length == 2) {
-//                    List<String> commands = PermissionManager.getEffectsUserHasPermissionFor((Player) sender);
-//                    StringUtil.copyPartialMatches(args[1], commands, completions);
-//                } else if (args[0].equalsIgnoreCase("style") && args.length == 2) {
-//                    List<String> commands = PermissionManager.getStylesUserHasPermissionFor((Player) sender);
-//                    StringUtil.copyPartialMatches(args[1], commands, completions);
-//                } else if (args[0].equalsIgnoreCase("fixed") && args.length > 1) {
-//                    if (args.length == 2) {
-//                        List<String> commands = Arrays.asList(FIXED_COMMANDS);
-//                        StringUtil.copyPartialMatches(args[1], commands, completions);
-//                    } else if (args[1].equalsIgnoreCase("create")) {
-//                        completions.add("<x> <y> <z> <effect> <style> [data]");
-//                    } else if ((args[1].equalsIgnoreCase("remove") || args[1].equalsIgnoreCase("info")) && args.length == 3) {
-//                        completions.add("<id>");
-//                    } 
-//                } else if (args[0].equalsIgnoreCase("data")) {
-//                    PPlayer pplayer = DataManager.getPPlayer(((Player) sender).getUniqueId());
-//                    if (pplayer == null) {
-//                        completions.add(ChatColor.stripColor(Lang.NO_DATA_USAGE.get()));
-//                    } else if (pplayer.getParticleEffect().hasProperty(ParticleProperty.REQUIRES_MATERIAL_DATA) && args.length == 2) {
-//                        if (pplayer.getParticleEffect() == ParticleEffect.ITEM) {
-//                            StringUtil.copyPartialMatches(args[1], ITEM_MATERIALS, completions);
-//                        } else {
-//                            StringUtil.copyPartialMatches(args[1], BLOCK_MATERIALS, completions);
-//                        }
-//                    } else if (pplayer.getParticleEffect().hasProperty(ParticleProperty.COLORABLE)) {
-//                        if (pplayer.getParticleEffect() == ParticleEffect.NOTE && args.length == 2) {
-//                            completions.add("<0-23>");
-//                            StringUtil.copyPartialMatches(args[args.length - 1], Arrays.asList(new String[] { "rainbow" }), completions);
-//                        } else if (pplayer.getParticleEffect() != ParticleEffect.NOTE && args.length > 1 && args.length < 5) {
-//                            completions.add("<0-255>");
-//                            if (args.length == 2) {
-//                                StringUtil.copyPartialMatches(args[args.length - 1], Arrays.asList(new String[] { "rainbow" }), completions);
-//                            }
-//                        }
-//                    } else if (args.length == 2) {
-//                        completions.add(ChatColor.stripColor(Lang.NO_DATA_USAGE.get()));
-//                    }
-//                }
-//            } else {
-//                List<String> commands = new ArrayList<String>(Arrays.asList(COMMANDS));
-//                StringUtil.copyPartialMatches(args[0], commands, completions);
-              }
+                CommandModule commandModule = findMatchingCommand(args[0]);
+                String[] cmdArgs = Arrays.copyOfRange(args, 1, args.length);
+                if (commandModule != null) {
+                    return commandModule.onTabComplete(DataManager.getPPlayer(((Player) sender).getUniqueId()), cmdArgs);
+                }
+            }
         }
         return null;
     }
 
-//    public static String[] getCommandsList() {
-//        return COMMANDS;
-//    }
-
 }
diff --git a/src/com/esophose/playerparticles/command/ResetCommandModule.java b/src/com/esophose/playerparticles/command/ResetCommandModule.java
index e63285e..0b8135f 100644
--- a/src/com/esophose/playerparticles/command/ResetCommandModule.java
+++ b/src/com/esophose/playerparticles/command/ResetCommandModule.java
@@ -2,12 +2,8 @@ package com.esophose.playerparticles.command;
 
 import java.util.List;
 
-import org.bukkit.Bukkit;
-import org.bukkit.entity.Player;
-
 import com.esophose.playerparticles.manager.DataManager;
 import com.esophose.playerparticles.manager.LangManager;
-import com.esophose.playerparticles.manager.PermissionManager;
 import com.esophose.playerparticles.manager.LangManager.Lang;
 import com.esophose.playerparticles.particles.PPlayer;
 import com.esophose.playerparticles.particles.ParticleGroup;
@@ -15,32 +11,9 @@ import com.esophose.playerparticles.particles.ParticleGroup;
 public class ResetCommandModule implements CommandModule {
 
     public void onCommandExecute(PPlayer pplayer, String[] args) {
-        if (args.length >= 1) {
-            String altPlayerName = args[0];
-            if (!PermissionManager.canUseForceReset(pplayer.getPlayer())) {
-                LangManager.sendMessage(pplayer, Lang.EXECUTE_FAILED_NO_PERMISSION, altPlayerName);
-            } else {
-                Player targetPlayer = null;
-                for (Player p : Bukkit.getOnlinePlayers()) {
-                    if (p.getName().toLowerCase().contains(altPlayerName.toLowerCase())) {
-                        targetPlayer = p;
-                        break;
-                    }
-                }
-
-                if (targetPlayer == null) {
-                    LangManager.sendMessage(pplayer, Lang.EXECUTE_FAILED_NOT_FOUND, altPlayerName);
-                } else {
-                    DataManager.saveParticleGroup(targetPlayer.getUniqueId(), ParticleGroup.getDefaultGroup());
-                    LangManager.sendMessage(targetPlayer, Lang.RESET_SUCCESS);
-
-                    LangManager.sendMessage(pplayer, Lang.EXECUTE_SUCCESS, targetPlayer.getName());
-                }
-            }
-        } else {
-            DataManager.saveParticleGroup(pplayer.getUniqueId(), ParticleGroup.getDefaultGroup());
-            LangManager.sendMessage(pplayer, Lang.RESET_SUCCESS);
-        }
+        int particleCount = pplayer.getActiveParticles().size();
+        DataManager.saveParticleGroup(pplayer.getUniqueId(), ParticleGroup.getDefaultGroup());
+        LangManager.sendMessage(pplayer, Lang.RESET_SUCCESS, particleCount);
     }
 
     public List<String> onTabComplete(PPlayer pplayer, String[] args) {
diff --git a/src/com/esophose/playerparticles/manager/DataManager.java b/src/com/esophose/playerparticles/manager/DataManager.java
index 5ee0c83..521fdb2 100644
--- a/src/com/esophose/playerparticles/manager/DataManager.java
+++ b/src/com/esophose/playerparticles/manager/DataManager.java
@@ -88,10 +88,9 @@ public class DataManager {
         					   	    "WHERE g.owner_uuid = ?"; // @formatter:on
                 try (PreparedStatement statement = connection.prepareStatement(groupQuery)) {
                     statement.setString(1, playerUUID.toString());
-                    System.out.println("Executing particle query");
+
                     ResultSet result = statement.executeQuery();
                     while (result.next()) {
-                        System.out.println("Particle result found");
                         // Group properties
                         String groupName = result.getString("name");
 
@@ -104,15 +103,6 @@ public class DataManager {
                         NoteColor noteColor = new NoteColor(result.getInt("note"));
                         OrdinaryColor color = new OrdinaryColor(result.getInt("r"), result.getInt("g"), result.getInt("b"));
                         ParticlePair particle = new ParticlePair(playerUUID, id, effect, style, itemMaterial, blockMaterial, color, noteColor);
-                        
-                        System.out.println("==============================");
-                        System.out.println(result.getInt("id"));
-                        System.out.println(result.getString("effect"));
-                        System.out.println(result.getString("style"));
-                        System.out.println(result.getString("item_material"));
-                        System.out.println(result.getString("block_material"));
-                        System.out.println(result.getInt("note"));
-                        System.out.println(result.getInt("r") + " " + result.getInt("g") + " " + result.getInt("b"));
 
                         // Try to add particle to an existing group
                         boolean groupAlreadyExists = false;
diff --git a/src/com/esophose/playerparticles/manager/LangManager.java b/src/com/esophose/playerparticles/manager/LangManager.java
index 4b95762..6e41ee2 100644
--- a/src/com/esophose/playerparticles/manager/LangManager.java
+++ b/src/com/esophose/playerparticles/manager/LangManager.java
@@ -20,7 +20,7 @@ public class LangManager {
     /**
      * Contains the location in the .lang file of every chat message
      */
-    public static enum Lang {
+    public static enum Lang { // @formatter:off
         
         // Command Errors
         COMMAND_ERROR_NO_EFFECTS,
@@ -59,6 +59,9 @@ public class LangManager {
         // Add Command
         COMMAND_ADD_PARTICLE_APPLIED,
         
+        // Data Command
+        COMMAND_DATA_NO_ARGS,
+        
         // Effects
         EFFECT_NO_PERMISSION,
         EFFECT_INVALID,
@@ -88,11 +91,6 @@ public class LangManager {
         DISABLED_WORLDS,
         DISABLED_WORLDS_NONE,
         
-        // Alternate Player Command Execution
-        EXECUTE_SUCCESS,
-        EXECUTE_FAILED_NOT_FOUND,
-        EXECUTE_FAILED_NO_PERMISSION,
-        
         // Reset
         RESET_SUCCESS,
         
@@ -134,10 +132,10 @@ public class LangManager {
         GUI_ICON_SETS_TO,
         GUI_ICON_SET_YOUR,
         GUI_NO_ACCESS_TO,
-        GUI_NO_DATA;
+        GUI_NO_DATA; // @formatter:on
 
         private String message;
-        
+
         private String getConfigName() {
             return this.name().toLowerCase().replaceAll("_", "-");
         }
@@ -235,7 +233,7 @@ public class LangManager {
 
         return YamlConfiguration.loadConfiguration(targetLangFile);
     }
-    
+
     /**
      * Gets a formatted and replaced message
      * 
@@ -254,7 +252,7 @@ public class LangManager {
      */
     public static void sendMessage(Player player, Lang messageType, Object... replacements) {
         if (!messagesEnabled) return;
-        
+
         String message = messageType.get(replacements);
 
         if (message.length() == 0) return;
diff --git a/src/com/esophose/playerparticles/manager/PermissionManager.java b/src/com/esophose/playerparticles/manager/PermissionManager.java
index 08b3539..e98f05b 100644
--- a/src/com/esophose/playerparticles/manager/PermissionManager.java
+++ b/src/com/esophose/playerparticles/manager/PermissionManager.java
@@ -70,16 +70,6 @@ public class PermissionManager {
         return list;
     }
 
-    /**
-     * Checks if a player has permission to execute commands for other players
-     * 
-     * @param player The player to check the permission for
-     * @return True if the player has permission
-     */
-    public static boolean canUseForceReset(Player player) {
-        return player.hasPermission("playerparticles.forcereset");
-    }
-
     /**
      * Checks if a player has permission to created fixed effects
      * 
diff --git a/src/com/esophose/playerparticles/styles/ParticleStyleBlockBreak.java b/src/com/esophose/playerparticles/styles/ParticleStyleBlockBreak.java
index d931465..48691b0 100644
--- a/src/com/esophose/playerparticles/styles/ParticleStyleBlockBreak.java
+++ b/src/com/esophose/playerparticles/styles/ParticleStyleBlockBreak.java
@@ -21,9 +21,11 @@ public class ParticleStyleBlockBreak implements ParticleStyle, Listener {
 
     public List<PParticle> getParticles(ParticlePair particle, Location location) {
         List<PParticle> particles = new ArrayList<PParticle>();
+        
+        location.add(0.5, 0.5, 0.5); // Center around the block
 
         for (int i = 0; i < 15; i++)
-            particles.add(new PParticle(location.add(0.5, 0.5, 0.5), 0.5F, 0.5F, 0.5F, 0.05F));
+            particles.add(new PParticle(location, 0.5F, 0.5F, 0.5F, 0.05F));
 
         return particles;
     }
diff --git a/src/com/esophose/playerparticles/styles/ParticleStyleBlockPlace.java b/src/com/esophose/playerparticles/styles/ParticleStyleBlockPlace.java
index 8911c4f..4595ec0 100644
--- a/src/com/esophose/playerparticles/styles/ParticleStyleBlockPlace.java
+++ b/src/com/esophose/playerparticles/styles/ParticleStyleBlockPlace.java
@@ -21,9 +21,11 @@ public class ParticleStyleBlockPlace implements ParticleStyle, Listener {
 
     public List<PParticle> getParticles(ParticlePair particle, Location location) {
         List<PParticle> particles = new ArrayList<PParticle>();
+        
+        location.add(0.5, 0.5, 0.5); // Center around the block
 
         for (int i = 0; i < 15; i++)
-            particles.add(new PParticle(location.add(0.5, 0.5, 0.5), 0.75F, 0.75F, 0.75F, 0.05F));
+            particles.add(new PParticle(location, 0.75F, 0.75F, 0.75F, 0.05F));
 
         return particles;
     }
diff --git a/src/config.yml b/src/config.yml
index e19daa7..0ebf03e 100644
--- a/src/config.yml
+++ b/src/config.yml
@@ -263,4 +263,4 @@ gui-icon:
     WINGS: ELYTRA
 
 # That's everything! You reached the end of the configuration.
-# Enjoy the plugin!
\ No newline at end of file
+# Enjoy the plugin!
diff --git a/src/lang/default.lang b/src/lang/default.lang
index d950581..f3e7ec5 100644
--- a/src/lang/default.lang
+++ b/src/lang/default.lang
@@ -45,6 +45,9 @@ command-description-group-remove: '&e/pp group remove <name> - Removes all parti
 # Add Command
 command-add-particle-applied: '&aA new particle has been applied with the effect &b{0}&a, style &b{1}&a, and data &b{2}&a!'
 
+# Data Command
+command-data-no-args: '&cMissing argument for effect. Command usage: &b/pp data <effect>'
+
 # Effects
 effect-no-permission: '&cYou do not have permission to use the effect &b{0} &c!'
 effect-invalid: '&cThe effect &b{0} &cdoes not exist. Use &b/pp effects &cfor a list of effects you can use.'
@@ -56,15 +59,15 @@ style-invalid: '&cThe style &b{0} &cdoes not exist. Use &b/pp styles &cfor a lis
 style-list: '&eYou can use the following styles: &b{0}'
 
 # Data
-data-usage-none: '&eYour current effect does not use any data!'
-data-usage-block: '&eThe effect &b{0} requires &bblock &edata! &bFormat: <blockName>'
-data-usage-item: '&eThe effect &b{0} requires &bitem &edata! &bFormat: <itemName>'
-data-usage-color: '&eThe effect &b{0} requires &bcolor &edata! &bFormat: [<0-255> <0-255> <0-255>]|[<rainbow>]'
-data-usage-note: '&eThe effect &b{0} requires &bnote &edata! &bFormat: <0-23>/<rainbow>'
+data-usage-none: '&eThe effect &b{0} &edoes not use any data!'
+data-usage-block: '&eThe effect &b{0} &erequires &bblock &edata! &bFormat: <blockName>'
+data-usage-item: '&eThe effect &b{0} &erequires &bitem &edata! &bFormat: <itemName>'
+data-usage-color: '&eThe effect &b{0} &erequires &bcolor &edata! &bFormat: <<0-255> <0-255> <0-255>>|<rainbow>'
+data-usage-note: '&eThe effect &b{0} &erequires &bnote &edata! &bFormat: <0-23>|<rainbow>'
 data-invalid-block: '&cThe &bblock &cdata you entered is invalid! &bFormat: <blockName>'
 data-invalid-item: '&cThe &bitem &cdata you entered is invalid! &bFormat: <itemName>'
-data-invalid-color: '&cThe &bcolor &cdata you entered is invalid! &bFormat: [<0-255> <0-255> <0-255>]|[<rainbow>]'
-data-invalid-note: '&cThe &bitem &cdata you entered is invalid! &bFormat: <0-23>/<rainbow>'
+data-invalid-color: '&cThe &bcolor &cdata you entered is invalid! &bFormat: <<0-255> <0-255> <0-255>>|<rainbow>'
+data-invalid-note: '&cThe &bnote &cdata you entered is invalid! &bFormat: <0-23>|<rainbow>'
 data-invalid-material-not-item: '&cThe &bitem &cmaterial &b{0} &cyou entered is not an item!'
 data-invalid-material-not-block: '&cThe &bblock &cmaterial &b{0} &cyou entered is not a block!'
 data-invalid-material-item: '&cThe &bitem &cmaterial &b{0} you entered does not exist!'
@@ -74,13 +77,8 @@ data-invalid-material-block: '&cThe &bblock &cmaterial &b{0} you entered does no
 disabled-worlds: '&eParticles are disabled in these worlds: &b{0}'
 disabled-worlds-none: '&eParticles are not disabled in any worlds!'
 
-# Alternate Player Command Execution
-execute-success: '&aSuccessfully executed &b{0} &aexecuted for &b{1}'
-execute-failed-not-found: '&cFailed to execute &b{0} &cfor &b{1}&c! Player not found!'
-execute-failed-no-permission: '&cFailed to execute &b{0} for &b{1}&c! You do not have permission!'
-
 # Reset
-reset-success: '&aAll your active particles have been removed.'
+reset-success: '&aRemoved &b{0} &aactive particles.'
 
 # Fixed Effects
 fixed-create-missing-args: '&cUnable to create fixed effect, you are missing &b{0} &crequired arguments!'
diff --git a/src/plugin.yml b/src/plugin.yml
index af5508c..58efa7d 100644
--- a/src/plugin.yml
+++ b/src/plugin.yml
@@ -7,4 +7,4 @@ author: Esophose
 website: https://dev.bukkit.org/projects/playerparticles
 commands:
     pp:
-        description: The main PlayerParticles command. By default, opens the GUI.
\ No newline at end of file
+        description: The main PlayerParticles command. By default, opens the GUI.