Bug #20294
Ne pas produire les variables de substitution quand la réponse n'est pas de l'html
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
27 novembre 2017
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Constaté via #20292 on génère les variables de substitution pour une réponse qui est une image, c'est totalement inutile.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-misc-don-t-generate-substitution-variables-for-non-h.patch 0001-misc-don-t-generate-substitution-variables-for-non-h.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 6 ans
Pour l'instant j'ai un peu de mal à comprendre à quel moment c'est appelé, vu que :
class CompatWcsPublisher(WcsPublisher):
def filter_output(self, request, output):
response = self.get_request().response
if response.status_code == 304:
# clients don't like to receive content with a 304
return ''
if response.content_type != 'text/html':
return output
if not hasattr(response, 'filter') or not response.filter:
return output
if request.META.get('HTTP_X_POPUP') == 'true':
return '<div class="popup-content">%s</div>' % output
if response.filter and response.filter.get('admin_ezt'):
return self.render_response(output)
...
Tu m'éclaires ?
Mis à jour par Frédéric Péters il y a plus de 6 ans
Les URL en /backoffice/ passent par cette vue (Backoffice(compat.TemplateWithFallbackView)).
Mis à jour par Frédéric Péters il y a plus de 6 ans
... et pas par le filter_output du CompatWcsPublisher.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit c4fc21a21a476f318a3e75da06f2dbeaeef9720a
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Mon Nov 27 11:18:58 2017 +0100
misc: don't generate substitution variables for non-html responses (#20294)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: don't generate substitution variables for non-html responses (#20294)