I may be missing something but the docs say passing a callback to db.update is optiona

Ditto. <div class="highlight highlight-source-js notranslate position-relative ove

db.update callback optional? I'm getting an error if I skip it... about nedb HOT 3 CLOSED

louischatriot commented on July 21, 2024

db.update callback optional? I'm getting an error if I skip it...

from nedb.

Comments (3)

kennethgreen commented on July 21, 2024

Ditto.

var Datastore = require('nedb');
var db = {};
db.users = new Datastore('/temp/users.db');
db.users.loadDatabase();

Run this (Win7, node 0.10.12, nedb 0.7.10) and I get the same error as somewhatdog:

C:\node\sites\cto_nat_tools\node_modules\nedb\lib\executor.js:27
      task.arguments.push(callback);
                     ^
TypeError: Object #<Object> has no method 'push'
    at C:\node\sites\cto_nat_tools\node_modules\nedb\lib\executor.js:27:22
    at Object.q.process [as _onImmediate] (C:\node\sites\cto_nat_tools\node_modules\nedb\node_modules\async\lib\async.js
:728:21)
    at processImmediate [as _immediateCallback] (timers.js:330:15)

from nedb.

kennethgreen commented on July 21, 2024

It seems the callback is not optional, not a problem but the doco should be updated.

var util      = require('util');
var Datastore = require('nedb');
var db = {};
db.users = new Datastore('/temp/users.db');
db.users.loadDatabase(function (err) {
    if (null === err)
    {
        console.log('All OK, no error detected');
    }
    else
    {
        console.log('Ouch!\n' + err);
    }
});

Results in: