-
Notifications
You must be signed in to change notification settings - Fork 278
File System API Migration
Here are the API changes that will result from the [File System Evolution](File System) plan:
("Usage" column = how many extensions currently use the API)
These APIs are either unchanged or "drop-in compatible" - code using them probably does not need to change at all, though in some unusual usage scenarios the APIs may behave differently from before.
| Old API | New API | Notes | Usage |
| FileEntry DirectoryEntry |
File Directory |
Drop-in compatible (same fields/members) | (lots) |
| entry.fullPath, entry.isFile, entry.isDirectory | (unchanged) | Properties are now read-only | fullPath: many isFile/Directory: 9 |
| FileUtils.readAsText(fileEntry) | FileUtils.readAsText(file) | Drop-in compatible | 13 |
| FileUtils.writeText(fileEntry) | FileUtils.writeText(file) | Drop-in compatible | 5 |
| Concatenation: folderPath + "/" + filename | (unchanged) | Doubled "/"es are now normalized out by fs APIs | |
| Concatenation: folderPath + filename | (unchanged) | Directory.fullPath always ends in "/", just like DirectoryEntry.fullPath | Several |
| Substrings: relpath = fullPath.slice(dirFullPath.length) | (unchanged) | Directory.fullPath always ends in "/", just like DirectoryEntry.fullPath |
These APIs are completely gone - code using them will throw exceptions. Extensions using these APIs will be broken immediately, and should be fixed ASAP.
| Old API | New API | Notes | Usage |
| fileEntry.createWriter()... writer.write(text) | file.write(text) | 5 (unused in 3) | |
| fileEntry.file()... new NativeFileSystem.FileReader().readAsText(fileObj) | file.readAsText() | None | |
| directoryEntry.createReader().readEntries() | directory.getContents() | 5 | |
| DirectoryEntry.getFile(relpath, {create:true}) | filesystem.getFileForPath(fullPath).write("") | 2 | |
| DirectoryEntry.getDirectory(relpath, {create:true}) | filesystem.getDirectoryForPath(fullPath).create() | 2 | |
| ProjectManager "projectFilesChange" event | FileSystem "change" event | 2 | |
| fileEntry.getMetadata() | file.stat() | 1 | |
| NativeFileError.* | FileSystemError.* | 1 | |
| instanceof NativeFileSystem.InaccessibleFileEntry | instanceof InMemoryFile | 1 | |
| entry.remove() | entry.moveToTrash() | None | |
| entry.filesystem | n/a | None | |
| NativeFileSystem.Encodings.* | (none) | None | |
| NativeFileSystem.isRelativePath() | (none) | None |
These APIs will temporarily continue to behave as before, but will be removed in the near future. Extensions using these APIs should be fixed relatively soon, within the next sprint or two.
Under the hood, the old API has been reimplemented in terns of the new APIs, so there may be subtle differences noticeable in some edge use cases. (Also - the shim code may be a helpful reference when migrating over to the new APIs).
| Old API | New API | Notes | Usage |
| new NativeFileSystem.FileEntry(fullPath) new NativeFileSystem.DirectoryEntry(fullPath) |
filesystem.getFileForPath(fullPath) filesystem.getDirectoryForPath(fullPath) |
These now return a File / Directory object using the new APIs instead of constructing a FileEntry / DirectoryEntry. (You never directly construct a File / Directory). | 19 |
| DirectoryEntry.getFile(relpath) | filesystem.resolve(dirFullPath + relpath) | 4 | |
| NativeFileSystem.resolveNativeFileSystemPath(fullPath) | filesystem.resolve(path) | 4 | |
| NativeFileSystem.requestNativeFileSystem(fullPath)... fs.root | filesystem.resolve(fullPath) | 6 | |
| NativeFileSystem.showOpenDialog() NativeFileSystem.showSaveDialog() |
filesystem.showOpenDialog() filesystem.showSaveDialog() |
4 | |
| FileIndexManager.getFileInfoList("all") FileIndexManager.getFileInfoList("...") |
ProjectManager.getAllFiles() ProjectManager.getAllFiles(filter) |
Returns an array of Files, but they provide same properties as the old FileInfos | 7 |
| FileIndexManager.getFilenameMatches("...", filename) | ProjectManager.getAllFiles(filter) | None | |
| brackets.fs.*() | (various) | These low-level APIs continue to exist, but have never been officially supported. They may break at any time. | Several |
The following 7 extensions use at least one API on the "Removed" list:
- angularui.angularjs -- (FileEntry.getMetadata())
- bsirlinger.github-access -- (DirectoryEntry.getFile({create}), DirectoryEntry.getDirectory({create}), FileEntry.createWriter())
- xunit -- (DirectoryEntry.getFile({create}), DirectoryEntry.getDirectory({create})
- zaggino.brackets-git (won't crash, but notable bug) -- ("projectFilesChange" event)
- variousimprovements (won't crash, minor bug) -- ("projectFilesChange" event)
- pflynn.brackets.editor.nav (won't crash, very minor bug) -- (InaccessibleFileEntry)
- interactive-linter (no effect on behavior) -- (NativeFileError)
We will file bugs with all extensions that are using removed APIs to make sure authors are aware of the change.