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

Remove lock from document if Document:close fails in specific ways #66

Merged
merged 2 commits into from
Aug 24, 2024

Conversation

nezuo
Copy link
Owner

@nezuo nezuo commented Aug 24, 2024

The lock will be removed if Document:close fails for the following reasons:

  • beforeSave/beforeClose threw an error.
  • validate threw an error.
  • validate didn't return true.

The lock is only removed if the lock still matches the one the Document has. game:BindToClose waits for these removals to finish.

@nezuo nezuo merged commit 20eef10 into master Aug 24, 2024
@nezuo nezuo deleted the remove-lock-after-failed-close branch August 24, 2024 01:04
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.

1 participant