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

don't alter file when there is not enough free space #3562

Merged

Conversation

grnd-alt
Copy link
Member

@grnd-alt grnd-alt commented Apr 4, 2024

Summary

When Updating files via WOPI the file used to be written from the beginning just to realize mid writing that there's not enough space and throw an error. Therefore the entire doc gets broken and mostly unusable.

Now: When a File is to be updated it gets checked if it would exceed the free space on storage/quota, if not an error is thrown and the file is left as is.

@grnd-alt grnd-alt linked an issue Apr 4, 2024 that may be closed by this pull request
@grnd-alt grnd-alt force-pushed the fix/3281-file-gets-truncated-on-save-when-quota-is-exceeded branch from cd6535b to cf33ed4 Compare April 8, 2024 13:59
@grnd-alt grnd-alt force-pushed the fix/3281-file-gets-truncated-on-save-when-quota-is-exceeded branch 2 times, most recently from 97d2459 to b1ef0b3 Compare April 10, 2024 14:27
Copy link
Member

@juliushaertl juliushaertl left a comment

Choose a reason for hiding this comment

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

Small comments, but otherwise tested and works

lib/Controller/WopiController.php Outdated Show resolved Hide resolved
@juliushaertl
Copy link
Member

Can you squash the commit to have a clean history?

@grnd-alt grnd-alt force-pushed the fix/3281-file-gets-truncated-on-save-when-quota-is-exceeded branch from b9dc420 to b72d576 Compare April 11, 2024 09:18
@grnd-alt grnd-alt merged commit f3644e4 into main Apr 11, 2024
51 checks passed
@delete-merged-branch delete-merged-branch bot deleted the fix/3281-file-gets-truncated-on-save-when-quota-is-exceeded branch April 11, 2024 09:48
@juliushaertl juliushaertl added the bug Something isn't working label Apr 11, 2024
@grnd-alt
Copy link
Member Author

/backport to stable27

@grnd-alt
Copy link
Member Author

/backport to stable28

@juliushaertl
Copy link
Member

/backport to stable29

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

Successfully merging this pull request may close these issues.

File gets truncated on save when quota is exceeded
3 participants