Многократное выполнение Google Script, даже с триггером Lock on form - PullRequest
Новая
       8

Многократное выполнение Google Script, даже с триггером Lock on form

0 голосов
/ 22 февраля 2019

Я изо всех сил пытаюсь понять, почему этот скрипт выполняет 2, 3, даже 4 раза одновременно каждый раз при отправке формы.Я добавил LockService, но, хотя он иногда не позволяет отправлять несколько электронных писем, он не препятствует одновременному выполнению.

Все остальное работает нормально -Я просто не хочу, чтобы сбойные выполнения выполнялись каждый раз, потому что они также нарушают последующие законные триггерные выполнения из-за тайм-аутов скриптов.

Я должен выполнить с листа, потому что я вызываю данные, которые извлекаютсялист на основе формы ответов.Лист не является общим, и триггеры «осиротевших» отсутствуют.Я удалил / заново создал триггер и даже заново создал лист и скрипт, но все еще получаю дополнительные исполнения.

Заранее благодарен за любые предложения! 

function zoneRepEmail(e) {

  var lock = LockService.getScriptLock();
  lock.waitLock(30000);  // wait 30 seconds before conceding defeat.
  // got the lock, you may now proceed
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = ss.getSheetByName('SupervisorApprovals');
  var lookupRow = e.range.getRow(); 
  var rowRange = sheet.getRange(1,1,sheet.getLastRow(),37);
  
  var appEmailSentLog = rowRange.getCell(lookupRow,36);
  var repEmailSentLog = rowRange.getCell(lookupRow,37); 
   Logger.log(appEmailSentLog);
  
  var form = FormApp.openById('1efkzyAgHf3PjEnPXSXP4P5mTCnII_BQaf5jf2Apr8gM');
   
  var startRow = 3;
  var headers = sheet.getRange(2, 1, 1, sheet.getLastColumn()).getValues()[0];
 
  var timestampColumn = sheet.getRange('K'+lookupRow);
  var dateColumns = sheet.getRangeList(['X'+lookupRow, 'Y'+lookupRow]);
  var pctColumns = sheet.getRange('Z'+lookupRow);
  var phoneColumns = sheet.getRangeList(['P'+lookupRow,'Q'+lookupRow,'U'+lookupRow]);
  var dataRange = sheet.getRange(lookupRow,1,sheet.getLastRow(),37);
  var data = dataRange.getValues();
    
    
  for (var i = 0; i < data.length; ++i){
      var row = data[i]; 
      var timestamp = e.namedValues["Timestamp"];
      var columnOfEditUrl = 46;
      var email_address = "email@email.com";
      var columnOfHomeUnit = 44;
      var columnOfAppId = 45;
      var columnOfZoneRep = 47;
      var columnOfZoneRepEmail = 48;
      var colEmplLName = 2;
      var emplLName = sheet.getRange(lookupRow, 12).getValue();
      var colEmplFName = 3;
      var emplFName = sheet.getRange(lookupRow, 13).getValue();
      var emplMName = sheet.getRange(lookupRow, 14).getValue();
      var locDisp = sheet.getRange(lookupRow, 15).getValue();
      var empPhone = row[15];
      var emplAltPhone = row[16];
      var emplEmail = row[17];
      var emplClass = row[18];
      var emplSup = row[19];
      var supPhone = row[20];
      var supEmail = row[21];
      var trainPosition = row[22];
      var firstAssign = row[23];
      var lastAssign = row[24];
      var tbPercent = row[25];
      var trainIMT = row[26];
      var imtName = row[27];
      var reqTraining = row[28];
      var trainNeeded = row[29];
      var appComments = row[30];
      var homeUnit = row[31];
      var appId = row[1];
      var zoneRep = row[33];
      var zoneRepEmail = sheet.getRange(lookupRow, 35).getValue();
      var appStatus = row[6];
      var statusReason = row[7];
      var statusComment = row[8];
      
    timestampColumn.setNumberFormat("mm/dd/yyyy hh:mm:ss");
    pctColumns.setNumberFormat("###%");
    dateColumns.setNumberFormat("mm/dd/yyyy");
    phoneColumns.setNumberFormat("###-###-####");
    
      var emailTo = emplEmail;
      var message = "The following application to the Rocky Mountain Area Priority Trainee Program has been "+appStatus+" for "+emplFName+" "+emplMName+" "+emplLName+". Please carefully review all details in the application and update your zone sheet accordingly."+"\n"+"If corrections are required, please contact the applicant directly."+"\n"+"A status email has been forwarded to the applicant and supervisor listed."+"\n"+"\n"+"--------------------------------------------------------------"+"\n"+"\n"+"Applicant Name: "+emplFName+" "+emplMName+" "+emplLName+"\n"+"Application Reference ID: "+appId+"\n"+"Dispatch: "+locDisp+"\n"+"Employee Phone: "+empPhone+"\n"+"Employee Alternate Phone: "+emplAltPhone+"\n"+"Employee Email: "+emplEmail+"\n"+"Home Unit ID: "+homeUnit+"\n"+"Employment Class: "+emplClass+"\n"+"\n"+"Supervisor Name: "+emplSup+"\n"+"Supervisor Phone: "+supPhone+"\n"+"Supervisor Email: "+supEmail+"\n"+"\n"+"Trainee Position: "+trainPosition+"\n"+"Taskbook Percentage Complete: "+tbPercent+"\n"+"Taskbook First Assignment: "+firstAssign+"\n"+"\n"+"IMT Trainee: "+trainIMT+"\n"+"Assigned to IMT: "+imtName+"\n"+"\n"+"All Required Training Completed?: "+reqTraining+"\n"+"Training Still Needed for Position: "+trainNeeded+"\n"+"\n"+"Comments: "+appComments+"\n"+"\n"+"Application Status: "+appStatus+"\n"+"Reason for Denial (if applicable): "+statusReason+"\n"+"Supervisor Comments: "+statusComment;
      var appMessage = "The following application to the Rocky Mountain Area Priority Trainee Program has been "+appStatus+" for "+emplFName+" "+emplMName+" "+emplLName+". The application has been forwarded to the Zone Training Representative for processing."+"\n"+"\n"+"--------------------------------------------------------------"+"\n"+"\n"+"Applicant Name: "+emplFName+" "+emplMName+" "+emplLName+"\n"+"Application Reference ID: "+appId+"\n"+"Dispatch: "+locDisp+"\n"+"Employee Phone: "+empPhone+"\n"+"Employee Alternate Phone: "+emplAltPhone+"\n"+"Employee Email: "+emplEmail+"\n"+"Home Unit ID: "+homeUnit+"\n"+"Employment Class: "+emplClass+"\n"+"\n"+"Supervisor Name: "+emplSup+"\n"+"Supervisor Phone: "+supPhone+"\n"+"Supervisor Email: "+supEmail+"\n"+"\n"+"Trainee Position: "+trainPosition+"\n"+"Taskbook Percentage Complete: "+tbPercent+"\n"+"Taskbook First Assignment: "+firstAssign+"\n"+"\n"+"IMT Trainee: "+trainIMT+"\n"+"Assigned to IMT: "+imtName+"\n"+"\n"+"All Required Training Completed?: "+reqTraining+"\n"+"Training Still Needed for Position: "+trainNeeded+"\n"+"\n"+"Comments: "+appComments+"\n"+"\n"+"Application Status: "+appStatus+"\n"+"Reason for Denial (if applicable): "+statusReason+"\n"+"Supervisor Comments: "+statusComment;
      var subject = appStatus+": PT App for "+emplFName+" "+emplMName+" "+emplLName+" // "+trainPosition+" // "+timestamp;

       
    Utilities.sleep(5000);//allow all column formats to be applied, then proceed.
    
 if (repEmailSentLog.isBlank()){
   GmailApp.sendEmail(zoneRepEmail, subject, message,{noReply:true});
    repEmailSentLog.setValue('EmailSent');
 }
    if (appEmailSentLog.isBlank()){
      GmailApp.sendEmail(emplEmail, subject, appMessage,{cc:supEmail,noReply:true});
    appEmailSentLog.setValue('EmailSent');
    }
    
  }
   lock.releaseLock(); Utilities.sleep(10000);
}

1 Ответ

0 голосов
/ 22 февраля 2019

Я бы определенно удалил строку: 

Utilities.sleep(10000);

Нет причин ждать.

Если вы хотите избежать ошибок ограничения скорости, вызванных ограничениями Apps Script, тогда звонитеслужбы внутри цикла for и ждут только в случае ошибки.Таким образом, код не ждет, если в этом нет необходимости, а только ждет, если в этом есть необходимость.

И я бы увеличил время ожидания.Обычно код снимает блокировку, поэтому длительный тайм-аут не влияет на эту ситуацию, но если тайм-аут заканчивается до того, как код будет создан, это может вызвать проблему.

И если возникает ошибка,затем отпустите замок.Итак, следующий код добавляет блок try / catch для обработки этого.

Кроме того, вы должны проверить полученную блокировку перед продолжением, и если блокировка не получена, выйдите и уведомите кого-нибудь, что былаошибка. 

function zoneRepEmail(e) {
try{
  var lock = LockService.getScriptLock();
  lock.waitLock(60000);  // wait 60 seconds before timing out

  if (!lock.hasLock()) {//Failed to get lock

    MailApp.sendEmail(Session.getEffectiveUser().getEmail(),
     'Code Failed', 'The code for ABC failed');
    //Logger.log('Could not obtain lock');
    return;
  }

  for (i=1;i<4;i++) {//Try up to 3 times
    try{
      //Rate limited service call code here
      break;
    }catch(e){
      if (i!==3){Utilities.sleep(i*2000);}
      if (i>=3) {
        MailApp.sendEmail(Session.getEffectiveUser().getEmail(), 
         'Code Failed', 'The code for ABC failed - ' + e.message + "\n\n" +
          e.stack);
      }
    };
  }

  //Code here

  lock.releaseLock();


} catch(e) {
  lock.releaseLock();//Release the lock if there is an error
  MailApp.sendEmail(Session.getEffectiveUser().getEmail(), 
    'Code Failed', 'The code for ABC failed - ' + e.message + "\n\n" +
     e.stack);
  //Logger.log('error: ' + e.message + ' stack: ' + e.stack);

}
}
...