Skip to content

Commit

Permalink
Don't set double drop if result is already a full stack (#4263)
Browse files Browse the repository at this point in the history
Thanks, merging because I can't think of anything bad that would result from this change.
  • Loading branch information
FrankHeijden committed Aug 17, 2020
1 parent a2f2614 commit dca2661
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ public ItemStack smeltProcessing(ItemStack smelting, ItemStack result) {
applyXpGain(Smelting.getResourceXp(smelting), XPGainReason.PVE, XPGainSource.PASSIVE);

if (Config.getInstance().getDoubleDropsEnabled(PrimarySkillType.SMELTING, result.getType())
&& isSecondSmeltSuccessful()) {
&& isSecondSmeltSuccessful() && result.getAmount() < 64) {
ItemStack newResult = result.clone();

newResult.setAmount(result.getAmount() + 1);
Expand Down

0 comments on commit dca2661

Please sign in to comment.