Refactoring url rewriting + amélioration SEO

[Grummfy]

remplacer
http://www.fire-soft-board.com/fsb/sujet-X-Y.html
par
http://www.fire-soft-board.com/fsb/sujet/X_Y-nom-du-sujet.html

remplacer
http://www.fire-soft-board.com/fsb/forum-X-Y.html
par
http://www.fire-soft-board.com/fsb/forum/X_Y-nom-du-forum.html

Le numéro de page doit toujours être présent dans l'url, si l'url rewritting est activé!

Il faut aussi ajouter un lien "rel=canonical" dans le head html et "rel='bookmark" dans les url des message unique

cf : http://www.fire-soft-board.com/fsb/sujet-9442-1.html et http://www.fire-soft-board.com/fsb/sujet-9917-1.html

[MGaetan89]

Je vais travailler sur ça quand j'aurais du temps libre.
Je commiterais de temps en temps pour que vous puissiez voir comment ça avance.

[arcalys]

Ok ! Merci beaucoup =)

[MGaetan89]

J'ai commencé à ajouté les règles de réécriture pour les pages qui n'en ont pas (Recherche, FAQ, ...) pour unifier le tout, plutôt que d'avoir que certaines pages qui sont réécrites. Ça vous convient ?

[Grummfy]

pas de problème

[arcalys]

C'est même mieux :)

[MGaetan89]

Vous pouvez voir les changements ici : https://github.com/MGaetan89/Fire-Soft-Board-2/compare/dev...feature_174
Je ferais un seul PR quand tout sera en place.
J'ai encore pas mal d'URL à ajouter et je vais voir s'il est possible de définir la variable tpl BASE avec la configuration du forum (voir index.php).

[arcalys]

Et bien, ça fait pas mal de changements ça :)

[MGaetan89]

Oui, et il y a encore beaucoup à faire^^

Je dois encore trouver comment intégrer le titre du sujet/forum/...

[arcalys]

T'es courageux, moi j'aime vraiment pas ça :/
Enfin, si jamais tu avais besoin, n'hésite pas, j'essayerai de t'aider.

[MGaetan89]

C'est pas bien méchant. Un gros copier/coller en adaptant les paramètres à chaque fois.

Je tenterais une règle générale pour voir ce que ça donne. Par exemple si on a /index.php?p=forum&c=1&page=2, on arrive sur /forum/c-1/page-2.html

[MGaetan89]

Ah ben non je dis des bêtises, puisqu'on ne connait pas le nombre de paramètre de la page, je ne peux pas faire une règle globale, je pense.

[arcalys]

Il doit bien y avoir moyen de généraliser, mais je ne sais pas comment :/

[MGaetan89]

J'ai ajouté les URLs pour la lecture d'un sujet et remit les anciennes règles pour assurer la compatibilité avec les anciennes versions.
Si vous avez le temps de tester et de me dire si vous pensez qu'il faut changer certaines URLs, n'hésitez pas :)
J'ai encore deux bugs à corriger (le lien pour remonter en haut de la page et le lien pour consulter son profil ne fonctionnent pas) et la partie modération (si elle est à faire).

Pour la version 2.1 peut-être voir pour trouver un système d'URL plus souple. Comme on le voit dans les Framework par exemple, peut-être :

index:
    std: 'index.php'
    rewrite: 'index.html'

portal:
    std: 'index.php?p=portal'
    rewrite: 'portal.html'

Mais ça demanderait beaucoup de travail, c'est pour ça que je le verrais plus dans la version 2.1 (ou plus). L'avantage serait que tout serait au sein d'un seul fichier, et pas besoin de modifier le htaccess et le fichier main/globals.php pour modifier une URL (sans compter tous les autres fichiers si on veut modifier l'URL classique).

Par contre j'ai merge les tags dans mon fork, du coup il faudrait tester sur une nouvelle installation ou faire les requêtes pour les tags dans votre forum de test ;)

[arcalys]

Je vais cloner ta version et la tester directement. Pourquoi avoir mergé les tags au passage ? C'est pas vraiment sur qu'on les garde comme je les ai fait actuellement :P

[MGaetan89]

Erreur de ma part. Je voulais le merge dans la branche tags et j'étais encore dans la branche pour l'URL rewrite...
Je ferais un pull request quand les tags seront finit comme ça il n'y aura pas de soucis ;-)

[arcalys]

Je viens de l'installer, alors :

• Dès que j'arrive sur la page d'index, tous les styles sont bousillés, j'ai une page blanche...
• La moitié des liens me redirigent incorrectement en modifiant le path (dans mon cas localhost/test/url devient localhost/url), notamment les catégories, les profils utilisateurs,...

[arcalys]

En essayant de me connecter, je me retrouve avec une URL : login-userlist&g_id=2.html ? Pour la redirection ? En tout cas, je ne peux plus me connecter :/

[MGaetan89]

Etrange, chez moi tout fonctionne très bien. Mais je ne suis pas dans un sous-dossier. Est-ce que tu as testé en incluant mon dernier commit : a439a33 ?

[arcalys]

Ouaip, j'ai pris le contenu de la branche en question pour tester.

[MGaetan89]

Je vais faire une installation dans un dossier chez moi pour voir si le soucis vient de la. Je te tiens au courant

[MGaetan89]

Bon j'ai trouvé pourquoi. J'avais juste oublié de mettre un slash à la fin de la variable BASE dans le template 288ad22
Pour l'URL de connexion c'est bien pour la redirection.
Dit moi si ça fonctionne maintenant.

[arcalys]

Quelques petits trucs :

• Quand tu cliques sur la FAQ dans le menu principal, tu arrives sur "section-forum/keyword-.html". Ca fait un peu bizarre avec le -. sans mot clé. Toujours dans la faq, si tu lances la recherche via le bouton "rechercher", tu as plus d'URl rewriting (voulu ou pas ?).
• Le calendrier ne fonctionne plus (impossible de changer de mois).
• Quand tu es dans "Mon profil", si tu cliques sur "Voir mon profil", il y a un petit foirage.

Le reste à l'air bon jusqu'à présent :)

[arcalys]

Un autre truc cool, serait de remplacer /mp/ par /pm/ pour les messages privés et portail.html par portal.html :)

[arcalys]

Petite question, pourquoi toujours mettre le id- ? Ne peut-on pas faire un truc du genre /forum/999/lereste.html ?

[MGaetan89]

Pour la recherche dans la FAQ, j'ai simplement oublié de la tester. Je vais donc corriger ça, et idem pour le calendrier ;)
Le profil je suis au courant du bug, mais je n'ai pas encore trouvé la solution. Il y en a aussi un quand on clique sur la flèche pour remonter en haut d'un sujet.
/mp/ et /pm/ semble logique oui^^
J'ai mit des préfixes pour la clartés surtout. Ca permet de savoir à quoi sert chaque paramètre quand il y en a plusieurs. Je peux l'enlever s'il faut.

[arcalys]

Non non pas besoin de l'enlever, c'était juste par curiosité =)

[MGaetan89]

Okay :)

Je viens de faire un commit qui prends en compte tes remarques. Je m'occuperais des deux bugs et de la partie modération plus tard.

[arcalys]

Manque plus que le portail.html => portal.html =D
Sinon, sur ce même portail, je vois qu'il y'a des URL avec des modes (newsletter), on pourrait peut-être les prendre en compte non ?

[MGaetan89]

Bien vu, je viens de le rajouter :)

[arcalys]

Cool. Le reste ça a l'air ok, mais je referai un test ce soir au cas ou certains trucs traîneraient encore !

[Taluu]

Hello,

Pense quand même à faire un git pull upstream dev sur ta branche dev et un git rebase une fois de temps en temps, car j'ai vu dans ton diff des commits qui sont pas vraiment liés à la feature :)

(Je jetterai également un oeil vite fait aux commits quand j'aurai un peu de temps...)

[arcalys]

Il y a encore des choses à faire sur ce truc ci ?

[MGaetan89]

Ca fait un moment que j'y ai plus touché. Mais il me semble qu'il manque juste encore la partie modération. Je le finirais dès que j'aurais un moment ;)

[arcalys]

Ok merci, car j'aimerais bien préparer la version de test ce mois-ci :)

[MGaetan89]

Je devrais pouvoir m'en occuper avant la fin du mois :)

[arcalys]

Ok merci :)

[MGaetan89]

J'aurais pas le temps de m'en charger avant la fin du mois, désolé.
Si tu veux absolument sortir une nouvelle version rapidement, il faudrait que quelqu'un d'autre s'en charge, ou alors déplacer cette issue dans une autre milestone.

[arcalys]

Pas de soucis, je déplace au milestone suivant ;)