Retrieve Objects Ids by default when useDisplayPaths is set to False about flysystem-google-drive-ext HOT 6 CLOSED

Hi,
this driver is focused on the case when useDisplayPaths==true so the support for a false case might be lacking. However, when you call the createDir function you get back an array. The path should contain the id. Does it not?
See this snippet from the createDir:

...
        $folder = $this->createDirectory($name, $pdir !== '' ? basename($pdir) : $pdir);
        if ($folder !== null) {
            $itemId = $folder->getId();
            $this->cacheFileObjects[$itemId] = $folder;
            $this->cacheHasDirs[$itemId] = false;
            $this->cacheObjects([$itemId => $folder]);
            $path_parts = $this->splitFileExtension($name);
            $result = [
                'path' => Util::normalizeDirname($pdir).'/'.($this->useDisplayPaths ? $name : $itemId), // <-here the $itemId should be the folder ID
                'filename' => $path_parts['filename'],
                'extension' => $path_parts['extension']
            ];
            return $result;
        }

        return false;
...

from flysystem-google-drive-ext.

Also acording Flysystems V1 Docs you have to use createDir, not 'createDirectory', Flysystems Docs

but if you are using createDirectory , maybe with $result->getId();

from flysystem-google-drive-ext.

@masbug sorry I didn't mentioned it I was asking for branch 2.x which I think uses Flysystems V2 where createDir and CreateDirectory functions are the opposite from version 1.x

from flysystem-google-drive-ext.

So, you can use: $adapter->getMetadata or $adapter->listContents and iterate a list of FileAttributes

$file=$adapter->getMetadata('path/to/file.txt'); //can be directory
if($file){
    $id=$file->getMetadata()['id']
}

because: $result = ['id' => $id]; and
return new FileAttributes($result['display_path'],$object->getSize(),$visibility, $object->getModifiedTime(),$object->mimeType,$result);

There is a PR for more direct access, but it's a breaking change github.com/thephpleague/flysystem/pull/1324

If something doesn't work or can be improved, i can make a PR

from flysystem-google-drive-ext.

I see I need to provide a path to this getMetadata function but the way google drive woks is with Ids. I would need to migrate my current database to use display paths instead of storing Ids I don't think so. Then I guess I will have to stay in V1 of flysystem.
Thank you for your help @erikn69

from flysystem-google-drive-ext.

V2 its a headacke with the throws and this package is focused on the case when useDisplayPaths==true

from flysystem-google-drive-ext.