Re : Urlrewrite sur xmcontent [Module xmcontent] - Mon XOOPS (https://www.monxoops.fr)

Register

Urlrewrite sur xmcontent

Grégory M 184
2021/6/29 21:50

#1

Grégory M 184
2021/6/29 21:50

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:

XmfMetagen::generateSeoTitle()

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

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

Re : Urlrewrite sur xmcontent

Nicolas 71
2021/6/29 22:19

#2

Nicolas 71
2021/6/29 22:19

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

Re : Urlrewrite sur xmcontent

Grégory M 184
2021/6/29 22:24

#3

Grégory M 184
2021/6/29 22:24

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

Re : Urlrewrite sur xmcontent

Grégory M 184
2021/6/29 22:24

#4

Grégory M 184
2021/6/29 22:24

Quote:

Nicolas a écrit :
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

Je vais intégrer le code pour voir aussi!

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

Re : Urlrewrite sur xmcontent

Nicolas 71
2021/6/29 22:31

#5

Nicolas 71
2021/6/29 22:31

Je ne trouve pas cette class XmfMetagen dans Xmf

Re : Urlrewrite sur xmcontent

Grégory M 184
2021/6/29 22:40

#6

Grégory M 184
2021/6/29 22:40

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

Re : Urlrewrite sur xmcontent

Nicolas 71
2021/6/29 22:50

#7

Nicolas 71
2021/6/29 22:50

Cela me va très bien cette petite classe

Merci

Nicolas (ForMuss)

Re : Urlrewrite sur xmcontent

Grégory M 184
2021/6/29 22:56

#8

Grégory M 184
2021/6/29 22:56

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

Re : Urlrewrite sur xmcontent

Nicolas 71
2021/6/29 23:08

#9

Nicolas 71
2021/6/29 23:08

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)

Re : Urlrewrite sur xmcontent

Nicolas 71
2021/6/29 23:12

#10

Nicolas 71
2021/6/29 23:12

Pour infos,

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

 public function render() 
    { 
        $ret = '' . NWLINE; 
        foreach ($this->getElements() as $ele) { 
            if (!$ele->isHidden()) { 
                $ret .= ''; 
                $ret .= '' . $ele->getCaption() . ''; 
                $eleDesc = $ele->getDescription(); 
                if (isset($eleDesc)) { 
                    $ret .= '
' . $eleDesc . ''; 
                } 
                $ret .= '' . $ele->render() . ''; 
            } else { 
                $ret .= $ele->render(); 
            } 
        } 
        $ret .= ''; 
 
        return $ret; 
    }

Urlrewrite sur xmcontent

Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Re : Urlrewrite sur xmcontent

Recent forum posts

Recent Comments