From: test test <yoanl@pilotsystems.net>
Date: Thu, 24 Jun 2021 09:44:29 +0000 (+0200)
Subject: Fix "send message" link from profile : https://trackers.pilotsystems.net/probarreau... 
X-Git-Tag: 2.1~45
X-Git-Url: https://git.parisson.com/?a=commitdiff_plain;h=e046e5ea34517f2d799186f4776833bf5adff391;p=teleforma.git

Fix "send message" link from profile : https://trackers.pilotsystems.net/probarreau/1117
---

diff --git a/teleforma/forms.py b/teleforma/forms.py
index 38c8373f..01674afd 100644
--- a/teleforma/forms.py
+++ b/teleforma/forms.py
@@ -319,5 +319,4 @@ class WriteForm(PostmanWriteForm):
     def clean_recipients(self):
         """compute recipient if 'auto' is set"""
         recipients = self.cleaned_data['recipients']
-        course = self.cleaned_data.get('course')
         return recipients
diff --git a/teleforma/templates/postman/base_write.html b/teleforma/templates/postman/base_write.html
index 15b067fb..af0ac2a7 100644
--- a/teleforma/templates/postman/base_write.html
+++ b/teleforma/templates/postman/base_write.html
@@ -29,15 +29,15 @@
 
 
                 {{ form.errors.recipients }}
-                <input type="radio" id="recipient_category-admin" name="recipient_category" value="admin" {% if request.POST.recipient_category == 'admin' %}checked="checked"{% endif %}/><label
+                <input type="radio" id="recipient_category-admin" name="recipient_category" value="admin" {% if recipient_category == 'admin' %}checked="checked"{% endif %}/><label
                     for="recipient_category-admin">A un administrateur</label>
-                <input type="radio" id="recipient_category-prof" name="recipient_category" value="prof" {% if request.POST.recipient_category == 'prof' %}checked="checked"{% endif %}/><label
+                <input type="radio" id="recipient_category-prof" name="recipient_category" value="prof" {% if recipient_category == 'prof' %}checked="checked"{% endif %}/><label
                     for="recipient_category-prof">A un professeur</label>
-                <input type="radio" id="recipient_category-corrector" name="recipient_category" value="corrector" {% if request.POST.recipient_category == 'corrector' %}checked="checked"{% endif %}/><label
+                <input type="radio" id="recipient_category-corrector" name="recipient_category" value="corrector" {% if recipient_category == 'corrector' %}checked="checked"{% endif %}/><label
                     for="recipient_category-corrector">A un correcteur</label>
 
                 {% if not user.student.all %}
-                    <input type="radio" id="recipient_category-other" name="recipient_category" value="other" {% if request.POST.recipient_category == 'other' %}checked="checked"{% endif %}/><label
+                    <input type="radio" id="recipient_category-other" name="recipient_category" value="other" {% if recipient_category == 'other' %}checked="checked"{% endif %}/><label
                         for="recipient_category-other">A un étudiant</label>
                 {% endif %}
 
@@ -135,7 +135,7 @@
             });
         }
 
-        function update_desk_messages(event) {
+        function update_desk_messages(event, noReset) {
 
             // reset values
             if(event){
@@ -156,7 +156,9 @@
 
             // show or hide field depending on what is selected
             var recipientCategory = $('[name="recipient_category"]:checked').val();
-            $("#id_recipients").val('');
+
+            if(!noReset)
+                $("#id_recipients").val('');
             $('#category-admin').hide();
             $('#category-prof').hide();
             $('#category-corrector').hide();
@@ -216,7 +218,7 @@
         $('[name="recipient_category"]').bind('change', update_desk_messages);
         $('[name="course"], [name="corrector_course"]').bind('change', update_desk_messages);
         $('#_professorSelect, #_correctorSelect, #_adminSelect').bind('change', update_desk_messages);
-        update_desk_messages();
+        update_desk_messages(null, true);
 
         document.getElementById("id_subject").focus();
         $('#send_button').click(function () {
diff --git a/teleforma/views/crfpa.py b/teleforma/views/crfpa.py
index 80ff879b..62899bd0 100644
--- a/teleforma/views/crfpa.py
+++ b/teleforma/views/crfpa.py
@@ -1128,6 +1128,14 @@ class WriteView(PostmanWriteView):
     form_classes = (WriteForm, AnonymousWriteForm)
     success_url = "postman:sent"
 
+    def get_context_data(self, **kwargs):
+        context = super().get_context_data(**kwargs)
+        context['recipient_category'] = self.request.POST.get('recipient_category', None)
+        if self.kwargs.get('recipients') and not context['recipient_category']:
+            context['recipient_category'] = 'other'
+
+        return context
+
 class CRFPAProfileView(ProfileView):
     """Provide Collections web UI methods"""