developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#1240 closed bug (fixed)

Plugin image et attribut alt

Reported by: Rob2 Owned by:
Priority: normal Milestone: Jelix 1.1.8
Component: jelix:plugins:tpl Version: 1.2RC2
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Dans le code du plugin image, l'attribut alt est automatiquement ajouté s'il n'existe pas. Or, dans le foreach qui suit, il y a une ligne qui vérifie que les attributs ne sont pas vides, donc l'attribut alt est supprimé. Voici le code en question :

if (!array_key_exists('alt',$att))
	$att['alt']=''; // Ici on ajoute l'attribut alt...

// generating hmtl tag img
echo '<img';
foreach( $att as $key => $val ) {
	if( !empty($val) ) // ...et ici on le supprime.
		echo ' '.$key.'="'.htmlspecialchars($val).'"';
}

Attachments (1)

monpatch.diff (496 bytes) - added by Rob2 10 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 10 years ago by Lipki

A ça c'est très bizarre. je me souviens avoir fait en sorte que alt soit toujours présent même vide. Peut-être une régression.

comment:2 Changed 10 years ago by Lipki

c'est une régression qui date de 2 ans :)

Tu peut créer le patch ?

Changed 10 years ago by Rob2

comment:3 Changed 10 years ago by Rob2

J'ai ajouté un patch. ;)

comment:4 Changed 10 years ago by laurentj

  • Milestone changed from Jelix 1.2 to Jelix 1.1.8
  • Resolution set to fixed
  • review set to review+
  • Status changed from new to closed

comment:5 Changed 10 years ago by Rob2

Ouais, c'est bien moi sur Bitbucket. ;)

Note: See TracTickets for help on using tickets.