Removal of unnecessary logs.

Contributed by Sahar.
This commit is contained in:
MobiusDevelopment
2019-10-10 22:10:51 +00:00
parent beff746963
commit 92d7275afa
91 changed files with 16 additions and 183 deletions
@@ -177,7 +177,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -190,31 +189,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -222,7 +216,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -788,7 +788,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -129,7 +129,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -177,7 +177,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -190,31 +189,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -222,7 +216,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -784,7 +784,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -129,7 +129,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -177,7 +177,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -190,31 +189,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -222,7 +216,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -784,7 +784,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -129,7 +129,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -179,7 +179,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -192,31 +191,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -224,7 +218,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -784,7 +784,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -129,7 +129,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -179,7 +179,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -192,31 +191,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -224,7 +218,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -784,7 +784,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -128,7 +128,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -179,7 +179,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -192,31 +191,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -224,7 +218,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -784,7 +784,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -128,7 +128,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -179,7 +179,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -192,31 +191,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -224,7 +218,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -784,7 +784,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -128,7 +128,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -179,7 +179,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -192,31 +191,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -224,7 +218,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -784,7 +784,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -128,7 +128,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -179,7 +179,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -192,31 +191,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -224,7 +218,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -741,7 +741,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -501,7 +501,6 @@ public class VillageMasterInstance extends NpcInstance
*/ */
if (!player.getFloodProtectors().getSubclass().tryPerformAction("add subclass")) if (!player.getFloodProtectors().getSubclass().tryPerformAction("add subclass"))
{ {
LOGGER.warning(VillageMasterInstance.class.getName() + ": Player " + player.getName() + " has performed a subclass change too fast");
return; return;
} }
@@ -569,7 +568,6 @@ public class VillageMasterInstance extends NpcInstance
*/ */
if (!player.getFloodProtectors().getSubclass().tryPerformAction("change class")) if (!player.getFloodProtectors().getSubclass().tryPerformAction("change class"))
{ {
LOGGER.warning(VillageMasterInstance.class.getName() + ": Player " + player.getName() + " has performed a subclass change too fast");
return; return;
} }
@@ -660,7 +658,6 @@ public class VillageMasterInstance extends NpcInstance
*/ */
if (!player.getFloodProtectors().getSubclass().tryPerformAction("change class")) if (!player.getFloodProtectors().getSubclass().tryPerformAction("change class"))
{ {
LOGGER.warning(VillageMasterInstance.class.getName() + ": Player " + player.getName() + " has performed a subclass change too fast");
return; return;
} }
@@ -19,6 +19,7 @@ package org.l2jmobius.gameserver.model.skills;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.List; import java.util.List;
import java.util.concurrent.ScheduledFuture; import java.util.concurrent.ScheduledFuture;
import java.util.logging.Level;
import java.util.logging.Logger; import java.util.logging.Logger;
import org.l2jmobius.commons.concurrent.ThreadPool; import org.l2jmobius.commons.concurrent.ThreadPool;
@@ -226,7 +227,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.warning("Error while channelizing skill: " + _skill + " channelizer: " + _channelizer + " channelized: " + _channelized + "; " + e.getMessage()); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + _skill + " channelizer: " + _channelizer + " channelized: " + _channelized + "; ", e);
} }
} }
} }
@@ -119,7 +119,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
else if (prevSkillLevel != (_level - 1)) else if (prevSkillLevel != (_level - 1))
@@ -103,7 +103,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -88,7 +88,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -177,7 +177,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -190,31 +189,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -222,7 +216,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -784,7 +784,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -129,7 +129,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -179,7 +179,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -192,31 +191,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -224,7 +218,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -784,7 +784,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -129,7 +129,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -179,7 +179,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -192,31 +191,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -224,7 +218,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -784,7 +784,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -129,7 +129,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -179,7 +179,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -192,31 +191,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -224,7 +218,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -806,7 +806,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -129,7 +129,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -179,7 +179,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -192,31 +191,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -224,7 +218,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -806,7 +806,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -129,7 +129,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }
@@ -179,7 +179,6 @@ public class TradeList
{ {
if (_locked) if (_locked)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to modify locked TradeList!");
return null; return null;
} }
@@ -192,31 +191,26 @@ public class TradeList
final ItemInstance item = (ItemInstance) o; final ItemInstance item = (ItemInstance) o;
if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem()) if (!(item.isTradeable() || (_owner.isGM() && Config.GM_TRADE_RESTRICTED_ITEMS)) || item.isQuestItem())
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add a restricted item!");
return null; return null;
} }
if (!_owner.getInventory().canManipulateWithItemId(item.getId())) if (!_owner.getInventory().canManipulateWithItemId(item.getId()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that can't manipualte!");
return null; return null;
} }
if ((count <= 0) || (count > item.getCount())) if ((count <= 0) || (count > item.getCount()))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item with invalid item count!");
return null; return null;
} }
if (!item.isStackable() && (count > 1)) if (!item.isStackable() && (count > 1))
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add non-stackable item to TradeList with count > 1!");
return null; return null;
} }
if ((MAX_ADENA / count) < price) if ((MAX_ADENA / count) < price)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to overflow adena !");
return null; return null;
} }
@@ -224,7 +218,6 @@ public class TradeList
{ {
if (checkitem.getObjectId() == objectId) if (checkitem.getObjectId() == objectId)
{ {
LOGGER.warning(_owner.getName() + ": Attempt to add an item that is already present!");
return null; return null;
} }
} }
@@ -806,7 +806,6 @@ public class Attackable extends Npc
final AggroInfo ai = _aggroList.get(target); final AggroInfo ai = _aggroList.get(target);
if (ai == null) if (ai == null)
{ {
LOGGER.info("Target " + target + " not present in aggro list of " + this);
return; return;
} }
@@ -231,7 +231,7 @@ public class SkillChannelizer implements Runnable
} }
catch (Exception e) catch (Exception e)
{ {
LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized, e); LOGGER.log(Level.WARNING, "Error while channelizing skill: " + skill + " channelizer: " + _channelizer + " channelized: " + channelized + "; ", e);
} }
} }
} }
@@ -120,7 +120,6 @@ public class RequestAcquireSkill implements IClientIncomingPacket
{ {
if (prevSkillLevel == _level) if (prevSkillLevel == _level)
{ {
LOGGER.warning("Player " + player.getName() + " is trying to learn a skill that already knows, Id: " + _id + " level: " + _level + "!");
return; return;
} }
@@ -108,7 +108,6 @@ public class RequestBypassToServer implements IClientIncomingPacket
bypassOriginId = player.validateHtmlAction(_command); bypassOriginId = player.validateHtmlAction(_command);
if (bypassOriginId == -1) if (bypassOriginId == -1)
{ {
LOGGER.warning("Player " + player.getName() + " sent non cached bypass: '" + _command + "'");
return; return;
} }
@@ -92,7 +92,6 @@ public class RequestRestartPoint implements IClientIncomingPacket
} }
else if (!player.isDead()) else if (!player.isDead())
{ {
LOGGER.warning("Living player [" + player.getName() + "] called RestartPointPacket! Ban this player!");
return; return;
} }