Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix Book Binder #3299

Closed
wants to merge 8 commits into from
Closed

Fix Book Binder #3299

Changes from 2 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
fd5beed
Fix Book Binder
svr333 Oct 3, 2021
58f845b
Remove unused import
svr333 Oct 3, 2021
03e3225
Update changelog
svr333 Nov 6, 2021
6594012
Merge branch 'master' into FixBookBinder
svr333 Nov 6, 2021
0b26396
Merge branch master
svr333 Jan 19, 2022
fd280d9
Merge branch master
svr333 Jan 19, 2022
e242ccd
Fix changelog
svr333 Jan 19, 2022
7af05a9
Resolve merge conflict
svr333 Jan 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 6 additions & 7 deletions ...hebusybiscuit/slimefun4/implementation/items/electric/machines/enchanting/BookBinder.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
import io.github.thebusybiscuit.slimefun4.api.items.SlimefunItemStack;
import io.github.thebusybiscuit.slimefun4.api.items.settings.IntRangeSetting;
import io.github.thebusybiscuit.slimefun4.api.recipes.RecipeType;

import me.mrCookieSlime.Slimefun.Objects.SlimefunItem.abstractItems.AContainer;
import me.mrCookieSlime.Slimefun.Objects.SlimefunItem.abstractItems.MachineRecipe;
import me.mrCookieSlime.Slimefun.api.inventory.BlockMenu;
Expand Down Expand Up @@ -129,13 +128,14 @@ private Map<Enchantment, Integer> combineEnchantments(Map<Enchantment, Integer>
if (!hasConflicts) {
enchantments.merge(entry.getKey(), entry.getValue(), (a, b) -> {
int maxLevel = entry.getKey().getMaxLevel();
return combineEnchantmentLevels(maxLevel, a, b);
int newLevel = combineEnchantmentLevels(maxLevel, a, b);

return newLevel == 0 ? null : newLevel;
});
}
}

return enchantments;

}

private int combineEnchantmentLevels(int maxLevel, int lvl1, int lvl2) {
Expand All @@ -145,9 +145,9 @@ private int combineEnchantmentLevels(int maxLevel, int lvl1, int lvl2) {
* unless it uses bypass-vanilla-max-level
*/
if (maxLevel <= lvl1 && !bypassVanillaMaxLevel.getValue()) {
return maxLevel;
return 0;
} else if (hasCustomMaxLevel.getValue()) {
return lvl1 + 1 > customMaxLevel.getValue() ? customMaxLevel.getValue() : lvl1 + 1;
return lvl1 + 1 > customMaxLevel.getValue() ? 0 : lvl1 + 1;
} else {
return lvl1 + 1;
}
Expand All @@ -165,7 +165,6 @@ private int combineEnchantmentLevels(int maxLevel, int lvl1, int lvl2) {
} else {
return highestLevel;
}

}
}
}
}