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

TypeError: Argument 4 passed to OCA\GroupFolders\Versions\GroupVersion::__construct() #1186

Closed
trvrcr opened this issue Dec 21, 2020 · 9 comments
Labels
0. Needs triage Issues that need to be triaged bug error_log feature: versions Items related to file versioning low low priority

Comments

@trvrcr
Copy link

trvrcr commented Dec 21, 2020

TypeError: Argument 4 passed to OCA\GroupFolders\Versions\GroupVersion::__construct() must be of the type int, float given, called in /var/www/nextcloud/apps/groupfolders/lib/Versions/VersionsBackend.php on line 66

NC 20.0.4
php7.4
MariaDB 10.3

@pierreozoux
Copy link
Member

Do you have user error? or just server error?

@pierreozoux pierreozoux added the 0. Needs triage Issues that need to be triaged label Mar 11, 2021
@trvrcr
Copy link
Author

trvrcr commented Mar 14, 2021

Do you have user error? or just server error?

what do you mean with user error?
The error is on the server so it is certainly a server error.

I already updated to version NC 21.0.0.18 and the problem is persisting

@pierreozoux
Copy link
Member

I mean, if there is just an error in the log, and no strange behavior for the end user, the priority of fixing this error is lower than end user errors. You see what I mean?

@trvrcr
Copy link
Author

trvrcr commented Mar 14, 2021

I see what you mean, I have not yet encountered any problems in the user space that can be linked to this.
Just want to make you aware of the issue.
I'm also willing to test out any propositions to fix the issue.

(I already tried fixing it by changing the code and requesting float instead of Int but that created another error)
With a little coaching I might help fix the issue...

@pierreozoux
Copy link
Member

I'm not the php person, just trying to help triage the issues here :)

@trvrcr
Copy link
Author

trvrcr commented Mar 14, 2021

Maybe you can redirect to the correct person

@trvrcr
Copy link
Author

trvrcr commented Sep 22, 2021

Still present in version 21.0.4

@joshtrichards joshtrichards added the feature: versions Items related to file versioning label Sep 30, 2023
@joshtrichards
Copy link
Member

This code no longer exists as of #2543.

If this issue still exists today it'll manifest itself differently, if at all. I'm going to close this out.

Out of curiosity, were you perhaps running NC on a 32-bit platform at the time of your report?

@joshtrichards joshtrichards closed this as not planned Won't fix, can't repro, duplicate, stale Oct 5, 2023
@trvrcr
Copy link
Author

trvrcr commented Oct 5, 2023

It was running on 32-bit at a certain time but I upgraded to 64bit.
I cannot recall if it was before or after the upgrade of the system.
Will keep an eye on if it still persists/manifests in another way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Issues that need to be triaged bug error_log feature: versions Items related to file versioning low low priority
Projects
None yet
Development

No branches or pull requests

3 participants