スプシの変更内容を一定期間毎にDiscordへ通知したい

Question

実現したいこと

GAS（JS）初心者です。
以下サイト記載の構文をコピペにて実行したのですが、デバッグ時に幾つかの型がundefinedとなり通らない。

前提

件名記載要件を満たしそうなサイトが以下にあったので、こちらのソースコードをコピペ。
7行目にて当該エラー発生。

https://qiita.com/daijin/items/5de520af98702f15f432

発生している問題・エラーメッセージ

TypeError: Cannot read properties of undefined (reading 'source')
recordToProperty	@ task.gs:7

該当のソースコード

////////////////////////////////////////////////////////////////////////////////
// 編集時に発行されるイベント
////////////////////////////////////////////////////////////////////////////////
function recordToProperty(e) {
  // 変更されたのが指定したシートの場合のみ有効
  // ★★★編集を監視したいシート名に置き換える
  if (e.source.getSheetName() != 'シート1') {
    return;
  }

  // 変更時刻
  const time = new Date().getTime();

  // メッセージに変更された範囲、新しい値を入れる
  const message = [e.range.getA1Notation(), e.value];

  // 変更があったことをスクリプトプロパティに追加。keyは変更時刻
  const script_property = PropertiesService.getScriptProperties();
  script_property.setProperty(time, JSON.stringify(message));
}

////////////////////////////////////////////////////////////////////////////////
// 定期実行されるイベント
////////////////////////////////////////////////////////////////////////////////
function notifyToDiscord(e) {
  // スクリプトプロパティの変更をチェック。
  const script_property = PropertiesService.getScriptProperties();
  const properties = script_property.getProperties();
  const array = Object.entries(properties);

  // スクリプトプロパティに1件もなければ、この期間に変更なしと見なす。
  if (array.length == 0) {
    return;
  }

  // プロパティがあったので変更あり。Webhookを送信する。
  const WEBHOOK_URL = 'https://discord.com/api/webhooks/xxxxxx/xxxxxx';  // ★★★DiscordのWebhook URL

  // メッセージを作成
  let message_body = 'シートに変更があったよ!!\n';
  let embeds = {
    fields: [],
  };

  array.forEach((prop) => {
    const key = prop[0];              // time
    const val = JSON.parse(prop[1]);  // message
    embeds.fields.push({
      name: 'セル: ' + val[0],
      value: '変更後の内容: ' + val[1],
    });
  });

  const payload = {
    'username': /* ★★★投稿時のユーザー名。指定しない（デフォのままで良い）場合は、行ごと削除する */,
    'content': message_body,
    'tts': false,
    'embeds': [embeds],
  }

  const param = {
    'method': 'POST',
    'headers': {
        'Content-type': 'application/json'
      },
    'payload': JSON.stringify(payload)
  }
  UrlFetchApp.fetch(WEBHOOK_URL, param);

  // 送信成功後、プロパティを全削除
  script_property.deleteAllProperties();
}

試したこと

  if (e.source.getSheetName() != 'シート1') {
    return;
  }

を削るなどでの動作確認。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[atlanticSalmon]

> 件名記載要件を満たしそうなサイトが以下にあったので、こちらのソースコードをコピペ。
ということですので、
そのサイトの説明をよく読んでみることをお勧めします。

動作確認
1.スプレッドシートを編集する
2.notifyToDiscord 関数を手動で「実行」する
これでDiscordの指定したチャンネルへBotの書き込みが投稿されれば :ok: 。

実行すべきはrecordToPropertyではないですよね。

[YellowGreen]

GAS(JavaScript)初心者ということなので、ご説明を加えておきます。

お示しのサイトの説明どおりトリガーを設定しているのであれば、
recordToPropertyは、
スプレッドシートのシート1を編集すると、
Googleサーバーのそのスクリプト用の格納領域(スクリプトプロパティ)に
編集した日時をキーとして[編集したセル名, 編集後のセルの値]の配列を保存します。
このスクリプトは、セルを編集したら起動するようにトリガー設定してあり、
編集された値に基づいて処理を行うスクリプトなので、
エディタからは実行しても編集された値がない（undefined）のでエラーになります。

notifyToDiscordの方は、時間指定で起動するトリガーを設定してあるので、
定期的にGoogleサーバーのスクリプトプロパティを読み込んで、
保存されたものがあるならその値に基づき投稿し、
投稿後はスクリプトプロパティを削除します。

なので、動作確認をしたいならば、
スプレッドシートを編集してrecordPropertyに編集内容をサーバーに保存させてから
notifyToDiscordを手動で（エディタから）実行して保存した編集内容を取得して
投稿しているかどうかを確認することで動作確認できるということです。

recordToPropertyの方は、
トリガーでの実行を前提としているので、
エディタからはデバッグ実行もできませんから、
console.logなどで適宜変数の値をログに表示するようにして、
シートの編集後にエディタの実行数のペインからログを確認するようになります。

YAmaGNZ様のご提案のようにテスト用の関数の方を実行するときは、
次のように書き換えてみるとエラーが出ないと思います。

function Test_recordToProperty() {
//A1セルを編集したときのテスト。A1は★のところを書き換えて適当なセルに設定
let sheet = SpreadsheetApp.openById('XXXXXX').getSheets()[0];//XXXXXXを書換える（左端のシートを取得）
let range = sheet.getRange('A1');//★
let value = range.getValue();
let e = {
source: sheet,
range: range,
value: value
}
recordToProperty(e);
}

※　いろいろ書いているうちに解決されたのですね。良かったです。

Answer 1

デバッグ時ということはエディタで実行されているのだと思います。
エディタで実行する場合は関数の引数は設定されません。
なので、function recordToProperty(e)のeの中身がnullになります。

引数がある関数をテストで実行したい場合はテスト用の関数を作成し、そこから呼ぶようにしてください。

function recordToProperty(e) {
  // 変更されたのが指定したシートの場合のみ有効
  // ★★★編集を監視したいシート名に置き換える
  if (e.source.getSheetName() != 'シート1') {
    return;
  }
}

function Test_recordToProperty() {
  // 本当の引数がどのようなものなのか知りませんが、実行できるように適当に作成しました。
  // 本来の引数になるように作成してください。
  let sheet = SpreadsheetApp.openById('XXXXXX').getSheets()[0];
  let e = {source:sheet}
  recordToProperty(e);
}

Comments

[SatouSio]

ご回答ありがとうございます。
解決いたしました！

[YAmaGNZ]

recordToProperty関数に関してはスプレッドシート内だけで完結するものなので実際に実行させてもいいのではないかとは思います。