[spip-dev] [spip-commit] r11844 - in spip: . ecrire/urls

Dev
1

Pourquoi ce niveau "article/" dans l'URL ?

-Nicolas

2

c'est un choix, mais cela reste personnalisable via le pipeline qui va bien :
cela peut etre a/titrearticle ou meme titrearticle tout court
Cédric

3

OK, super !

J'avais proposé ma propre généralisation voilà quelques mois -- http://trac.rezo.net/trac/spip/ticket/1148 -- avec des fonctionnalités qu'il me semblerait intéressant d'ajouter à ta version :

- reprise du groupe de mot clef comme arborescence d'un mot clef
- possibilité d'avoir au choix des URL en minuscules ou conservant la casse

Sinon, il y a un soucis avec les URL des rubriques quand on définie ceci :
define('_terminaison_urls_propres', '.html');

On obtient par exemple ceci :
http://spip-svn.local/Rubrique-1/.html

Alors que ceci serait plus adapté :
http://spip-svn.local/Rubrique-1/

-Nicolas

4

Autre remarque sur tes URL arborescentes, tu ne stockes en base que la "fin" de l'URL correspondant, ce qui empêche d'avoir des contenus de même titre avec une URL vraiment propre (sans id) même s'ils sont dans des rubriques différentes.

Par exemple, j'obtiens ça :
http://spip-svn.local/Rubrique-1/article/Exemple-d-article.html
http://spip-svn.local/Rubrique-2/article/Exemple-d-article-4.html

Alors que j'aurais préféré ça :
http://spip-svn.local/Rubrique-1/article/Exemple-d-article.html
http://spip-svn.local/Rubrique-2/article/Exemple-d-article.html

-Nicolas

5

Sinon, il y a un soucis avec les URL des rubriques quand on définie ceci :
define('_terminaison_urls_propres', '.html');

On obtient par exemple ceci :
http://spip-svn.local/Rubrique-1/.html

Alors que ceci serait plus adapté :
http://spip-svn.local/Rubrique-1/

Ah, d'ailleurs, ça foire même sur les articles. L'URL se termine bien avec .html, mais elle n'est pas reconnue, on se prend une erreur 404.

-Nicolas

6

Mais a contrario, si on renomme une rubrique, cela n'oblige pas à recalculer les url de tous les enfants, ce qui etait un défaut de ta proposition, sous peine de se retrouver avec des urls arbo inconsistantes, ou de devoir recalculer d'un coup toutes les urls du site (exemple : renommage du secteur).

Sur le fond, je crois qu'avec un peu d'astuce on pourrait arriver à ton résultat en gardant la méthode actuelle.
Cédric

7

oui j'ai laissé l'option mais je l'ai pas testée, donc il faut la debug !
Cédric

8

Mais a contrario, si on renomme une rubrique, cela n'oblige pas à recalculer les url de tous les enfants, ce qui etait un défaut de ta proposition, sous peine de se retrouver avec des urls arbo inconsistantes, ou de devoir recalculer d'un coup toutes les urls du site (exemple : renommage du secteur).

Effectivement, je n'y avais pas pensé.

Mais c'est normalement géré proprement par les redirections 301 faites si on demande une ancienne URL. Je ne sais pas pourquoi tu as mis ce code très utile en commentaires.

Sur le fond, je crois qu'avec un peu d'astuce on pourrait arriver à ton résultat en gardant la méthode actuelle.

Je ne vois pas l'astuce sur le moment, mais peut-être en cherchant un peu... :wink:

-Nicolas

9

oui j'ai laissé l'option mais je l'ai pas testée, donc il faut la debug !

Ah, OK.