rub a écrit:

PS: Bientôt, les modifs dans ce topic pour avoir la nouvelle icône dans les version 1.6.x

Pour avoir la nouvelle petite icône de notification sur les catégories parentes (fonctionne uniquement sur les menus).

Copier l'image
Dans le répetoire F:\Dev\branch-1_6\template\yoga\icon sous le nom recent_by_child.png

Dans le fichier functions_html.inc.php:
Chercher la fonction get_html_menu_category

    if ($category['nb_images'] > 0)
    {
      $menu.= "\n".'<span class="menuInfoCat"';
      $menu.= ' title="'.$category['nb_images'];
      $menu.= ' '.$lang['images_available'].'">';
      $menu.= '['.$category['nb_images'].']';
      $menu.= '</span>';
      $menu.= get_icon($category['date_last']);
    }

à remplacer par

    if ($category['nb_images'] > 0)
    {
      $menu.= "\n".'<span class="menuInfoCat"';
      $menu.= ' title="'.$category['nb_images'];
      $menu.= ' '.$lang['images_available'].'">';
      $menu.= '['.$category['nb_images'].']';
      $menu.= '</span>';
    }
    $menu.= get_icon($category['date_last'], $category['is_child_date_last']);

Dans le fichier functions_html.inc.php:
Chercher la fonction get_icon

function get_icon($date)
{
  ...
}

à remplacer par

function get_icon($date, $is_child_date = false)
{
  global $page, $user, $conf, $lang;

  if (empty($date))
  {
    $date = 'NULL';
  }

  if (isset($page['get_icon_cache'][$is_child_date][$date]))
  {
    return $page['get_icon_cache'][$is_child_date][$date];
  }

  if (!preg_match('/^(\d{4})-(\d{2})-(\d{2})/', $date, $matches))
  {
    // date can be empty, no icon to display
    $page['get_icon_cache'][$is_child_date][$date] = '';
    return $page['get_icon_cache'][$is_child_date][$date];
  }

  list($devnull, $year, $month, $day) = $matches;
  $unixtime = mktime( 0, 0, 0, $month, $day, $year );

  if ($unixtime === false  // PHP 5.1.0 and above
      or $unixtime === -1) // PHP prior to 5.1.0
  {
    $page['get_icon_cache'][$is_child_date][$date] = '';
    return $page['get_icon_cache'][$is_child_date][$date];
  }

  $diff = time() - $unixtime;
  $day_in_seconds = 24*60*60;
  $output = '';
  $title = $lang['recent_image'].' ';
  if ( $diff < $user['recent_period'] * $day_in_seconds )
  {
    $icon_url = get_themeconf('icon_dir').'/'.($is_child_date ? 'recent_by_child.png' : 'recent.png');
    $title .= $user['recent_period'];
    $title .=  '&nbsp;'.$lang['days'];
    $size = getimagesize( PHPWG_ROOT_PATH.$icon_url );
    $icon_url = get_root_url().$icon_url;
    $output = '<img title="'.$title.'" src="'.$icon_url.'" class="icon" style="border:0;';
    $output.= 'height:'.$size[1].'px;width:'.$size[0].'px" alt="(!)" />';
  }

  $page['get_icon_cache'][$is_child_date][$date] = $output;

  return $page['get_icon_cache'][$is_child_date][$date];
}

Dans le fichier functions_category.inc.php:
Chercher la fonction get_categories_menu

function get_categories_menu()
{
  global $page,$user;
...
}

à remplacer par ces 2 fonctions

function compute_branch_cat_date_last(&$cats, &$list_cat_id, &$level, &$ref_level)
{
  $date = '';
  do
  {
    $cat_id = array_pop($list_cat_id);
    if (!is_null($cat_id))
    {
      if ((empty($cats[$cat_id]['date_last'])) or ($cats[$cat_id]['date_last'] < $date))
      {
        $cats[$cat_id]['date_last'] = $date;
        $cats[$cat_id]['is_child_date_last'] = true;
      }
      else
      {
        $date = $cats[$cat_id]['date_last'];
      }
      $ref_level = substr_count($cats[$cat_id]['global_rank'], '.') + 1;
    }
    else
    {
      $ref_level = 0;
    }
  } while ($level <= $ref_level);

  // Last cat updating must be added to list for next branch
  if ($ref_level <> 0)
  {
    array_push($list_cat_id, $cat_id);
  }
}

function get_categories_menu()
{
  global $page,$user;

  $infos = array('');

  $query = '
SELECT name,id,date_last,nb_images,global_rank,id_uppercat
  FROM '.CATEGORIES_TABLE.'
  WHERE 1 = 1'; // stupid but permit using AND after it !
  /*if (!$user['expand'])
  {
    $query.= '
    AND (id_uppercat is NULL';
    if (isset($page['category']))
    {
      $query.= ' OR id_uppercat IN ('.$page['uppercats'].')';
    }
    $query.= ')';
  }*/
  if ($user['forbidden_categories'] != '')
  {
    $query.= '
    AND id NOT IN ('.$user['forbidden_categories'].')';
  }
  $query.= '
;';

  $result = pwg_query($query);
  $cats = array();
  while ($row = mysql_fetch_array($result))
  {
    $cats += array($row['id'] => $row);
  }
  usort($cats, 'global_rank_compare');

  $ref_level = 0;
  $level = 0;
  $list_cat_id = array();

  foreach ($cats as $id => $category)
  {
    // Update field
    $cats[$id]['is_child_date_last'] = false;

    $level = substr_count($category['global_rank'], '.') + 1;
    if ($level > $ref_level)
    {
      array_push($list_cat_id, $id);
    }
    else
    {
      compute_branch_cat_date_last($cats, $list_cat_id, $level, $ref_level);
      array_push($list_cat_id, $id);
    }
    $ref_level = $level;
  }

  $level = 1;
  compute_branch_cat_date_last($cats, $list_cat_id, $level, $ref_level);

  if (!$user['expand'])
  {
    $no_expand_cats = array();
    $list_id_uppercat = array();
    if (isset($page['category']))
    {
      $list_id_uppercat = explode(',', $page['uppercats']);
    }
    foreach ($cats as $id => $category)
    {
      if (empty($category['id_uppercat']) or in_array($category['id_uppercat'], $list_id_uppercat))
      {
        $no_expand_cats += array($category['id'] => $category);
      }
    }
    $cats = $no_expand_cats;
  }

  return get_html_menu_category($cats);
}

Salut

et merci car effectivement c etait ca il fallait que je fasse un tour dans l admin !

nickel

en fait au depart j avais pas vu que c etait du dev
et vu que je faisais partie des testeurs j ai voulu continué  ;o)

a+

au fait c est nickel

Oui le pire c est que j ai un envirronement de test !!! c est ca le pire  ;o)

et oui si on a besoin moi je veux bien tester la bsf en cours pas de soucis !

et sinon bahhh nickel l icone et le fonctionnement

a+

rub a écrit:

VDigital a écrit:

Je suis surpris de la présence du user_id dans la table et que cette table est unique, c'est tout.

Mais ca ne veut pas dire que c'est calculer pour tous à chaque fois d'un coup.
C'est calculé, si nécessaire, en même temps que le  forbidden_categories pour un utilisateur en particulier.

Unicité de la table? c'est à dire?

VDigital a écrit:

Si tu crées un table incluant le user_id, cela signifie pour moi que la last_date est recalculée pour chaque catégorie et pour chaque user.
Obligatoirement.

Oui mais pas en même temps, seulement selon les besoins.

VDigital a écrit:

Une grande galerie comme celle de v_images qui contient 1900 catégories et 360 utilisateurs, même en supposant que seuls 36 utilisateurs sont actifs et que ceux-ci ne voient guère plus de la moitié des catégories, tu calcules tout de même 36*950 last_dates.

si on faisait tout en même temps comme dans les versions précédentes à ce qu'il parait (cf. Pierrick).
Mais la, ce n'est pas le cas.

J'ai fait mes tests avec 439 catégories (le nombre d'utilisateurs n'influe pas sur les performances) est les temps de réponses sont correctes.

VDigital a écrit:

Je te proposais dans mes explications de ne pas retenir le user_id pour n'avoir n'avoir que 1900 lignes ou moins.
Les catégories parentes ou vides n'ayant pas de ligne dans la table ou une last_date à NULL.
Je crois que c'est plus léger et que cela fonctionnait quand même.

Cela impliquait au moins une petite modif à l'issue de la synchro (que je ne vois pas).
Ton bel algo s'appuyant sur cette table pour calculer les dates de catégories parentes en fonction des droits du user.

On n'est pas parti sur le même algo, ca c'est sur... ;-)
Explique un peu ton algo et ta modification de la synchro?

VDigital a écrit:

Je n'ai pas cherché encore à comprendre ce que tu as fait (il faudra que je le fasse pour pouvoir comparer).

C'est la que la bat blesse car...

VDigital a écrit:

Cependant, je reste persuadé que je me suis bien mal exprimé pour n'avoir pas su faire comprendre que les dates devaient être calculées en deux temps.
Un premier pour les catégories quelques soient les droits.
Puis un second temps, à partir du résultat précédent, en fonction des droits de l'utilisateur en ligne.

... car c'est bien ce qui a été fait....
Il y a bien 2  calculs, les mêmes que ceux tu te décris.

VDigital a écrit:

Mais cela n'a aucune importance au final, puisque cela marche.
Quand j'aurai un moment, je reviendrai expliquer le processus (qui ne remet pas en cause l'algo).

Si, c'est important d'en discuter car au final, on a bien la même idée en tête, 2 calculs distincts.
C'est simplement la mise en place qui est différente, d'ou l'incompréhension.

PS: Bientôt, les modifs dans ce topic pour avoir la nouvelle icône dans les version 1.6.x

Un petit up pour VDigital, car j'essaie de comprendre ce que je n'ai pas compris et je n'y arrive pas...

Et je ne vois pas non plus le truc de la synchro...

Ca me turlupine à donf...

Sinon, et c'est peut-être ce que tu as voulu dire, mon algo se base sur le champ forbidden_categories et la nouvelle table se met à jour à chaque mise de forbidden_categories.
Le champ forbidden_categories est mis à jour à chaque qu'on va dans l'admin (par le champ need_update à true).
Une optimisation serait de mettre à jour forbidden_categories uniquement si nécessaire (synchro, ajout d'images, etc...)
Après bien sur, on peut optimiser de plus en plus pour vraiment le faire si nécessaire mais ca va devenir de plus en plus compliquer.

Bref, je suis rester sur le même principe, l'optimisation du cache est un autre problème, disons une autre discussion.

rub a écrit:

Dernier point, quand on clique sur les catégories récentes, l'icône n'est pas présente (alors qu'elle l'est pour les images récentes). Pour être uniformes entre les images et les catégories, je supprimerais bien ce comportement qui n'apporte pas grand chose à mon gout.

Un petit up!
Sans avis, contraire, je le ferais prochainement en BSF.

Rub, j'ai regarde ton code et je me suis pose la question sur l'interet de #user_cache_categories.is_child_date_last dans la base de donnees.
Car on peut le deduire tres facilement a partir de #user_cache_categories.max_date_last et #categories.date_last sans aucune requete (un simple test). Ca simplifierait un peu le code et ca vire une colonne non necessaire de la table.

Aussi il y a moyen d'optimiser un peu get_icon pour ne pas appeler getimagesize chaque fois que necessaire:
- on sauve la premiere fois que c'est necessaire $page['get_icon_cache']['recent']='<img....' et $page['get_icon_cache']['recent_by_child']=...
- ensuite on utilise directement une de ces 2 valeurs

PS: Si t'es d'accord pour la virer, mais t'as pas envie de le faire, alors je le ferai

Dernière modification par rvelices (2006-12-07 06:44:32)

Voila c'est fait.
Mais j'ai un petit souci avec les categories virtuelles:
Exemple: j'ai la categorie virtuelle Gens qui contient rub et rvelices. J'ai une photo de rub et rvelices qui est dans les 2 categories filles.
Maintenant on me dit que Gens a 2 photos, alors que j'ai une seule photo dans toute la galerie ...

C'est voulu comme ca?

rub a écrit:

rvelices a écrit:

Voila c'est fait.

A priori les icones sont inversées

Pas le temps de faire le post que hop, c'est corrigé...
It's the Radu's touch ;-)

[HS]
Bon, il est tard, dodo...
[/HS]