diff --git a/src/fr/neatmonster/nocheatplus/actions/types/ActionWithParameters.java b/src/fr/neatmonster/nocheatplus/actions/types/ActionWithParameters.java index a9eeb378..51ce7c53 100644 --- a/src/fr/neatmonster/nocheatplus/actions/types/ActionWithParameters.java +++ b/src/fr/neatmonster/nocheatplus/actions/types/ActionWithParameters.java @@ -52,7 +52,7 @@ public abstract class ActionWithParameters extends Action { } /** - * Get a string with all the wildcards replaced with dataFactory from LogData. + * Get a string with all the wildcards replaced with data from LogData. * * @param check * the check diff --git a/src/fr/neatmonster/nocheatplus/actions/types/CommandAction.java b/src/fr/neatmonster/nocheatplus/actions/types/CommandAction.java index c029128e..7fbf1d35 100644 --- a/src/fr/neatmonster/nocheatplus/actions/types/CommandAction.java +++ b/src/fr/neatmonster/nocheatplus/actions/types/CommandAction.java @@ -47,7 +47,7 @@ public class CommandAction extends ActionWithParameters { * that can be directly used as a command in the console. * * @param check - * The check that is used to fill in missing dataFactory. + * The check that is used to fill in missing . * @param violationData * the violation data * @return The complete, ready to use, command. @@ -57,7 +57,7 @@ public class CommandAction extends ActionWithParameters { } /** - * Convert the commands dataFactory into a string that can be used in the configuration files. + * Convert the commands into a string that can be used in the configuration files. * * @return the string */ diff --git a/src/fr/neatmonster/nocheatplus/actions/types/LogAction.java b/src/fr/neatmonster/nocheatplus/actions/types/LogAction.java index 1edf2aa0..fd3b4a06 100644 --- a/src/fr/neatmonster/nocheatplus/actions/types/LogAction.java +++ b/src/fr/neatmonster/nocheatplus/actions/types/LogAction.java @@ -54,7 +54,7 @@ public class LogAction extends ActionWithParameters { } /** - * Parse the final log message out of various dataFactory from the player and check that triggered the action. + * Parse the final log message out of various data from the player and check that triggered the action. * * @param player * The player that is used as a source for the log message. diff --git a/src/fr/neatmonster/nocheatplus/checks/ViolationData.java b/src/fr/neatmonster/nocheatplus/checks/ViolationData.java index 7cc9c00a..a61ba08b 100644 --- a/src/fr/neatmonster/nocheatplus/checks/ViolationData.java +++ b/src/fr/neatmonster/nocheatplus/checks/ViolationData.java @@ -14,7 +14,7 @@ import fr.neatmonster.nocheatplus.actions.types.ActionList; * MMMMMMMMMMM MMMMMMMMMMM */ /** - * Violation specific dataFactory, for executing actions.
+ * Violation specific data, for executing actions.
* This is meant to capture a violation incident in a potentially thread safe way. * * @author asofold diff --git a/src/fr/neatmonster/nocheatplus/checks/blockinteract/BlockInteractData.java b/src/fr/neatmonster/nocheatplus/checks/blockinteract/BlockInteractData.java index 65eac7f1..5d62e8e0 100644 --- a/src/fr/neatmonster/nocheatplus/checks/blockinteract/BlockInteractData.java +++ b/src/fr/neatmonster/nocheatplus/checks/blockinteract/BlockInteractData.java @@ -26,7 +26,7 @@ import fr.neatmonster.nocheatplus.checks.CheckDataFactory; * MMMMMMMMMMM */ /** - * Player specific dataFactory for the block interact checks. + * Player specific data for the block interact checks. */ public class BlockInteractData implements CheckData { @@ -38,15 +38,15 @@ public class BlockInteractData implements CheckData { } }; - /** The map containing the dataFactory per players. */ + /** The map containing the data per players. */ private static Map playersMap = new HashMap(); /** - * Gets the dataFactory of a specified player. + * Gets the data of a specified player. * * @param player * the player - * @return the dataFactory + * @return the data */ public static BlockInteractData getData(final Player player) { if (!playersMap.containsKey(player.getName())) diff --git a/src/fr/neatmonster/nocheatplus/checks/blockplace/BlockPlaceData.java b/src/fr/neatmonster/nocheatplus/checks/blockplace/BlockPlaceData.java index 8c342bee..e74757b9 100644 --- a/src/fr/neatmonster/nocheatplus/checks/blockplace/BlockPlaceData.java +++ b/src/fr/neatmonster/nocheatplus/checks/blockplace/BlockPlaceData.java @@ -38,15 +38,15 @@ public class BlockPlaceData implements CheckData { } }; - /** The map containing the dataFactory per players. */ + /** The map containing the data per players. */ private static Map playersMap = new HashMap(); /** - * Gets the dataFactory of a specified player. + * Gets the data of a specified player. * * @param player * the player - * @return the dataFactory + * @return the data */ public static BlockPlaceData getData(final Player player) { if (!playersMap.containsKey(player.getName())) diff --git a/src/fr/neatmonster/nocheatplus/checks/chat/ChatData.java b/src/fr/neatmonster/nocheatplus/checks/chat/ChatData.java index 81f23944..d3b5dfe6 100644 --- a/src/fr/neatmonster/nocheatplus/checks/chat/ChatData.java +++ b/src/fr/neatmonster/nocheatplus/checks/chat/ChatData.java @@ -19,7 +19,7 @@ import fr.neatmonster.nocheatplus.checks.CheckDataFactory; * MMMMMMMMMMM MMMMMMMMMMM */ /** - * Player specific dataFactory for the chat checks. + * Player specific data for the chat checks. */ public class ChatData implements CheckData { @@ -31,15 +31,15 @@ public class ChatData implements CheckData { } }; - /** The map containing the dataFactory per players. */ + /** The map containing the data per players. */ private static Map playersMap = new HashMap(); /** - * Gets the dataFactory of a specified player. + * Gets the data of a specified player. * * @param player * the player - * @return the dataFactory + * @return the data */ public synchronized static ChatData getData(final Player player) { if (!playersMap.containsKey(player.getName())) @@ -67,7 +67,7 @@ public class ChatData implements CheckData { public long noPwnageReloginWarningTime; /** - * Clear the dataFactory of the no pwnage check. + * Clear the data of the no pwnage check. */ public synchronized void clearNoPwnageData() { // TODO: re-think this sync [keep related to ChatData/NoPwnage/Color used lock.] diff --git a/src/fr/neatmonster/nocheatplus/checks/chat/NoPwnage.java b/src/fr/neatmonster/nocheatplus/checks/chat/NoPwnage.java index eed39c24..1268ccf5 100644 --- a/src/fr/neatmonster/nocheatplus/checks/chat/NoPwnage.java +++ b/src/fr/neatmonster/nocheatplus/checks/chat/NoPwnage.java @@ -179,7 +179,7 @@ public class NoPwnage extends Check { if (cc.noPwnageCaptchaCheck && data.noPwnageHasStartedCaptcha) { // Correct answer to the captcha? if (message.equals(data.noPwnageGeneratedCaptcha)) { - // Yes, clear his dataFactory and do not worry anymore about him. + // Yes, clear his data and do not worry anymore about him. data.clearNoPwnageData(); data.noPwnageHasFilledCaptcha = true; player.sendMessage(replaceColors(cc.noPwnageCaptchaSuccess)); @@ -288,7 +288,7 @@ public class NoPwnage extends Check { cancel = executeActionsThreadSafe(player, data.noPwnageVL, cc.noPwnageActions, isMainThread); } - // Store the message and some other dataFactory. + // Store the message and some other data. data.noPwnageLastMessage = message; data.noPwnageLastMessageTime = now; lastGlobalMessage = message; @@ -330,7 +330,7 @@ public class NoPwnage extends Check { cancel = executeActionsThreadSafe(player, data.noPwnageVL, cc.noPwnageActions, true); } - // Store his location and some other dataFactory. + // Store his location and some other data. data.noPwnageLastLocation = player.getLocation(); data.noPwnageJoinTime = now; diff --git a/src/fr/neatmonster/nocheatplus/checks/fight/Angle.java b/src/fr/neatmonster/nocheatplus/checks/fight/Angle.java index 01937086..880067ff 100644 --- a/src/fr/neatmonster/nocheatplus/checks/fight/Angle.java +++ b/src/fr/neatmonster/nocheatplus/checks/fight/Angle.java @@ -54,7 +54,7 @@ public class Angle extends Check { // Add the new location to the map. data.angleHits.put(System.currentTimeMillis(), player.getLocation()); - // Not enough dataFactory to calculate deltas. + // Not enough data to calculate deltas. if (data.angleHits.size() < 2) return false; diff --git a/src/fr/neatmonster/nocheatplus/checks/fight/FightData.java b/src/fr/neatmonster/nocheatplus/checks/fight/FightData.java index 3ff45f2e..53dae9a1 100644 --- a/src/fr/neatmonster/nocheatplus/checks/fight/FightData.java +++ b/src/fr/neatmonster/nocheatplus/checks/fight/FightData.java @@ -21,7 +21,7 @@ import fr.neatmonster.nocheatplus.checks.CheckDataFactory; * d8888P */ /** - * Player specific dataFactory for the fight checks. + * Player specific data for the fight checks. */ public class FightData implements CheckData { @@ -33,15 +33,15 @@ public class FightData implements CheckData { } }; - /** The map containing the dataFactory per players. */ + /** The map containing the data per players. */ private static Map playersMap = new HashMap(); /** - * Gets the dataFactory of a specified player. + * Gets the data of a specified player. * * @param player * the player - * @return the dataFactory + * @return the data */ public static FightData getData(final Player player) { if (!playersMap.containsKey(player.getName())) diff --git a/src/fr/neatmonster/nocheatplus/checks/inventory/InventoryData.java b/src/fr/neatmonster/nocheatplus/checks/inventory/InventoryData.java index 08bd2d92..164f5306 100644 --- a/src/fr/neatmonster/nocheatplus/checks/inventory/InventoryData.java +++ b/src/fr/neatmonster/nocheatplus/checks/inventory/InventoryData.java @@ -32,15 +32,15 @@ public class InventoryData implements CheckData { } }; - /** The map containing the dataFactory per players. */ + /** The map containing the data per players. */ private static Map playersMap = new HashMap(); /** - * Gets the dataFactory of a specified player. + * Gets the data of a specified player. * * @param player * the player - * @return the dataFactory + * @return the data */ public static InventoryData getData(final Player player) { if (!playersMap.containsKey(player.getName())) diff --git a/src/fr/neatmonster/nocheatplus/checks/inventory/InventoryListener.java b/src/fr/neatmonster/nocheatplus/checks/inventory/InventoryListener.java index 08701ca2..db2061ca 100644 --- a/src/fr/neatmonster/nocheatplus/checks/inventory/InventoryListener.java +++ b/src/fr/neatmonster/nocheatplus/checks/inventory/InventoryListener.java @@ -159,7 +159,7 @@ public class InventoryListener implements Listener { data.instantEatFood = event.getItem().getType(); data.instantEatLastTime = System.currentTimeMillis(); } else { - // Nothing that we are interested in, reset dataFactory. + // Nothing that we are interested in, reset data. data.instantBowLastTime = 0; data.instantEatLastTime = 0; data.instantEatFood = null; diff --git a/src/fr/neatmonster/nocheatplus/checks/moving/MovingData.java b/src/fr/neatmonster/nocheatplus/checks/moving/MovingData.java index 6a1248ba..b64546de 100644 --- a/src/fr/neatmonster/nocheatplus/checks/moving/MovingData.java +++ b/src/fr/neatmonster/nocheatplus/checks/moving/MovingData.java @@ -20,7 +20,7 @@ import fr.neatmonster.nocheatplus.checks.CheckDataFactory; * d8888P */ /** - * Player specific dataFactory for the moving checks. + * Player specific data for the moving checks. */ public class MovingData implements CheckData { @@ -32,15 +32,15 @@ public class MovingData implements CheckData { } }; - /** The map containing the dataFactory per players. */ + /** The map containing the data per players. */ private static Map playersMap = new HashMap(); /** - * Gets the dataFactory of a specified player. + * Gets the data of a specified player. * * @param player * the player - * @return the dataFactory + * @return the data */ public static MovingData getData(final Player player) { if (!playersMap.containsKey(player.getName())) @@ -103,7 +103,7 @@ public class MovingData implements CheckData { public Location teleported; /** - * Clear the dataFactory of the fly checks. + * Clear the data of the fly checks. */ public void clearFlyData() { bunnyhopDelay = 0; @@ -114,7 +114,7 @@ public class MovingData implements CheckData { } /** - * Clear the dataFactory of the more packets checks. + * Clear the data of the more packets checks. */ public void clearMorePacketsData() { morePacketsSetback = null; diff --git a/src/fr/neatmonster/nocheatplus/checks/moving/MovingListener.java b/src/fr/neatmonster/nocheatplus/checks/moving/MovingListener.java index a1162928..8abc3d1d 100644 --- a/src/fr/neatmonster/nocheatplus/checks/moving/MovingListener.java +++ b/src/fr/neatmonster/nocheatplus/checks/moving/MovingListener.java @@ -177,7 +177,7 @@ public class MovingListener implements Listener { } /** - * Just for security, if a player switches between worlds, reset the fly and more packets checks dataFactory, + * Just for security, if a player switches between worlds, reset the fly and more packets checks data, * because it is definitely invalid now. * * @param event @@ -278,7 +278,7 @@ public class MovingListener implements Listener { final Player player = event.getPlayer(); // Don't care for movements that are very high distance, to another world (such that it is very likely the event - // dataFactory was modified by another plugin before we got it) or if the player is inside a vehicle. + // data was modified by another plugin before we got it) or if the player is inside a vehicle. if (!event.getFrom().getWorld().equals(event.getTo().getWorld()) || event.getFrom().distanceSquared(event.getTo()) > 400D || player.isInsideVehicle()) return; @@ -286,7 +286,7 @@ public class MovingListener implements Listener { final MovingData data = MovingData.getData(player); // Just try to estimate velocities over time. Not very precise, but works good enough most of the time. Do - // general dataFactory modifications one for each event. + // general data modifications one for each event. if (data.horizontalVelocityCounter > 0D) data.horizontalVelocityCounter--; else if (data.horizontalFreedom > 0.001D) @@ -324,14 +324,14 @@ public class MovingListener implements Listener { if (newTo == null && noFall.isEnabled(player)) noFall.check(player, from, to); } else - // He isn't handled by any fly check, clear his dataFactory. + // He isn't handled by any fly check, clear his data. data.clearFlyData(); if (newTo == null && morePackets.isEnabled(player)) // If he hasn't been stopped by any other check and is handled by the more packets check, execute it. newTo = morePackets.check(player, from, to); else - // Otherwise we need to clear his dataFactory. + // Otherwise we need to clear his data. data.clearMorePacketsData(); // Did one of the checks decide we need a new "to"-location? @@ -389,9 +389,9 @@ public class MovingListener implements Listener { /** * If a player gets teleported, it may have two reasons. Either it was NoCheat or another plugin. If it was - * NoCheatPlus, the target location should match the "dataFactory.teleportedTo" value. + * NoCheatPlus, the target location should match the "data.teleportedTo" value. * - * On teleports, reset some movement related dataFactory that gets invalid. + * On teleports, reset some movement related data that gets invalid. * * @param event * the event @@ -414,12 +414,12 @@ public class MovingListener implements Listener { if (data.teleported != null && data.teleported.equals(event.getTo())) event.setCancelled(false); else - // Only if it wasn't NoCheatPlus, drop dataFactory from more packets check. If it was NoCheatPlus, we don't + // Only if it wasn't NoCheatPlus, drop data from more packets check. If it was NoCheatPlus, we don't // want - // players to exploit the fly check teleporting to get rid of the "morepackets" dataFactory. + // players to exploit the fly check teleporting to get rid of the "morepackets" data. data.clearMorePacketsData(); - // Always drop dataFactory from fly checks, as it always loses its validity after teleports. Always! + // Always drop data from fly checks, as it always loses its validity after teleports. Always! data.teleported = null; data.clearFlyData(); } @@ -478,7 +478,7 @@ public class MovingListener implements Listener { * \_/ \___|_| |_|_|\___|_|\___| |_| |_|\___/ \_/ \___| */ // Don't care if a player isn't inside the vehicle, for movements that are very high distance or to another - // world (such that it is very likely the event dataFactory was modified by another plugin before we got it). + // world (such that it is very likely the event data was modified by another plugin before we got it). if (event.getVehicle().getPassenger() == null || !(event.getVehicle().getPassenger() instanceof Player) || !event.getFrom().getWorld().equals(event.getTo().getWorld()) || event.getFrom().distanceSquared(event.getTo()) > 400D) @@ -492,7 +492,7 @@ public class MovingListener implements Listener { // If the player is handled by the more packets vehicle check, execute it. newTo = morePacketsVehicle.check(player, event.getFrom(), event.getTo()); else - // Otherwise we need to clear his dataFactory. + // Otherwise we need to clear his data. MovingData.getData(player).clearMorePacketsData(); // Did one of the checks decide we need a new "to"-location? diff --git a/src/fr/neatmonster/nocheatplus/config/ConfigFile.java b/src/fr/neatmonster/nocheatplus/config/ConfigFile.java index 707152b7..dfcfeec5 100644 --- a/src/fr/neatmonster/nocheatplus/config/ConfigFile.java +++ b/src/fr/neatmonster/nocheatplus/config/ConfigFile.java @@ -43,7 +43,7 @@ public class ConfigFile extends YamlConfiguration { } /** - * Do this after reading new dataFactory. + * Do this after reading new data. */ public void regenerateActionLists() { factory = new ActionFactory(((MemorySection) this.get(ConfPaths.STRINGS)).getValues(false)); diff --git a/src/fr/neatmonster/nocheatplus/players/ExecutionHistory.java b/src/fr/neatmonster/nocheatplus/players/ExecutionHistory.java index 4137ed41..d0244f52 100644 --- a/src/fr/neatmonster/nocheatplus/players/ExecutionHistory.java +++ b/src/fr/neatmonster/nocheatplus/players/ExecutionHistory.java @@ -130,7 +130,7 @@ public class ExecutionHistory { } } - /** Store dataFactory between events (time + action + action-counter). **/ + /** Store data between events (time + action + action-counter). **/ private final Map> executionHistories; /**