typo and adding sql query to get duplicate broadcast notifications

Author: Sachin Maheshwari

migrations/v2.0.4.sql

@@ -27,3 +27,11 @@ AND type = 'admin.notification.broadcast';
 
 -- make unique column to avoid duplicate insert 
 ALTER TABLE bulk_message_user_refs ADD UNIQUE (bulk_message_id, user_id);
+
+-- get duplicate broadcast rows
+SELECT * FROM "Notifications" AS a
+   LEFT JOIN "bulk_message_user_refs" AS b
+   ON a.id=b.notification_id
+   WHERE a.type='admin.notification.broadcast'
+   AND b.id IS NULL;
+

src/hooks/hookBulkMessage.js

@@ -136,7 +136,7 @@ async function insertUserRefs(userId, bulkMessageId, notificationObj) {
         if (notificationId && notificationObj) {
              try {
                  await notificationObj.destroy()
-                 logger.info(`Deleted duplicate/ref-transaction failed, broadcast notification ${notificationId} for user: ${userid}`)
+                 logger.info(`Deleted/reverted duplicate/ref-transaction failed, broadcast notification ${notificationId} for user: ${userId}`)
              } catch (error) {
                  logger.error(`Error in deleting duplicate notification record, ${error}`)
              }