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

Editor: Fixed camera UUID lost when recovering #28337

Merged
merged 1 commit into from
May 13, 2024

Conversation

ycw
Copy link
Contributor

@ycw ycw commented May 11, 2024

The issue: When recovering from IndexedDB, all properties are copied to existing default camera, except for .uuid , in turn commands, which hold object uuid referencing to the default camera, will fail to undo/redo, because they wont be able to get back the camera by UUID during recovery, for example SetValueCommand's object state will be undefined after recovery.

This PR fixed that by copying .uuid to existing default camera, also updating editor.cameras.

@ycw ycw force-pushed the editor-fully-copy-camera branch from 7759fc3 to a564aa9 Compare May 11, 2024 19:28
editor/js/Editor.js Outdated Show resolved Hide resolved
@ycw ycw requested a review from Mugen87 May 12, 2024 12:13
@ycw ycw force-pushed the editor-fully-copy-camera branch from 91f624b to 12b9bda Compare May 12, 2024 12:18
@Mugen87 Mugen87 added this to the r165 milestone May 13, 2024
@Mugen87 Mugen87 merged commit db047d1 into mrdoob:dev May 13, 2024
11 checks passed
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.

2 participants