Dropped isMageCaster method from AutoUseTaskManager.

This commit is contained in:
MobiusDevelopment 2021-06-14 21:45:27 +00:00
parent 18ebd4f620
commit a81169820a
6 changed files with 6 additions and 103 deletions

View File

@ -24,7 +24,6 @@ import java.util.concurrent.ConcurrentHashMap;
import org.l2jmobius.Config;
import org.l2jmobius.commons.concurrent.ThreadPool;
import org.l2jmobius.gameserver.data.xml.ActionData;
import org.l2jmobius.gameserver.enums.Race;
import org.l2jmobius.gameserver.handler.IItemHandler;
import org.l2jmobius.gameserver.handler.IPlayerActionHandler;
import org.l2jmobius.gameserver.handler.ItemHandler;
@ -256,14 +255,7 @@ public class AutoUseTaskManager
if (!player.hasSkillReuse(skill.getReuseHashCode()) && skill.checkCondition(player, target, false))
{
if (isMageCaster(player))
{
player.useMagic(skill, null, true, false);
}
else
{
player.doCast(skill);
}
player.useMagic(skill, null, true, false);
}
}
@ -373,18 +365,6 @@ public class AutoUseTaskManager
stopAutoUseTask(player);
}
private boolean isMageCaster(PlayerInstance player)
{
// Iss classes considered fighters.
final int classId = player.getActiveClass();
if ((classId > 170) && (classId < 176))
{
return false;
}
return player.isMageClass() && (player.getRace() != Race.ORC);
}
public static AutoUseTaskManager getInstance()
{
return SingletonHolder.INSTANCE;

View File

@ -24,7 +24,6 @@ import java.util.concurrent.ConcurrentHashMap;
import org.l2jmobius.Config;
import org.l2jmobius.commons.concurrent.ThreadPool;
import org.l2jmobius.gameserver.data.xml.ActionData;
import org.l2jmobius.gameserver.enums.Race;
import org.l2jmobius.gameserver.handler.IItemHandler;
import org.l2jmobius.gameserver.handler.IPlayerActionHandler;
import org.l2jmobius.gameserver.handler.ItemHandler;
@ -256,14 +255,7 @@ public class AutoUseTaskManager
if (!player.hasSkillReuse(skill.getReuseHashCode()) && skill.checkCondition(player, target, false))
{
if (isMageCaster(player))
{
player.useMagic(skill, null, true, false);
}
else
{
player.doCast(skill);
}
player.useMagic(skill, null, true, false);
}
}
@ -373,18 +365,6 @@ public class AutoUseTaskManager
stopAutoUseTask(player);
}
private boolean isMageCaster(PlayerInstance player)
{
// Iss classes considered fighters.
final int classId = player.getActiveClass();
if ((classId > 170) && (classId < 176))
{
return false;
}
return player.isMageClass() && (player.getRace() != Race.ORC);
}
public static AutoUseTaskManager getInstance()
{
return SingletonHolder.INSTANCE;

View File

@ -24,7 +24,6 @@ import java.util.concurrent.ConcurrentHashMap;
import org.l2jmobius.Config;
import org.l2jmobius.commons.concurrent.ThreadPool;
import org.l2jmobius.gameserver.data.xml.ActionData;
import org.l2jmobius.gameserver.enums.Race;
import org.l2jmobius.gameserver.handler.IItemHandler;
import org.l2jmobius.gameserver.handler.IPlayerActionHandler;
import org.l2jmobius.gameserver.handler.ItemHandler;
@ -256,14 +255,7 @@ public class AutoUseTaskManager
if (!player.hasSkillReuse(skill.getReuseHashCode()) && skill.checkCondition(player, target, false))
{
if (isMageCaster(player))
{
player.useMagic(skill, null, true, false);
}
else
{
player.doCast(skill);
}
player.useMagic(skill, null, true, false);
}
}
@ -373,18 +365,6 @@ public class AutoUseTaskManager
stopAutoUseTask(player);
}
private boolean isMageCaster(PlayerInstance player)
{
// Iss classes considered fighters.
final int classId = player.getActiveClass();
if ((classId > 170) && (classId < 176))
{
return false;
}
return player.isMageClass() && (player.getRace() != Race.ORC);
}
public static AutoUseTaskManager getInstance()
{
return SingletonHolder.INSTANCE;

View File

@ -24,7 +24,6 @@ import java.util.concurrent.ConcurrentHashMap;
import org.l2jmobius.Config;
import org.l2jmobius.commons.concurrent.ThreadPool;
import org.l2jmobius.gameserver.data.xml.ActionData;
import org.l2jmobius.gameserver.enums.Race;
import org.l2jmobius.gameserver.handler.IItemHandler;
import org.l2jmobius.gameserver.handler.IPlayerActionHandler;
import org.l2jmobius.gameserver.handler.ItemHandler;
@ -256,14 +255,7 @@ public class AutoUseTaskManager
if (!player.hasSkillReuse(skill.getReuseHashCode()) && skill.checkCondition(player, target, false))
{
if (isMageCaster(player))
{
player.useMagic(skill, null, true, false);
}
else
{
player.doCast(skill);
}
player.useMagic(skill, null, true, false);
}
}
@ -373,11 +365,6 @@ public class AutoUseTaskManager
stopAutoUseTask(player);
}
private boolean isMageCaster(PlayerInstance player)
{
return player.isMageClass() && (player.getRace() != Race.ORC);
}
public static AutoUseTaskManager getInstance()
{
return SingletonHolder.INSTANCE;

View File

@ -255,14 +255,7 @@ public class AutoUseTaskManager
if (!player.hasSkillReuse(skill.getReuseHashCode()) && skill.checkCondition(player, target, false))
{
if (isMageCaster(player))
{
player.useMagic(skill, null, true, false);
}
else
{
player.doCast(skill);
}
player.useMagic(skill, null, true, false);
}
}
@ -372,11 +365,6 @@ public class AutoUseTaskManager
stopAutoUseTask(player);
}
private boolean isMageCaster(PlayerInstance player)
{
return player.isMageClass() /* && (player.getRace() != Race.ORC) - On Essence orc mages have caster skills. */;
}
public static AutoUseTaskManager getInstance()
{
return SingletonHolder.INSTANCE;

View File

@ -255,14 +255,7 @@ public class AutoUseTaskManager
if (!player.hasSkillReuse(skill.getReuseHashCode()) && skill.checkCondition(player, target, false))
{
if (isMageCaster(player))
{
player.useMagic(skill, null, true, false);
}
else
{
player.doCast(skill);
}
player.useMagic(skill, null, true, false);
}
}
@ -372,11 +365,6 @@ public class AutoUseTaskManager
stopAutoUseTask(player);
}
private boolean isMageCaster(PlayerInstance player)
{
return player.isMageClass() /* && (player.getRace() != Race.ORC) - On Essence orc mages have caster skills. */;
}
public static AutoUseTaskManager getInstance()
{
return SingletonHolder.INSTANCE;