[SPIP Zone] Patch pour plugin Multiflex

BERTRAND_Joel · Décembre 13, 2018, 2:10

Bonjour à tous,

J'ai commité ce matin un patch pour Multiflex afin de le rendre
compatible Spip 3.2.

Je vois ce patch ici :
https://zone.spip.net/trac/spip-zone/log/spip-zone/squelettes/oswd/3626_multiflex-3

Faut-il faire quelque chose de particulier pour que le plugin soit mis
à jour sur https://plugins.spip.net/squelette_multiflex.html ? J'ai cru
lire que c'était fait automatiquement toutes les heures, mais je suis
comme Soeur Anne, je ne vois rien venir.

Bien cordialement,

JB

Florence_HENRY · Mars 22, 2019, 9:30

Bonjour

Je viens de passer un site en 3.2 et ça coince avec Multiflex, malgré le patch évoqué ci-dessous. J'ai dû repasser le site en 3.1 avec lequel ça marche.

Pour comprendre le problème, j'ai déballé un spip 3.2 tout neuf dans un coin et ai installé multiflex en 0.4.2. J'ai le même problème : aucun style n'est appliqué.

Voilà l'erreur que j'obtiens dans les logs :

PHP Fatal error: Call to undefined function couleur_dec_to_hex() in /Users/flo/Sites/test-multiflex/plugins/auto/squelette_multiflex/v0.4.2/squelette_multiflex_fonctions.php on line 58, referer: http://localhost/~flo/test-multiflex/?var_mode=recalcul

Moyennant de rajouter :
include_spip('inc/filtres_images_mini');
au début du fichier
squelette_multiflex_fonctions.php
ça marche.

Quelqu'un pourrait-il contribuer cette modification ?

Cordialement
Florence

Le 13/12/2018 à 15:10, BERTRAND Joël a écrit :

  Bonjour à tous,

  J'ai commité ce matin un patch pour Multiflex afin de le rendre
compatible Spip 3.2.

  Je vois ce patch ici :
https://zone.spip.net/trac/spip-zone/log/spip-zone/_squelettes_/oswd/3626_multiflex-3

  Faut-il faire quelque chose de particulier pour que le plugin soit mis
à jour sur https://plugins.spip.net/squelette_multiflex.html ? J'ai cru
lire que c'était fait automatiquement toutes les heures, mais je suis
comme Soeur Anne, je ne vois rien venir.

  Bien cordialement,

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

--
Florence HENRY
LESIA - CNRS / Observatoire de Paris

BERTRAND_Joel · Mai 2, 2019, 8:45

Florence HENRY a écrit :

Bonjour

Je viens de passer un site en 3.2 et ça coince avec Multiflex, malgré le
patch évoqué ci-dessous. J'ai dû repasser le site en 3.1 avec lequel ça
marche.

Pour comprendre le problème, j'ai déballé un spip 3.2 tout neuf dans un
coin et ai installé multiflex en 0.4.2. J'ai le même problème : aucun
style n'est appliqué.

Voilà l'erreur que j'obtiens dans les logs :

PHP Fatal error: Call to undefined function couleur_dec_to_hex() in
/Users/flo/Sites/test-multiflex/plugins/auto/squelette_multiflex/v0.4.2/squelette_multiflex_fonctions.php
on line 58, referer:
http://localhost/~flo/test-multiflex/?var_mode=recalcul

Moyennant de rajouter :
include_spip('inc/filtres_images_mini');
au début du fichier
squelette_multiflex_fonctions.php
ça marche.

Quelqu'un pourrait-il contribuer cette modification ?

Bonjour,

Je peux Néanmoins, je n'arrive pas à reproduire le problème sur ma
machine de test. Serait-il possible d'avoir un accès à la vôtre ?

Bien cordialement,

JKB

Florence_HENRY · Mai 7, 2019, 12:48

Bonjour

Ce n'est pas techniquement faisable (firewall)...

J'ai pourtant le même problème sur chaque spip 3.2 que j'installe (sur des machines différentes).

Avez-vous déballé un spip 3.2 tout neuf depuis svn ?

Cordialement
Florence

Le 02/05/2019 à 10:45, BERTRAND Joël a écrit :

Florence HENRY a écrit :

Bonjour

Je viens de passer un site en 3.2 et ça coince avec Multiflex, malgré le
patch évoqué ci-dessous. J'ai dû repasser le site en 3.1 avec lequel ça
marche.

Pour comprendre le problème, j'ai déballé un spip 3.2 tout neuf dans un
coin et ai installé multiflex en 0.4.2. J'ai le même problème : aucun
style n'est appliqué.

Voilà l'erreur que j'obtiens dans les logs :

PHP Fatal error: Call to undefined function couleur_dec_to_hex() in
/Users/flo/Sites/test-multiflex/plugins/auto/squelette_multiflex/v0.4.2/squelette_multiflex_fonctions.php
on line 58, referer:
http://localhost/~flo/test-multiflex/?var_mode=recalcul

Moyennant de rajouter :
include_spip('inc/filtres_images_mini');
au début du fichier
squelette_multiflex_fonctions.php
ça marche.

Quelqu'un pourrait-il contribuer cette modification ?

  Bonjour,

  Je peux Néanmoins, je n'arrive pas à reproduire le problème sur ma
machine de test. Serait-il possible d'avoir un accès à la vôtre ?

  Bien cordialement,

  JKB

--
Florence HENRY
LESIA - CNRS / Observatoire de Paris

BERTRAND_Joel · Mai 7, 2019, 2:26

Florence HENRY a écrit :

Bonjour

Bonjour,

Ce n'est pas techniquement faisable (firewall)...

J'ai pourtant le même problème sur chaque spip 3.2 que j'installe (sur
des machines différentes).

Avez-vous déballé un spip 3.2 tout neuf depuis svn ?

Non, pas depuis le SVN. J'ai utilisé la dernière version stable, mais
je l'ai fait sur un site complet sur une machine de test et il est
possible qu'il y ait un effet de bord avec un autre plugin.

Dans ce cas, je vous demanderai de tester la modification suivante
avant que je la remonte sur le plugin.

Dans le fichier squelette_multiflex_fonctions.php, rajoutez simplement
après

include_spip('inc/filtres_images');

(ligne 29) ceci :
include_spip('inc/filtres_images_mini');

La fonction devrait commencer comme ceci :

function calculer_balise_KRVB($r,$v,$b,$kr,$kv,$kb){
include_spip('inc/filtres_images');
include_spip('inc/filtres_images_mini');

if ($r == 255 && $v == 255 && $b == 255) return '#ffffff';
if ($r == 0 && $v == 0 && $b == 0) return '#000000';

    if ($kr > 1) $kr = 1;
    if ($kr < -1) $kr = -1;
    if ($kv > 1) $kv = 1;
    if ($kv < -1) $kv = -1;
    if ($kb > 1) $kb = 1;
    if ($kb < -1) $kb = -1;

Bien cordialement,

JKB

Florence_HENRY · Mai 7, 2019, 2:41

Le 07/05/2019 à 16:26, BERTRAND Joël a écrit :

  Dans ce cas, je vous demanderai de tester la modification suivante
avant que je la remonte sur le plugin.

  Dans le fichier squelette_multiflex_fonctions.php, rajoutez simplement
après

  include_spip('inc/filtres_images');

(ligne 29) ceci :
  include_spip('inc/filtres_images_mini');

Ca marche avec cette modification (et donc en ayant retiré la mienne).

Merci
Florence

--
Florence HENRY
LESIA - CNRS / Observatoire de Paris

BERTRAND_Joel · Mai 7, 2019, 3:46

Florence HENRY a écrit :

Le 07/05/2019 à 16:26, BERTRAND Joël a écrit :
Dans ce cas, je vous demanderai de tester la modification suivante
avant que je la remonte sur le plugin.
Dans le fichier squelette\_multiflex\_fonctions\.php, rajoutez
simplement
après
include\_spip$&#39;inc/filtres\_images&#39;$;
(ligne 29) ceci :
include_spip('inc/filtres_images_mini');
Ca marche avec cette modification (et donc en ayant retiré la mienne).

Merci, je committe.

JKB

gilcot · Mai 7, 2019, 8:59

Salut Joël

La fonction devrait commencer comme ceci :

function calculer_balise_KRVB($r,$v,$b,$kr,$kv,$kb){
include_spip(‹ inc/filtres_images ›);
include_spip(‹ inc/filtres_images_mini ›);

if ($r == 255 && $v == 255 && $b == 255) return ‹ #ffffff ›;
if ($r == 0 && $v == 0 && $b == 0) return ‹ #000000 ›;

if ($kr > 1) $kr = 1;
if ($kr < -1) $kr = -1;
if ($kv > 1) $kv = 1;
if ($kv < -1) $kv = -1;
if ($kb > 1) $kb = 1;
if ($kb < -1) $kb = -1;

Ceci m’interpelle : pourquoi répéter le duo de lignes trois fois ?

BERTRAND_Joel · Mai 20, 2019, 9:02

Gildas Cotomale a écrit :

Salut Joël

     La fonction devrait commencer comme ceci :

    function calculer_balise_KRVB($r,$v,$b,$kr,$kv,$kb){
     include_spip('inc/filtres_images');
     include_spip('inc/filtres_images_mini');

     if ($r == 255 && $v == 255 && $b == 255) return '#ffffff';
     if ($r == 0 && $v == 0 && $b == 0) return '#000000';

     if ($kr > 1) $kr = 1;
     if ($kr < -1) $kr = -1;
     if ($kv > 1) $kv = 1;
     if ($kv < -1) $kv = -1;
     if ($kb > 1) $kb = 1;
     if ($kb < -1) $kb = -1;

Ceci m'interpelle : pourquoi répéter le duo de lignes trois fois ?

Bonjour Gildas,

Parce que je n'ai pas touché à ce bout de code

On pourrait effectivement factoriser un peu le tout, mais je n'ai pas
trop le temps de faire ce genre de chose actuellement. Pour l'instant,
ça fonctionne comme ça et je pense le laisser ainsi, il y a des choses
plus urgentes à régler dans le plugin.

Bien cordialement,

JKB