Skip to content

Fix quota type to int - #34019

Merged
PVince81 merged 1 commit into
masterfrom
fix/quota-type
Sep 12, 2022
Merged

Fix quota type to int#34019
PVince81 merged 1 commit into
masterfrom
fix/quota-type

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

Change the quota to int from float, since the quota is a number of bits and a bits can not be splitted.

Fix#34010

Change the quota to int from float, since the quota is a number of bits
and a bits can not be splitted.
Fix#34010
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwanCarlSchwan added this to the Nextcloud 25 milestone Sep 12, 2022
@CarlSchwan
CarlSchwan requested a review from a teamSeptember 12, 2022 07:27
@CarlSchwanCarlSchwan self-assigned this Sep 12, 2022
@CarlSchwan
CarlSchwan requested review from PVince81, icewind1991 and skjnldsv and removed request for a teamSeptember 12, 2022 07:28
@solracsf

Copy link
Copy Markdown
Member

Backport?

@CarlSchwan

Copy link
Copy Markdown
MemberAuthor

/backport to stable24

@PVince81PVince81 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.

👍

@PVince81

Copy link
Copy Markdown
Member

reverted on NC 24 and NC 25 as it broke quotas on 32-bit architecture:

@PVince81

Copy link
Copy Markdown
Member

we keep it on NC 26 / master where we deprecate 32-bit support

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

Labels

3. to reviewWaiting for reviewsbugregression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Internal server error after upgrade to NC25beta: Typed property OC\Files\Storage\Wrapper\Quota::$quota must be int or null, float used in file

4 participants

@CarlSchwan@solracsf@PVince81@come-nc