@inject(FileSystem) will inject the type instead of the actual object

[slhultgren]

I know that the FileSystem is deprecated, however it is currently broken since the bad thing is injected.

https://github.com/eclipse-theia/theia/blob/master/packages/filesystem/src/browser/filesystem-frontend-module.ts#L85

Shouldn't this return a new instance instead of just a class?

When running this code:
@Inject(FileSystem) private fileSystem: FileSystem;
...
console.log("fs.exists: " + this.fileSystem.exists);

You will get "fs.exists: undefined"

However, (this.fileSystem as any).prototype.exists does exist.

[vince-fugnitto]

cc @akosyakov

[akosyakov]

Good catch! Please send a PR.