op_by_name crashes server

kirjorjos commented 3 months ago

Issue type:

:bug: Bug

Short description:

Using the op_by_name operator caused a server crash, after reboot, display panel behaves inconsistently (as shown in 1378) even if card is taken out and put back in. In order to cause a 2nd crash, network must be updated by placing any network part on it or destroying any part of the network.

Steps to reproduce the problem:

Get "Concat" as a string card
Put it in the op_by_name "tab" in the LP
Put the result in a display panel and "Concat" anywhere in the network

Expected behaviour:

If op_by_name isn't expecting that particular string, it should just display an X in the display panel.

Versions:

This mod: Cyclops Core: 1.19.5
CommonCapabilities: 2.9.2
IntegratedDynamics: 1.23.0
Minecraft: 1.19.2
Forge: Client: 43.3.5 Server: 43.4.2

Log file:

crash-2024-08-14_06.18.56-server.txt

rubensworks commented 3 months ago

Thanks for reporting!

rubensworks commented 3 months ago

Note to self:

java.lang.NullPointerException: Cannot invoke "net.minecraft.resources.ResourceLocation.toString()" because "uniqueName" is null
    at org.cyclops.integrateddynamics.core.evaluate.operator.OperatorRegistry.getOperator(OperatorRegistry.java:112) ~[IntegratedDynamics-1.19.2-1.23.0.jar%23116!/:1.23.0] {re:classloading}
    at org.cyclops.integrateddynamics.core.evaluate.operator.Operators.lambda$static$197(Operators.java:2942) ~[IntegratedDynamics-1.19.2-1.23.0.jar%23116!/:1.23.0] {re:classloading}
    at org.cyclops.integrateddynamics.core.evaluate.operator.OperatorBase.evaluate(OperatorBase.java:214) ~[IntegratedDynamics-1.19.2-1.23.0.jar%23116!/:1.23.0] {re:classloading}

CyclopsMC / IntegratedDynamics