r11090 - in spip: dist/modeles ecrire/inc

Author: esj@rezo.net
Date: 2008-01-17 17:44:09 +0100 (jeu, 17 jan 2008)
New Revision: 11090

Log:
Utilisation du filtre ''contenu_document'' revu dans le modele Text (aurait du faire partie de [11089].

Ajout de Camille: révision de ''filtre text_csv'' qui considère qu'un fichier CSV commence toujours par une ligne nommant les colonnes, et donc la typographie différement (mais c'est plutôt des TH qu'il faudrait faire produire au raccourci de tableau là).

Modified:
   spip/dist/modeles/text.html
   spip/ecrire/inc/filtres.php

Details: http://trac.rezo.net/trac/spip/changeset/11090

S'lt

Merci pour le commit

Mais comme tu le faisais remarquer on n'a pas possibilité de savoir si
il y a ou non l'entête.
Du coup pourquoi n'as tu pas gardé l'option avecentete.
Cela offrirait au moins au webmestre un cas possible :

[(#URL_DOCUMENT|contenu_document|filtre_text_csv{0})]

Cela est effectivement un cas très spécifique mais ça ne mange pas de pain.

(mais c'est plutôt des TH qu'il faudrait faire produire au raccourci
de tableau là).

SPIP génère bien un thead avec th pour l'entête. A moins que je ne
comprenne pas la remarque.

Km

Le 18 janv. 08 à 10:43, cam.lafit@azerttyu.net a écrit :

Mais comme tu le faisais remarquer on n'a pas possibilité de savoir si
il y a ou non l'entête.
Du coup pourquoi n'as tu pas gardé l'option avecentete.
Cela offrirait au moins au webmestre un cas possible :

[(#URL_DOCUMENT|contenu_document|filtre_text_csv{0})]

Ce qui m'ennuie c'est que ce filtre se nomme ainsi parce qu'il est appelable génériquement par appliquer_filtre
comme dans le modèle text.html:

[(#URL_DOCUMENT|contenu_document|appliquer_filtre{#MIME_TYPE,filtre_text_txt_dist}]

bien que PHP, contrairement à beaucoup de langages, accepte que l'ensemble des fonctions appelables soit hétérogène sur le plan des signatures, accepter une telle généralité empêche d'avance certains catégories d'optimisation.
Je préfère donc ne pas ouvrir la porte à ça a priori; c'est peut-être une prudence excessive mais la gêne est trop minime pour y renoncer au moins pour le moment.

(mais c'est plutôt des TH qu'il faudrait faire produire au raccourci
de tableau là).

SPIP génère bien un thead avec th pour l'entête. A moins que je ne
comprenne pas la remarque.

Ah tiens, j'avais le cas d'une colonne vide dans ma première ligne et ça a suffit pour que SPIP sorte autre chose.
Effectivement quand toutes les colonnes sont remplies ça marche.

Committo,Ergo:Sum

S'lt

Ce qui m'ennuie c'est que ce filtre se nomme ainsi parce qu'il est
appelable génériquement par appliquer_filtre
comme dans le modèle text.html:

[...]

l'ensemble des fonctions appelables soit hétérogène sur le plan
des signatures, accepter une telle généralité empêche d'avance
certains catégories d'optimisation.
Je préfère donc ne pas ouvrir la porte à ça a priori; c'est peut-être
une prudence excessive mais la gêne est trop minime pour y
renoncer au moins pour le moment.

Pas de pb :slight_smile:
Égoïstement je dirais du moment que j'ai mes entêtes :slight_smile:

Sérieusement, Je comprends que même si PHP est permissif il vaut mieux
garder une cohérence dans l'écriture des fonctions.
Et tant qu'il y a une unique exception, ça serait dommage de tout
casser juste pour ça.

>> (mais c'est plutôt des TH qu'il faudrait faire produire au
>> raccourci de tableau là).
>
> SPIP génère bien un thead avec th pour l'entête.

Ah tiens, j'avais le cas d'une colonne vide dans ma première ligne > et ça a suffit pour que SPIP sorte autre chose.

Pas testé ce cas de figure. Si c'est le cas c'est un peu ballot.

Merci

Km