如何运行多个onEdit函数

Question

I have two onEdit functions that work individually, but I cannot seem to combine them properly to run at the same time.

My goals is to timestamp the cell in column 9 AND copy the whole row to a new sheet when the checkbox is checked in column 1.

I have followed the advice posted elsewhere on Stackoverflow regarding triggers, renaming the functions, etc, but at most I can only get the first function to run.

function onEdit(e) {

var colToWatch = 1, colToStamp = 9;
var valueToWatch = "TRUE"; 
if (e.range.columnStart === colToWatch && (e.value === valueToWatch || 
typeof e.value == 'object'))
e.source.getActiveSheet()
    .getRange(e.range.rowStart, colToStamp)
    .setValue(typeof e.value === 'object' ? null : new Date());
}

function onEdit(event) {
// assumes source data in sheet named Active
// target sheet of move to named Found 
// test column with yes/no is col 1 or A
var ss = SpreadsheetApp.getActiveSpreadsheet();
var s = event.source.getActiveSheet();
var r = event.source.getActiveRange();

if(s.getName() == "Jessica" && r.getColumn() == 1 && r.getValue() == true) {
var row = r.getRow();
var numColumns = s.getLastColumn();
var targetSheet = ss.getSheetByName("Done");
var target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
s.getRange(row, 1, 1, numColumns).copyTo(target);
}

}

I expected both functions to run, but I fear I've taken too many disparate pieces of advice, and now neither function runs on edit.

Answer 1

You cannot have two functions sharing the same name, and therefore will encounter issues trying to have two separate onEdit() functions. Rewrite your script so that you have a single function; and also give a read through the guide on triggers

Answer 2

In the timestamp function, you are using two undefined properties of Range : e.range.columnStart and e.range.rowStart . Your code works when you replace them with e.range.getColumn() and e.range.getRow() .

I suggest renaming the functions to describe their function, and then make a new onEdit that calls these functions in the correct order.

Here is a working example:

function onEdit(e) {
  timestamp(e);
  copyRow(e);
}

function timestamp(e) {
  var colToWatch = 1, colToStamp = 9;
  var valueToWatch = "TRUE"; 
  if (e.range.getColumn() === colToWatch &&
     (e.value === valueToWatch || typeof e.value == 'object')) {
    e.source.getActiveSheet()
      .getRange(e.range.getRow(), colToStamp)
      .setValue(typeof e.value === 'object' ? null : new Date());
  }
}

function copyRow(event) {
  // assumes source data in sheet named Active
  // target sheet of move to named Found 
  // test column with yes/no is col 1 or A
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var s = event.source.getActiveSheet();
  var r = event.source.getActiveRange();

  if (s.getName() == "Jessica" &&
      r.getColumn() == 1 &&
      r.getValue() == true) {
    var row = r.getRow();
    var numColumns = s.getLastColumn();
    var targetSheet = ss.getSheetByName("Done");
    var target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
    s.getRange(row, 1, 1, numColumns).copyTo(target);
  }
}