Skip to content

fix(files): skip quota check when opening files for reading - #65769

Open
cristianscheid wants to merge 1 commit into
masterfrom
fix/noid/quota-allow-reads
Open

cristianscheid wants to merge 1 commit into
masterfrom
fix/noid/quota-allow-reads

Conversation

@cristianscheid

@cristianscheid cristianscheid commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Summary

Skip quota/free space check when opening files for reading, since reads don't consume any space.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@cristianscheid

Copy link
Copy Markdown
Member Author

/backport to stable35

@cristianscheid

Copy link
Copy Markdown
Member Author

/backport to stable34

@cristianscheid

Copy link
Copy Markdown
Member Author

/backport to stable33

@cristianscheid

Copy link
Copy Markdown
Member Author

/backport to stable32

@joshtrichards joshtrichards left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Makes sense.

Though could use some unit test coverage.

@joshtrichards joshtrichards added this to the Nextcloud 36 milestone Oct 22, 2026
Comment thread lib/private/Files/Storage/Wrapper/Quota.php Outdated
Comment thread lib/private/Files/Storage/Wrapper/Quota.php

@skjnldsv skjnldsv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Slight request for the next dev looking at this code, otherwise good

@cristianscheid
cristianscheid force-pushed the fix/noid/quota-allow-reads branch from a541a5f to 9165236 Compare October 23, 2026 13:31
Signed-off-by: Cristian Scheid <cristianscheid@gmail.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants