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

Console Gets Spammed with this #6

Closed
alexbegt opened this issue Jul 31, 2013 · 16 comments
Closed

Console Gets Spammed with this #6

alexbegt opened this issue Jul 31, 2013 · 16 comments

Comments

@alexbegt
Copy link

http://pastebin.com/Y59bNSuz

I think you forgot to register the TE's

@Chicken-Bones
Copy link
Owner

What do you have in those parts? I need a full FML log as well.

@alexbegt
Copy link
Author

Ok, ill get you one.

@OniBait
Copy link

OniBait commented Aug 7, 2013

Here's one where I'm getting it -- not sure what is being placed there.
https://gist.github.com/OniBait/6170538

Note: I'm running MCPC+, but I also develop for it so I can look into it further if needed.

@Chicken-Bones
Copy link
Owner

I'm not seeing the spam in that log you just posted.

@OniBait
Copy link

OniBait commented Aug 7, 2013

View the full file. Line 20333 (not spam per se, but the block isn't being saved).

I'm not seeing in your code where you are registering the TE with forge or adding the mapping.

@jahvazi
Copy link

jahvazi commented Aug 7, 2013

Error code : http://pastebin.com/dkjc2Ahm
FML log : https://dl.dropboxusercontent.com/u/20714042/Error%20Log%20Lines%206817-6832.txt
Blocks are not being saved on server (works on SSP strangley). 1.5.2 version of Minecraft. Multipart version : ForgeMultipart-universal-1.5.2-1.0.0.119.jar

@Iscariah
Copy link

Iscariah commented Aug 7, 2013

Same here, we're using MCPC+ and the wireless redstone blocks are gone after restarting the server. I'm not sure about the micro blocks from Multi-part, since we use Immibis microblocks for that. (It came out long before multi-part came out, that's the reason) but I can imagine the same will happen with them.

@Chicken-Bones
Copy link
Owner

This issue has been reported a few times. Unfortunately I can't do anything about non-forge servers, please report it to the MCPC+ devs.

@alexbegt
Copy link
Author

alexbegt commented Aug 7, 2013

Well it seems you don't register the TE's in your code anywhere.

@OniBait
Copy link

OniBait commented Aug 7, 2013

I am an MCPC+ dev. I'm looking into it. ;) Sorry to bug you Chicken-Bones. I'll report back shortly.

@OniBait
Copy link

OniBait commented Aug 8, 2013

Problem is here:
https://github.com/Chicken-Bones/ForgeMultipart/blob/master/codechicken/multipart/handler/MultipartSaveLoad.scala#L46

You are relying on fields to be in the specific order instead of using the field name like you did above with the nameToClassMap:

https://github.com/Chicken-Bones/ForgeMultipart/blob/master/codechicken/multipart/handler/MultipartSaveLoad.scala#L30-32

Verifying the fix on the MCPC+ side by moving a spigot field down.

@Chicken-Bones
Copy link
Owner

Urf, my apologies, let me fix that. I should be using the name in both cases.

@Chicken-Bones
Copy link
Owner

Fixed on my end as of 78abc37 (1.6.2-1.0.0.120) and e618af7 (1.5.2-1.0.0.121)

@Iscariah
Copy link

Iscariah commented Aug 8, 2013

Thank you Chickenbones for the fast fix. That's awesome. Diamonds for you. I'm gonna try it in about 30 minutes or so. I'll let know if it works this time.

EDIT: I almost forget to thank you too OniBait. :) So, thank you as well!

EDIT 2: Confirmed! Yay! it works!!! :D Thanks so much again!

@alexbegt
Copy link
Author

alexbegt commented Aug 9, 2013

ChickenBones, Standing on the Microblocks gets you kicked for: "end of stream" here is a log: http://pastebin.com/8k7ENYTD

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants