Supprimer les fonctions CSV ? #1

Open
opened 2 years ago by rastapopoulos · 5 comments
Owner

Elles sont déjà dans le core de SPIP 3, et dernièrement encore mises à jour pour correspondre à celles de Bonux. Pourquoi sont-elles encore gardées dans Bonux 3 (pour le SPIP qui les a déjà donc) ? On peut les supprimer ?

Elles sont déjà dans le core de SPIP 3, et dernièrement encore mises à jour pour correspondre à celles de Bonux. Pourquoi sont-elles encore gardées dans Bonux 3 (pour le SPIP qui les a déjà donc) ? On peut les supprimer ?
Poster
Owner

Je viens encore de voir une mise à jour, où on synchronise les deux… mais pourquoi diable ? Version du core à jour, et basta non ? Ensuite on vire ça de Bonux…

Je viens encore de voir une mise à jour, où on synchronise les deux… mais pourquoi diable ? Version du core à jour, et basta non ? Ensuite on vire ça de Bonux…
Owner

Parce que je viens d'envoyer des grosses évolutions, qui seront a priori pas dans la 3.2 (et y sont pas pour le moment en tout cas) et que c'est très bien de pouvoir tester et roder dans le plugin plutot que dans le core ?

Parce que je viens d'envoyer des grosses évolutions, qui seront a priori pas dans la 3.2 (et y sont pas pour le moment en tout cas) et que c'est très bien de pouvoir tester et roder dans le plugin plutot que dans le core ?
Poster
Owner

ok mais vu que ça fait plus de 10 ans qu'on "rode" ces fonctions dans Bonux… quelles seraient les conditions d'après toi pour qu'on arrive à vider ce plugin de sa moelle ?

avant on n'avait pas les branches Git et les PR pour montrer facilement aux autres, mais de nos jours ?

il me semblait que beaucoup de gens (équipe noyau ou pas) étaient plutôt d'accord pour dire que ce serait cool d'arriver à intégrer le max et à pouvoir virer ou au moins vider au mieux ce plugin fourre-tout qu'on traine depuis si longtemps… mais c'est peut-être pas possible, je sais pas…

on a ouvert la boite de Pandore et on n'arrive plus à la refermer… :)

ok mais vu que ça fait plus de 10 ans qu'on "rode" ces fonctions dans Bonux… quelles seraient les conditions d'après toi pour qu'on arrive à vider ce plugin de sa moelle ? avant on n'avait pas les branches Git et les PR pour montrer facilement aux autres, mais de nos jours ? il me semblait que beaucoup de gens (équipe noyau ou pas) étaient plutôt d'accord pour dire que ce serait cool d'arriver à intégrer le max et à pouvoir virer ou au moins vider au mieux ce plugin fourre-tout qu'on traine depuis si longtemps… mais c'est peut-être pas possible, je sais pas… on a ouvert la boite de Pandore et on n'arrive plus à la refermer… :)
Owner

le revers de la médaille des fonctions dans le core, c'est qu'une grosse modif n'est possible que avec une version majeure de SPIP, donc tous les 5 ans. Quand tu as besoin de la modif entre temps sur une version stable, pas d'autre choix que de forward-porter dans un plugin en attendant la release.

Donc bon. Je vois pas le problème : l'implémentation du core fait référence, mais on a la possibilité d'utiliser les features en avance ici.

Il me semble qu'il y a bien d'autres sujet et problèmes plus importants à régler que ça...

le revers de la médaille des fonctions dans le core, c'est qu'une grosse modif n'est possible que avec une version majeure de SPIP, donc tous les 5 ans. Quand tu as besoin de la modif entre temps sur une version stable, pas d'autre choix que de forward-porter dans un plugin en attendant la release. Donc bon. Je vois pas le problème : l'implémentation du core fait référence, mais on a la possibilité d'utiliser les features en avance ici. Il me semble qu'il y a bien d'autres sujet et problèmes plus importants à régler que ça...
Collaborator

On s’est dit qu’on allait faire des versions plus fréquentes de SPIP :)
Il faut y croire !

Toujours est-il que les fonctions d’export / import CSV ont divergé entre le core et bonux et que ça rerait même peut être bien d’unifier tout ça, et si besoin de reporter en 3.2 donc :)

On s’est dit qu’on allait faire des versions plus fréquentes de SPIP :) Il faut y croire ! Toujours est-il que les fonctions d’export / import CSV ont divergé entre le core et bonux et que ça rerait même peut être bien d’unifier tout ça, et si besoin de reporter en 3.2 donc :)
Sign in to join this conversation.
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.