[SPIP Zone] flux et reflux

Archives Zone
1

dans http://zone.spip.org/trac/spip-zone/changeset/84020
je vois arriver

function switcher_insert_head($flux){
         $flux .= switcher_insert_head_css($flux); // au cas ou il n'est pas implemente
         return $flux;
}

alors que switcher_insert_head($flux) renvoie le $flux modifié.

ça ne me semble pas souhaitable.

JL

2

En effet, il ne faut pas.
Le code correspond à une ancienne pratique.
Depuis SPIP 3, si il n'est pas présent dans le squelette, #INSERT_HEAD_CSS est automatiquement inséré dans #INSERT_HEAD.
On peut donc de manière sure se contenter d'injecter dans le pipeline insert_head_css

Cédric

Le 4 août 2014 à 23:22, JLuc <jluc@no-log.org> a écrit :

dans Connexion · GitLab
je vois arriver

function switcher_insert_head($flux){
       $flux .= switcher_insert_head_css($flux); // au cas ou il n'est pas implemente
       return $flux;
}

alors que switcher_insert_head($flux) renvoie le $flux modifié.

ça ne me semble pas souhaitable.

JL

----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

3

Yep, mais Switcher est compatible de 1.9 vers 3.0, ça pose pas de soucis ?

–
Yohann Prigent

On 5 Aug 2014 at 00:34:23, Cédric Morin (cedric@yterium.com) wrote:

En effet, il ne faut pas.
Le code correspond à une ancienne pratique.
Depuis SPIP 3, si il n’est pas présent dans le squelette, #INSERT_HEAD_CSS est automatiquement inséré dans #INSERT_HEAD.
On peut donc de manière sure se contenter d’injecter dans le pipeline insert_head_css

Cédric

Le 4 août 2014 à 23:22, JLuc jluc@no-log.org a écrit :

dans Connexion · GitLab
je vois arriver

function switcher_insert_head($flux){
$flux .= switcher_insert_head_css($flux); // au cas ou il n’est pas implemente
return $flux;
}

alors que switcher_insert_head($flux) renvoie le $flux modifié.

ça ne me semble pas souhaitable.

JL

spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

4

Alors il faut faire une branche, ou utiliser paquet.xml pour injecter via insert_head_css en SPIP 3 uniquement.
Mais il ne faut plus utiliser ce code basé sur une statique, c’est bugogène.

Cédric

Le 5 août 2014 à 00:35, Yohann Prigent <prigent.yohann@gmail.com> a écrit :

Yep, mais Switcher est compatible de 1.9 vers 3.0, ça pose pas de soucis ?

–
Yohann Prigent

On 5 Aug 2014 at 00:34:23, Cédric Morin (cedric@yterium.com) wrote:

En effet, il ne faut pas.
Le code correspond à une ancienne pratique.
Depuis SPIP 3, si il n’est pas présent dans le squelette, #INSERT_HEAD_CSS est automatiquement inséré dans #INSERT_HEAD.
On peut donc de manière sure se contenter d’injecter dans le pipeline insert_head_css

Cédric

Le 4 août 2014 à 23:22, JLuc <jluc@no-log.org> a écrit :

dans http://zone.spip.org/trac/spip-zone/changeset/84020
je vois arriver

function switcher_insert_head($flux){
$flux .= switcher_insert_head_css($flux); // au cas ou il n’est pas implemente
return $flux;
}

alors que switcher_insert_head($flux) renvoie le $flux modifié.

ça ne me semble pas souhaitable.

JL

spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

5

Hop : http://zone.spip.org/trac/spip-zone/changeset/84023

–
Yohann Prigent

On 5 Aug 2014 at 00:38:42, Cédric Morin (cedric@yterium.com) wrote:

Alors il faut faire une branche, ou utiliser paquet.xml pour injecter via insert_head_css en SPIP 3 uniquement.
Mais il ne faut plus utiliser ce code basé sur une statique, c’est bugogène.

Cédric

Le 5 août 2014 à 00:35, Yohann Prigent <prigent.yohann@gmail.com> a écrit :

Yep, mais Switcher est compatible de 1.9 vers 3.0, ça pose pas de soucis ?

–
Yohann Prigent

On 5 Aug 2014 at 00:34:23, Cédric Morin (cedric@yterium.com) wrote:

En effet, il ne faut pas.
Le code correspond à une ancienne pratique.
Depuis SPIP 3, si il n’est pas présent dans le squelette, #INSERT_HEAD_CSS est automatiquement inséré dans #INSERT_HEAD.
On peut donc de manière sure se contenter d’injecter dans le pipeline insert_head_css

Cédric

Le 4 août 2014 à 23:22, JLuc <jluc@no-log.org> a écrit :

dans http://zone.spip.org/trac/spip-zone/changeset/84020
je vois arriver

function switcher_insert_head($flux){
$flux .= switcher_insert_head_css($flux); // au cas ou il n’est pas implemente
return $flux;
}

alors que switcher_insert_head($flux) renvoie le $flux modifié.

ça ne me semble pas souhaitable.

JL

spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

6

Tant mieux si ma remarque a levé un lièvre.

Moi ce qui m'interpelait, c'est avec le code corrigé depuis,
on avait l'équivalent de $flux .= $flux,
ce qui double le flux au lieu de le compléter...
à moins qu'il soit vide initialement, ce qui n'est pas vérifié.

JLuc

Le 05/08/2014 00:45, Yohann Prigent a écrit :

Hop : Connexion · GitLab

--
Yohann Prigent

On 5 Aug 2014 at 00:38:42, Cédric Morin (cedric@yterium.com <mailto:cedric@yterium.com>) wrote:

Alors il faut faire une branche, ou utiliser paquet.xml pour injecter via insert_head_css en SPIP 3 uniquement.
Mais il ne faut plus utiliser ce code basé sur une statique, c'est bugogène.

Cédric

Le 5 août 2014 à 00:35, Yohann Prigent <prigent.yohann@gmail.com <mailto:prigent.yohann@gmail.com>> a écrit :

Yep, mais Switcher est compatible de 1.9 vers 3.0, ça pose pas de soucis ?

--
Yohann Prigent

On 5 Aug 2014 at 00:34:23, Cédric Morin (cedric@yterium.com <mailto:cedric@yterium.com>) wrote:

En effet, il ne faut pas.
Le code correspond à une ancienne pratique.
Depuis SPIP 3, si il n'est pas présent dans le squelette, #INSERT_HEAD_CSS est automatiquement inséré dans
#INSERT_HEAD.
On peut donc de manière sure se contenter d'injecter dans le pipeline insert_head_css

Cédric

Le 4 août 2014 à 23:22, JLuc <jluc@no-log.org <mailto:jluc@no-log.org>> a écrit :

> danshttp://zone.spip.org/trac/spip-zone/changeset/84020
> je vois arriver
>
> function switcher_insert_head($flux){
> $flux .= switcher_insert_head_css($flux); // au cas ou il n'est pas implemente
> return $flux;
> }
>
> alors que switcher_insert_head($flux) renvoie le $flux modifié.
>
> ça ne me semble pas souhaitable.
>
> JL
>
> ----
>spip-zone@rezo.net <mailto:spip-zone@rezo.net> - http://listes.rezo.net/mailman/listinfo/spip-zone

----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone