Patch: Make embedded note recipients include current agent_id

Author: xaviqv

src/Controllers/TicketsController.php

@@ -1215,7 +1215,7 @@ public function edit($id, $parameters = null)
         // Default notification recipients
         $all_c = clone $all_comments;
         $a_reply = [(!is_null($ticket->owner) ? $ticket->owner->id : $ticket->user_id)];
-        $a_note = [$ticket->agent->id];
+        $a_note = [];
         foreach($all_c->get() as $comm){
             if ($comm->type == 'reply'){
                 $a_reply = array_merge($a_reply, $comm->notifications->pluck('member_id')->toArray());

src/Views/tickets/partials/comments/embedded_form.blade.php

@@ -30,7 +30,7 @@
                 <div class="col-lg-10">
                     <select class="form-control note_recipients" name="comment_x_recipients[]" multiple="multiple" style="display: none; width: 100%" disabled="disabled">
                         @foreach ($c_members->filter(function($q)use($u){ return $q->id != $u->id; }) as $member)
-                            <option value="{{ $member->id }}" {{ in_array($member->id, $a_notifications['note']) ? 'selected="selected"' : '' }}>{{ $member->name . ($member->email == "" ? ' ' . trans('panichd::lang.ticket-owner-no-email') : ' - ' . $member->email) }}
+                            <option value="{{ $member->id }}">{{ $member->name . ($member->email == "" ? ' ' . trans('panichd::lang.ticket-owner-no-email') : ' - ' . $member->email) }}
                             @if ($setting->grab('departments_notices_feature'))
                                 @if ($member->ticketit_department == '0')
                                     {{ ' - ' . trans('panichd::lang.create-ticket-notices') . ' ' . trans('panichd::lang.all-depts')}}

src/Views/tickets/partials/comments/embedded_scripts.blade.php

@@ -4,12 +4,23 @@
     $('#add_comment').click(function(e){
        e.preventDefault();
 
+       // Notification members
+       var a_notifications_note = [{{ implode(',', $a_notifications['note']) }}];
+
        var _cloned = $('#comment_template').clone();
        var _num = $('#comments .comment_block').length + 1;
        _cloned.prop('id', 'comment_' + _num);
        _cloned.find('.input_comment_num').val(_num);
        _cloned.find('.input_response_type').prop('disabled', false).attr('name', 'response_' + _num);
         @if($setting->grab('custom_recipients'))
+            $(a_notifications_note).each(function(i,v){
+                // Add other notes recipients
+                _cloned.find('.note_recipients option[value=' + v + ']').prop('selected', true);
+            });
+            // Add current agent id
+            _cloned.find('.note_recipients option[value=' + $('#agent_id').val() + ']').prop('selected', true);
+
+            // Activate notification recipients selects
             _cloned.find('.note_recipients').prop('disabled', false).attr('name', 'comment_' + _num + '_recipients[]');
             _cloned.find('.reply_recipients').attr('name', 'comment_' + _num + '_recipients[]');
         @endif