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

Better Placement + CarryOn: taking entity causing instantly drop it #558

Open
Krutoy242 opened this issue Sep 29, 2024 · 6 comments
Open
Labels
bug Something isn't working question Further information is requested

Comments

@Krutoy242
Copy link

When Better Placement is turned ON and CarryOn is installed, taking entity causing to drop it right next tick.
It feels like Better Placement overrides some CarryOn code, since without this option, picked up entities not dropped untill click again.

There is a video demonstration. I'm pressing and holding Shift + Right Click to take a cow. Normally it would only take a cow, but with option enabled it will drop it next tick, and then take again.

javaw_Wol1EdoTNQ.mp4
@jchung01
Copy link
Contributor

Does this occur if Force New Location=false for the Better Placement tweak?

@ACGaming ACGaming added bug Something isn't working question Further information is requested labels Sep 30, 2024
@Krutoy242
Copy link
Author

When Force New Location=false entity will be picked up and placed indefinitely:

mpc-hc64_P5OakZ24RT

@jchung01
Copy link
Contributor

jchung01 commented Oct 1, 2024

I tested CarryOn by itself without UT, and it seems the 2nd clip is the default behavior. It looks like 1.12 CarryOn, unlike its modern versions, doesn't have the proper checks to prevent the behavior of constant picking up/placing while holding the button. The only reason it seems like it works in the 1st clip is because of unintended interaction with Force New Location=true. You can easily check this is the case by continuing holding the button while looking at a new block.
So there is a bug when Force New Location=true because it doesn't match default CarryOn behavior, but the entity being instantly dropped is a problem with CarryOn itself.

@ChromaPIE
Copy link

Can reproduce, but I'm having the same issue with Blocks, but not entities, which is super annoying bc Carry On is 95% about picking up blocks.

@Krutoy242
Copy link
Author

Krutoy242 commented Nov 1, 2024

Can reproduce, but I'm having the same issue with Blocks, but not entities, which is super annoying bc Carry On is 95% about picking up blocks.

Actually true. With Better Placement picking up blocks is even worse - when entities placed back next tick, blocks placed back next frame, which means you need to release mouse button in 10ms or block will be placed down.

@ChromaPIE
Copy link

The old Better Placement mod does it much better

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants