Deck 1.13. error in log after upgrade to NC 29

ℹ️ Support
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

3 Likes
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!

1 Like
4

After the 29.0.2 update, the error is back. I don’t know what to do from here. Anyone?

5

There are some issues.
NC29 Optional parameter $userId declared before required parameter $path is implicitly treated as a required parameter · Issue #5938 · nextcloud/deck · GitHub

6

I saw it. I think it will be an official release to fix this soon.

7

I just updated to Nextcloud Hub 8 (29.0.3) and the error is still present.

1 Like