Botanical Addons

Botanical Addons

69.7k Downloads

Crashing on Server Start

TheWhiteWolves opened this issue ยท 19 comments

commented

Getting a crash on server start due to a reference to a client side file that obviously doesn't exist.

Log

Class Crash Occurs In

If you need anymore info, ask

commented

I forgot a sideonly, didn't I.

commented

probably, for now i just disabled it til it can be fixed, once it is ill re-add and get to try out the new stuff you guys been showing off

commented

ok, I'm going to test with a server as well as a client from now on >.<

commented

lol don't worry we've done it before, vaz keeps forgetting to check server side aswell

commented

Side aware is strange for me. Plus Kotlin is stingy about null safety.

Do we need to register on the bus for that class? It doesn't look like we are registering any events.

commented

yes, we are. I'm checking it out, don't worry.

commented

Cool I'll leave it too you then. Luckily github is a valid worktime activity.

commented

what you think i do for half the time that i have tests running at work ;P

commented

So, this will be fixed once my local changes aren't pertaining to a local version of botania .-.

namely, they'll only compile with botania 236 + my PR

commented

ok, 236 is now working with these changes, so i'll work on this again

commented

wait, how does VAZKII's code run? it's doing the same thing we are o.o

commented

tfw eula.txt prevents server loading for tests >.<

commented

once build 236 comes out I'll push this release.

commented

Also I think vazkii is using black magic or something to make her code compile

commented

Its cos we set it in the base class of all the blocks here

It then just gets overridden in inheritors but stays marked as client side

commented

It should be set in the base class in ours as well here. I'm wondering if kotlins annotation processor is having issues propagating it for overrides.

commented

Oh right, the log didn't have the error i'm talking about
it occurred when one of the rainbow ones was EVENT_BUSsed on the serverside, because it couldn't process importing the IIconRegister. :/

commented

Oh, thats good to know. I was worried there for a min, because I've had issues with kapt before and I thought it was fixed already.

commented

Yeah, I fixed those by checking the side before EVENT_BUSsing it.