Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

FileUtils.canonicalizeFolderPath() is confusing (now deprecated - need to remove) #2818

Closed
peterflynn opened this issue Feb 8, 2013 · 4 comments

Comments

@peterflynn
Copy link
Member

This utility function "canonicalizes" paths by making sure paths pointing to folders don't end in a trailing slash. However, the canonical format of NativeFileSystem.DirectoryEntry (and thus most paths in the Brackets code), is the opposite of this -- directories always have a trailing slash.

Looking at references to this API, it appears to only be used by code related to Recent Projects. Perhaps we should clean up that code and then remove this API?

@ghost ghost assigned njx Feb 8, 2013
@njx
Copy link

njx commented Feb 8, 2013

To me, low priority.

@njx
Copy link

njx commented Feb 9, 2013

Reviewed.

@peterflynn
Copy link
Member Author

This was deprecated as part of the FileSystem migration. Assigning to me to track eventually removing it (along with the other deprecated FS-related APIs - #6127).

@peterflynn peterflynn assigned peterflynn and unassigned njx Mar 21, 2014
@peterflynn peterflynn changed the title FileUtils.canonicalizeFolderPath() is confusing FileUtils.canonicalizeFolderPath() is confusing (now deprecated - need to remove) Apr 23, 2014
@le717
Copy link
Contributor

le717 commented Feb 14, 2015

@peterflynn This can be closed. The API was removed with #8992.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants