[SPIP Zone] [Spip-zone-commit] r114639 - in _plugins_/webfonts/branches/webfonts2

Archives Zone
1

Hello :blush:
Possible que je me trompe, mais je pense que tu ferais mieux de mettre cette version dans le trunk et le trunk dans une branche (vu que le numéro du trunk est plus petit) mais cela n'a rien d'obligatoire...
Ensuite concernant ton commit, Je pense qu'il y a un conflit.
Tu devais mettre spip 3.2 comme version mini (quitte à faire un y ou un x+1 dans la version du plug) car tu as supprimé la balise "spip", résultat, ceux en spip 3.0 et sans doute 3.1 risque d'avoir un problème, car la détection de module php n'est dans spip que depuis spip 3.2 https://core.spip.net/issues/3819

Enfin, concernant le conflit, je pense qu'il faut que tu supprime cette ligne https://zone.spip.net/trac/spip-zone/browser/spip-zone/plugins/webfonts/branches/webfonts2/paquet.xml#L24 car tu indique qu'il faut spip 3.2 mini, mais en même temps, tu n'as pas changer https://zone.spip.net/trac/spip-zone/browser/spip-zone/plugins/webfonts/branches/webfonts2/paquet.xml#L6
Franck

-----Message d'origine-----
De : spip-zone-commit@rezo.net <spip-zone-commit@rezo.net>
Envoyé : mardi 26 mars 2019 13:17
À : spip-zone-commit@rezo.net
Objet : [Spip-zone-commit] r114639 - in _plugins_/webfonts/branches/webfonts2

Author: arnaud.berard@mister-graphx.com
Date: 2019-03-26 12:17:11 +0000 (Tue, 26 Mar 2019) New Revision: 114639

Modified:
   _plugins_/webfonts/branches/webfonts2/formulaires/configurer_webfonts2.html
   _plugins_/webfonts/branches/webfonts2/formulaires/configurer_webfonts2.php
   _plugins_/webfonts/branches/webfonts2/lang/webfonts2_fr.php
   _plugins_/webfonts/branches/webfonts2/paquet.xml
   _plugins_/webfonts/branches/webfonts2/webfonts2_pipelines.php
Log:
v0.2.5

- ajout d'une configuration pour inssérer les webfonts dans l'espace privé (pour la preview/wysiwyg)

+ en test/dev : les fonts qui sont inssérées via la pipeline font_list sont récupérable avec scssphp dans les variables injectées.

+ validation du paquet xml qui ne permettait plus l'installation et
+ provoquais une erreur

Details: https://zone.spip.org/trac/spip-zone/changeset/114639

_______________________________________________
Spip-zone-commit@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone-commit

2

Hello franck

Le 26/03/2019 à 22:04, Franck a écrit :

Hello :blush:
Possible que je me trompe, mais je pense que tu ferais mieux de mettre cette version dans le trunk et le trunk dans une branche (vu que le numéro du trunk est plus petit) mais cela n'a rien d'obligatoire...

Disons qu'on est dans le cas ou la branche continue a évoluer et plus le trunk, donc effectivement ça pourrais être …

Ensuite concernant ton commit, Je pense qu'il y a un conflit.
Tu devais mettre spip 3.2 comme version mini (quitte à faire un y ou un x+1 dans la version du plug) car tu as supprimé la balise "spip", résultat, ceux en spip 3.0 et sans doute 3.1 risque d'avoir un problème, car la détection de module php n'est dans spip que depuis spip 3.2 https://core.spip.net/issues/3819

oui la vérification du module php est la a titre informatif quand dès fois sur de hébergements cpanel curl est pas activé par defaut .

mais seulement si on veux récupérer le json des googlefonts et utiliser le plugin comme catalogue.

j'ai viré la balise spip car en fait le plugin ne s'installait plus du tout, erreur de validation du paquet. il me fallait juste rétablir l'installation , et commiter mes modifs.

Enfin, concernant le conflit, je pense qu'il faut que tu supprime cette ligne https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/webfonts/branches/webfonts2/paquet.xml#L24 car tu indique qu'il faut spip 3.2 mini, mais en même temps, tu n'as pas changer https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/webfonts/branches/webfonts2/paquet.xml#L6
Franck

tu as raison, je me suis fais la réflexion aussi en comitant, que ça faisait doublon.

le truc c'est en tout cas la balise <spip></spip> ne passe pas au validateur et je n'en ai trouvé explication nulle part dans https://plugins.spip.net/redaction-du-paquet-xml.html

j'ai donc utilisé le validateur de paquet (ayant eut des surprise avec des crochets ^^ je fais souvent ça quand je touche a un paquet.xml),

https://plugins.spip.net/spip.php?page=valider_paquet

et comme c'est ça qui plantais la validation et plantais l'installe du plugin… , j'ai viré

après je pense que hormis cette histoire de curl dans le paquet le plugin pourrais être compatible 3.X.X,

Merci de tes retours et précisions en tous cas.

Arnaud

-----Message d'origine-----
De : spip-zone-commit@rezo.net <spip-zone-commit@rezo.net>
Envoyé : mardi 26 mars 2019 13:17
À : spip-zone-commit@rezo.net
Objet : [Spip-zone-commit] r114639 - in _plugins_/webfonts/branches/webfonts2

Author: arnaud.berard@mister-graphx.com
Date: 2019-03-26 12:17:11 +0000 (Tue, 26 Mar 2019) New Revision: 114639

Modified:
    _plugins_/webfonts/branches/webfonts2/formulaires/configurer_webfonts2.html
    _plugins_/webfonts/branches/webfonts2/formulaires/configurer_webfonts2.php
    _plugins_/webfonts/branches/webfonts2/lang/webfonts2_fr.php
    _plugins_/webfonts/branches/webfonts2/paquet.xml
    _plugins_/webfonts/branches/webfonts2/webfonts2_pipelines.php
Log:
v0.2.5

- ajout d'une configuration pour inssérer les webfonts dans l'espace privé (pour la preview/wysiwyg)

+ en test/dev : les fonts qui sont inssérées via la pipeline font_list sont récupérable avec scssphp dans les variables injectées.

+ validation du paquet xml qui ne permettait plus l'installation et
+ provoquais une erreur

Details: https://zone.spip.org/trac/spip-zone/changeset/114639

_______________________________________________
Spip-zone-commit@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone-commit