text_truncate est généreusement foireux #3

Closed
opened 2 years ago by JLuc · 0 comments
JLuc commented 2 years ago
Collaborator

Suite à un commentaire sur contrib https://contrib.spip.net/Bonux-pour-SPIP3#forum505774 j'ai regardé le source de text_truncate, et il semble

  • la variable $ending n'est pas définie. Elle il faut l'initialier à $options['ending'] après le merge

  • la variable $exact n'est pas définie. Si on fait comme pour $ending, il faut l'initialier avec $options['exact'].

  • la variable $html n'est pas définie, si bien que l'effet est seulement celui d'un substr. Si on faisait comme pour $ending, il faudrait l'initialier avec $options['html'], ou alors c'est plus complexe car il y a aussi l'argument $text. Peut être faut il détecter si cest du html ou du texte d'abord ?

Le source linké est HS mais retrouvable sur archive.org :
https://web.archive.org/web/20110902013547/http://book.cakephp.org/view/1469/Text
on voit que c'est cakePHP et il y a la doc d'une fonction truncate là 👍 https://api.cakephp.org/4.0/class-Cake.Utility.Text.html#truncate
Globalement ça confirme les intuitions énoncées. Et $html c'est un booléen.

Suite à un commentaire sur contrib https://contrib.spip.net/Bonux-pour-SPIP3#forum505774 j'ai regardé le source de text_truncate, et il semble - la variable $ending n'est pas définie. Elle il faut l'initialier à $options['ending'] après le merge - la variable $exact n'est pas définie. Si on fait comme pour $ending, il faut l'initialier avec $options['exact']. - la variable $html n'est pas définie, si bien que l'effet est seulement celui d'un substr. Si on faisait comme pour $ending, il faudrait l'initialier avec $options['html'], ou alors c'est plus complexe car il y a aussi l'argument $text. Peut être faut il détecter si cest du html ou du texte d'abord ? Le source linké est HS mais retrouvable sur archive.org : https://web.archive.org/web/20110902013547/http://book.cakephp.org/view/1469/Text on voit que c'est cakePHP et il y a la doc d'une fonction truncate là :+1: https://api.cakephp.org/4.0/class-Cake.Utility.Text.html#truncate Globalement ça confirme les intuitions énoncées. Et $html c'est un booléen.
JLuc closed this issue 2 years ago
Sign in to join this conversation.
No Label
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.