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

Fixes bugs with copy entities from Position A to Position B #2195

Open
wants to merge 29 commits into
base: main
Choose a base branch
from

Conversation

TheMeinerLP
Copy link
Member

@TheMeinerLP TheMeinerLP commented Apr 29, 2023

Overview

Fixes #2140
Fixes #2427

Description

This PR included the fix for passenger of entities and also leashs for other entities.
That means now //copy -e and //paste -e will work to 99% correct now for entities except some edge cases.

Submitter Checklist

  • Make sure you are opening from a topic branch (/feature/fix/docs/ branch (right side)) and not your main branch.
  • Ensure that the pull request title represents the desired changelog entry.
  • New public fields and methods are annotated with @since TODO.
  • I read and followed the contribution guidelines.

@TheMeinerLP TheMeinerLP requested a review from a team as a code owner April 29, 2023 14:21
@github-actions github-actions bot added the Bugfix This PR fixes a bug label Apr 29, 2023
@TheMeinerLP
Copy link
Member Author

I tested each of them on a Plot world

@LedariaTim1
Copy link

LedariaTim1 commented May 13, 2023

i just compiled your bugfix and tested it, the entities including passanger are now copied, but they are now both independent entities and no longer mounted. in my case the pig is no longer sitting on the invisible armorstand and can be moved around
image
image

@TheMeinerLP
Copy link
Member Author

Bump!
Tested with 1.20.1

@TheMeinerLP TheMeinerLP closed this Dec 2, 2023
@TheMeinerLP TheMeinerLP deleted the bugfix/2140 branch December 2, 2023 11:12
@TheMeinerLP TheMeinerLP restored the bugfix/2140 branch December 2, 2023 11:13
@TheMeinerLP TheMeinerLP reopened this Dec 2, 2023
@TheMeinerLP TheMeinerLP requested a review from SirYwell December 2, 2023 11:43
@Hy3z
Copy link

Hy3z commented Jan 12, 2024

Hey, do we have any news about this? This fix would be very useful for builders who use Block Display Studio to create structures.

Copy link

Please take a moment and address the merge conflicts of your pull request. Thanks!

Copy link

Please take a moment and address the merge conflicts of your pull request. Thanks!

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