-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
[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 #34010
Comments
This comment was marked as outdated.
This comment was marked as outdated.
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 <[email protected]>
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 <[email protected]>
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 <[email protected]> Signed-off-by: Côme Chilliet <[email protected]>
Thanks for the quick fix. I applied the patch and everything works as expected. |
Thanks for testing 😀 |
Is it possible, that this fix broke 32-bit compatibility again? I just upgraded to 25.0.2 on my ARMv7 server (32 bit) and got this error:
Changing the type of quota from int to float in Files/Storage/Wrapper/Quota.php seems to fix the problem. |
+1 same for me |
could you please post what did you change; filenames, nr. of line and the edited content - it what be very helpful! |
TypeError: Cannot assign float to property OC\Files\Storage\Wrapper\Quota::$quota of type ?int
|
I changed following lines and it worked out for me. Referring to following code: Line 44: protected ?int $quota; -> ?int to ?float |
i changed following lines and it worked for me too (based on this post: /legacy/OC_Helper.php lib/private/legacy/OC_Util.php I didn't change "/private/Files/Storage/Wrapper/Quota.php" at all! thanks a lot for your quick replays! |
this one solved it for me! |
can you confirm if #35734 is enough on top of 25.0.2 to fix the quota issues on 32-bit systems ? |
the quota issue on my 32-bit system (raspbian/pi4) got fixed by executing following steps: /legacy/OC_Helper.php lib/private/legacy/OC_Util.php |
I upgraded today to "Nextcloud Hub 3 (25.0.2)" via Stable Channel and also get this error now. Running NC on a Raspi 3b. Can confirm, that the fix is working. |
I've updated the patch for 25.0.2, please check if it #35734 (comment) |
Sorry for not replying earlier. Busy times... and the return type of the method getQuota() from int to float That has solved the immediate issue on my 32 bit system and I haven't observed any side effects so far. Neither have I tried whether changing both lines is actually necessary. I was simply happy the instance was running again since I currently don't have the bandwidth to deal with a broken cloud. BTW, I'm not a php programmer. Please excuse if I may have used wrong terminology. |
Those two changes indeed made the quota load again for my users so they could actually see content in their iOS NC apps again and upload stuff using WebDAV. Gosh that 32 bit thing... luckily clock is ticking, but releasing NC v25 for 32 bit was a huuuuuuuuge mistake, obviously. When users try to load gallery view in the app I see plenty of those in the logs, is there a issue for that one too? Other 32 bit users should suffer from this too right? |
Also change line 71 |
Bug description
After an upgrade to NC25 Beta 6 from NC24 Nextcloud throws an 500 internal server error for logged in users. I switched the debug level to 0, but wasn't unable to finde more information than this trace which doesn't help me to get an idea, what might trigger the problem.
Steps to reproduce
Haven't tried to reproduce it on other instances. Error occured directly after installing Beta5 with the web installer. Tried to upgrade to Beta6 with a manual upgrade. Nothing changed.
Expected behavior
200 OK instead of 500 Internal Server Error.
Installation method
Community Manual installation with Archive
Operating system
Debian/Ubuntu
PHP engine version
PHP 7.4
Web server
Nginx
Database engine version
MySQL
Is this bug present after an update or on a fresh install?
Updated to a major version (ex. 22.2.3 to 23.0.1)
Are you using the Nextcloud Server Encryption module?
No response
What user-backends are you using?
Configuration report
List of activated Apps
Nextcloud Signing status
No response
Nextcloud Logs
Additional info
No response
The text was updated successfully, but these errors were encountered: