Eviter duplication de contenu - Récupérer ID des sujets dans index.php

vin100 · 30-12-2006 00:50:32

c'est ce que je disait il faudrait enregistrer last_topic_id en plus de ce last_post_id mais cela engage la structure de la base de données ainsi que plusieurs fonctions php

bertimus · 30-12-2006 03:50:16

Y'a vraiment pas plus simple ?
Parce que très concrètement, sur ce sujet, on arrive à 2 pages.
Sur l'index sont ainsi apparus les chiffres 1 et 2 pointant respectivement vers viewtopic.php?id=4046&p=1 et viewtopic.php?id=4046&p=2. Puisque ces liens sont apparus c'est donc qu'il y a bien un moyen de récupérer sur l'index le tid et le N° de page non ?
L'enjeu est donc, arrêtez moi si je me trompe :
1) de savoir où sont piochées ces 2 id ("id=" et "p=")
2) de les coupler à l'URL présente dans la colonne "Dernier message" viewtopic.php?pid=32278#p32278 pour ainsi créer une url tip top pour google : viewtopic.php?id=4046&p=2#p32278

Dernière modification par bertimus (30-12-2006 05:43:46)

Ju · 30-12-2006 04:34:43

Un problème plein de bonnes intentions : j'adhére.

Commence par virer soigneusement toutes tes modifications bertimus : on va travailler proprement en utilisant le buffer. C'est un poil plus lourd niveau ressources (de l'ordre de la micro-seconde happy ) mais ça a l'avantage de garder le code de PunBB à l'état d'origine pour les update ou autres, d'appliquer la mod sur toutes les pages du forum d'un coup, et d'être libre de désactiver/supprimer la mod en 2 secondes.

Le résultat : CtrlAltSuppr.com (avec mon UR)

La solution :
MAJ : http://www.punbb.fr/mods/mod.php?id=71

J'ai tenté autant que faire ce peut d'aller au plus court et économique niveau requêtes et autres, tout en gardant un code dans l'esprit de PunBB.

Tu peux noter que la toute dernière ligne réalise une autre modification qui va également dans le sens de ton problème : ça vire les adresses de premières pages en double (qui précisent ou non que c'est bien la première page qu'on souhaite).

Enfin, pour ton URL Rewriting, soit il se réalise également par une fonction placée dans le footer (c'est mon cas), auquel cas tu ne modifies pas mon code mais tu le places avant cette fonction, soit tu as effectué les modifications directement sur les fichiers, auquel cas je te laisse adapter mon code à ton grès (première ligne et avant-dernière ligne, en sachant que ([0-9]+) correspond précisemment au pid).

Je serais tenté de conseiller cette mod à tout le monde.

Dernière modification par Ju (24-04-2007 21:57:03)

bertimus · 30-12-2006 05:26:54

Zorro Ju est arrivé wé é, sans se presser wé é, le bon Zorro Ju U, le vrai Zorro Ju U, avec son mod et LA solution :P

Bon je regarde tout ça demain car il se fait tard (tu vois je commence même à partir en vrille là)

Mais bon, ça m'a l'air nickel... Merci M'sieur ;)

[J'ai effectivement rewrité chacune des urls directement dans les fichiers tout simplement car je n'ai pas les compétences pour créer une fonction dédiée dans le footer...]

Et je partage ton avis sur le fait que tout le monde devrait appliquer ces modifications...

PS : ton forum est superbe, bien joué :)

PS 2 : Un jour faudra aussi que tu nous expliques comment t'as réussi à mettre une meta description statique sur la page d'accueil de ton forum tout en générant parallèlement des meta description dynamiques sur les discussions. Moi j'ai juste réussi à mettre en place les meta description dynamiques en suivant ce post... Bref, bref, ceci n'est qu'un détail face au bond que tu vas me/nous permettre de faire avec ton intervention précédente (quoique beaucoup n'en ont pas encore conscience :P)...

Dernière modification par bertimus (30-12-2006 05:46:32)

Pandark · 30-12-2006 05:44:52

Ca a l'air d'être une bonne mod...
Mais on pourrait quand même faire pars du problème de duplication des adresses à la team de punBB.org pour la version 1.3 (qui va bien arriver un jour tongue ) histoire que ce soit géré de base (parce qu'autant on a entendu parlé d'url rewriting, autant ça... pas vraiment).
J'vais ajouter la mod de Ju au wiki... (à moins qu'il ne préfère la soumettre dans la section mods.. l'un n'empêchant pas l'autre)

http://www.punbb.fr/aide/doku.php/astuc … ation_urls

vin100 · 30-12-2006 10:24:24

excellent ju, je n'aurais jamais pensé à prendre le problème par ce bout :canon:

ouaip une discussion sur le .org serait pas superflu...

bon aller, à l'année prochaine ! salut !

romain9441 · 30-12-2006 11:00:09

Pandark a écrit :

http://www.punbb.fr/aide/doku.php/astuc … ation_urls

J'ai corrigé une petite faute de frape. wink

vin100 a écrit :

bon aller, à l'année prochaine ! salut !

Pourquoi l'année prochaine ? blink happy

Sinon Ju c'est super ! :canon:

Je vais essayer ça. wink

EDIT : Je l'ai installé et ça tourne pour le moment.

Dernière modification par romain9441 (30-12-2006 11:52:04)

romain9441 · 30-12-2006 11:43:22

J'ai trouvé le problème sur mon forum :

Pour la même page de discussion, j'ai 3 pages similaires dont l'URL est différente dans Google.

EDIT : Je l'ai installé et ça tourne pour le moment.

Dernière modification par romain9441 (30-12-2006 11:52:14)

romain9441 · 30-12-2006 11:58:45

Petite modifiaction de ma part pour les personnes possédant PunBB entièrement en français :

/* Début - Empêcher la duplication des adresses */

// On commence par récupérer les pid de tous les liens qui appellent un message précis
preg_match_all('#viewtopic\.php\?pid\=([0-9]+)#', $tpl_main, $pids);
$pids = array_unique($pids[1]);

if(!empty($pids)) {

    // On récupére ensuite les id des topics qui correspondent aux différents pid
    $result = $db->query('SELECT id, topic_id FROM '.$db->prefix.'posts WHERE id='.implode(' OR id=', $pids)) or error('Impossible de retrouver les informations du message', __FILE__, __LINE__, $db->error());
    if (!$db->num_rows($result))
        message($lang_common['Bad request']);

    while ($pid_id = $db->fetch_assoc($result))
    {
        $adresse[$pid_id['id']] = Array('topic_id' => $pid_id['topic_id']);

        if (isset($query))
            $query .= ' OR topic_id='.$pid_id['topic_id'];
        else
            $query = $pid_id['topic_id'];
    }

    // On récupére et on classe ensuite les pid de tous les posts de tous les topics concernés
    $result = $db->query('SELECT topic_id, id FROM '.$db->prefix.'posts WHERE topic_id='.$query.' ORDER BY posted') or error('Impossible de retrouver les informations du message', __FILE__, __LINE__, $db->error());
    while ($id_pid = $db->fetch_assoc($result))
        $posts[$id_pid['topic_id']][] = $id_pid['id'];

    // On recherche les pid des liens dans les pid des topics et on récupére la place qu'ils occupent pour calculer leur page
    foreach ($posts AS $id => $pids)
    {
        foreach(array_keys($adresse) AS $pid) {
            if(($position = array_search($pid, $pids)) !== false)
                $adresse[$pid]['p'] = ceil(($position + 1) / $pun_user['disp_posts']);
        }
    }
}

// On remplace enfin les adresses avec les pid par des adresses uniques et directes avec l'id du topic en précisant la page
foreach($adresse AS $pid => $url)
    $tpl_main = str_replace('viewtopic.php?pid='.$pid, 'viewtopic.php?id='.$url['topic_id'].'&p='.$url['p'], $tpl_main);

// On en profite pour virer tous les ?p=1 ou &p=1 qui doublonnent aussi inutilement les pages
$tpl_main = preg_replace('#(\?|&)p\=1#', NULL, $tpl_main);

/* Fin dupplication */

Les requètes sont en français. wink

romain9441 · 30-12-2006 12:18:20

Ju, avec ta modification, sur certaines pages comme là où il y a les règles, il y a une erreur :

Warning: Invalid argument supplied for foreach() in /mnt/103/sdb/c/a/boardfun/footer.php on line 211

Ju · 30-12-2006 13:34:59

romain9441 a écrit :

Ju, avec ta modification, sur certaines pages comme là où il y a les règles, il y a une erreur :
Warning: Invalid argument supplied for foreach() in /mnt/103/sdb/c/a/boardfun/footer.php on line 211

Effectivement, et c'est probablement le cas sur toutes les pages qui ne contiennent pas de liens avec des pid. Mon hébergeur étant extrémement permissif, je n'avais pas remarqué mon erreur.

/* Début - Empêcher la duplication des adresses */

// On commence par récupérer les pid de tous les liens qui appellent un message précis
$pattern = 'viewtopic.php?pid=<PID>'; // URL Rewriting ?
preg_match_all('#'.str_replace('\<PID\>', '([0-9]+)', preg_quote($pattern, '#')).'#', $tpl_main, $pids);
$pids = array_unique($pids[1]);

if(!empty($pids)) {

    // On récupére ensuite les id des topics qui correspondent aux différents pid
    $result = $db->query('SELECT id, topic_id FROM '.$db->prefix.'posts WHERE id='.implode(' OR id=', $pids)) or error('Unable to fetch post info', __FILE__, __LINE__, $db->error());
    if (!$db->num_rows($result))
        message($lang_common['Bad request']);

    while ($pid_id = $db->fetch_assoc($result))
    {
        $adresse[$pid_id['id']] = Array('topic_id' => $pid_id['topic_id']);

        if (isset($query))
            $query .= ' OR topic_id='.$pid_id['topic_id'];
        else
            $query = $pid_id['topic_id'];
    }

    // On récupére et on classe ensuite les pid de tous les posts de tous les topics concernés
    $result = $db->query('SELECT topic_id, id FROM '.$db->prefix.'posts WHERE topic_id='.$query.' ORDER BY posted') or error('Unable to fetch post info', __FILE__, __LINE__, $db->error());
    while ($id_pid = $db->fetch_assoc($result))
        $posts[$id_pid['topic_id']][] = $id_pid['id'];

    // On recherche les pid des liens dans les pid des topics et on récupére la place qu'ils occupent pour calculer leur page
    foreach ($posts AS $id => $pids)
    {
        foreach(array_keys($adresse) AS $pid) {
            if(($position = array_search($pid, $pids)) !== false)
                $adresse[$pid]['p'] = ceil(($position + 1) / $pun_user['disp_posts']);
        }
    }

    // On remplace enfin les adresses avec les pid par des adresses uniques et directes avec l'id du topic en précisant la page
    foreach($adresse AS $pid => $url)
        $tpl_main = str_replace(str_replace('<PID>', $pid, $pattern), 'viewtopic.php?id='.$url['topic_id'].'&p='.$url['p'], $tpl_main);
}

// On en profite pour virer tous les ?p=1 ou &p=1 qui doublonnent aussi inutilement les pages
$tpl_main = preg_replace('#(\?|&)p\=1#', NULL, $tpl_main);

/* Fin dupplication */

-> J'ai élargi le bloc du premier if à après le dernier foreach.
-> J'en ai profité pour créer une variable $pattern pour faciliter l'UR en dur.

@bertimus :
pour ton URL Rewriting modifie juste le pattern à souhait en mettant <PID> où tu veux dedans (message-<PID>.html dans ton cas).

Dernière modification par Ju (30-12-2006 13:56:36)

romain9441 · 30-12-2006 13:44:20

C'est nikel ! wink

Merci Ju !

J'ai corrigé dans la documentation et j'ai mis la version française :

/* Début - Empêcher la duplication des adresses */

// On commence par récupérer les pid de tous les liens qui appellent un message précis
$pattern = 'viewtopic.php?pid=<PID>'; // URL Rewriting ?
preg_match_all('#'.str_replace('\<PID\>', '([0-9]+)', preg_quote($pattern, '#')).'#', $tpl_main, $pids);
$pids = array_unique($pids[1]);

if(!empty($pids)) {

    // On récupére ensuite les id des topics qui correspondent aux différents pid
    $result = $db->query('SELECT id, topic_id FROM '.$db->prefix.'posts WHERE id='.implode(' OR id=', $pids)) or error('Impossible de retrouver les informations du message', __FILE__, __LINE__, $db->error());
    if (!$db->num_rows($result))
        message($lang_common['Bad request']);

    while ($pid_id = $db->fetch_assoc($result))
    {
        $adresse[$pid_id['id']] = Array('topic_id' => $pid_id['topic_id']);

        if (isset($query))
            $query .= ' OR topic_id='.$pid_id['topic_id'];
        else
            $query = $pid_id['topic_id'];
    }

    // On récupére et on classe ensuite les pid de tous les posts de tous les topics concernés
    $result = $db->query('SELECT topic_id, id FROM '.$db->prefix.'posts WHERE topic_id='.$query.' ORDER BY posted') or error('Impossible de retrouver les informations du message', __FILE__, __LINE__, $db->error());
    while ($id_pid = $db->fetch_assoc($result))
        $posts[$id_pid['topic_id']][] = $id_pid['id'];

    // On recherche les pid des liens dans les pid des topics et on récupére la place qu'ils occupent pour calculer leur page
    foreach ($posts AS $id => $pids)
    {
        foreach(array_keys($adresse) AS $pid) {
            if(($position = array_search($pid, $pids)) !== false)
                $adresse[$pid]['p'] = ceil(($position + 1) / $pun_user['disp_posts']);
        }
    }

    // On remplace enfin les adresses avec les pid par des adresses uniques et directes avec l'id du topic en précisant la page
    foreach($adresse AS $pid => $url)
        $tpl_main = str_replace(str_replace('<PID>', $pid, $pattern), 'viewtopic.php?id='.$url['topic_id'].'&p='.$url['p'], $tpl_main);
}

// On en profite pour virer tous les ?p=1 ou &p=1 qui doublonnent aussi inutilement les pages
$tpl_main = preg_replace('#(\?|&)p\=1#', NULL, $tpl_main);

/* Fin dupplication */

Ce serais sympa d'en faire une mod. wink

Dernière modification par Ju (30-12-2006 13:56:22)

Ju · 30-12-2006 13:51:21

bertimus a écrit :

PS 2 : Un jour faudra aussi que tu nous expliques comment t'as réussi à mettre une meta description statique sur la page d'accueil de ton forum tout en générant parallèlement des meta description dynamiques sur les discussions. Moi j'ai juste réussi à mettre en place les meta description dynamiques en suivant ce post...

J'ai juste rajouté :

<meta name="description" content="<?php echo $page_title ?>" />

Après :

<title><?php echo $page_title ?></title>

Dans le fichier header.php.
Et pour en mettre des statiques, suffit de les ajouter en dessous wink .

Ju · 30-12-2006 13:52:36

@Romain :
Oups tu es passé avant que j'ai édité (ajouté l'histoire du <PID>).
Tu peux mettre à jour ? (je mets à jour ton post pour éviter les confusions)
Merci.

EDIT: Tu traduis pas "Bad Request" ?

Dernière modification par Ju (30-12-2006 13:56:02)

bertimus · 30-12-2006 13:52:55

Je confirme c'est nickel, merci à tous wink

@ Ju : ben dans le dernier code de Romain9441, j'ai juste remplacé viewtopic.php?pid= par message- et viewtopic.php?id= par sujet- et ça m'a l'air bien, cf. ici... c'est bon non ?

Dernière modification par bertimus (30-12-2006 13:54:13)

Ju · 30-12-2006 14:34:53

Si les balézes en anglais pouvaient m'aider, me corriger avant que je propose sur PunRES :

##
## MOD TITLE :
## Prevent the doubloons of address in search engines
##
## Mod version : 1
## Works on PunBB : 1.2.* (and probably before)
## Release date : 30-12-06
## Author : Julien VAUBOURG (Ju) [CtrlAltSuppr.com]
##
## DESCRIPTION
##
## On a same topic :
## http://FORUMS/viewtopic.php?pid=XXX#pXXX
## http://FORUMS/viewtopic.php?pid=YYY#pYYY
## The addresses of post are considered by search engine as two
## different pages while they have the same contents.
## This mod prevents the duplication of the contents in engine.
##
## Affects DB : No
## Affected file : footer.php
##
## Note : Also work with URL Rewriting.
##
## DISCLAIMER : Please note that 'mods' are not officially supported by
## PunBB. Installation of this modification is done at your
## own risk. Backup your forum database and any and all
## applicable files before proceeding.
##

#
#---------[ 1. OPEN ]---------------------------------------------------
#
footer.php
#
#---------[ 2. FIND (line: 157) ]---------------------------------------------------
#
// Close the db connection (and free up any result data)
$db->close();
#
#---------[ 3. BEFORE, ADD ]---------------------------------------------------
#
/* Début - Empêcher la duplication des adresses */
// URL Rewriting ?
$pattern_in = 'viewtopic.php?pid=<PID>';
$pattern_out = 'viewtopic.php?id=<ID>&p=<P>';
// On commence par récupérer les pid de tous les liens qui appellent un message précis
preg_match_all('#'.str_replace('\<PID\>', '([0-9]+)', preg_quote($pattern_in, '#')).'#', $tpl_main, $pids);
$pids = array_unique($pids[1]);
if(!empty($pids)) {
// On récupére ensuite les id des topics qui correspondent aux différents pid
$result = $db->query('SELECT id, topic_id FROM '.$db->prefix.'posts WHERE id='.implode(' OR id=', $pids)) or error('Unable to fetch post info', __FILE__, __LINE__, $db->error());
if (!$db->num_rows($result))
message($lang_common['Bad request']);
while ($pid_id = $db->fetch_assoc($result))
{
$adresse[$pid_id['id']] = Array('topic_id' => $pid_id['topic_id']);
if (isset($query))
$query .= ' OR topic_id='.$pid_id['topic_id'];
else
$query = $pid_id['topic_id'];
}
// On récupére et on classe ensuite les pid de tous les posts de tous les topics concernés
$result = $db->query('SELECT topic_id, id FROM '.$db->prefix.'posts WHERE topic_id='.$query.' ORDER BY posted') or error('Unable to fetch post info', __FILE__, __LINE__, $db->error());
while ($id_pid = $db->fetch_assoc($result))
$posts[$id_pid['topic_id']][] = $id_pid['id'];
// On recherche les pid des liens dans les pid des topics et on récupére la place qu'ils occupent pour calculer leur page
foreach ($posts AS $id => $pids)
{
foreach(array_keys($adresse) AS $pid) {
if(($position = array_search($pid, $pids)) !== false)
$adresse[$pid]['p'] = ceil(($position + 1) / $pun_user['disp_posts']);
}
}
// On remplace enfin les adresses avec les pid par des adresses uniques et directes avec l'id du topic en précisant la page
foreach($adresse AS $pid => $url)
$tpl_main = str_replace(str_replace('<PID>', $pid, $pattern_in), htmlentities(str_replace(Array('<ID>', '<P>'), Array($url['topic_id'], $url['p']), $pattern_out)), $tpl_main);
}
// On en profite pour virer tous les ?p=1 ou &p=1 qui doublonnent aussi inutilement les pages
$tpl_main = preg_replace('#(\?|&)p\=1#', NULL, $tpl_main);
/* Fin dupplication */
#
#---------[ 4. SAVE/UPLOAD ]---------------------------------------------------
#

Merci smile

romain9441 · 30-12-2006 14:39:46

Ju a écrit :

EDIT: Tu traduis pas "Bad Request" ?

Il n'a pas besoin d'être traduit ? Non ?
Puisque il fait appel au fichier lang/*****/common.php.

Ju · 30-12-2006 16:24:36

La mod sur PunRES
Ouverture de débat sur le .org

Quant à mon anglais, ils feront avec.

romain9441 · 30-12-2006 16:32:54

Et tu ne la proposes pas ici ? roll

Thiery · 30-12-2006 17:33:27

Bonjour à tous,

Une question bête mais seuls les experts peuvent me répondre.

Est-ce Google et autres moteurs indexent les pages d'un forum si celui-ci est accessible aux seuls inscrits?

Bonne fin d'année à tous,

Thiery

Pandark · 30-12-2006 18:04:37

Pas besoin d'être expert, il suffit de réfléchir deux secondes : est-ce qu'un moteur peut indexer ce qu'il ne voit pas ? Non.

PS: C'est assez hors sujet, cette question...

Thiery · 30-12-2006 18:24:25

Merci pour ta réponse qui est claire.
Mais justement je ne crois pas qu'elle soit hors sujet. En effet, je trouve l'intérêt de ce mod très utile pour le référencement, et je pensais l'installer. Je le ferais le jour ou j'ouvrirai mon forum.

romain9441 · 30-12-2006 18:48:54

Cette mod ! tongue

Keulig · 30-12-2006 18:51:49

Pourquoi ne pas simplement mettre un rel="nofollow" dans le lien ? J'ai fait ça et pas de contenu dupliqué smile

Pandark · 30-12-2006 19:17:52

Keulig a écrit :

Pourquoi ne pas simplement mettre un rel="nofollow" dans le lien ? J'ai fait ça et pas de contenu dupliqué

Heu... tu veux dire pourquoi ne pas supprimer tous les liens internes ?... eh bien parce que c'est totalement absurde étant donné que le nombre de liens vers une page fait partie des critères qui déterminent l'importance de cette page.. et que si aucun lien ne pointe vers une page, elle risque fort de ne jamais être vue par le moteur de recherche...

Archives FluxBB.fr

#26 30-12-2006 00:50:32

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#27 30-12-2006 03:50:16

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#28 30-12-2006 04:34:43

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#29 30-12-2006 05:26:54

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#30 30-12-2006 05:44:52

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#31 30-12-2006 10:24:24

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#32 30-12-2006 11:00:09

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#33 30-12-2006 11:43:22

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#34 30-12-2006 11:58:45

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#35 30-12-2006 12:18:20

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#36 30-12-2006 13:34:59

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#37 30-12-2006 13:44:20

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#38 30-12-2006 13:51:21

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#39 30-12-2006 13:52:36

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#40 30-12-2006 13:52:55

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#41 30-12-2006 14:34:53

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#42 30-12-2006 14:39:46

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#43 30-12-2006 16:24:36

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#44 30-12-2006 16:32:54

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#45 30-12-2006 17:33:27

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#46 30-12-2006 18:04:37

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#47 30-12-2006 18:24:25

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#48 30-12-2006 18:48:54

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#49 30-12-2006 18:51:49

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

#50 30-12-2006 19:17:52

Re : Eviter duplication de contenu - Récupérer ID des sujets dans index.php

Pied de page des forums