Fork/mmocore
2
0
Fork 0
forked from Upstream/mmocore
Code Pull Requests Activity

Reduced confusion for fishing drops config

Browse Source
This commit is contained in:
Indyuce 2022-10-14 00:38:24 +02:00
parent c5177c8fc2
commit 719ce614bc
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
MMOCore-API/src/main/java/net/Indyuce/mmocore/loot/fishing/FishingDropItem.java
View File

@ -15,18 +15,22 @@ public class FishingDropItem implements Weighted {
private final DropItem dropItem; private final DropItem dropItem;
public FishingDropItem(MMOLineConfig config) { public FishingDropItem(MMOLineConfig config) {
config.validate("tugs", "experience"); config.validateKeys("tugs", "experience");
tugs = new RandomAmount(config.getString("tugs")); tugs = new RandomAmount(config.getString("tugs"));
experience = new RandomAmount(config.getString("experience")); experience = new RandomAmount(config.getString("experience"));
dropItem = MMOCore.plugin.loadManager.loadDropItem(config); dropItem = MMOCore.plugin.loadManager.loadDropItem(config);
Validate.isTrue(dropItem.getWeight() > 0, "A fishing drop table item must have a strictly positive weight");
} }
/**
* An item cannot have a negative weight. Since drop items have 0 weight
* by default, MMOCore takes 1 as minimum value if the item weight is
* negative or equal to 0
*/
@Override @Override
public double getWeight() { public double getWeight() {
return dropItem.getWeight(); return dropItem.getWeight() <= 0 ? 1 : 0;
} }
public DropItem getItem() { public DropItem getItem() {