From f392494754df32b0cc3e10628c805bf121fa3a00 Mon Sep 17 00:00:00 2001 From: Jens Scheffler <95105677+jscheffl@users.noreply.github.com> Date: Tue, 31 Dec 2024 20:56:09 +0100 Subject: [PATCH] [v2-10-test] Fix update issues for object and advanced-arrays fields when empty default (#45313) (cherry picked from commit 6eab1f24c2fe070b6d68a1435eadaccd807ed1c0) Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com> --- airflow/www/static/js/trigger.js | 4 ++-- airflow/www/templates/airflow/trigger.html | 4 +++- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/airflow/www/static/js/trigger.js b/airflow/www/static/js/trigger.js index 7a3444f460f41..9242984e75bdf 100644 --- a/airflow/www/static/js/trigger.js +++ b/airflow/www/static/js/trigger.js @@ -59,8 +59,6 @@ function updateJSONconf() { } } params[keyName] = values.length === 0 ? null : values; - } else if (elements[i].value.length === 0) { - params[keyName] = null; } else if ( elements[i].attributes.valuetype && (elements[i].attributes.valuetype.value === "object" || @@ -81,6 +79,8 @@ function updateJSONconf() { // ignore JSON parsing errors // we don't want to bother users during entry, error will be displayed before submit } + } else if (elements[i].value.length === 0) { + params[keyName] = null; } else if (Number.isNaN(elements[i].value)) { params[keyName] = elements[i].value; } else if ( diff --git a/airflow/www/templates/airflow/trigger.html b/airflow/www/templates/airflow/trigger.html index 158b54cabaf69..20019db6903ae 100644 --- a/airflow/www/templates/airflow/trigger.html +++ b/airflow/www/templates/airflow/trigger.html @@ -120,7 +120,9 @@ {% elif form_details.schema and "object" in form_details.schema.type %} {% elif form_details.schema and ("integer" in form_details.schema.type or "number" in form_details.schema.type) %}