Project

General

Profile

Bug #7585

Preview lp item not work +solution ( 1.9.10)

Added by Andre Boivin over 4 years ago. Updated over 4 years ago.

Status:
Bug resolved
Priority:
Normal
Category:
Learning paths / Lecciones
Target version:
Start date:
13/03/2015
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

In inc/lib/display.lib.php remove line 835
$url = htmlspecialchars($url, ENT_QUOTES, 'UTF-8');


Files

preview.png (130 KB) preview.png Andre Boivin, 04/04/2015 16:01

Associated revisions

Revision cc7bce69 (diff)
Added by Yannick Warnier over 4 years ago

Fix issue with & in URLs causing page link to break - refs #7585

Revision 1610e818 (diff)
Added by Yannick Warnier over 4 years ago

Replace & by & in learning paths - refs #7585

History

#1

Updated by Yannick Warnier over 4 years ago

  • Target version set to 1.10.0
#2

Updated by Yannick Warnier over 4 years ago

  • Status changed from New to Needs more info
  • Assignee set to Andre Boivin
  • % Done changed from 0 to 20

André, could you be more specific about how to reproduce the issue? (so we can confirm the fix)

#3

Updated by Andre Boivin over 4 years ago

Sur l'image de gauche, avec ligne 835 non commentée et l'image de droite avec
// $url = htmlspecialchars($url, ENT_QUOTES, 'UTF-8');
J'ai essayé avec des fichiers html en utf8 et c'est le même problème qu'avec mes vieux fichiers.

#4

Updated by Yannick Warnier over 4 years ago

OK. Bien que le patch proposé fonctionne, il ne règle pas vraiment le problème de base (et il en cause potentiellement un autre avec les fichiers dont les noms ne sont pas en UTF-8), qui est simplement que nous utilisons (pour des raisons historiques peu claires) trop de ```& amp;``` (sans l'espace) dans les URLs. J'ai corrigé ça pour tout l'outil learning paths et fait une série de tests qui ont l'air de démontrer que c'est correct.
J'ai aussi ajouté, dans la fonction url() de display.lib.php, un filtre pour changer les ```& amp;``` en &

#5

Updated by Yannick Warnier over 4 years ago

  • Status changed from Needs more info to Bug resolved
  • Assignee changed from Andre Boivin to Yannick Warnier
  • % Done changed from 20 to 100

J'ai fait ça dans 1.10.x, et dans 1.9.x j'ai juste envoyé le petit patch de preg_match() dans url().
Je considère ceci comme terminé, mais je reste attentif aux éventuelles conséquences imprévues.

Also available in: Atom PDF