[Rt-commit] rt branch, 4.4-trunk, updated. rt-4.4.4-131-g31ca5c2ab2

[? sunnavy]

The branch, 4.4-trunk has been updated
       via  31ca5c2ab297d4593102e6fb28f5ae18644109d0 (commit)
      from  b83819374cd5ebdc5c8ddf3e3e1bd168a03e3e0c (commit)

Summary of changes:
 share/html/Ticket/Elements/UpdateCc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

- Log -----------------------------------------------------------------
commit 31ca5c2ab297d4593102e6fb28f5ae18644109d0
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Sat Feb 23 02:47:16 2019 +0800

    Fix wrongly duplicated one-time addresses on ticket update page
    
    When user clicks address checkboxes below one-time inputs, RT will
    add/remove corresponding addresess to/from the corresponding one-time
    inputs, and when it's done, UpdateIgnoreAddressCheckboxes will be set to
    true to indicate RT to ignore those checkboxes at backend as they are
    already synced to one-time inputs.
    
    It didn't consider the case where people may submit form a few times
    before really updating the ticket, e.g. via searching/selecting
    articles, adding more attachments(the old UI), etc. If we don't keep the
    submitted UpdateIgnoreAddressCheckboxes value, it could be finally
    submitted with value 0 and RT will wrongly handle those checkboxes at
    backend, which could cause duplicated Cc/Bcc email addresses in outgoing
    emails.

diff --git a/share/html/Ticket/Elements/UpdateCc b/share/html/Ticket/Elements/UpdateCc
index a1fa20a364..5b11913c17 100644
--- a/share/html/Ticket/Elements/UpdateCc
+++ b/share/html/Ticket/Elements/UpdateCc
@@ -48,7 +48,7 @@
 % $m->callback(CallbackName => 'BeforeCc', ARGSRef => \%ARGS, Ticket => $TicketObj, one_time_Ccs => \@one_time_Ccs, txn_addresses => \%txn_addresses);
 
 <tr><td class="label"><&|/l&>One-time Cc</&>:</td><td><& /Elements/EmailInput, Name => 'UpdateCc', Size => undef, Default => $ARGS{UpdateCc}, AutocompleteMultiple => 1 &>
-<input type="hidden" id="UpdateIgnoreAddressCheckboxes" name="UpdateIgnoreAddressCheckboxes" value="0">
+<input type="hidden" id="UpdateIgnoreAddressCheckboxes" name="UpdateIgnoreAddressCheckboxes" value="<% $ARGS{UpdateIgnoreAddressCheckboxes} || 0 %>">
         <br />
 
 %if (scalar @one_time_Ccs) {

-----------------------------------------------------------------------