V8.0.0 mergée dans master

[Bertrand]

Salut à tous !

Je viens de merder l'ensemble des branches master et v8.0.0.
Toutes les branches master de base et modules sont donc à jour.

Je suis très preneur de tests d'installation et d'upgrade !
Merci par avance pour vos retours !

B.

[marsante]

Salut,

J'espère que tu as juste mergé .

Je venais de mettre en place une image docker sur la v8 pour tester cette aprem. Je pourrais tester tout ça à partir de mercredi.

[Bertrand]

J'ai pas compris, j'ai fait une bêtise ?

B.

[marsante]

Je viens de merder l'ensemble des branches master et v8.0.0.

Non une correction auto dans le post du forum. Quand j'ai lu ça je l'ai pris premier degré, j'étais déjà en train de chercher les commandes pour revert tout ça

[Bertrand]

Mort de rire ! J'avais pas vu ... on va laisser comme ça pour le fun

[marsante]

J'ai apporté une correction pour la mise à jour. Ça casse par contre le module chiro et osteo. Il faut que je regarde et adapte les fichiers php dans class et controller, c'est bien ça ?

[Bertrand]

Merci pour le PR, c'est mergé. Effectivement les noms de fichiers .. hum .. hum ...

Normalement le master de chiro est à jour pour utilisation de la v8.0.0.
Ca veut pas ?
Chez moi ça veut.

B.

[marsante]

A non je n'avais pas vu qu'il y avait des nouveaux commits sur le chiro. Normalement si je répercute sur le osteo ça devrait être bon

[Bertrand]

Je viens de regarder. En théorie y'a rien qui doit être cassant ...
Le changement majeur, c'est le changement de class de connexion à mariaDB. On passe de mysqli à PDO car mysqli ne permet pas l'utilisation de marqueurs nommés.
Mais la compatibilité ensuite est théoriquement totale car le paramètre qui passe le tableau des marqueurs dans les méthodes de msSQL.php est optionnel et on peut très bien passé une requête formatée à l'ancienne avec concaténation.
Bref, très intéressé de savoir ce qui a cassé le module.

B.

[marsante]

Oui pour l'instant rien n'est cassant à part mon interface chaise clavier. Je n'avais pas vu que le module chiro avait été mis à jour récemment. J'ai fait un pull pour le récupérer et ça fonctionne (je ne me souviens plus si je l'ai installé avant ou après la mise à jour 8.0.0 il faudra que je teste, car sur le module ostéo le comportement et différent.).

Pour le module ostéo j'ai tenté de répercuter les changements, mais l'interface des formulaires disparait, j'ai probablement loupé un truc dans la conversion du yaml. Ce qui aurait tendance à me faire confirmer la chose, c'est que si j'installe le module ostéo non mis à jour sur une 7.3.2 et qu'on passe à la 8.0.0 ça fonctionne. La fameuse moulinette fonctionne sur le module ostéo . Je pense qu'à tête reposée, je vais reprendre le code de création sql généré par l'interface web qui répercute la bonne syntaxe yaml après le passage de la moulinette, ça évitera l'erreur chaise clavier.

Par contre, je ne sais pas si c'est important ou pas, mais le module qui génère le code de création récupère les requêtes avec , '1') et non , 1) dans une version module ostéo update 7.3.2 vers 8.0.0 alors que le code yaml est bien converti. Est-ce que ça a une grosse importance pour le fonctionnement avec PDO ou les deux syntaxes sont supportées ? Dans le cas d'un changement, c'est facile à rechercher, remplacer.

Donc plus qu'à retenter la réécriture du module ostéo et de retester le module chiro sur une installation vs une mise à jour comme je ne me souviens plus de ce que j'ai fait .