Deck 1.13. error in log after upgrade to NC 29

🍱 Features & apps deck
1

Hi,
I did upgrade my NC and using deck. (1.13 in use). The NC log file show now the following error:

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

Is it something I can/need to fix via config?

2

Thanks for your report, this is already reported here: 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 · Issue #5793 · nextcloud/deck · GitHub

1 Like
3

I got the same error, so I edit the file as recommended in the GitHub link: fix: Avoid optional before required parameter by juliushaertl · Pull Request #5794 · nextcloud/deck · GitHub

 /var/www/***/apps/deck/lib/Service/ConfigService.php

Seems like the issue is gone.

I’m sharing this just to help newbies like me where exactly to look.

Thanks!