Simplification de code #3505

Closed
opened 8 years ago by RealET · 1 comments
RealET commented 8 years ago

Dans inc/filtres_ecrire.php il y a :

function lien_article_virtuel($virtuel){
	include_spip('inc/lien');
  if (!$virtuel = virtuel_redirige($virtuel))
	  return '';
  return propre("[->".$virtuel."]");
}

Mais virtuel_redirige() ne fait rien que renvoyer ce qu'elle reçoit (contrairement à ses commentaires)

et propre() pourrait avantageusement être remplacé par expanser_liens()

Ci-joint le patch.

Dans inc/filtres_ecrire.php il y a : <pre> function lien_article_virtuel($virtuel){ include_spip('inc/lien'); if (!$virtuel = virtuel_redirige($virtuel)) return ''; return propre("[->".$virtuel."]"); } </pre> Mais virtuel_redirige() ne fait rien que renvoyer ce qu'elle reçoit (contrairement à ses commentaires) et propre() pourrait avantageusement être remplacé par expanser_liens() Ci-joint le patch.
Owner

Mauvaise lecture du code : la fonction virtuel_redirige appelée est celle de TextWheel qui surcharges inc/lien, et qui fait bien quelque chose. Le remplacement de propre par expanser_liens est en effet possible, mais pas forcément génial car propre est une api plus stable (expanser_liens est une fonction interne de traitement a propre)
Statut changé à Fermé

Mauvaise lecture du code : la fonction virtuel_redirige appelée est celle de TextWheel qui surcharges inc/lien, et qui fait bien quelque chose. Le remplacement de propre par expanser_liens est en effet possible, mais pas forcément génial car propre est une api plus stable (expanser_liens est une fonction interne de traitement a propre) **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.