MineWeb / MineWebCMS

🚀 A French Minecraft CMS since 2015 (used by +1k websites)
https://mineweb.org
GNU General Public License v3.0
93 stars 41 forks source link

feat. cakephp 2.5.3 -> 2.10.20 #195

Closed nivcoo closed 4 years ago

nivcoo commented 4 years ago

Je viens d'apprendre que CakePhP 2.x était toujours maintenu donc je l'ai update, j'ai remonté tout les commits des modifications que tu avais faites sur les libs et j'ai modifié, j'ai enlevé les choses inutiles comme le système de licences et de customs messages. J'ai testé pas mal de fonctionnalités du CMS pour voir si cela marchais bien, thème, plugins, ajout dans une table via un plugin, envoie de notification, j'ai regardé les sessions, le setFlash etc. Faudra testé le systeme de Mise à jour du CMS en lui même même si je ne pense pas que cela puisse poser un probleme, le bug présent des Mise à Jour n'est pas réglé avec cela mais je m'y pencherai car je ne savais pas que tu avais modifier les Libs de cakephp au niveau du Shell. j'ai également ajouté dans le footer du panel admin un check de la version de cakephp.

La maj 2.10.20 date du 16/01/2020

MaximeMichaud commented 4 years ago

Je l'avais vu, depuis longtemps, mais honnêtement aucune idée si cela est une bonne chose de l'update juste comme ça. Personnellement, si cela peut fonctionné tant mieux, mais je ne testerai pas à ta place une aussi grosse modification.

nivcoo commented 4 years ago

J'ai pas compris ta dernière phrase

MaximeMichaud commented 4 years ago

J'ai pas compris ta dernière phrase

Ne fait juste pas comme la 1.8.5 & 1.8.6 encore.

Edit : J'ai testé, car je veux vraiment que cela marche. Tout semble fonctionner :D Stv mon VPS privée, écrit moi discord.

Edit : @nivcoo @Eywek Je crois que c'est une bonne occasion de sortir la 2.0.0 avec cette update. Puisqu'il n'y aura rien de comparable dans le futur afin de justifier une 2.0.0. Par contre, je demanderai plus de temps afin de tester. Et ainsi que du temps afin de mettre à jour si possible tous les thèmes & plugins. D'en plus de faire le ménage dans ceux-ci. (Je pense notamment à Forum qui doit être archivé)

nivcoo commented 4 years ago

C'est vraiment bizarre je recois un mail sur deux :x, Oui par logique tout marche et pour la 2.0.0 jsuis d'accord les thème et les pl ont rien a voir avec le CMS

MaximeMichaud commented 4 years ago

C'est vraiment bizarre je recois un mail sur deux :x, Oui par logique tout marche et pour la 2.0.0 jsuis d'accord les thème et les pl ont rien a voir avec le CMS

Je n'ai pas complètement compris ton message. D'accord pour la 2.0.0, mais pas pour la mise à jour des thèmes ? C'est bien ça ? En fait, je veux faire du ménage dans les thèmes & plugins, pour rendre la 2.0.0 plus "justifiable" Y'a beaucoup de code dans les thèmes qui n'est plus utiliser & tant qu'a faire de sortir une 2.0.0 .. Pourquoi ne pas corrigé.

nivcoo commented 4 years ago

Après avoir vérifié il n'y aucune erreur avec les mises a jour avec la nouvelle maj de CakePhp. Maxime non j'ai dis que au final on peut maj le cms sans forcement maj les pl/th même si c'est le mieux

MaximeMichaud commented 4 years ago

Après avoir vérifié il n'y aucune erreur avec les mises a jour avec la nouvelle maj de CakePhp. Maxime non j'ai dis que au final on peut maj le cms sans forcement maj les pl/th même si c'est le mieux

Oui, on peut très bien, mais j'aimerais bien que MW démontre que le projet est toujours solide. Je manque un peu de temps présentement d'en plus des nouveaux projets ambitieux que je participe. Je ne suis pas dev Web et en effet y'a des thèmes qui sont monstrueux.

Eywek commented 4 years ago

Il y a pas de raison de sortir une 2.0.0 alors que il y a aucun changement notable pour l'utilisateur finale et qu'on fait 0 breaking changes (cf. https://semver.org/)

nivcoo commented 4 years ago

Oui je sais mais ça permet de montrer au gens que à partir de cette version il n’y a pas de problème de maj et une maj du framework

nivcoo commented 4 years ago

Car il y aura jamais un truc enorme donc au fur et a mesure on va y arriver maj après maj et ça va passer à la 2.0.0

Eywek commented 4 years ago

Nope, si on follow le semver on passe a la 2.0 seulement si il y a des breaking changes, ce qui est pas le cas là. (apres la 1.9.0 c'est pas 2.0.0 c'est 1.10.0)

nivcoo commented 4 years ago

D’accord c’est spécial à suivre mais ouai

nivcoo commented 4 years ago

Oui mais même si on passe pas en 2.0.0 faut quand même maj ahah

Eywek commented 4 years ago

je release pas ce soir (au cas ou faudrait rollback), je m'en occupe demain matin

nivcoo commented 4 years ago

Okey demain mais pour moi y'aura pas besoin de back on verra bien

MaximeMichaud commented 4 years ago

Pour le moment, il n'y a aucun problème relatif à ce pull. Par contre, je souhaitais avoir plus de temps pour tester davantage. À toi de voir si tu veux releases.

Eywek commented 4 years ago

Ça dépend dans combien de temps tu test, ça fait déjà un petit moment que ça commence à traîner

nivcoo commented 4 years ago

Il y a pas besoin de tester 20 ans j’ai déjà testé chaque fonctionnalité du CMS et j’ai dis que ça marchait

nivcoo commented 4 years ago

Je l’ai sur mon site perso depuis 1 semaine sans aucun soucis également

MaximeMichaud commented 4 years ago

Bof, cela traîne. Il y a des choses qui ont traîner des mois et qui n'était pas de taille à ça. Bref, Si nivcoo a tout testé. 👍 Je n'avais pas eu de problème, mais côté plugins/thèmes aucune idée. Même s'il devrait pas y avoir de problème normalement.