[SPIP Zone] [Spip-zone-commit] r103476 - in _plugins_

b_b · Mars 24, 2017, 10:52

Hop,

Le 24/03/2017 à 09:47, spip-zone-commit@rezo.net a écrit :

Ajout du plugin tabcsv

Details: Connexion · GitLab

J'ai peut-être mal compris la description du plugin mais on dirait bien qu'il fait doublon avec la fonctionnalité native de SPIP qui permet d'afficher un tableau CSV avec le modèle emb des documents.

Je me trompe ?

++
b_b

Robert_Sebille · Mars 24, 2017, 11:44

Bonjour,

oui et non. tabcsv est plus spécialisé:
- possibilité de choisir titre ou non
- possibilité de choisir le séparateur de champ
- tous les tableaux générés ont aussi une classe tabcsv qui permet de formater ces tableaux-là seulement
- chaque tableau généré peut avoir sa propre classe css (permet par exemple de diminuer la taille de la fonte d'un tableau "un peu trop large" sans affecter les autres tableaux.
- interprète le csv écrit (ou copié/collé) directement dans le texte

Mais il serait sans doute utile d'indiquer en début de documentation quelque chose comme "N'utilisez ce plugin que si vous avez besoin des fonctionnalités ci-dessus, sinon, utilisez la balise "emb" de spip"

bat
rT

-------- Message original --------
Sujet : Re: [Spip-zone-commit] r103476 - in _plugins_
De : Bruno Bergot <brunobergot@gmail.com>
Pour : spip-zone@rezo.net, robert.sebille@cassiopea.org
Date : 24/03/17 11:52

Hop,

Le 24/03/2017 à 09:47, spip-zone-commit@rezo.net a écrit :

Ajout du plugin tabcsv

Details: Connexion · GitLab

J'ai peut-être mal compris la description du plugin mais on dirait bien
qu'il fait doublon avec la fonctionnalité native de SPIP qui permet
d'afficher un tableau CSV avec le modèle emb des documents.

Je me trompe ?

++
b_b

--
♯♪
Cannelle et safran <https://www.youtube.com/watch?v=RCmcyKbOHGY>, curry,
curry.
Genièvre et origan, curry, curry.
♫

b_b · Mars 24, 2017, 11:58

Hop,

Le 24/03/2017 à 12:44, Robert a écrit :

Bonjour,

oui et non. tabcsv est plus spécialisé:
- possibilité de choisir titre ou non
- possibilité de choisir le séparateur de champ
- tous les tableaux générés ont aussi une classe tabcsv qui permet de
formater ces tableaux-là seulement
- chaque tableau généré peut avoir sa propre classe css (permet par
exemple de diminuer la taille de la fonte d'un tableau "un peu trop
large" sans affecter les autres tableaux.
- interprète le csv écrit (ou copié/collé) directement dans le texte

Mais il serait sans doute utile d'indiquer en début de documentation
quelque chose comme "N'utilisez ce plugin que si vous avez besoin des
fonctionnalités ci-dessus, sinon, utilisez la balise "emb" de spip"

Ha ok, merci pour les détails. Juste une dernière remarque, ou un conseil, tu devrais peut-être utiliser un filtre perso plutôt que de coller du php directement dans ton modèle.

++
b_b

Robert_Sebille · Mars 24, 2017, 12:59

Oui, si j'ai le temps ;).

Note que ici, le "mal" est modéré, vu que dans un modèle, c'est le résultat du calcul du php qui est mis en cache (et non le php lui-même comme avec les squelettes)

bat
rT

-------- Message original --------
Sujet : Re: [Spip-zone-commit] r103476 - in _plugins_
De : Bruno Bergot <brunobergot@gmail.com>
Pour : Robert <robert.sebille@cassiopea.org>, spip-zone@rezo.net
Date : 24/03/17 12:58

Hop,

Le 24/03/2017 à 12:44, Robert a écrit :

Bonjour,

oui et non. tabcsv est plus spécialisé:
- possibilité de choisir titre ou non
- possibilité de choisir le séparateur de champ
- tous les tableaux générés ont aussi une classe tabcsv qui permet de
formater ces tableaux-là seulement
- chaque tableau généré peut avoir sa propre classe css (permet par
exemple de diminuer la taille de la fonte d'un tableau "un peu trop
large" sans affecter les autres tableaux.
- interprète le csv écrit (ou copié/collé) directement dans le texte

Mais il serait sans doute utile d'indiquer en début de documentation
quelque chose comme "N'utilisez ce plugin que si vous avez besoin des
fonctionnalités ci-dessus, sinon, utilisez la balise "emb" de spip"

Ha ok, merci pour les détails. Juste une dernière remarque, ou un
conseil, tu devrais peut-être utiliser un filtre perso plutôt que de
coller du php directement dans ton modèle.

++
b_b

--
♯♪
Cannelle et safran <https://www.youtube.com/watch?v=RCmcyKbOHGY>, curry,
curry.
Genièvre et origan, curry, curry.
♫

BoOz · Mars 24, 2017, 3:33

Il ya la boucle data source csv de SPIP qui fait ca déjà non ?

http://www.spip.net/fr_article5443.html

BoOz · Mars 24, 2017, 6:15

Ah oui voila, avec le nom du fichier qui arrive dans #ENV{tsv}

https://github.com/BoOz/simple-spip/blob/master/inclure/tableau_tsv.html

Robert_Sebille · Mars 24, 2017, 7:39

Bonjour,

je ne comprends pas vos remarques.

où votre boucle permet-elle de:

- de choisir titre ou non pour le tableau
- de choisir le séparateur de champ
- que tous les tableaux générés aient aussi une classe qui permet de formater ces tableaux-là seulement
- que chaque tableau généré puisse avoir sa propre classe css (qui permette par exemple de diminuer la taille de la fonte d'un tableau "un peu trop large" sans affecter les autres tableaux).
- d'interpréter le csv écrit (ou copié/collé) directement dans le texte

cela dans un modèle compatible avec les squelettes par défaut sans en créer de nouveau?

bav
Rt

-------- Message original --------
Sujet : Re: [SPIP Zone] [Spip-zone-commit] r103476 - in _plugins_
De : BoOz <booz@rezo.net>
Pour : Bruno Bergot <brunobergot@gmail.com>
Copie à : Robert <robert.sebille@cassiopea.org>, spip zone <spip-zone@rezo.net>
Date : 24/03/17 19:15

Ah oui voila, avec le nom du fichier qui arrive dans #ENV{tsv}

simple-spip/inclure/tableau_tsv.html at master · BoOz/simple-spip · GitHub

Le 24 mars 2017 à 16:33, BoOz <booz@rezo.net <mailto:booz@rezo.net>> a écrit :

    Il ya la boucle data source csv de SPIP qui fait ca déjà non ?

    Exemples de `BOUCLE(DATA)` - SPIP
    <http://www.spip.net/fr_article5443.html>

    Le 24 mars 2017 à 12:58, Bruno Bergot <brunobergot@gmail.com
    <mailto:brunobergot@gmail.com>> a écrit :

        Hop,

        Le 24/03/2017 à 12:44, Robert a écrit :

            Bonjour,

            oui et non. tabcsv est plus spécialisé:
            - possibilité de choisir titre ou non
            - possibilité de choisir le séparateur de champ
            - tous les tableaux générés ont aussi une classe tabcsv qui permet de
            formater ces tableaux-là seulement
            - chaque tableau généré peut avoir sa propre classe css (permet par
            exemple de diminuer la taille de la fonte d'un tableau "un peu trop
            large" sans affecter les autres tableaux.
            - interprète le csv écrit (ou copié/collé) directement dans le texte

            Mais il serait sans doute utile d'indiquer en début de documentation
            quelque chose comme "N'utilisez ce plugin que si vous avez besoin des
            fonctionnalités ci-dessus, sinon, utilisez la balise "emb" de spip"

        Ha ok, merci pour les détails. Juste une dernière remarque, ou un
        conseil, tu devrais peut-être utiliser un filtre perso plutôt que de
        coller du php directement dans ton modèle.

        ++
        b_b
        ----
        spip-zone@rezo.net <mailto:spip-zone@rezo.net> -
        http://listes.rezo.net/mailman/listinfo/spip-zone
        <http://listes.rezo.net/mailman/listinfo/spip-zone>

--
♯♪
Cannelle et safran <https://www.youtube.com/watch?v=RCmcyKbOHGY>, curry,
curry.
Genièvre et origan, curry, curry.
♫

Mist_GraphX · Mars 25, 2017, 9:14

Le 24/03/2017 à 20:39, Robert a écrit :

- de choisir le séparateur de champ

je crois que la fonction native de spip détecte toute seule le séparateur :

<https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/csv.php#L34>

--
Bonne journée
Arnaud B. (Mist. GraphX)

Beurt1 · Mars 25, 2017, 10:06

Oué en DATA c’est possible, une autre possibilité intéressante (par rapport au core) serait de faire l’interprétation du CSV en JS (http://papaparse.com/ → fonctionne super !).

Robert_Sebille · Mars 25, 2017, 11:29

Effectivement, mais, de ce que je comprends du code, sa méthode de détection compte le nombre de , de ; ou de \t et considère que le plus nombreux est le séparateur.

Si on ne peut imposer le séparateur, une rangée comme:
une virgule:, ;deux virgules:,
posera problème.

bat
rT

-------- Message original --------
Sujet : Re: [SPIP Zone] [Spip-zone-commit] r103476 - in _plugins_
De : Mist. GraphX <arnaud.berard@mister-graphx.com>
Pour : spip-zone@rezo.net
Date : 25/03/17 10:14

Le 24/03/2017 à 20:39, Robert a écrit :

- de choisir le séparateur de champ

je crois que la fonction native de spip détecte toute seule le séparateur :

<https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/csv.php#L34>

--
♯♪
Cannelle et safran <https://www.youtube.com/watch?v=RCmcyKbOHGY>, curry,
curry.
Genièvre et origan, curry, curry.
♫

Mist_GraphX · Mars 27, 2017, 7:49

Hello,

Effectivement, mais, de ce que je comprends du code, sa méthode de détection compte le nombre de , de ; ou de \t et considère que le plus nombreux est le séparateur.
Si on ne peut imposer le séparateur, une rangée comme:
une virgule:, ;deux virgules:,
posera problème.

Ha oui, ça peut être aléatoire, effectivement… il serait peut être bien de faire évoluer la fonction _dist pour que tout le monde en profite.
En fait j'utilise toujours les séparateur '';" , bon a savoir que ça existe dans certains cas.

Merci de tes précisions

--
Bonne journée
Arnaud B. (Mist. GraphX)

Le 25/03/2017 à 12:29, Robert a écrit :

Effectivement, mais, de ce que je comprends du code, sa méthode de détection compte le nombre de , de ; ou de \t et considère que le plus nombreux est le séparateur.

Si on ne peut imposer le séparateur, une rangée comme:
une virgule:, ;deux virgules:,
posera problème.

bat
rT

-------- Message original --------
Sujet : Re: [SPIP Zone] [Spip-zone-commit] r103476 - in _plugins_
De : Mist. GraphX <arnaud.berard@mister-graphx.com>
Pour : spip-zone@rezo.net
Date : 25/03/17 10:14

Le 24/03/2017 à 20:39, Robert a écrit :

- de choisir le séparateur de champ

je crois que la fonction native de spip détecte toute seule le séparateur :

<https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/csv.php#L34>

BoOz · Mars 27, 2017, 9:01

Hello,

Pardon j’ai trop peu détaillé ma réponse. La boucle que je donne est une méthode en SPIP pur pour afficher un tableau cvs / tsv / ssv. Elle peut servir de base pour ton modèle qui ajoute les options que tu as mise en place, mais sans recoder tout en PHP.

BoOz

Robert_Sebille · Mars 27, 2017, 12:56

Ah, merci; je comprends mieux!

Je vais essayer de regarder de ce côté dès que j'ai un peu de temps.

Robert.

-------- Message original --------
Sujet : Re: [SPIP Zone] [Spip-zone-commit] r103476 - in _plugins_
De : BoOz <booz@rezo.net>
Pour : Robert <robert.sebille@cassiopea.org>
Copie à : Bruno Bergot <brunobergot@gmail.com>, spip zone <spip-zone@rezo.net>
Date : 27/03/17 11:01

Hello,

Pardon j'ai trop peu détaillé ma réponse. La boucle que je donne est une méthode
en SPIP pur pour afficher un tableau cvs / tsv / ssv. Elle peut servir de base
pour ton modèle qui ajoute les options que tu as mise en place, mais sans
recoder tout en PHP.

BoOz

Le 24 mars 2017 à 20:39, Robert <robert.sebille@cassiopea.org
<mailto:robert.sebille@cassiopea.org>> a écrit :

    Bonjour,

    je ne comprends pas vos remarques.

    où votre boucle permet-elle de:

    - de choisir titre ou non pour le tableau
    - de choisir le séparateur de champ
    - que tous les tableaux générés aient aussi une classe qui permet de
    formater ces tableaux-là seulement
    - que chaque tableau généré puisse avoir sa propre classe css (qui permette
    par exemple de diminuer la taille de la fonte d'un tableau "un peu trop
    large" sans affecter les autres tableaux).
    - d'interpréter le csv écrit (ou copié/collé) directement dans le texte

    cela dans un modèle compatible avec les squelettes par défaut sans en créer
    de nouveau?

    bav
    Rt

    -------- Message original --------
    Sujet : Re: [SPIP Zone] [Spip-zone-commit] r103476 - in _plugins_
    De : BoOz <booz@rezo.net <mailto:booz@rezo.net>>
    Pour : Bruno Bergot <brunobergot@gmail.com <mailto:brunobergot@gmail.com>>
    Copie à : Robert <robert.sebille@cassiopea.org
    <mailto:robert.sebille@cassiopea.org>>, spip zone <spip-zone@rezo.net
    <mailto:spip-zone@rezo.net>>
    Date : 24/03/17 19:15

        Ah oui voila, avec le nom du fichier qui arrive dans #ENV{tsv}

        simple-spip/inclure/tableau_tsv.html at master · BoOz/simple-spip · GitHub
        <https://github.com/BoOz/simple-spip/blob/master/inclure/tableau_tsv.html>

--
♯♪
Cannelle et safran <https://www.youtube.com/watch?v=RCmcyKbOHGY>, curry,
curry.
Genièvre et origan, curry, curry.
♫

Robert_Sebille · Avril 3, 2017, 10:20

Bonjour,

j'ai regardé d'un peu plus près, comme déjà répondu à un précédent message, le problème avec la boucle data est qu'elle ne permet pas de choisir le séparateur; sa méthode de détection compte le nombre de , de ; ou de \t et considère que le plus nombreux est le séparateur.

Donc, dans une rangée comme:
une virgule:, ;deux virgules:,
la boucle data choisira la virgule comme séparateur et je ne peux pas décider que c'est le point-virgule.

NB: Il ne faudrait pas grand chose je pense pour faire évoluer le code de la boucle dans ce sens, quelque part semble-t-il dans function analyse_csv($t) dans <https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/csv.php#L34>, à partir de function inc_csv_to_array_dist($u) dans <https://core.spip.net/projects/spip/repository/entry/spip/ecrire/iterateur/data.php>
mais là, c'est du core de spip où je ne toucherai pas.

bien à toi,
Robert.

-------- Message original --------
Sujet : Re: [SPIP Zone] [Spip-zone-commit] r103476 - in _plugins_
De : Robert <robert.sebille@cassiopea.org>
Pour : BoOz <booz@rezo.net>
Copie à : spip zone <spip-zone@rezo.net>
Date : 27/03/17 14:56

Ah, merci; je comprends mieux!

Je vais essayer de regarder de ce côté dès que j'ai un peu de temps.

Robert.

-------- Message original --------
Sujet : Re: [SPIP Zone] [Spip-zone-commit] r103476 - in _plugins_
De : BoOz <booz@rezo.net>
Pour : Robert <robert.sebille@cassiopea.org>
Copie à : Bruno Bergot <brunobergot@gmail.com>, spip zone
<spip-zone@rezo.net>
Date : 27/03/17 11:01

Hello,

Pardon j'ai trop peu détaillé ma réponse. La boucle que je donne est
une méthode
en SPIP pur pour afficher un tableau cvs / tsv / ssv. Elle peut servir
de base
pour ton modèle qui ajoute les options que tu as mise en place, mais sans
recoder tout en PHP.

BoOz

Le 24 mars 2017 à 20:39, Robert <robert.sebille@cassiopea.org
<mailto:robert.sebille@cassiopea.org>> a écrit :

    Bonjour,

    je ne comprends pas vos remarques.

    où votre boucle permet-elle de:

    - de choisir titre ou non pour le tableau
    - de choisir le séparateur de champ
    - que tous les tableaux générés aient aussi une classe qui permet de
    formater ces tableaux-là seulement
    - que chaque tableau généré puisse avoir sa propre classe css (qui
permette
    par exemple de diminuer la taille de la fonte d'un tableau "un peu
trop
    large" sans affecter les autres tableaux).
    - d'interpréter le csv écrit (ou copié/collé) directement dans le
texte

    cela dans un modèle compatible avec les squelettes par défaut sans
en créer
    de nouveau?

    bav
    Rt

    -------- Message original --------
    Sujet : Re: [SPIP Zone] [Spip-zone-commit] r103476 - in _plugins_
    De : BoOz <booz@rezo.net <mailto:booz@rezo.net>>
    Pour : Bruno Bergot <brunobergot@gmail.com
<mailto:brunobergot@gmail.com>>
    Copie à : Robert <robert.sebille@cassiopea.org
    <mailto:robert.sebille@cassiopea.org>>, spip zone <spip-zone@rezo.net
    <mailto:spip-zone@rezo.net>>
    Date : 24/03/17 19:15

        Ah oui voila, avec le nom du fichier qui arrive dans #ENV{tsv}

simple-spip/inclure/tableau_tsv.html at master · BoOz/simple-spip · GitHub

<https://github.com/BoOz/simple-spip/blob/master/inclure/tableau_tsv.html>

--
♯♪
Cannelle et safran <https://www.youtube.com/watch?v=RCmcyKbOHGY>, curry,
curry.
Genièvre et origan, curry, curry.
♫

b_b · Avril 3, 2017, 11:42

Hop,

Le 03/04/2017 à 12:20, Robert a écrit :

NB: Il ne faudrait pas grand chose je pense pour faire évoluer le code
de la boucle dans ce sens, quelque part semble-t-il dans function
analyse_csv($t) dans
<https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/csv.php#L34>,
à partir de function inc_csv_to_array_dist($u) dans
<https://core.spip.net/projects/spip/repository/entry/spip/ecrire/iterateur/data.php>

mais là, c'est du core de spip où je ne toucherai pas.

Tu ne pourrais pas commiter sur le core directement, mais n'hésite pas à proposer une évolution sur ce point dans un ticket :

https://core.spip.net/

++
b_b