Can't get getActiveSheet() from undefined - google-apps-script

Question

I'm trying to copy a row from sheet "Name1" to sheet "Name2" when entering "ok" in cell 1 (column 1) but I couldn't fix this error in the title (Cannot call method "getActiveSheet" from undefined). I have tried all those commented lines but no luck, I'm editing the cell and enter "ok" in that cell but not working.
function onEdit2(e) {
//var ss = SpreadsheetApp.getActiveSpreadsheet();
var ss = e.source;
//var s = SpreadsheetApp.openById("1yAJY3gO46A7Wr_fYtVHAB_wSElUdG2kkiH1clRg9o8k").getSheetByName('Name1');
var s = ss.getActiveSheet();
var r = e.source.getActiveRange();
if(s.getName() == "Name1" && r.getColumn() == 1 && r.getValue() == "ok") {
var row = r.getRow();
var numColumns = s.getLastColumn();
var targetSheet = ss.getSheetByName("Name2");
var target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
s.getRange(row, 1, 1, numColumns).moveTo(target);
s.deleteRow(row);
}
}
the error I'm getting is
TypeError: Can not call method "getActiveSheet" from undefined. (Row 135).
I edited those first lines to those
var ss = SpreadsheetApp.getActiveSpreadsheet();
// var ss = e.source;
//var s = SpreadsheetApp.openById("1yAJY3gO46A7Wr_fYtVHAB_wSElUdG2kkiH1clRg8o9k").getSheetByName('Lista');
var s = ss.getActiveSheet();
var r = e.source.getActiveRange(); // I get error now here can't call getactiverange from undefined .

Answer 1

At this line:
var targetSheet = s.getSheetByName("Name2");
You're running the getSheetByName() on the Sheet and it needs to run on the Spreadsheet to be able to get the other Sheet Name2 , so just change that line to:
var targetSheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName("Name2");
Also rename your function from onEdit2() to the reserved function name onEdit() so it gets triggered, or create an Installable Trigger
Your function should look like this:
function onEdit(e) {
var ss = e.source;
var s = ss.getActiveSheet();
var r = e.source.getActiveRange();
if(s.getName() == "Name1" && r.getColumn() == 1 && r.getValue() == "ok") {
var row = r.getRow();
var numColumns = s.getLastColumn();
var targetSheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName("Name2"); // Changed
var target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
s.getRange(row, 1, 1, numColumns).moveTo(target);
s.deleteRow(row);
}
}
I created an example Spreadsheet here.

Answer 2

When a sheet is edited, the function onEdit() is called, not onEdit2(). onEdit2 seems to be your custom function, so you must be calling that yourself from somewhere else. (maybe from within onEdit()?). If so, are you passing the event object e while calling it? (You have to call it like this:
onEdit2(e);
If you're not passing the parameter e, then you will get the error you're seeing, since ss will be undefined.
ref: GAS-trigger-events