Attempts to fix issue #mmocore-0417. Adds limit on amount of skills that can be bound.

This commit is contained in:
Ethan 2021-02-10 18:22:53 -05:00
parent 76af128d8f
commit 4ad5bfb933
3 changed files with 5 additions and 6 deletions

View File

@ -66,7 +66,7 @@ public class PlayerData extends OfflinePlayerData {
private final List<UUID> friends = new ArrayList<>();
private final Set<String> waypoints = new HashSet<>();
private final Map<String, Integer> skills = new HashMap<>();
private final List<SkillInfo> boundSkills = new ArrayList<>();
private final List<SkillInfo> boundSkills = new ArrayList<>(5);
private final PlayerProfessions collectSkills = new PlayerProfessions(this);
private final PlayerSkillData skillData = new PlayerSkillData(this);
private final PlayerAttributes attributes = new PlayerAttributes(this);
@ -688,14 +688,12 @@ public class PlayerData extends OfflinePlayerData {
}
public SkillInfo getBoundSkill(int slot) {
slot = Math.max(0, slot);
return slot >= boundSkills.size() ? null : boundSkills.get(slot);
}
public void setBoundSkill(int slot, SkillInfo skill) {
if (boundSkills.size() < 6)
boundSkills.add(skill);
else
boundSkills.set(slot, skill);
boundSkills.set(Math.min(5, Math.max(0, slot)), skill);
}
public void unbindSkill(int slot) {

View File

@ -7,6 +7,7 @@ import java.util.Map;
import java.util.UUID;
import java.util.function.Consumer;
import net.Indyuce.mmocore.api.experience.EXPSource;
import org.bukkit.entity.Player;
import net.Indyuce.mmocore.MMOCore;

View File

@ -130,7 +130,7 @@ public abstract class Skill {
}
/*
* not overriden for passive skills therefore not abstract.
* not overridden for passive skills therefore not abstract.
*/
public SkillResult whenCast(PlayerData data, SkillInfo skill) {
return new SkillResult(data, skill);