Upstream/mmocore
1
0
Fork 1
mirror of https://gitlab.com/phoenix-dvpmt/mmocore.git synced 2024-11-28 00:55:29 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

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

Browse Source
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
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/main/java/net/Indyuce/mmocore/api/player/PlayerData.java
View File

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

1
src/main/java/net/Indyuce/mmocore/api/player/social/Party.java
View File

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

2
src/main/java/net/Indyuce/mmocore/api/skill/Skill.java
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) { public SkillResult whenCast(PlayerData data, SkillInfo skill) {
return new SkillResult(data, skill); return new SkillResult(data, skill);