added lightningcraft compat#283
Conversation
New WR adding a gs compat
WaitingIdly
left a comment
WaitingIdly
left a comment
There was a problem hiding this comment.
there is also lightning transforming recipes, according to MoreTweaker, although those might deserve a similar treatment to Pneu explosion (creation of native functionality + suggestion to use that instead of custom compat)
|
I've heard the LC's own lightning transform is really buggy (see: DJ2 where you can only craft electricium in a specific order), which is why I haven't implemented it. Do not mind trying to implement lightning transform properly under in-world crafting. |
brachy84
added
the
mod compat
|
updated |
| public @Nullable LightningInfusionRecipe register() { | ||
| if (!validate()) return null; | ||
|
|
||
| ItemStack centerItem = this.centerItem.getMatchingStacks()[0]; |
There was a problem hiding this comment.
should loop for each of getMatchingStacks() instead of only taking the first.
| if (!validate()) return null; | ||
|
|
||
| ItemStack centerItem = this.centerItem.getMatchingStacks()[0]; | ||
| ItemStack[] inputs = input.stream().map(i -> i.getMatchingStacks()[0]).toArray(ItemStack[]::new); |
There was a problem hiding this comment.
while it is not currently on main and is still just in the Betweenlands PR, IngredientHelper.cartesianProductItemStacks would be very useful here to, once again, use all values of getMatchingStacks() instead of just the first.
| @Property(defaultValue = "determined automatically based on the input items") | ||
| private boolean nbtSensitive = false; |
There was a problem hiding this comment.
is there a reason why having this be directly configurable by the player is required? if it can be automatically determined, why not simply not expose it.
There was a problem hiding this comment.
Because there are cases where it's automatically configured value is wrong. For example, if you want to allow only input item that has no NBT in it (say, it's some tool that stores durability in NBT). You need to enable nbtSensitive = true, and there's not really another way to do it
|
going to wait until betweenlands is merged for the cartesian product thing because I do not feel like reinventing the wheel and also because I feel like I'll have to rewrite to it anyway. |
|
the betweenlands PR has been merged, so now |
|
rip in peace |
|
Closed due to inactivity. Can be reopened if someone wants to continue. |
4 participants
literally no one asked for this but here we go
oh wait Lord_Chaos did