Merge pull request #2613 from MTES-MCT/bugfix/2610-bo-email-rel-noopener

[BO - email] mettre rel="noopener" dans les liens des mails
MTES-MCT · May 31, 2024 · e9bcbbf · e9bcbbf
2 parents 3910c4e + b18d783
commit e9bcbbf
Show file tree

Hide file tree

Showing 9 changed files with 10 additions and 10 deletions.
diff --git a/src/Manager/SuiviManager.php b/src/Manager/SuiviManager.php
@@ -167,7 +167,7 @@ public function createInstanceForFilesSignalement(User $user, Signalement $signa
                 UrlGeneratorInterface::ABSOLUTE_URL
             ).'?t=___TOKEN___';
 
-            $linkFile = '<li><a class="fr-link" target="_blank" href="'.$fileUrl.'">'.$file->getTitle().'</a>';
+            $linkFile = '<li><a class="fr-link" target="_blank" rel="noopener" href="'.$fileUrl.'">'.$file->getTitle().'</a>';
             if (DocumentType::PHOTO_SITUATION === $file->getDocumentType() && null !== $file->getDesordreSlug()) {
                 $desordreCritere = $this->desordreCritereRepository->findOneBy(
                     ['slugCritere' => $file->getDesordreSlug()]

diff --git a/src/Service/Signalement/SignalementFileProcessor.php b/src/Service/Signalement/SignalementFileProcessor.php
@@ -157,7 +157,7 @@ public function generateListItemDescription(
         )
             .$queryTokenUrl;
 
-        return '<li><a class="fr-link" target="_blank" href="'
+        return '<li><a class="fr-link" target="_blank" rel="noopener" href="'
             .$fileUrl
             .'">'
             .$title

diff --git a/templates/back/signalement/view/address-qualifications.html.twig b/templates/back/signalement/view/address-qualifications.html.twig
@@ -21,7 +21,7 @@
         {{ signalement.cpOccupant ~' '~ signalement.villeOccupant|upper }}
         <br>
         {% if signalement.geoloc.lat is defined and signalement.geoloc.lng is defined %}
-            <a target="_blank" rel="noreferrer" class="force-link-color"
+            <a target="_blank" rel="noreferrer noopener" class="force-link-color"
                 href="http://www.openstreetmap.org/?mlat={{ signalement.geoloc.lat }}&mlon={{ signalement.geoloc.lng }}#map=18/{{ signalement.geoloc.lat }}/{{ signalement.geoloc.lng }}">
                 Voir sur la carte
             </a>

diff --git a/templates/base.html.twig b/templates/base.html.twig
@@ -176,7 +176,7 @@
                             type: 'menuitem',
                             text: '{{ doc.title }}',
                             onAction: function () {
-                                editor.insertContent('&nbsp;<a href="{{ asset('_up/'~doc.filename) }}?t=___TOKEN___" class="fr-btn fr-fi-eye-fill fr-btn--icon-left" title="Afficher le document" target="_blank">Consulter "{{ doc.title }}"</a>');
+                                editor.insertContent('&nbsp;<a href="{{ asset('_up/'~doc.filename) }}?t=___TOKEN___" class="fr-btn fr-fi-eye-fill fr-btn--icon-left" title="Afficher le document" target="_blank" rel="noopener">Consulter "{{ doc.title }}"</a>');
                             }
                         },
                         {% endfor %}
@@ -189,7 +189,7 @@
                                     type: 'menuitem',
                                     text: "{{ doc.title|raw }}",
                                     onAction: function () {
-                                        editor.insertContent('&nbsp;<a href="{{ asset(files.path~doc.file) }}" class="fr-link" title="Afficher le document" target="_blank">Consulter "{{ doc.title }}"</a></br>');
+                                        editor.insertContent('&nbsp;<a href="{{ asset(files.path~doc.file) }}" class="fr-link" title="Afficher le document" target="_blank" rel="noopener">Consulter "{{ doc.title }}"</a></br>');
                                     }
                                 })     
                             {% endif %}   

diff --git a/templates/emails/affectation_email.html.twig b/templates/emails/affectation_email.html.twig
@@ -13,7 +13,7 @@
                     <tbody>
                     <tr style="text-align: center;width: 100%">
                         <td>
-                            <a href="{{ link|raw }}" target="_blank">{{ btntext }}</a></td>
+                            <a href="{{ link|raw }}" target="_blank" rel="noopener">{{ btntext }}</a></td>
                     </tr>
                     </tbody>
                 </table>

diff --git a/templates/emails/lost_pass_email.html.twig b/templates/emails/lost_pass_email.html.twig
@@ -13,7 +13,7 @@
                     <tbody>
                     <tr style="text-align: center;width: 100%">
                         <td>
-                            <a href="{{ link|raw }}" target="_blank">{{ btntext }}</a></td>
+                            <a href="{{ link|raw }}" target="_blank" rel="noopener">{{ btntext }}</a></td>
                     </tr>
                     </tbody>
                 </table>

diff --git a/templates/emails/new_signalement_email.html.twig b/templates/emails/new_signalement_email.html.twig
@@ -15,7 +15,7 @@
                     <tbody>
                     <tr style="text-align: center;width: 100%">
                         <td>
-                            <a href="{{ link|raw }}" target="_blank">{{ btntext }}</a></td>
+                            <a href="{{ link|raw }}" target="_blank" rel="noopener">{{ btntext }}</a></td>
                     </tr>
                     </tbody>
                 </table>

diff --git a/templates/emails/nouveau_suivi_signalement_back_email.html.twig b/templates/emails/nouveau_suivi_signalement_back_email.html.twig
@@ -20,7 +20,7 @@
                     <tbody>
                     <tr style="text-align: center;width: 100%">
                         <td>
-                            <a href="{{ link|raw }}" target="_blank">{{ btntext }}</a></td>
+                            <a href="{{ link|raw }}" target="_blank" rel="noopener">{{ btntext }}</a></td>
                     </tr>
                     </tbody>
                 </table>

diff --git a/templates/emails/validation_signalement_email.html.twig b/templates/emails/validation_signalement_email.html.twig
@@ -18,7 +18,7 @@
                     <tbody>
                     <tr style="text-align: center;width: 100%">
                         <td>
-                            <a href="{{ lien_suivi|raw }}" target="_blank">{{ btntext|capitalize }}</a></td>
+                            <a href="{{ lien_suivi|raw }}" target="_blank" rel="noopener">{{ btntext|capitalize }}</a></td>
                     </tr>
                     </tbody>
                 </table>