[suggestions] Affichage du nom des tags

fab56

Bonjour,

Une suggestion d'amélioration concernant l'affichage du nom des tags dans la balise title et dans le tags.php du thème par défaut.

Le nom du tag est affiché de la façon suivante :

[== Indéfini ==]
<p class="directory">
<strong><?php echo $plxShow->plxMotor->cible; ?></strong>
</p>

Or le titre obtenu de cette manière pose des problèmes si nom du tags contient des accents ou espaces.
Cela va par exemple afficher "energies-renouvelables" à la place d'"énergies renouvelables".

J'obtiens le nom correctement écrit via le code :

[== Indéfini ==]
<h4>
<?php echo $plxShow->tagName(); ?>
</h4>

Par contre, pour la balise title, je n'ai pas directement la main car c'est la même fonction qui est utilisée sur toutes les pages :

[== Indéfini ==]
<title><?php $plxShow->pageTitle(); ?></title>

J'ai bidouillé comme ça :

[== Indéfini ==]
<title>
<?php
if($plxMotor->mode=='tags')
  {
  $plxShow->tagName();
  }
else
  {
  $plxShow->pageTitle();
  }
?>
</title>

Mais ça serait mieux que la méthode pageTitle() traite directement le problème.
A priori ça se passe en ligne 231 du script class.plx.show.php mais je ne connais pas encore assez bien le code pour proposer la solution

Fab

Stéphane

Bonjour

Ha oui bien vu !.
J'ai donc bien pris en compte tes remarques.

Les bonnes corrections sont:
dans le fichier tags.php du theme, remplacer

<strong><?php echo $plxShow->plxMotor->cible; ?></strong>

par

<strong><?php $plxShow->tagName(); ?></strong>

dans le fichier class.plx.show.php, fonction pageTitle()

if($this->plxMotor->mode == 'tags') {
	echo plxUtils::strCheck(L_PAGETITLE_TAG.' '.$this->plxMotor->cible.' - '.$this->plxMotor->aConf['title']);
	return;
}

par

if($this->plxMotor->mode == 'tags') {
	echo plxUtils::strCheck(L_PAGETITLE_TAG.' '.$this->plxMotor->cibleName.' - '.$this->plxMotor->aConf['title']);
	return;
}

Ces modifs sont intégrées pour la prochaine version de PluXml

Merci

fab56

OK super

On retrouve la même erreur dans le plugin "filAriane".

En ligne 43, il vaut mieux écrire :

[== PHP ==]
echo plxUtils::strCheck($plxShow->tagName());

Moins important mais il n'y a pas de balise meta description prévue par défaut pour les pages tags (parce que l'on a rien d'autre que le nom je suppose).