Vérifier ID et titre dans une url

Version imprimable

Bonjour,

Voila j'utilise ce code sur ma page news.php pour afficher mes news par leur ID (par exemple news.php?id=10). Je réécris mes url en injectant le titre de la news dedans ce qui me donne par exemple 10-titre_de_la_news.php.
Le soucis c'est que n'importe quel titre marche tant que l'ID est bon, 10-mauvais_titre_de_la_news.php marche tout aussi bien que le bon titre. Si j'ai à modifier le titre de la news, cela me créera donc un nouveau lien qui sera tout aussi valide que le précédent.
J'ai donc peur que cela me fasse du duplicate content chez Google puisqu'il visitera l'ancienne news indexée puis la nouvelle : 2 url, contenu identique = pas bon.

J'aimerai donc modifier mon script pour qu'il vérifie que le titre de la news dans la BDD corresponde bien à l'ID demandée et que si ce n'est pas le cas qu'il redirige vers la bonne adresse avec le titre actuel.

Code de la page news.php:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 $newsId= ( isset($_GET["id"])) ? $_GET['id'] : FALSE; if ( is_numeric($newsId)) { $sql = 'SELECT * FROM news WHERE newsId = ' . $newsId; $rc = mysql_query($sql); if ( $data = mysql_fetch_assoc($rc) ) { //afficher la news }else { //mauvaise id de news } }else { //ce n'est pas une id valide }
Code du fichier .htaccess:

Code:

RewriteRule ^([0-9]+)-([a-z0-9_-])*\.php$ news.php?id=$1 [QSA,L]

Merci par avance de votre aide, j'y ai passé un après-midi complet cette semaine et je n'ai pas réussi à le faire.

Guillaume

10/05/2008, 17h45
Invité

perso je te conseillerai de générertes pages : dans ton cms, tu a un bouton "generer" qui crée les pages html de tes articles, ça chargera beaucoup moin ta bd :) (a la place de l'url redirect)

Sinon

a tu essayé

RewriteRule ^([0-9]+)-([a-z0-9_-])*\.php$ news.php?id=$1&name=$2 [QSA,L]

et verifier l'id et le name dans ta requète ?

Je n'utilise pas de CMS sur mon site, c'est du fais à la "mano" ;)

Sinon j'ai essayé de vérifier le titre en plus de l'ID
Comme ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
$newsId= ( isset($_GET["id"])) ? $_GET['id'] : FALSE;
$titre= ( isset($_GET["titre"])) ? $_GET['titre'] : FALSE;
    if (  is_numeric($newsId) && is_string(filter($titre)) ) {
    $sql = 'SELECT * FROM news WHERE newsId = ' . $newsId . ' AND titre =  ' . $titre;   
    $rc = mysql_query($sql);
    if ( $data = mysql_fetch_assoc($rc) ) {
   //afficher la news
   }else
    {
   //mauvaise id de news
   }
    }else
    {
   //ce n'est pas une id valide
   }

Mais cela ne marche pas, la fonction filter sur le 'titre' sert à retirer les caractères spéciaux pour en faire une url propre

Code:

1
2
3
4
5
function filter($in) {
    $search = array ('@[B]@i','@[D]@i','@[F]@i','@[G]@i','@[H]@i','@[J]@i','@[K]@i','@[L]@i','@[M]@i','@[N]@i','@[P]@i','@[Q]@i','@[R]@i','@[S]@i','@[T]@i','@[V]@i','@[W]@i','@[X]@i','@[Y]@i','@[Z]@i','@[éèêëÊËE]@i','@[àâäÂÄA]@i','@[îïÎÏI]@i','@[ûùüÛÜU]@i','@[ôöÔÖO]@i','@[çC]@i','@[ ]@i','@[^a-zA-Z0-9_]@');
    $replace = array ('b','d','f','g','h','j','k','l','m','n','p','q','r','s','t','v','w','x','y','z','e','a','i','u','o','c','_','');
    return preg_replace($search, $replace, $in);
}

10/05/2008, 17h59
Invité

tu n'utiliser pas un cms, mais c'est un cms que tu fait : un logiciel de gestion de contenu :)

est ce que ton url redirect fonctionne bien ? cad est ce que $_GET['titre'] est renseigné ?
10/05/2008, 18h08
Dark Jukka

Vi, enfin je voulais dire que je n'utilise pas de CMS tout fait quoi ^^

Pour le moment je test sans le redirect sur une page d'essai et cela ne marche pas, l'ID n'est pas bonne (alors que je teste avec une existante).

Mon url est formatée comme cela news.php?id=33&titre=titre_de_la_news_correspondante
10/05/2008, 19h22
Invité

peut tu me donner le nouveau code de verif coté serveur stp ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
function filter($in) {
        $search = array ('@[b]@i','@[D]@i','@[F]@i','@[G]@i','@[H]@i','@[J]@i','@[K]@i','@[L]@i','@[M]@i','@[N]@i','@[P]@i','@[Q]@i','@[R]@i','@[S]@i','@[T]@i','@[V]@i','@[W]@i','@[X]@i','@[Y]@i','@[Z]@i','@[éèêëÊËE]@i','@[àâäÂÄA]@i','@[îïÎÏI]@i','@[ûùüÛÜU]@i','@[ôöÔÖO]@i','@[çC]@i','@[ ]@i','@[^a-zA-Z0-9_]@');
        $replace = array ('b','d','f','g','h','j','k','l','m','n','p','q','r','s','t','v','w','x','y','z','e','a','i','u','o','c','_','');
        return preg_replace($search, $replace, $in);
    }
 
$newsId= ( isset($_GET["id"])) ? $_GET['id'] : FALSE;
$titre= ( isset($_GET["titre"])) ? $_GET['titre'] : FALSE;
    if (  is_numeric($newsId) && is_string(filter($titre)) ) {
    $sql = 'SELECT * FROM news WHERE newsId = ' . $newsId . ' AND titre =  ' . $titre;   
    $rc = mysql_query($sql);
    if ( $data = mysql_fetch_assoc($rc) ) {
   //afficher la news
   }else
    {
   //mauvaise id de news
   }
    }else
    {
   //ce n'est pas une id valide
   }

11/05/2008, 21h11
Invité

c'est pas l'inverse de la fonction filter qu'il te faut faire ? la tu remplace les " " par des "_" mais ça c'est fait pour créer le lien pa pour le lire , non ?
11/05/2008, 21h17
Dark Jukka

Je m'en suis rendu compte dans l'après midi que je me plantais (avec l'aide d'un amis), j'ai donc réussi à faire ce que je voulais!
J'ai simplement ajouté un champ 'titre_url' dans ma table de news qui contient le titre de la news comme dans l'url demandée et ça roule maintenant.