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.

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

add Additional Enchanted Miner compat #277

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
1 change: 1 addition & 0 deletions dependencies.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@
// Sorted by alphabetical name of the `project.debug_[this]` value
final def mod_dependencies = [
'actually-additions-228404:3117927' : [project.debug_actually_additions],
'additionalenchantedminer-282837:3851282' : [project.debug_additional_enchanted_miner],
'advancedmortars-283777:2780626' : [project.debug_adv_mortars],
'advanced-rocketry-236542:4671856' : [project.debug_advanced_rocketry],
'libvulpes-236541:3801015' : [project.debug_advanced_rocketry],
Expand Down
1 change: 1 addition & 0 deletions gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ debug_generate_and_crash = false
# SECTION: debug mod compat

debug_actually_additions = false
debug_additional_enchanted_miner = false
debug_adv_mortars = false
debug_advanced_rocketry = false
debug_aether = false
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
import com.cleanroommc.groovyscript.api.GroovyBlacklist;
import com.cleanroommc.groovyscript.api.GroovyPlugin;
import com.cleanroommc.groovyscript.compat.mods.actuallyadditions.ActuallyAdditions;
import com.cleanroommc.groovyscript.compat.mods.additionalenchantedminer.AdditionalEnchantedMiner;
import com.cleanroommc.groovyscript.compat.mods.advancedmortars.AdvancedMortars;
import com.cleanroommc.groovyscript.compat.mods.advancedrocketry.AdvancedRocketry;
import com.cleanroommc.groovyscript.compat.mods.aetherlegacy.Aether;
Expand Down Expand Up @@ -82,6 +83,7 @@ public class ModSupport {
public static final ModSupport INSTANCE = new ModSupport(); // Just for Binding purposes

public static final GroovyContainer<ActuallyAdditions> ACTUALLY_ADDITIONS = new InternalModContainer<>("actuallyadditions", "Actually Additions", ActuallyAdditions::new, "aa");
public static final GroovyContainer<AdditionalEnchantedMiner> ADDITIONAL_ENCHANTED_MINER = new InternalModContainer<>("quarryplus", "Additional Enchanted Miner", AdditionalEnchantedMiner::new, "aem");
WaitingIdly marked this conversation as resolved.
Show resolved Hide resolved
public static final GroovyContainer<AdvancedMortars> ADVANCED_MORTARS = new InternalModContainer<>("advancedmortars", "Advanced Mortars", AdvancedMortars::new);
public static final GroovyContainer<AdvancedRocketry> ADVANCED_ROCKETRY = new InternalModContainer<>("advancedrocketry", "Advanced Rocketry", AdvancedRocketry::new);
public static final GroovyContainer<Aether> AETHER = new InternalModContainer<>("aether_legacy", "Aether Legacy", Aether::new, "aether");
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
package com.cleanroommc.groovyscript.compat.mods.additionalenchantedminer;

import com.cleanroommc.groovyscript.compat.mods.GroovyPropertyContainer;

public class AdditionalEnchantedMiner extends GroovyPropertyContainer {
public final WorkBenchPlus WorkBenchPlus = new WorkBenchPlus();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fields should be in camelCase - WorkBenchPlus is PascalCase, it should be workBenchPlus

}
Original file line number Diff line number Diff line change
@@ -0,0 +1,100 @@
package com.cleanroommc.groovyscript.compat.mods.additionalenchantedminer;

import com.cleanroommc.groovyscript.api.GroovyLog;
import com.cleanroommc.groovyscript.api.documentation.annotations.*;
import com.cleanroommc.groovyscript.helper.recipe.AbstractRecipeBuilder;
import com.cleanroommc.groovyscript.registry.VirtualizedRegistry;
import com.yogpc.qp.recipe.WorkbenchRecipe;
import com.yogpc.qp.tile.ItemDamage;
import net.minecraft.item.ItemStack;
import net.minecraft.util.ResourceLocation;
import org.jetbrains.annotations.Nullable;
import scala.Option;
import scala.collection.JavaConversions;
import scala.collection.Map;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you have to use scala here?


@RegistryDescription
public class WorkBenchPlus extends VirtualizedRegistry<WorkbenchPlusRecipe> {

@Override
public void onReload() {
removeScripted().forEach(recipe -> WorkbenchPlusRecipe.removeById(recipe.getLocation().toString()));
restoreFromBackup().forEach(WorkbenchPlusRecipe::addRecipe);
}

@MethodDescription(example = @Example("item('quarryplus:quarry')"))
public Boolean removeByOutput(ItemStack output) {
WaitingIdly marked this conversation as resolved.
Show resolved Hide resolved
ItemDamage itemDamage = ItemDamage.apply(output);
Map<ResourceLocation, WorkbenchRecipe> recipeMap = WorkbenchRecipe.getRecipeMap();
Iterable<WorkbenchRecipe> iterable = JavaConversions.asJavaIterable(recipeMap.values());
iterable.forEach(recipe -> {
if (recipe.key().equals(itemDamage)) {
addBackup(new WorkbenchPlusRecipe(recipe.inputs(), recipe.getOutput(), recipe.energy(), recipe.location()));
}
});
return WorkbenchPlusRecipe.removeByOutput(output);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you have direct access to the Map, why do you need to call a different method?

}

public Boolean removeById(String id) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be used by people using GroovyScript be calling this method? if so, use @MethodDescription and state what it does. otherwise, private or @GroovyBlacklist. same with removeAll.

Map<ResourceLocation, WorkbenchRecipe> OptinalList = WorkbenchRecipe.getRecipeMap();
WaitingIdly marked this conversation as resolved.
Show resolved Hide resolved
ResourceLocation location = new ResourceLocation(id);
Option<WorkbenchRecipe> recipe = OptinalList.get(location);
if (recipe.isDefined()) addBackup(new WorkbenchPlusRecipe(recipe.get().inputs(), recipe.get().getOutput(), recipe.get().energy(), recipe.get().location()));
return WorkbenchPlusRecipe.removeById(id);
}

public void removeAll() {
Map<ResourceLocation, WorkbenchRecipe> OptinalList = WorkbenchRecipe.getRecipeMap();
Iterable<ResourceLocation> iterable = JavaConversions.asJavaIterable(OptinalList.keys());
iterable.forEach(
recipe -> removeById(recipe.toString())
);
}

@RecipeBuilderDescription(example =
@Example(".output(item('minecraft:nether_star')).input(item('minecraft:diamond'),item('minecraft:gold_ingot')).energy(10000)")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you provide a second example?

)
public RecipeBuilder recipeBuilder(){return new RecipeBuilder();}

@Property(property = "input", comp = @Comp(not = "null", gte = 1 , lte = 27))
@Property(property = "output", comp = @Comp(not = "null" , eq = 1))
WaitingIdly marked this conversation as resolved.
Show resolved Hide resolved
public class RecipeBuilder extends AbstractRecipeBuilder<WorkbenchPlusRecipe> {
WaitingIdly marked this conversation as resolved.
Show resolved Hide resolved

@Property(property = "energy", comp = @Comp(gt = 0))
WaitingIdly marked this conversation as resolved.
Show resolved Hide resolved
private double energy;

@RecipeBuilderMethodDescription
public RecipeBuilder energy(double energy) {
this.energy = energy;
return this;
}

@Override
public String getRecipeNamePrefix() {
return "additionalenchantedminer_workbenchplus_";
}


@Override
public String getErrorMsg() {
return "Error adding AdditionalEnchantedMiner WorkbenchPlus recipe";
code-onigiri marked this conversation as resolved.
Show resolved Hide resolved
}

@Override
public void validate(GroovyLog.Msg msg) {
validateName();
validateItems(msg, 1, 27, 1, 1);
msg.add(energy <= 0, "energy must be an integer greater than 0, yet it was {}", energy);
}

@Override
@RecipeBuilderRegistrationMethod
public @Nullable WorkbenchPlusRecipe register() {
if (!validate()) return null;
WorkbenchPlusRecipe recipe = new WorkbenchPlusRecipe(this.input, this.output.get(0), this.energy, name);
WaitingIdly marked this conversation as resolved.
Show resolved Hide resolved
addScripted(recipe);
WorkbenchPlusRecipe.addRecipe(recipe);
return recipe;
}
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,69 @@
package com.cleanroommc.groovyscript.compat.mods.additionalenchantedminer;

import com.cleanroommc.groovyscript.api.IIngredient;
import com.yogpc.qp.recipe.WorkbenchRecipe;
import com.yogpc.qp.tile.ItemDamage;
import com.yogpc.qp.utils.IngredientWithCount;
import net.minecraft.item.ItemStack;
import net.minecraft.util.ResourceLocation;
import scala.Option;
import scala.collection.JavaConversions;
import scala.collection.Map;
import scala.collection.Seq;

import java.util.Collection;
import java.util.Collections;
import java.util.List;
import java.util.stream.Collectors;

import static com.yogpc.qp.recipe.WorkbenchRecipe.addIngredientRecipe;

public class WorkbenchPlusRecipe {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the point of this class? what purpose does it serve that it couldnt be replaced by some code in the register() method?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now that I think about it, I realize that I don't need WorkbenchPlusRecipe, I'll fix it.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nothing changed here.

private final List<List<IngredientWithCount>> input;
private final ItemStack output;
private final Double energy;
private final ResourceLocation location;

public WorkbenchPlusRecipe(Collection<IIngredient> input, ItemStack output, Double energy , ResourceLocation location) {

this.input = input.stream()
.map(i -> new IngredientWithCount(i.toMcIngredient(), i.getAmount()))
.map(Collections::singletonList)
.collect(Collectors.toList());
this.output = output;
this.energy = energy;
this.location = location;
}

public WorkbenchPlusRecipe(Seq<Seq<IngredientWithCount>> input, ItemStack output, Double energy , ResourceLocation location) {
this.input = JavaConversions.seqAsJavaList(input).stream()
.map(JavaConversions::seqAsJavaList)
.collect(Collectors.toList());
this.output = output;
this.energy = energy;
this.location = location;

}


public static WorkbenchPlusRecipe addRecipe(WorkbenchPlusRecipe recipe) {
addIngredientRecipe(recipe.location, recipe.output, recipe.energy, recipe.input, true);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not WorkbenchRecipe.addIngredientRecipe instead of importing a static just for this

return recipe;
}

public static Boolean removeByOutput(ItemStack output) {
ItemDamage itemDamage = ItemDamage.apply(output);
WorkbenchRecipe.removeRecipe(itemDamage);
return true;
}

public static Boolean removeById(String id) {
ResourceLocation resourceLocation = new ResourceLocation(id);
WorkbenchRecipe.removeRecipe(resourceLocation);
return true;
}

public ResourceLocation getLocation() {
return location;
}
}