AEnterprise / Eureka

Learning as you go
GNU Lesser General Public License v3.0
13 stars 4 forks source link

Tick Loop (AE2) #13

Closed Shadowhunter82 closed 10 years ago

Shadowhunter82 commented 10 years ago

Hello Found ah new bug!

http://pastebin.com/7SddPEni

Speiger commented 10 years ago

@Shadowhunter82 when did that happen? When your AESystem did craft something?

Shadowhunter82 commented 10 years ago

Thats the main problem i don´t know, It happend on my server. and there are ah view people on it with the autocrafting from AE

Speiger commented 10 years ago

Test it in singelplayer. or make a server on your pc on put the mods in and try to crash it with this bug.

iTitus commented 10 years ago

AE2 is throwing the CraftEvent but with a null-Playerobject, @AEnterprise should implement a null-Check there.

Am 07.09.2014 um 17:18 schrieb Speiger notifications@github.com:

Test it in singelplayer. or make a server on your pc on put the mods in and try to crash it with this bug.

— Reply to this email directly or view it on GitHub.

Speiger commented 10 years ago

@iTitus i did think that too but could be possible that the fakeplayer of them had some problems.

iTitus commented 10 years ago

Trying to reproduce now

iTitus commented 10 years ago

Ok, its definetely a broken FakePlayer: the playerNetServerHandler is null while it tries to send a message in "addChatMessage(IChatComponent)"

AEnterprise commented 10 years ago

@iTitus so this has to be reported to AE2 then? should i do a null check for that?

iTitus commented 10 years ago

Yes, I think so, I was able to recreate this with only AE2, BC6 and BCA. When I created the right amount of Tools via Autocrafting it crashed with the same crash log.

iTitus commented 10 years ago

You could check if player.playerNetServerHandler is null before sending a message but this should be done at AE2's end, I think

AEnterprise commented 10 years ago

the right amount, so did you make progresss if you look in your book?

iTitus commented 10 years ago

I will check now, I have to restart MC

iTitus commented 10 years ago

My Player did not make any progress. But it keeps crashing as long as the crafting operation is running (I requested 60 Wooden Pickaxes...)

BigXplosion commented 10 years ago

You should just add a null check like iTitus said, but you can't really blame AE2 (and other mods that can cause this problem) for this, as I don't think there is a better way of throwing a CraftEvent without having a normal player.

AEnterprise commented 10 years ago

alright will add a null check

BigXplosion commented 10 years ago

Also it's normal that auto crafting stuff via AE2 or other mods doesn't progress the player as there is no player specified. the same thing is happening for the HQM quest if you have a crafting task, only the player crafted items help you progress.

AEnterprise commented 10 years ago

did it with a try catch as there might be other issues with fake players, this will take care of all of them

iTitus commented 10 years ago

Also a possible solution ;)

Speiger commented 10 years ago

Why not make a FakePlayerTestfunction?

AEnterprise commented 10 years ago

@Speiger this just prevent all possible crashes, easier solution and it works