How to Promisify this function - nodejs

Question

I have an ajax call which needs to return a promise. The function is as follows

client.tickets.create(ticket,  function(err, req, result) {
  if (err) {    
    logger.error(err);

    return false;
  }

  return JSON.stringify(result);
});

I have to wait for this function to execute before I can perform the next action. How can I promisify this function ?

I tried the following and it gave me an error saying Cannot call method then of undefined:

return client.tickets.create(ticket).then(function(result){
    return JSON.stringify(result);
},function(err){
    logger.error(err);
    return false;
});

du u use any promise api? – Raja Sekar Jan 12 '16 at 06:16 — Raja Sekar, Jan 12 '16 at 06:16

Shanoor · Accepted Answer · 2016-01-12T06:38:00.667

You have the error because create() is not a Promise. Promisifying an async function is quite easy (nodejs has a built-in Promise support nowadays):

function createTicket(ticket) {
    // 1 - Create a new Promise
    return new Promise(function (resolve, reject) {
        // 2 - Copy-paste your code inside this function
        client.tickets.create(ticket, function (err, req, result) {
            // 3 - in your async function's callback
            // replace return by reject (for the errors) and resolve (for the results)
            if (err) {
                reject(err);
            } else {
                resolve(JSON.stringify(result));
            }
        });
    });
}

// 4 - consume your promise with then() (resolved promise) and catch (rejected promise)
createTicket(ticket).then(function (result) {
    // deal with result here
}).catch(function (err) {
    // deal with error here
});

In release candidate for Node.js 8.0.0, there's a new utility, `util.promisify`, that encapsulates the capacity of promisifying whatever function. It is not much more different from the approaches suggested in the other answer, but has the advantage of being a core method. You may want to read more [in this post](https://brunoscopelliti.com/new-util-promisify-in-nodejs/) — Bruno, May 16 '17 at 05:20

mido · Answer 2 · 2017-05-30T07:17:19.173

rather than manually wrapping async code into promises, I would advice using libraries like Bluebird to do that for you:

var Bluebird = require('bluebird');

//either 
client.tickets = Bluebird.promisifyAll(client.tickets);
//or 
client.tickets.createAsync = Bluebird.promisify(client.tickets.create);
...
    return client.tickets.createAsync(ticket)
      .then(JSON.stringify)
      .catch(err => {
        logger.error(error); 
        return false
      });

score -3 · Answer 3 · answered Jan 12 '16 at 06:22

function createTicket(){
var deferred = Q.defer()
client.tickets.create(ticket,  function(err, req, result) {
     if (err) { 
                logger.error(err);
                return deferred.reject(err)
            }
   return deferred.resolve(result)
});
}

createTicket().then(function(){
    //success here
}, function(){
    //failure here
});

Using Q api you can achive promise.

How to Promisify this function - nodejs

3 Answers3

Linked