Fix for the order dats are loaded (#455)

* Making sure that onupdate is not called before the return happened in order to prevent zalgo from hauting us at night.

* Fixing problems with not running update in zalgo style.

* Making sure that the dats are closed before trying to join the network.

lib/dat-manager.js

@@ -20,7 +20,9 @@ function createManager ({ multidat, dbPaused }, onupdate) {
   var dats = multidat.list()
   var speed = { up: 0, down: 0 }
   dats.forEach(initDat)
-  onupdate(null, dats, speed)
+  setImmediate(function () {
+    onupdate(null, dats, speed)
+  })
 
   return {
     create: create,
@@ -144,6 +146,7 @@ function createManager ({ multidat, dbPaused }, onupdate) {
     const key = encoding.toStr(dat.key)
     dbPaused.read((err, paused) => {
       if (err) throw err
+      if (dat._closed) return
       if (!paused[key]) {
         dat.joinNetwork()
         dat.network.on('connection', function (connection) {
@@ -158,6 +161,7 @@ function createManager ({ multidat, dbPaused }, onupdate) {
 
     dat.archive.readFile('/dat.json', function (err, blob) {
       if (err && !dat.writable) return
+      if (dat._closed) return
       if (err) {
         var json = datJson(dat)
         json.write(next)

models/dats.js

@@ -75,9 +75,9 @@ function datsModel (state, bus) {
         state.dats.speed = speed
         state.dats.ready = true
         bus.emit('render')
+        bus.emit('dats:loaded')
+        done()
       })
-      bus.emit('dats:loaded')
-      done()
     }
   ]
 

models/intro.js

@@ -10,7 +10,7 @@ function introModel (state, bus) {
 
   // FIXME: wait for DOMContentLoaded
   // requires some sort of global load event first
-  bus.on('dats:loaded', function () {
+  bus.once('dats:loaded', function () {
     if (state.dats.values.length) return
     state.intro.show = true
     bus.emit('render')