From abc2705c6404c80e691350f172af6e2379ac9471 Mon Sep 17 00:00:00 2001 From: Clemens Schwaighofer Date: Thu, 7 Mar 2024 14:31:30 +0900 Subject: [PATCH] Bug fix in Form Generate load list generation For binary 1/0 blocks make sure we use the actual data and not just the wrong binary 0/1 source Also add auto translations for all strings in there --- www/lib/CoreLibs/Output/Form/Generate.php | 23 ++++++++++++----------- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/www/lib/CoreLibs/Output/Form/Generate.php b/www/lib/CoreLibs/Output/Form/Generate.php index 85531874..3c406d4d 100644 --- a/www/lib/CoreLibs/Output/Form/Generate.php +++ b/www/lib/CoreLibs/Output/Form/Generate.php @@ -826,27 +826,28 @@ class Generate $pk_selected = $res[$this->int_pk_name]; } $t_string = ''; - foreach ($this->field_array as $i => $field_array) { + foreach ($this->field_array as $field_array) { if ($t_string) { $t_string .= ', '; } - if (isset($field_array['before_value'])) { - $t_string .= $field_array['before_value']; + if (!empty($field_array['before_value'])) { + $t_string .= $this->l->__($field_array['before_value']); } // must have res element set if ( - isset($field_array['name']) && + !empty($field_array['name']) && isset($res[$field_array['name']]) ) { - if (isset($field_array['binary'])) { - if (isset($field_array['binary'][0])) { - $t_string .= $field_array['binary'][0]; - } elseif (isset($field_array['binary'][1])) { - $t_string .= $field_array['binary'][1]; - } + $_t_value = ''; + // if we have a binary set, where 0 = YES and 1 = NO + if (!empty($field_array['binary'])) { + $_t_value = !empty($res[$field_array['name']]) ? + ($field_array['binary'][0] ?? 'Yes') : + ($field_array['binary'][1] ?? 'No'); } else { - $t_string .= $res[$field_array['name']]; + $_t_value = $res[$field_array['name']]; } + $t_string .= $this->l->__($_t_value); } } $pk_names[] = $t_string;