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

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 :slight_smile:

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 ! :slight_smile:

--
Cédric
Le 24 juil. 2019 à 20:37 +0200, spip-zone-commit@rezo.net, a écrit :

Author: spip.franck@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: Connexion · GitLab

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