P@t a écrit:
Autre solution (plus simple à mettre en place), ajouter l'image "en trop" dans le $page['items'].
On aura ainsi 16 images au lieu de 15 (ou $conf['top_number'] + 1), avec l'image en trop à la fin.
Moi j'aime bien cette solution plutot que l'url...
Hors ligne
rvelices a écrit:
P@t a écrit:
Autre solution (plus simple à mettre en place), ajouter l'image "en trop" dans le $page['items'].
On aura ainsi 16 images au lieu de 15 (ou $conf['top_number'] + 1), avec l'image en trop à la fin.Moi j'aime bien cette solution plutot que l'url...
Alors c'est vendu!
Hors ligne
Merci P@t.
8-)
Hors ligne
Bon, en fait, il fallait quand meme vérifier les droits de l'utilisateur pour l'image en plus...
Voila donc la modif que je propose.
Dites-moi quand meme si vous trouvez ca correct avant que je commit:
// if this image_id doesn't correspond to this category, an error message is
// displayed, and execution is stopped
if (!in_array($page['image_id'], $page['items']))
{
if ($page['section'] == 'categories')
{
page_not_found(
'The requested image does not belong to this image set',
duplicate_index_url()
);
}
else
{
// If image doesn't belong anymore to a special section
// we check access authorization and add it to $page['items']
$query ='
SELECT id
FROM '.IMAGES_TABLE.'
INNER JOIN '.IMAGE_CATEGORY_TABLE.' AS ic ON id = ic.image_id
WHERE id = '.$page['image_id'].'
'.$forbidden.'
LIMIT 1
;';
$result = pwg_query($query);
if (mysql_fetch_assoc($result))
{
array_push($page['items'], $page['image_id']);
}
else
{
access_denied();
}
}
}
En gras ce que j'ai ajouté...
Hors ligne
P@t a écrit:
Voila donc la modif que je propose.
2 points:
- ne pas utiliser $forbidden - mais le regenerer (c'est une variable globale effectivement, mais on ne devrait pas la voir ailleurs que section_init)
- j'aurais prefere un test exclusiv sur page['section']=='best_rated' (dans ton exemple si 'section'=='tags' tu me rajoute une image ...)
Dernière modification par rvelices (2008-07-10 18:05:01)
Hors ligne
rvelices a écrit:
P@t, en fait j'aimerais pousser la logique de ce truc un peu plus loin (most_visted) avec redirection ... Ca t'embete si je prends le dev (sinon je peux toujours modifier apres:-))?
Non non, pas du tout!
Je te le laisse ;-)
Hors ligne