Suggesting an autosave option

[fritx]

// Compose every function with fn
function composeAll(obj, fn) {
  for (var key in obj) {
    if (_.isFunction(obj[key])) {
      obj[key] = _.compose(fn, obj[key])
    }
  }
}

Hey, I doubt the composing on all functions including irrelevant ones (like isRegExp)
Shall we pick out the ones?
Or give it an autosave option which defaults to true, like:

var db = low(filename, { autosave: false })

[typicode]

Picking out the ones irrelevant, why not, it'll improve performances. However, when Node 0.12 will be out (no date though), it should be possible to listen to object changes with Object.observe so I don't know if it's very useful to implement right now.

Regarding the autosave, you can already but it's not very beautiful:

var db = low() // no filename, so there's no autosave
fs.writeFileSync(filename, JSON.stringify(db.object))

I think about making something like what you suggested:

var db = low(filename, { autosave: false })
db.save()
// or
var db = low()
db.save(filename)

[fritx]

Oh you're right.. :)
If you're not free, I'm glad to PR

[typicode]

It's okay, thanks for the offer and the feedback :)
I think I'll release today.

[typicode]

Just published v0.6.0 :)
It includes autosave option and I've added db.saveSync() for more flexibility.

var db = low(filename, { autosave: false })
db.saveSync()

var db = low()
db.saveSync(filename)

Both examples will do the same.

[fritx]

👍 I see the change, async defaults to true too, thanks

[typicode]

Hi,

Just as an update, I've released 0.7.0 and now it only writes to disk when db has changed (not composing every functions anymore :)).

Also, it has been updated to lodash 3.0.0.

[fritx]

Awesome!