Working with Shared Drives about flysystem-google-drive-ext HOT 13 CLOSED

Fixed on #63, please test that branch
Thanks @xel1045 for the suggestions

from flysystem-google-drive-ext.

I have the same problem on my side and it seems to be related with the setPathPrefix in method setTeamDriveId. When I update it to the following, listContents works just fine:

    public function setTeamDriveId($teamDriveId, $corpora = 'teamDrive')
    {
        $this->enableTeamDriveSupport();
        $this->optParams = array_merge_recursive($this->optParams, [
            'files.list' => [
                'corpora' => $corpora,
                'includeTeamDriveItems' => true,
                'teamDriveId' => $teamDriveId
            ]
        ]);

        if ($this->root === 'root' || $this->root === null) {
            $this->setPathPrefix(''); // $this->setPathPrefix($teamDriveId);
            $this->root = $teamDriveId;
        }
    }

Also, I see that the parameter includeTeamDriveItems is deprecated and should be replace by includeItemsFromAllDrives (see https://developers.google.com/drive/api/v3/reference/files/list)

from flysystem-google-drive-ext.

Good news, I'm really glad it helped!

from flysystem-google-drive-ext.

Yes, it is a hack.

from flysystem-google-drive-ext.

For the record, this is cross posted. I asked @erikn69 as well on their repo but we weren’t able to find a solve.

from flysystem-google-drive-ext.

But wich laravel versión? Package versión?

from flysystem-google-drive-ext.

Apologies! Laravel 9, Flysystem 3

from flysystem-google-drive-ext.

I already test team drives, and shared folders on Laravel 9, Flysystem 3, everything working as expected

from flysystem-google-drive-ext.

@xel1045 run the tests, with and without your change
(be carefull with the usePermanentDelete)

from flysystem-google-drive-ext.

@xel1045, thanks: it works for me.

from flysystem-google-drive-ext.

I have the exact same problem as @rjshreve. The fix proposed by @xel1045 works, thank you for the suggestion. Where do we go from here, @erikn69? Would it be appropriate to open a PR?

from flysystem-google-drive-ext.

Who needs a fix could apply the @xel1045's solution using the spatie/invade package like so:

$adapter = new GoogleDriveAdapter(
    service: new \Google_Service_Drive($client),
    root: '',
    options: [ 'teamDriveId' => $config['teamDriveId'] ]
);

invade($adapter)->setPathPrefix('');

from flysystem-google-drive-ext.

@masterix21 Yes but that is more of a hack than a fix, right?

Edit: Also, though I recognize the potential usefulness of spatie/invade in working around situations like these, I really feel it is bad practice to trespass on private properties (pun intended).

from flysystem-google-drive-ext.