Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatbilité SPIP 4 #292

Open
brunob opened this issue Nov 23, 2022 · 7 comments
Open

Compatbilité SPIP 4 #292

brunob opened this issue Nov 23, 2022 · 7 comments
Assignees

Comments

@brunob
Copy link
Member

brunob commented Nov 23, 2022

Le support de SPIP 3.2 se termine fin 2022 cf https://www.spip.net/fr_article6500.html

Il est grand temps de porter et tester seenthis et tous ses plugins en SPIP 4.1 et de faire la bascule en prod.

Des personnes motivées pour ça ?

@brunob brunob closed this as completed Nov 23, 2022
@brunob brunob reopened this Jan 26, 2023
@brunob
Copy link
Member Author

brunob commented Feb 28, 2024

Bon, je me lance sur le chantier on va commencer par un passage en 4.1. Vu le nombre de plugins qui n'ont pas encore de paquet.xml, je pense que je vais en profiter pour les brancher.

@brunob brunob self-assigned this Feb 28, 2024
@brunob
Copy link
Member Author

brunob commented Feb 28, 2024

TODO en vrac :

  • réparer les paginations fdd950b
  • bug avec le plugin soundmanager, le bouton play n'est pas stylé (il semble manquer <span class="ui-icon-play">play</span> dans le button). 3f3c4c6
  • vérifier que le changement de mot de passe depuis la page de profil fonctionne toujours
  • erreur lors d'un post comportant une image Fatal error: Uncaught Error: Call to undefined function copie_locale_safe() in /home/bb/github/seenthis_squelettes/action/creer_miniature.php:17 Stack trace: #0 /home/bb/sites/seenthis/ecrire/public/aiguiller.php(81): action_creer_miniature_dist() #1 /home/bb/sites/seenthis/ecrire/public.php(99): traiter_appels_actions() #2 /home/bb/sites/seenthis/spip.php(20): include('/home/bb/sites/...') #3 {main} thrown in /home/bb/github/seenthis_squelettes/action/creer_miniature.php on line 17 2a58564
  • sur la page d'une recherche Warning: Use of undefined constant INCLUREAJAXLOAD_affichemeta - assumed 'INCLUREAJAXLOAD_affichemeta' (this will throw an Error in a future version of PHP) in /home/bb/sites/seenthis/ecrire/public/composer.php(96) : eval()'d code on line 157 https://git.spip.net/spip-contrib-extensions/inclure-ajaxload/commit/a072add49e2c526d06418119a4ceed1f865af043

@brunob
Copy link
Member Author

brunob commented Feb 29, 2024

J'hésite à porter https://github.com/seenthis/seenthis_opencalais en SPIP 4, je crois me souvenir que le truc ne fonctionne plus en prod depuis un moment. Un avis @Fil & @martinarnaud ?

@brunob
Copy link
Member Author

brunob commented Feb 29, 2024

J'ai testé avec succès : un post avec une image, un embed bandcamp, l'ajout d'un post aux favoris, la suppression d'un post, la détection d'un lien déjà posté, la page d'un lien, la page d'un tag, un changement de mot de passe, bref tout me semble fonctionnel.

@brunob
Copy link
Member Author

brunob commented Mar 3, 2024

On est pas mal, mais il reste une palanquée de warnings à corriger en PHP 8.1, j'avance là dessus dès que j'ai un moment...

@brunob
Copy link
Member Author

brunob commented Mar 4, 2024

Bon voilà, au final sur les 2 jours de travail environ, j'aurais passé plus de temps à revoir la qualité du code et l'adapter à PHP 8.1, je marque les PRs comme prêtes à être revues et validées.

Je m'occuperai de la compat SPIP 4.2 & PHP 8.2 dans la foulée, mais pas tout de suite car il faut que je travaille un peu côté pro.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant