From 5bae51593d4a91591489c13df16ef678a4fb6a37 Mon Sep 17 00:00:00 2001
From: JediKev <kevin@enhancesoft.com>
Date: Mon, 11 Mar 2019 11:28:29 -0500
Subject: [PATCH] issue: Duplicate Form Titles
This addresses an issue mentioned in the forum where having more than one
custom field on a ticket shows the same title for all forms on the
client-side ticket view (after creation). This adds an array of the form
names indexed by sort order and displays them in the correct order with the
correct names.
---
include/client/view.inc.php | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/include/client/view.inc.php b/include/client/view.inc.php
index 2aa01d019..315f501ac 100644
--- a/include/client/view.inc.php
+++ b/include/client/view.inc.php
@@ -98,7 +98,7 @@ if ($thisclient && $thisclient->isGuest()
<td colspan="2">
<!-- Custom Data -->
<?php
-$sections = array();
+$sections = $forms = array();
foreach (DynamicFormEntry::forTicket($ticket->getId()) as $i=>$form) {
// Skip core fields shown earlier in the ticket view
$answers = $form->getAnswers()->exclude(Q::any(array(
@@ -111,11 +111,13 @@ foreach (DynamicFormEntry::forTicket($ticket->getId()) as $i=>$form) {
if ($v = $a->display())
$sections[$i][$j] = array($v, $a);
}
+ // Set form titles
+ $forms[$i] = $form->getTitle();
}
foreach ($sections as $i=>$answers) {
?>
<table class="custom-data" cellspacing="0" cellpadding="4" width="100%" border="0">
- <tr><td colspan="2" class="headline flush-left"><?php echo $form->getTitle(); ?></th></tr>
+ <tr><td colspan="2" class="headline flush-left"><?php echo $forms[$i]; ?></th></tr>
<?php foreach ($answers as $A) {
list($v, $a) = $A; ?>
<tr>
--
GitLab