Bulkmail_lib fix

entity/Employee_entity/recordcontainers/jdito/contentProcess.js

@@ -51,7 +51,7 @@ var names = [];
 
 for (let begin = 0, max = contactIds.length; begin < max; begin += idsPerPage)
 {
-    names = names.concat(newSelect("CONTACTID, " + selectExpression)
+    names = names.concat(newSelect(["CONTACTID", selectExpression])
                             .from("CONTACT")
                             .leftJoin("PERSON", "(PERSON.PERSONID = CONTACT.PERSON_ID)")
                             .whereIfSet("CONTACT.CONTACTID", contactIds.slice(begin, begin + idsPerPage), SqlBuilder.IN())

process/Bulkmail_lib/process.js

@@ -122,11 +122,11 @@ BulkMailUtils.sendBulkMail = function (pBulkMailId, pTestRecipients)
         }
     }
     
-    newWhereIfSet("BULKMAILRECIPIENT.BULKMAILRECIPIENTID", successIds, SqlBuilder.In())
+    newWhereIfSet("BULKMAILRECIPIENT.BULKMAILRECIPIENTID", successIds, SqlBuilder.IN())
         .updateData(true, "BULKMAILRECIPIENT", ["STATUS", "SENTDATE"], null, [$KeywordRegistry.bulkMailRecipientStatus$sent(), sentDate]);
     
     
-    newWhereIfSet("BULKMAILRECIPIENT.BULKMAILRECIPIENTID", failedIds, SqlBuilder.In())
+    newWhereIfSet("BULKMAILRECIPIENT.BULKMAILRECIPIENTID", failedIds, SqlBuilder.IN())
         .updateData(true, "BULKMAILRECIPIENT", ["STATUS", "SENTDATE"], null, [$KeywordRegistry.bulkMailRecipientStatus$failed(), sentDate]);
     
     if (!pTestRecipients) //if its just a test run, don't set the status to sent