v0.0.27

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "filer",
-  "version": "0.0.26",
+  "version": "0.0.27",
   "main": "dist/filer.js",
   "devDependencies": {
     "mocha": "1.17.1",

dist/filer.js

@@ -13812,10 +13812,13 @@ function validate_file_options(options, enc, fileMode){
 
 function pathCheck(path, callback) {
   var err;
-  if(isNullPath(path)) {
+
+  if(!path) {
+    err = new Errors.EINVAL('Path must be a string', path);
+  } else if(isNullPath(path)) {
     err = new Errors.EINVAL('Path must be a string without null bytes.', path);
   } else if(!isAbsolutePath(path)) {
-    err = new Errors.EINAVL('Path must be absolute.', path);
+    err = new Errors.EINVAL('Path must be absolute.', path);
   }
 
   if(err) {
@@ -14526,9 +14529,8 @@ function FileSystem(options, callback) {
   }
 
   // Open file system storage provider
-  provider.open(function(err, needsFormatting) {
+  provider.open(function(err) {
     function complete(error) {
-
       function wrappedContext(methodName) {
         var context = provider[methodName]();
         context.flags = flags;
@@ -14571,20 +14573,22 @@ function FileSystem(options, callback) {
       return complete(err);
     }
 
-    // If we don't need or want formatting, we're done
-    if(!(forceFormatting || needsFormatting)) {
-      return complete(null);
-    }
-    // otherwise format the fs first
     var context = provider.getReadWriteContext();
     context.guid = wrappedGuidFn(context);
-    context.clear(function(err) {
+
-      if(err) {
+    // Mount the filesystem, formatting if necessary
-        complete(err);
+    if(forceFormatting) {
-        return;
+      // Wipe the storage provider, then write root block
-      }
+      context.clear(function(err) {
+        if(err) {
+          return complete(err);
+        }
+        impl.ensureRootDirectory(context, complete);
+      });
+    } else {
+      // Use existing (or create new) root and mount
       impl.ensureRootDirectory(context, complete);
-    });
+    }
   });
 }
 
@@ -15184,15 +15188,10 @@ IndexedDB.prototype.open = function(callback) {
   var that = this;
 
   // Bail if we already have a db open
-  if( that.db ) {
+  if(that.db) {
-    callback(null, false);
+    return callback();
-    return;
   }
 
-  // Keep track of whether we're accessing this db for the first time
-  // and therefore needs to get formatted.
-  var firstAccess = false;
-
   // NOTE: we're not using versioned databases.
   var openRequest = indexedDB.open(that.name);
 
@@ -15204,13 +15203,11 @@ IndexedDB.prototype.open = function(callback) {
       db.deleteObjectStore(FILE_STORE_NAME);
     }
     db.createObjectStore(FILE_STORE_NAME);
-
-    firstAccess = true;
   };
 
   openRequest.onsuccess = function onsuccess(event) {
     that.db = event.target.result;
-    callback(null, firstAccess);
+    callback();
   };
   openRequest.onerror = function onerror(error) {
     callback(new Errors.EINVAL('IndexedDB cannot be accessed. If private browsing is enabled, disable it.'));
@@ -15241,14 +15238,10 @@ var asyncCallback = _dereq_('../../lib/async.js').setImmediate;
 var createDB = (function() {
   var pool = {};
   return function getOrCreate(name) {
-    var firstAccess = !pool.hasOwnProperty(name);
+    if(!pool.hasOwnProperty(name)) {
-    if(firstAccess) {
       pool[name] = {};
     }
-    return {
+    return pool[name];
-      firstAccess: firstAccess,
-      db: pool[name]
-    };
   };
 }());
 
@@ -15313,11 +15306,8 @@ Memory.isSupported = function() {
 };
 
 Memory.prototype.open = function(callback) {
-  var result = createDB(this.name);
+  this.db = createDB(this.name);
-  this.db = result.db;
+  asyncCallback(callback);
-  asyncCallback(function() {
-    callback(null, result.firstAccess);
-  });
 };
 Memory.prototype.getReadOnlyContext = function() {
   return new MemoryContext(this.db, true);
@@ -15462,8 +15452,7 @@ WebSQL.prototype.open = function(callback) {
 
   // Bail if we already have a db open
   if(that.db) {
-    callback(null, false);
+    return callback();
-    return;
   }
 
   var db = global.openDatabase(that.name, WSQL_VERSION, WSQL_DESC, WSQL_SIZE);
@@ -15480,18 +15469,7 @@ WebSQL.prototype.open = function(callback) {
   }
   function onSuccess(transaction, result) {
     that.db = db;
-
+    callback();
-    function gotCount(transaction, result) {
-      var firstAccess = result.rows.item(0).count === 0;
-      callback(null, firstAccess);
-    }
-    function onError(transaction, error) {
-      callback(error);
-    }
-    // Keep track of whether we're accessing this db for the first time
-    // and therefore needs to get formatted.
-    transaction.executeSql("SELECT COUNT(id) AS count FROM " + FILE_STORE_NAME + ";",
-                           [], gotCount, onError);
   }
 
   // Create the table and index we'll need to store the fs data.

package.json

@@ -11,7 +11,7 @@
     "idb",
     "websql"
   ],
-  "version": "0.0.26",
+  "version": "0.0.27",
   "author": "Alan K <ack@modeswitch.org> (http://blog.modeswitch.org)",
   "homepage": "http://js-platform.github.io/filer",
   "bugs": "https://github.com/js-platform/filer/issues",