From 13b3c2c76e584749b6600e748a0f417f531b5714 Mon Sep 17 00:00:00 2001 From: camer0n Date: Thu, 17 Apr 2025 05:42:07 -0700 Subject: [PATCH] Issue #5443 Trigger a warning for a missing 'title' key. --- e107_handlers/form_handler.php | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/e107_handlers/form_handler.php b/e107_handlers/form_handler.php index d64938ab9..e2e40fb74 100644 --- a/e107_handlers/form_handler.php +++ b/e107_handlers/form_handler.php @@ -4912,7 +4912,7 @@ var_dump($select_options);*/ * @param array $currentlist - eg $this->fieldpref * @param array $fieldvalues - eg. $row * @param string $pid - eg. table_id - * @return string + * @return string|null */ public function renderTableCells($fieldarray, $currentlist, $fieldvalues, $pid) { @@ -5349,7 +5349,7 @@ var_dump($select_options);*/ * @param string $field field name * @param mixed $value field value * @param array $attributes field attributes including render parameters, element options - see e_admin_ui::$fields for required format - * @return string + * @return string|null */ public function renderValue($field, $value, $attributes, $id = 0) { @@ -6250,7 +6250,10 @@ var_dump($select_options);*/ $styleClass = ($value === 1) ? 'admin-true-icon' : 'admin-false-icon'; } - + if(!isset($attributes['title'])) + { + trigger_error("$field is missing the 'title' key/attribute", E_USER_WARNING); + } return $this->renderInline($field, $id, $attributes['title'], $value, $dispValue, 'select', $wparms, array('class'=>'e-editable-boolean '.$styleClass)); }