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

elevation correction error #12

Closed
Synochy opened this issue Feb 2, 2023 · 4 comments
Closed

elevation correction error #12

Synochy opened this issue Feb 2, 2023 · 4 comments

Comments

@Synochy
Copy link

Synochy commented Feb 2, 2023

When using the elevation correction feature.
The feature fails and there is the following error in the server logbook:
Optional parameter $recursive declared before required parameter $directoryId is implicitly treated as a required parameter at /var/www/nextcloud/apps/gpxpod/lib/Service/ProcessService.php#803

@julien-nc
Copy link
Owner

Thanks for the bug report. Could you try to just remove the default value for $recursive on gpxpod/lib/Service/ProcessService.php line 803?

public function getGeoPicsFromFolder(string $userId, string $subfolder, bool $recursive = false, int $directoryId): array {

should become

public function getGeoPicsFromFolder(string $userId, string $subfolder, bool $recursive, int $directoryId): array {

And then confirm the issue goes away on your side?

julien-nc added a commit that referenced this issue Feb 2, 2023
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@Synochy
Copy link
Author

Synochy commented Feb 3, 2023

I do not have error in the logbook anymore but the elevation correction still fails with the following message:
image

@Synochy
Copy link
Author

Synochy commented Feb 23, 2023

Any other idea to fix the problem ?

@Synochy
Copy link
Author

Synochy commented Feb 25, 2023

I corrected my problem by unstalling and installing again the app

@Synochy Synochy closed this as completed Feb 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants