From b41fd720e6ca47d0234cebeb7b3d121897c07dd0 Mon Sep 17 00:00:00 2001
From: TheWitness <thewitness@cacti.net>
Date: Sun, 23 Feb 2025 19:01:12 -0500
Subject: [PATCH] QA: Fix: #6070 - Undefined variable

- Undefined variable due to the data input data returning no data.
---
 data_templates.php | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/data_templates.php b/data_templates.php
index 30c5eb3865..cadd8fd1f4 100644
--- a/data_templates.php
+++ b/data_templates.php
@@ -819,7 +819,13 @@ function template_edit() {
 		if ($input_type != 1 && $input_type != 5) {
 			unset($struct_data_source_item['data_input_field_id']);
 		} else {
-			$struct_data_source_item['data_input_field_id']['sql'] = "SELECT id, CONCAT(data_name, ' - ', name) AS name FROM data_input_fields WHERE data_input_id=" . $template_data['data_input_id'] . " AND input_output='out' AND update_rra='on' ORDER BY data_name, name";
+			$struct_data_source_item['data_input_field_id']['sql'] = "SELECT id,
+				CONCAT(data_name, ' - ', name) AS name
+				FROM data_input_fields
+				WHERE data_input_id = " . $template_data['data_input_id'] . "
+				AND input_output='out'
+				AND update_rra='on'
+				ORDER BY data_name, name";
 		}
 	}
 
@@ -888,7 +894,7 @@ function template_edit() {
 					$message = __esc('This value is disabled due to it either it value being derived from the Device or special Data Query object that keeps track of critical data Data Query associations.');
 
 					if (cacti_sizeof($data_input_data)) {
-						$old_value  = $data_input_data['value'];
+						$old_value  = $data_input_data['value'] ?? '';
 						$old_tvalue = 'on';
 						$disable    = 'disable';
 					} else {
@@ -899,15 +905,15 @@ function template_edit() {
 				} elseif ($field['type_code'] == 'host_id') {
 					$message = __esc('This value is disabled due to it being derived from the Device and read only.');
 
-					$old_value  = $data_input_data['value'];
+					$old_value  = $data_input_data['value'] ?? '';
 					$old_tvalue = '';
 					$disable    = 'disable';
 				} else {
 					$message = __esc('Check this checkbox if you wish to allow the user to override the value on the right during Data Source creation.');
 
 					if (cacti_sizeof($data_input_data)) {
-						$old_value  = $data_input_data['value'];
-						$old_tvalue = $data_input_data['t_value'];
+						$old_value  = $data_input_data['value'] ?? '';
+						$old_tvalue = $data_input_data['t_value'] ?? '';
 						$disable    = '';
 					} else {
 						$old_value  = '';