J'ouvre ce sujet pour faire suite à la discussion ici:

https://www.monxoops.fr/modules/newbb/viewtopic.php?start=0&topic_id=25&order=ASC&status=&mode=0

Une très bonne initiative, par contre de le faire hors github c'est pas top car la version proposée n’intègre pas la dernière version de mon master! Je vais faire la comparaison fichier par fichier pour ce module.

Pour un petit retour rapide, j'ai remarqué qu'il y a une nouvelle classe "XoopsRewrite", je pense que cette classe n'est pas utile car on pourrais utiliser:



Nicolas, tu penses que cette classe pourrai remplacer la tienne? Richard a intégré cette classe pour cela.

--------------------
http://www.monxoops.fr

Salut,

Je vais vais faire un fork de ton module sur le GIT que je viens d'ouvrir.
Je vais regarder cette classe et je reviens vers toi au plus vite.

PS : Je peux te passer un super logiciel de comparaison si besoin

Nicolas

J'ai ajouté une nouvelle branche avec le code de Nicolas:

https://github.com/GregMage/xmcontent/tree/rewriteurl

Ceux qui veulent tester c'est possible avec ce lien!

--------------------
http://www.monxoops.fr

Citation :

Je vais intégrer le code pour voir aussi!

--------------------
http://www.monxoops.fr

Je ne trouve pas cette class XmfMetagen dans Xmf

Salut,

J'ai intégré:

https://github.com/GregMage/xmcontent/commit/0e9ae283f0fd88e540d81887c400845ea24763d5

La balise code de xoops a enlevé les "\" dans mon message avant...

--------------------
http://www.monxoops.fr

Cela me va très bien cette petite classe

Merci

Nicolas (ForMuss)

J'ai fais quelques tests et cela semble bien fonctionner! Merci pour cette ajout, c'est un vrai plus pour ce module!

Dès que je serai sur, j'intègre cette branche dans le master!

--------------------
http://www.monxoops.fr

De rien, je vais pouvoir commencer à regarder pour xmnews du coup.
Si besoin je peux le faire sur une branch comme dans xmcontent, je te passerai mon pseudo Github

Nicolas (ForMuss)

Pour infos,

Dans le module cpslider on a repris la class "blockform.php"
Cela permet d'ajouter des description en utilisant setDescription
Le code :