Firebase Cloud功能会将多个相同的通知发送到同一设备

时间:2018-03-18 20:19:01

标签: android firebase firebase-cloud-messaging google-cloud-functions

我有一个firebase云功能,应该在新的"紧急情况下向最近的5个手机发送通知。被添加到数据库中。我写的代码确实会向最近的5部手机发送通知,但它会一遍又一遍地发送相同的通知。当我的用户登录或关闭时,它会变得更糟,因为它会发送更多内容。我很困惑为什么我的云功能不会只运行一次然后终止。

以下是我的云功能代码:

const functions = require('firebase-functions');
const admin = require('firebase-admin');
admin.initializeApp(functions.config().firebase);

exports.sendPushNotificationAdded = functions.database.ref('/emergencies/{id}').onCreate(event => {
    return admin.database().ref('/tokens').on('value', function(snapshot) {
        var efarArray = snapshotToArray(snapshot, event.data.child('latitude').val(), event.data.child('longitude').val());
        efarArray.sort(function(a, b) {
            return a.distance - b.distance;
        });
        var payload = {
            notification: {
                title: "NEW EMERGANCY!",
                body: "Message from Patient: " + event.data.child('other_info').val(),
                //badge: '1',
                sound: 'default',
            }
        };
        var options = {
          priority: "high",
          timeToLive: 0
        };
        tokens_to_send_to = [];
        if(efarArray.length >= 5){
            //only send to the 5 closest efars
            for (var i = 4; i >= 0; i--) {
                tokens_to_send_to.push(efarArray[i].token);
            }
        }else{
            for (var i = efarArray.length - 1; i >= 0; i--) {
                tokens_to_send_to.push(efarArray[i].token);
            }
        }
        //TODO: send a messaged back to patient if no efars respond or are found?
        return admin.messaging().sendToDevice(tokens_to_send_to, payload, options).then(response => {

        });
    });
});

//code for function below from https://ilikekillnerds.com/2017/05/convert-firebase-database-snapshotcollection-array-javascript/
function snapshotToArray(snapshot, incoming_latitude, incoming_longitude) {
    var returnArr = [];

    snapshot.forEach(function(childSnapshot) {
        var distance_to_efar = distance(childSnapshot.child('latitude').val(), childSnapshot.child('longitude').val(), incoming_latitude, incoming_longitude);
        var item = {
            latitude: childSnapshot.child('latitude').val(), 
            longitude: childSnapshot.child('longitude').val(),
            token: childSnapshot.key,
            distance: distance_to_efar
        };

        returnArr.push(item);
    });

    return returnArr;
};

如果需要更多说明或代码,请告诉我。我一直坚持这个......

1 个答案:

答案 0 :(得分:3)

不要将on()与云功能一起使用。这几乎从来都不是正确的用法,因为它增加了一个监听器,随着数据库的变化可以被调用多次。使用once()获取单个快照并对其进行操作。

此外,您必须从函数中返回一个promise,该函数在该函数中的所有异步工作完成时解析。 on()没有返回承诺,所以你的功能也没有。

您可能想学习一些official sample code并遵循那里建立的模式。