Emit Lob end event before close. Another fix by @bjouhier

Author: cjbj

CHANGELOG.md

@@ -8,7 +8,7 @@
 
 - Changed write-only attributes to allow console.log() on Connection objects. Note the attribute values will show as null; refer to the documentation.
 
-- Remove non-portable memory allocation for queries that return NULL.
+- Removed non-portable memory allocation for queries that return NULL.
 
 - Added check to make sure `maxRows` is greater than zero for non-ResultSet queries.
 
@@ -24,6 +24,8 @@
 
 - Added a `type` property to the Lob class to distinguish CLOB and BLOB types.
 
+- Corrected the order of Stream 'end' and 'close' events when reading a LOB.
+
 - Made installation halt sooner for Node.js versions currently known to be unusable.
 
 ## node-oracledb v1.1.0 (3 Sep 2015)

examples/blobstream2.js

@@ -86,6 +86,11 @@ oracledb.getConnection(
                    }
                  });
                });
+        lob.on('close',
+               function(chunk)
+               {
+                 console.log("lob.on 'close' event");
+               });
         lob.on('error',
                function(err)
                {

examples/clobstream2.js

@@ -81,6 +81,11 @@ oracledb.getConnection(
                     }
                   });
                 });
+        lob.on('close',
+               function(chunk)
+               {
+                 console.log("lob.on 'close' event");
+               });
         lob.on('error',
                 function(err)
                 {

lib/oracledb.js

@@ -66,10 +66,12 @@ Lob.prototype._read = function()
         self.emit('error', err);
         return;
       }
+      self.push(str);
       if (!str) {
-        self.close();
+        process.nextTick(function() {
+          self.close();
+        });
       }
-      self.push(str);
     });
 }