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

Optional parameter $userId declared before required parameter $path is implicitly treated as a required parameter at /var/www/apps/deck/lib/Service/ConfigService.php#234 #5793

Closed
snetAT opened this issue Apr 24, 2024 · 4 comments · Fixed by #5794

Comments

@snetAT
Copy link

snetAT commented Apr 24, 2024

After the update to nextcloud 29 the following is written to the logfile every minute:

Optional parameter $userId declared before required parameter $path is implicitly treated as a required parameter at /var/www/apps/deck/lib/Service/ConfigService.php#234

List of activated apps:
...
user_external ?

@juliushaertl
Copy link
Member

Thanks for reporting, fix is in #5794

@PrzemekSkw
Copy link

PrzemekSkw commented Jun 14, 2024

Thanks for reporting, fix is in #5794

Can You expalin please?
I don't see any fix there.
Regards.

@janomat
Copy link

janomat commented Jun 29, 2024

I received the error message mentioned above with every run of the cron jobs aftwer upgrading to NC 28.0.7 and deck app to Version 1.12.3 today.

Line 234 in /apps/deck/lib/Service/ConfigService.php needs to be edited.

Described here fix: Avoid optional before required parameter #5794

After editing ConfigService.php the error logs are clean.

Edited: added the deck app version

@snetAT
Copy link
Author

snetAT commented Jun 29, 2024

Update your deck app to 1.13.1
1.13.1 is only available for Nextcloud 29

Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants