search

pondersource / sciencemesh-php

Connect your Nextcloud server to Sciencemesh
MIT License
0 stars 1 forks source link

NC PR testing #134

Closed michielbdejong closed 1 year ago

michielbdejong commented 1 year ago

After fixing #132 there still is a generic "failed to create share" error.

michielbdejong commented 1 year ago

I think this was because I hadn't rebuilt the NC docker image, it couldn't find the getSciencemeshHelper function on the ShareAPIController, that's an issue I already fixed earlier this week.

michielbdejong commented 1 year ago

Next error:

This version of Nextcloud requires at least PHP 8.0
You are currently running 7.4.3-4ubuntu2.17. Please update your PHP version.
michielbdejong commented 1 year ago

Next error: Error creating the share: Sharing /einstein/files/asdf failed because the back end does not support sciencemesh shares

michielbdejong commented 1 year ago

Could not resolve OCA\ScienceMesh\Sharing\ShareAPIHelper! Class "OCA\ScienceMesh\Sharing\ShareAPIHelper" does not exist

michielbdejong commented 1 year ago

Could not resolve OCA\ScienceMesh\Sharing\ShareAPIHelper! Class "OCA\ScienceMesh\Sharing\ShareAPIHelper" does not exist

fixed with https://github.com/pondersource/nc-sciencemesh/commit/cad153f1531971951cf738359c47eb63b1447d7e

Apparently DI could not find OCA\ScienceMesh\Sharing\ShareAPIHelper when it was in lib/ShareProvider/ShareAPIHelper.php but it can find it if it's in lib/Sharing/ShareAPIHelper.php. Note the similarity between the file path of the code and the namespace path of the class.