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

Added support for ItemPhysic (again) #1563

Merged
merged 5 commits into from
Dec 13, 2020

Conversation

CreativeMD
Copy link
Contributor

Might not be the best prettiest solution. (We used it in 1.12).
If you have suggestions on how to make this better just let me know.

@HellFirePvP
Copy link
Owner

Here we go again :P


public EntityItemHighlighted(EntityType<? extends ItemEntity> type, World world) {
super(type, world);
if(skipPhysicRenderer != null)
try {
skipPhysicRenderer.setBoolean(this, true);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use ObfuscationHelper.setPrivateValue() instead; so we don't need to try/catch ourselves.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bad for performance, the field should be cached.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Derp. my bad. Thought for a moment it took a Field not just the fieldname.

@HellFirePvP HellFirePvP merged commit ac32dc1 into HellFirePvP:1.16-indev Dec 13, 2020
@yavincl
Copy link

yavincl commented Dec 13, 2020

By the way, why is this being merged into 1.16-indev instead of master?

@CreativeMD
Copy link
Contributor Author

ups

@CreativeMD CreativeMD deleted the patch-1 branch December 13, 2020 19:00
@Wolfstar1010
Copy link

Hello! Thanks for working on this! I saw that the issue was in the changelog for the most recent Astral update but I'm still having the issue occur in game after updating. Is there something special I'd have to do to fix this?

@CreativeMD
Copy link
Contributor Author

Have you update ItemPhysic as well?

@Wolfstar1010
Copy link

Yes, I just updated both.

@Wolfstar1010
Copy link

I have Item Physics 1.4.11 and Astral Sorcery 1.13.5

@yavincl
Copy link

yavincl commented Dec 15, 2020

Maybe the fix didn't work, or uhh the fix was not properly merged

@CreativeMD
Copy link
Contributor Author

My bad, should be fixed in the newest version of ItemPhysic.

@Wolfstar1010
Copy link

:( Sorry to be back here again, I'm still having the same issue. I wish I could easily share a video of what's going on... but here is a picture. The items placed are laying flat and frantically rotating around a central point. Please let me know if I'm doing something wrong... I updated both Astral and Item Physics just now before launching...

image

image

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

Successfully merging this pull request may close these issues.

None yet

4 participants