Moué, je suis pas certain que ce soit l’affaire du siècle là.
Ça part d’une très bonne intention, mais l’enfer est pavé de bonnes intentions 
:)

Qu’il n’y ait pas de bornes max n’est pas un indice suffisant : le plugin qui 
référence une version 1.x du plugin sans borne max ne garanti absolument pas 
que ça fonctionne avec la v2 ni avec la v3 du plugin saisies, tout simplement 
car au moment où il a été écrit personne ne savait ce qu’il y aurait dans la v2 
ou la v3.

Donc sauf à tester les plugins 1 par 1 pour vérifier (ou en tout cas un nombre 
suffisamment grand), je pense que là tu as plutôt cassé pas mal de plugins dans 
des branches figées pour des raisons X ou Y, et c’est assez contre productif.

En plus, est-ce qu’on est vraiment capable de dire qu’une version 3.x de 
saisies est plus sécure qu’une version 1.x ? Je ne crois pas.
On sait juste qu’on a peut-être bouché un trou qui affecte la version 3.x. Mais 
affectait-il la version 1 ?
La version 3 qui embarque plein de choses en plus n’est-elle pas plus trouée 
que la version 1 ?

Bref, je serai plutôt pour revert ce commit, et ne pas toucher aux 
xxx/branches/vx
Eventuellement juste sur les trunk/ parce qu’au moins ça concerne la version 
maintenue du plugin et on peut supposer que l’auteur corrigera si besoin pour 
permettre au plugin de fonctionner.

Mais bon, de façon générale attention à ces gros changements de compat à 
l’aveugle sans aucun test : on perd de l’information utile et vérifiée (un 
numéro de version dont on est sûr qu’il fonctionne avec) pour une demande dont 
on ne sait pas si elle est légitime ni possible.

Et quand tu arrives 5 ans plus tard pour utiliser une vieille version du plugin 
et que rien ne marche parce que le plugin demande une version de saisies qui ne 
tourne pas sur la version de SPIP que tu utilises, tu es bien dans la merde, et 
tu te retrouves à éplucher des logs pour retrouver quoi au lieu de juste 
pouvoir faire marcher ce vieux site que tu as récupéré…

(on a eu fait ça une fois sur la zone, un gros changement de compat en masse : 
on l’a laissé passé, et ça nous a pourri la vie pendant des années ensuite)

Bises et merci pour tout ton travail que ça n’invalide pas du tout ! :)


--
Cédric
Le 24 juil. 2019 à 20:37 +0200, spip-zone-com...@rezo.net, a écrit :
> Author: spip.fra...@lien-d-amis.net
> Date: 2019-07-24 18:37:23 +0000 (Wed, 24 Jul 2019)
> New Revision: 116113
>
> Modified:
> _plugins_/COG/trunk/paquet.xml
> _plugins_/abonnements/branches/v2/paquet.xml
> _plugins_/abonnements/branches/v3.0/paquet.xml
> _plugins_/abonnements/trunk/paquet.xml
> _plugins_/amap/trunk/paquet.xml
> _plugins_/autocompletion/paquet.xml
> _plugins_/bigbrother/trunk/paquet.xml
> _plugins_/cameras/paquet.xml
> _plugins_/campagnes/trunk/paquet.xml
> _plugins_/cartes/trunk/paquet.xml
> _plugins_/chants/branches/v_0_9/paquet.xml
> _plugins_/chants/trunk/paquet.xml
> _plugins_/chatbox/trunk/paquet.xml
> _plugins_/chmlatex/trunk/paquet.xml
> _plugins_/ckeditor-spip-plugin/trunk/paquet.xml
> _plugins_/clients/branches/v0/paquet.xml
> _plugins_/commandes/trunk/paquet.xml
> _plugins_/contacts_et_organisations/branches/v2/paquet.xml
> _plugins_/contacts_et_organisations/branches/v3/paquet.xml
> _plugins_/contacts_et_organisations/trunk/paquet.xml
> _plugins_/coordonnees/branches/v2/paquet.xml
> _plugins_/coordonnees/tags/v2.0.19/paquet.xml
> _plugins_/csv2spip/dev_spip3/paquet.xml
> _plugins_/dayfill/trunk/paquet.xml
> _plugins_/depublication/trunk/paquet.xml
> _plugins_/dictionnaires/trunk/paquet.xml
> _plugins_/diogene/diogene/branches/v1.4/paquet.xml
> _plugins_/diogene/diogene/branches/v1.7/paquet.xml
> _plugins_/diogene/diogene/trunk/paquet.xml
> _plugins_/diogene/diogene_complements/diogene_gerer_auteurs/branches/v1.4/paquet.xml
> _plugins_/diogene/diogene_complements/diogene_gerer_auteurs/trunk/paquet.xml
> _plugins_/diogene/diogene_complements/diogene_mots/trunk/paquet.xml
> _plugins_/doc2article/trunk/paquet.xml
> _plugins_/elasticsearch/branches/v1/paquet.xml
> _plugins_/elasticsearch/trunk/paquet.xml
> _plugins_/elements/paquet.xml
> _plugins_/emballe_medias/media_collections/trunk/paquet.xml
> _plugins_/encarts/branches/v2.0/paquet.xml
> _plugins_/evaluations/trunk/paquet.xml
> _plugins_/factures/trunk/paquet.xml
> _plugins_/fbantispam/paquet.xml
> _plugins_/formidable/branches/v1/paquet.xml
> _plugins_/forum_sectorise/paquet.xml
> _plugins_/gis/branches/v3/paquet.xml
> _plugins_/greves/paquet.xml
> _plugins_/groupes_mots_arborescents/paquet.xml
> _plugins_/guestbook/trunk/paquet.xml
> _plugins_/imap/paquet.xml
> _plugins_/import_ics/branches/v1/paquet.xml
> _plugins_/importateur_contacts/paquet.xml
> _plugins_/info_sites/branches/v0/paquet.xml
> _plugins_/info_spip/tags/0.7.3/paquet.xml
> _plugins_/info_spip/tags/0.9.5/paquet.xml
> _plugins_/info_spip/tags/0.9.6/paquet.xml
> _plugins_/info_spip/trunk/paquet.xml
> _plugins_/ingredient/trunk/paquet.xml
> _plugins_/inscription/branches/inscription3_5x/paquet.xml
> _plugins_/itineraires/trunk/paquet.xml
> _plugins_/joindre_auto_zone/trunk/paquet.xml
> _plugins_/jquery_corner/paquet.xml
> _plugins_/jquery_masonry/paquet.xml
> _plugins_/jquery_menu_superfish/paquet.xml
> _plugins_/jqvmap/trunk/paquet.xml
> _plugins_/kaye/trunk/paquet.xml
> _plugins_/legendes/branches/v1/paquet.xml
> _plugins_/lim/branches/V1/paquet.xml
> _plugins_/lim/branches/V2/paquet.xml
> _plugins_/lim/trunk/paquet.xml
> _plugins_/manuel_site/paquet.xml
> _plugins_/medias_nettoyage/branches/v0/paquet.xml
> _plugins_/medias_nettoyage/tags/0.10.4/paquet.xml
> _plugins_/medias_nettoyage/trunk/paquet.xml
> _plugins_/menu_evitement/trunk/paquet.xml
> _plugins_/menus/trunk/paquet.xml
> _plugins_/metas_opengraph/paquet.xml
> _plugins_/mots_arborescents/trunk/paquet.xml
> _plugins_/motus/trunk/paquet.xml
> _plugins_/move_events/trunk/paquet.xml
> _plugins_/multilang/branches/v0.7/paquet.xml
> _plugins_/notifications_avancees/trunk/paquet.xml
> _plugins_/ocr/trunk/paquet.xml
> _plugins_/orr/branches/v1/paquet.xml
> _plugins_/orr/branches/v2/paquet.xml
> _plugins_/orr/trunk/paquet.xml
> _plugins_/paravent/trunk/paquet.xml
> _plugins_/parrainage/trunk/paquet.xml
> _plugins_/partenaires/branches/v1/paquet.xml
> _plugins_/partenaires/trunk/paquet.xml
> _plugins_/photos_safe/paquet.xml
> _plugins_/pj2article/paquet.xml
> _plugins_/podcast/trunk/paquet.xml
> _plugins_/polyhierarchie_configurable/trunk/paquet.xml
> _plugins_/porte_plume_codes/trunk/paquet.xml
> _plugins_/prestashop_api/branches/v1/paquet.xml
> _plugins_/produits/trunk/paquet.xml
> _plugins_/produits_liens/trunk/paquet.xml
> _plugins_/projets_sites/branches/v0/paquet.xml
> _plugins_/rao/trunk/paquet.xml
> _plugins_/reglements/trunk/paquet.xml
> _plugins_/requeteursql/paquet.xml
> _plugins_/rubrique_preferee/trunk/paquet.xml
> _plugins_/saisie_liste/trunk/paquet.xml
> _plugins_/selections_editoriales/branches/selections_editoriales_dragdrop/paquet.xml
> _plugins_/selections_editoriales/trunk/paquet.xml
> _plugins_/seminaire/branches/v2.2/paquet.xml
> _plugins_/seminaire/trunk/paquet.xml
> _plugins_/seoptimizr/paquet.xml
> _plugins_/sjcycle/branches/v2/paquet.xml
> _plugins_/souhaits/trunk/paquet.xml
> _plugins_/spip-lettres/trunk/paquet.xml
> _plugins_/spip-sondages/trunk/paquet.xml
> _plugins_/spip_piwik/branches/spip_piwik_2_0/paquet.xml
> _plugins_/spip_piwik/trunk/paquet.xml
> _plugins_/spipad/branches/post_creation/paquet.xml
> _plugins_/spipad/extensions/ad_deux_roues/paquet.xml
> _plugins_/spipad/extensions/ad_service/paquet.xml
> _plugins_/spipad/trunk/paquet.xml
> _plugins_/spipicious_jquery/branches/v1/paquet.xml
> _plugins_/spipicious_jquery/trunk/paquet.xml
> _plugins_/spipimmo/paquet.xml
> _plugins_/spipmotion/branches/v1.3/paquet.xml
> _plugins_/spipmotion/branches/v1.5/paquet.xml
> _plugins_/spipmotion/trunk/paquet.xml
> _plugins_/switcher/trunk/paquet.xml
> _plugins_/taxonomie/tags/v0.2/paquet.xml
> _plugins_/test_url/trunk/paquet.xml
> _plugins_/tickets/trunk/paquet.xml
> _plugins_/titre_de_logo/trunk/paquet.xml
> _plugins_/tradsync/trunk/paquet.xml
> _plugins_/unsplash/trunk/paquet.xml
> _plugins_/updateheaders/paquet.xml
> _plugins_/wp_import/branches/V1/paquet.xml
> _plugins_/wp_import/trunk/paquet.xml
> Log:
> Ne devrait pas y avoir de problème car il n'y avait pas de borne max, et 
> comme ça, nous sommes sûr que les gens auront une version secu
>
>
> Details: https://zone.spip.org/trac/spip-zone/changeset/116113
>
> _______________________________________________
> spip-zone-com...@rezo.net - 
> https://listes.rezo.net/mailman/listinfo/spip-zone-commit
----
spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

Répondre à