gasでデータを貼り付けた後に、実行日時をA列に自動記入させたい

Question

Google Apps Scriptで質問があります。
宜しくお願い致します。

現在、以下の2がうまく動きません。

1.sheet1の特定範囲のデータをsheet2の最終行のB列に貼り付ける。（毎週）
2.さらに貼り付けた日時をA列に残したい。

1は下記URLを参照になんとか動くようになったのですが、
2の部分が1を実行しても、更新されません。
2はsheet2の方に書いており、トリガーは、スプレッドシートの値の変更にしています。

1の参照URL
https://teratail.com/questions/112154
https://teratail.com/questions/106446

2の参照URL
http://bonheur.hatenablog.jp/entry/2017/06/30/130437
https://qiita.com/Yamotty/items/888ae6f93a05a8cfc16f

初心者で表現がわかりづらいかもしれませんが、どなたかアドバイスをいただけないでしょうか。何卒よろしくお願いいたします。

2のコード

function insertLastUpdated() {
  var ss = SpreadsheetApp.getActiveSheet(); 
  var currentRow = ss.getActiveCell().getRow(); 
  var currentCell = ss.getActiveCell().getValue(); 
  var updateRange = ss.getRange('A' + currentRow) 
  Logger.log(updateRange);
  
  if(currentRow>1){ 
    if(currentCell) { 
      updateRange.setValue(new Date());
    }
  }
}

Comments

[退会済みユーザー]

とりあえずの回答はしておきましたが、１の処理も提示していただければもっと問題点が明確になると思いますよ。

Answer 1

意図があって処理を分割したのか、よくわからないけど分割して構築したか、ということも気になりますが、無駄が多いので、ロジックとしては以下のように一連の流れとして行うほうが良いと思いますよ。
※特定範囲が複数行である前提です。

1. sheet1から特定範囲のデータを取得するgetRange({特定範囲}).getValues()
2. 取得データは二次元配列なので、ループでそれぞれ先頭に本日の日付をunshift()する
3. sheet2の最終行以下に成形データをセットするsetValues({成形データ})

一応、追記・修正依頼もしておきますが、１の処理も提示していただければもっと問題点が明確になると思いますよ。

Comments

[ogumogu]

アドバイスありがとうございます。stshishoさんのご指摘通り、無駄が非常に多いかと思います。。
sheet1のコードは以下のとおりです。
（sheet1は複数シートあり、A1~EZ199までのデータを毎週、sheet2の最終行のB列にセットする。Sheet1のデータは、1~199まで全てにデータがあるわけではなく、空白セルや空白行も存在しています）

function copy() {

var ss_copyFrom = SpreadsheetApp.getActiveSpreadsheet();
var ss_copyTo = SpreadsheetApp.openById('sheet2');
var sheet_copyFrom = ss_copyFrom.getSheetByName('sheet1のファイル名');

var lr = sheet_copyFrom.getLastRow();
var lc = sheet_copyFrom.getLastColumn();
var copyRange = sheet_copyFrom.getRange(1,1,199,156);

var sheet_copyTo = ss_copyTo.getSheetByName('sheet2のファイル名');
var lastRow = sheet_copyTo.getLastRow();
var lastRow1 = lastRow + 1;

var copyValues = copyRange.getValues();
sheet_copyTo.getRange(lastRow1,2,199,156).setValues(copyValues);
}

[退会済みユーザー]

結論から申しますと、最初に提示された2のコードは不要です。
1のコードを当方が提示したロジックに沿って修正されれば良いかと思います。
具体的には以下の部分になりますが、空行を削除する必要がある場合はcopyValues.filter({空行削除処理})を挟みます。

//1. sheet1から特定範囲のデータを取得する
var copyValues = copyRange.getValues();

//1.5 必要ならば空行削除処理

//2. copyValuesに対しループでそれぞれ先頭に本日の日付をunshift()する
var today = new Date();
/* ループ処理 */

//3. sheet2の最終行以下に成形データをセットする
sheet_copyTo.getRange(lastRow1,1,{配列の長さ},{列数}).setValues(copyValues);

動作確認はしておりませんので、参考までに。
頑張ってみてください。