exports.getMentionedUsers = function(str) {
    return Promise.all(getUsernamesFromString(str).map(username => {
        return db.ref('/users').orderByChild('username').equalTo(username).once('value').then(snapshot => {
            return snapshot.val();
        });
    }));
}

Right now, if snapshot.val() is null, the element is still included in the final result.

How do I not insert a null element in the final map?

exports.getMentionedUsers = function(str) {
    return Promise.all(getUsernamesFromString(str).map(username => {
        return db.ref('/users').orderByChild('username').equalTo(username).once('value').then(snapshot => {
            return snapshot.val();
        });
    }));
}

Right now, if snapshot.val() is null, the element is still included in the final result.

How do I not insert a null element in the final map?

• javascript
• promise

Share Improve this question Follow asked Aug 29, 2017 at 22:04 TIMEXTIMEX 272k367367 gold badges802802 silver badges1.1k1.1k bronze badges 3

• if(snapshot.val()) return snapshot.val() – TimCodes Commented Aug 29, 2017 at 22:10
• Use Array filter function and filter the null elements from your array – sidanmor Commented Aug 29, 2017 at 22:12
• .then(arr => arr.filter(Boolean))? – Bergi Commented Aug 29, 2017 at 22:23

Add a ment  | 

1 Answer 1

Sorted by: Reset to default 8

Add then callback and use Array#filter to remove null elements:

exports.getMentionedUsers = function(str) {
    return Promise
        .all(getUsernamesFromString(str).map(username => {
            return db.ref('/users').orderByChild('username').equalTo(username).once('value').then(snapshot => {
              return snapshot.val();
            });
        }))
        .then(values => values.filter(v => v);
}

Update: If you need to delete elements exactly with null value, you should use this filter: v => v !== null.