Support for INSERT ... RETURNING about node-postgres HOT 7 CLOSED

RETURNING works. It triggers a "row" event just fine.

If you want to use event listeners, then do this:

query.on('row', function (row) {
  // do some stuff
})

Currently the result does not automatically accumulate rows unless you use a callback. If you want to accumulate rows using event listeners, you have to do this:

query.on('row', function (row, result) {
  result.addRow(row);
});
query.on('end', function (result) {
  if (result.rowCount > 0) {
    // RETURNING used
    doSomething(result.rows[0]);
  } else {
    // standard end event
  }
});

This issue could be closed.

from node-postgres.

While the FAQ seems to imply that support for INSERT.. RETURNING already exists (https://github.com/brianc/node-postgres/wiki/FAQ), that does not seem to be the case. The callback function is never called on non-error results (according to my scenarios).

from node-postgres.

I have the same experience, have to resort to using CURRVAL to get the last insert id :/ which is 1 extra query

from node-postgres.

Seems like the "INSERT... RETURNING" statement only works with a callback and not with event listeners. Take a look at this gist https://gist.github.com/1662362 for a working example.

query.on('end', function(row) {
   if(row) {
      // RETURNING uesd
   }
   else {
     // standard end event
   }
});

Keep in mind that the statement can return multiple values. Either change it to query.on('end', function(rows)... or use the row event if you add this feature.

Regards
Philipp

from node-postgres.

This is great to know.
For clarity item 7 in the FAQ should be updated, however: https://github.com/brianc/node-postgres/wiki/FAQ

from node-postgres.

Cool, I updated it.

(It is a wiki after all.)

from node-postgres.

Thank you! ❤️

from node-postgres.