Bug avec recently_viewed

Version stable courante
téléchargement osCSS 1.2.2 RC

Messagepar steph68 » Sam 01 Aoû 2009, 12:36

Bonjour,

Voici un bug que j'ai relevé en voulant faire fonctionner le box 'recently_viewed'.

Dans le fichier catalogproduct_info.php :

Code: Tout sélectionner
 if (!tep_session_is_registered('recently_viewed'))
      {
     tep_session_register('recently_viewed');
      $recently_viewed = $_GET['products_id'] . ';';
      //$recently_viewed = "";
      }
 $check_not_duplicate = $_GET['products_id'];
  $temp_recent = $recently_viewed; 
  if (ereg($check_not_duplicate, $temp_recent ) ) {
 $recently_viewed = ereg_replace($check_not_duplicate,"",$temp_recent);
  }
 $recently_viewed = $_GET['products_id'] . ';' . $recently_viewed ;

à remplacer par :

Code: Tout sélectionner
  $prod_id='#'.$_GET['products_id'].';';
 
  if (!tep_session_is_registered('recently_viewed'))
  {
    tep_session_register('recently_viewed');
   $recently_viewed = "";
  }
  else
    $recently_viewed = str_replace($prod_id,'',$recently_viewed);

  $recently_viewed = $prod_id . $recently_viewed;

si par exemple, le produit 134 était dans la chaine et que l'on visite par la suite le 34, ça évitera d'avoir un 1 qui traine dans la chaine

sinon à modifier aussi le box :

dans catalog/includes/boxes/recently_viewed.php :

Code: Tout sélectionner
if ((strlen($recent) >0) && ($counter < (MAX_ROWS*MAX_COLS)) && ($rows<MAX_ROWS)) {

Code: Tout sélectionner
if ((strlen($recent) >1) && ($counter < (MAX_ROWS*MAX_COLS)) && ($rows<MAX_ROWS)) {
  $recent=substr($recent,1);  // skip #

perso, j'ai plutot utilisé des break dans le foreach ... mais voilà sans trop faire de modif ...

@+
steph68
Membre actif
 
Messages: 17
Inscription: Sam 01 Aoû 2009, 12:25

Messagepar oscim » Lun 03 Aoû 2009, 11:14

Je fais le liens sur le tracker ici aussi:

http://dev.oscss.org/index.php?do=details&task_id=153
Pour un web au normes du W3C, pour un web plus propre !

- Download osCSS Toutes versions
- stable osCSS 2.1.0 (help|info|new dev) Official help|wiki|Tuto for osCSS 2 | doc doxygen
- Utiliser le tracker pour suivre les évolutions et développements
- Inscrivez vos sites sur oscss pour présenter vos réalisations
Avatar de l’utilisateur
oscim
Site Admin
 
Messages: 1643
Inscription: Mar 25 Sep 2007, 11:33

Messagepar j4k » Ven 13 Nov 2009, 19:16

corrigé et commité
~ osCSS 1.2.1 ~
Avatar de l’utilisateur
j4k
Site Admin
 
Messages: 1259
Inscription: Mer 27 Avr 2005, 09:13
Localisation: France

Messagepar oscim » Sam 14 Nov 2009, 00:18

Cool
Pour un web au normes du W3C, pour un web plus propre !

- Download osCSS Toutes versions
- stable osCSS 2.1.0 (help|info|new dev) Official help|wiki|Tuto for osCSS 2 | doc doxygen
- Utiliser le tracker pour suivre les évolutions et développements
- Inscrivez vos sites sur oscss pour présenter vos réalisations
Avatar de l’utilisateur
oscim
Site Admin
 
Messages: 1643
Inscription: Mar 25 Sep 2007, 11:33


Retourner vers osCSS 1.2.2 RC

Qui est en ligne

Utilisateurs parcourant ce forum: Aucun utilisateur enregistré et 1 invité

cron