MagicSkill packet related cleanups.

This commit is contained in:
MobiusDevelopment
2022-12-24 22:03:24 +00:00
parent 4632ecc3a8
commit 0ab71457ca
161 changed files with 1316 additions and 1658 deletions

View File

@@ -4108,7 +4108,7 @@ public abstract class Creature extends WorldObject implements ISkillsHolder, IDe
final SkillCaster skillCaster = getSkillCaster(SkillCaster::isAnyNormalType);
if ((skillCaster != null) && skillCaster.getSkill().isMagic())
{
// Abort the cast of the Creature and send Server->Client MagicSkillCanceld/ActionFailed packet.
// Abort the cast of the Creature and send Server->Client MagicSkillCanceled/ActionFailed packet.
skillCaster.stopCasting(true);
if (isPlayer())

View File

@@ -75,7 +75,7 @@ import org.l2jmobius.gameserver.network.serverpackets.ActionFailed;
import org.l2jmobius.gameserver.network.serverpackets.ExMagicSkillUseGround;
import org.l2jmobius.gameserver.network.serverpackets.ExRotation;
import org.l2jmobius.gameserver.network.serverpackets.FlyToLocation;
import org.l2jmobius.gameserver.network.serverpackets.MagicSkillCanceld;
import org.l2jmobius.gameserver.network.serverpackets.MagicSkillCanceled;
import org.l2jmobius.gameserver.network.serverpackets.MagicSkillLaunched;
import org.l2jmobius.gameserver.network.serverpackets.MagicSkillUse;
import org.l2jmobius.gameserver.network.serverpackets.MoveToPawn;
@@ -797,7 +797,7 @@ public class SkillCaster implements Runnable
// If aborted, broadcast casting aborted.
if (aborted)
{
caster.broadcastPacket(new MagicSkillCanceld(caster.getObjectId())); // broadcast packet to stop animations client-side
caster.broadcastPacket(new MagicSkillCanceled(caster.getObjectId())); // broadcast packet to stop animations client-side
caster.sendPacket(ActionFailed.get(_castingType)); // send an "action failed" packet to the caster
}

View File

@@ -18,11 +18,11 @@ package org.l2jmobius.gameserver.network.serverpackets;
import org.l2jmobius.gameserver.network.ServerPackets;
public class MagicSkillCanceld extends ServerPacket
public class MagicSkillCanceled extends ServerPacket
{
private final int _objectId;
public MagicSkillCanceld(int objectId)
public MagicSkillCanceled(int objectId)
{
_objectId = objectId;
}

View File

@@ -16,10 +16,6 @@
*/
package org.l2jmobius.gameserver.network.serverpackets;
import java.util.Arrays;
import java.util.Collections;
import java.util.List;
import org.l2jmobius.gameserver.model.Location;
import org.l2jmobius.gameserver.model.WorldObject;
import org.l2jmobius.gameserver.model.actor.Creature;
@@ -42,11 +38,11 @@ public class MagicSkillUse extends ServerPacket
private final Creature _creature;
private final WorldObject _target;
private final boolean _isGroundTargetSkill;
private final List<Location> _groundLocations;
private final Location _groundLocation;
public MagicSkillUse(Creature creature, WorldObject target, int skillId, int skillLevel, int hitTime, int reuseDelay, int reuseGroup, int actionId, SkillCastingType castingType, boolean isGroundTargetSkill)
{
super(84);
super(83);
_creature = creature;
_target = target;
_skillId = skillId;
@@ -57,8 +53,7 @@ public class MagicSkillUse extends ServerPacket
_actionId = actionId;
_castingType = castingType;
_isGroundTargetSkill = isGroundTargetSkill;
final Location skillWorldPos = creature.isPlayer() ? creature.getActingPlayer().getCurrentSkillWorldPosition() : null;
_groundLocations = skillWorldPos != null ? Arrays.asList(skillWorldPos) : Collections.emptyList();
_groundLocation = creature.isPlayer() ? creature.getActingPlayer().getCurrentSkillWorldPosition() : null;
}
public MagicSkillUse(Creature creature, WorldObject target, int skillId, int skillLevel, int hitTime, int reuseDelay, int reuseGroup, int actionId, SkillCastingType castingType)
@@ -91,20 +86,24 @@ public class MagicSkillUse extends ServerPacket
writeInt(_creature.getX());
writeInt(_creature.getY());
writeInt(_creature.getZ());
writeShort(_isGroundTargetSkill ? 0x10000 : 0);
writeShort(_groundLocations.size());
for (Location location : _groundLocations)
writeShort(_isGroundTargetSkill ? 65535 : 0);
if (_groundLocation == null)
{
writeInt(location.getX());
writeInt(location.getY());
writeInt(location.getZ());
writeShort(0);
}
else
{
writeShort(1);
writeInt(_groundLocation.getX());
writeInt(_groundLocation.getY());
writeInt(_groundLocation.getZ());
}
writeInt(_target.getX());
writeInt(_target.getY());
writeInt(_target.getZ());
writeInt(_actionId >= 0); // 1 when ID from RequestActionUse is used
writeInt(_actionId >= 0 ? _actionId : 0); // ID from RequestActionUse. Used to set cooldown on summon skills.
if (_groundLocations.isEmpty())
if (_groundLocation == null)
{
writeInt(-1); // 306
}