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: OCA\GroupFolders\ACL\ACLManagerFactory::getACLManager(): Argument #1 ($user) must be of type OCP\IUser #2473

Closed
proserver1 opened this issue Jul 18, 2023 · 14 comments · Fixed by #2476
Labels
3. to review Items that need to be reviewed bug regression

Comments

@proserver1
Copy link

since last update:

Updated 19 hours ago
(now: Tue Jul 18 03:02:05 PM CEST 2023)
Nextcloud 26 14.0.3

cronjobs throw errors:

TypeError: OCA\GroupFolders\ACL\ACLManagerFactory::getACLManager(): Argument #1 ($user) must be of type OCP\IUser, null given, called in /srv/foobar/nextcloud-26.0.3/apps_custom/groupfolders/lib/Mount/MountProvider.php on line 206 and defined in /srv/foobar/nextcloud-26.0.3/apps_custom/groupfolders/lib/ACL/ACLManagerFactory.php:37 Stack trace: #0 /srv/foobar/nextcloud-26.0.3/apps_custom/groupfolders/lib/Mount/MountProvider.php(206): OCA\GroupFolders\ACL\ACLManagerFactory->getACLManager() #1 /srv/foobar/nextcloud-26.0.3/apps_custom/groupfolders/lib/Versions/VersionsBackend.php(172): OCA\GroupFolders\Mount\MountProvider->getMount() #2 /srv/foobar/nextcloud-26.0.3/apps_custom/groupfolders/lib/Versions/GroupVersionsExpireManager.php(68): OCA\GroupFolders\Versions\VersionsBackend->getAllVersionedFiles() #3 /srv/foobar/nextcloud-26.0.3/apps_custom/groupfolders/lib/Versions/GroupVersionsExpireManager.php(59): OCA\GroupFolders\Versions\GroupVersionsExpireManager->expireFolder() #4 /srv/foobar/nextcloud-26.0.3/apps_custom/groupfolders/lib/BackgroundJob/ExpireGroupVersions.php(43): OCA\GroupFolders\Versions\GroupVersionsExpireManager->expireAll() #5 /srv/foobar/nextcloud-26.0.3/lib/public/BackgroundJob/Job.php(78): OCA\GroupFolders\BackgroundJob\ExpireGroupVersions->run() #6 /srv/foobar/nextcloud-26.0.3/lib/public/BackgroundJob/TimedJob.php(103): OCP\BackgroundJob\Job->start() #7 /srv/foobar/nextcloud-26.0.3/lib/public/BackgroundJob/TimedJob.php(93): OCP\BackgroundJob\TimedJob->start() #8 /srv/foobar/nextcloud-26.0.3/cron.php(152): OCP\BackgroundJob\TimedJob->execute() #9 {main}

happens on all instances
PHP 8.1.2-1ubuntu2.13

@proserver1 proserver1 added 0. Needs triage Issues that need to be triaged bug labels Jul 18, 2023
@DavidMndz
Copy link

I confirm the error is present in Nextcloud 27.0.0 as well.

@gregoa
Copy link

gregoa commented Jul 19, 2023

Maybe related to #2464

@BJKle
Copy link

BJKle commented Jul 19, 2023

This is unrelated to OS and NC Version. I have it it different instances.

@julian70400
Copy link

julian70400 commented Jul 19, 2023

Same here, Nextcloud 26.0.3, since new version of Group Folder.

Is there a any impact ? To me, it just show this error, but no issue users side.

@alphasixtyfive
Copy link

Same issue after updating to 26.0.3. No issues have been reported by the users though.

@koelle25
Copy link

koelle25 commented Jul 19, 2023

This problem also affects the occ groupfolders:scan command, which throws the exception mentioned in OP and does not work anymore.

www-data@38705baa1282:~/html$ ./occ groupfolders:scan 4
An unhandled exception has been thrown:
TypeError: OCA\GroupFolders\ACL\ACLManagerFactory::getACLManager(): Argument #1 ($user) must be of type OCP\IUser, null given, called in /var/www/html/custom_apps/groupfolders/lib/Mount/MountProvider.php on line 206 and defined in /var/www/html/custom_apps/groupfolders/lib/ACL/ACLManagerFactory.php:37
Stack trace:
#0 /var/www/html/custom_apps/groupfolders/lib/Mount/MountProvider.php(206): OCA\GroupFolders\ACL\ACLManagerFactory->getACLManager(NULL, 647)
#1 /var/www/html/custom_apps/groupfolders/lib/Command/Scan.php(82): OCA\GroupFolders\Mount\MountProvider->getMount(4, '/daad_project', 31, 107374182400)
#2 /var/www/html/3rdparty/symfony/console/Command/Command.php(255): OCA\GroupFolders\Command\Scan->execute(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#3 /var/www/html/core/Command/Base.php(177): Symfony\Component\Console\Command\Command->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#4 /var/www/html/3rdparty/symfony/console/Application.php(1009): OC\Core\Command\Base->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#5 /var/www/html/3rdparty/symfony/console/Application.php(273): Symfony\Component\Console\Application->doRunCommand(Object(OCA\GroupFolders\Command\Scan), Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#6 /var/www/html/3rdparty/symfony/console/Application.php(149): Symfony\Component\Console\Application->doRun(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#7 /var/www/html/lib/private/Console/Application.php(211): Symfony\Component\Console\Application->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#8 /var/www/html/console.php(100): OC\Console\Application->run()
#9 /var/www/html/occ(11): require_once('/var/www/html/c...')

Edit: I tried to roll back to v14.0.2, but then Nextcloud requires an occ upgrade, which in turn updates the groupfolders app back to v14.0.3. Any idea why?

@solracsf

This comment was marked as resolved.

@rkl110

This comment was marked as resolved.

@solracsf

This comment was marked as duplicate.

@ebdavison

This comment was marked as outdated.

@solracsf
Copy link
Member

See #2476

@ebdavison
Copy link

The fix in #2476 works for me with occ groupfolders:scan. Thanks.

@solracsf solracsf added 3. to review Items that need to be reviewed regression and removed 0. Needs triage Issues that need to be triaged labels Jul 19, 2023
@proserver1
Copy link
Author

fix in #2476 works with occ and cron. thank you

@solracsf
Copy link
Member

Fix has been released.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Items that need to be reviewed bug regression
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants