je ne vais pas pouvoir regarder tout de suite car direction les piste mais merci pour ton boulot c est sympa de t etre lancé sur ce plugin ;-]

a+ et bon courage pour tes amelioration ...

avec ddtddt et p@t tu es entre de bonnes mains pour nous faire un bon petit plugin !!!

Voici les quelques erreurs à corriger...

Dans le fichier main.inc.php:
Compatibilité avec les plugins UpToD@te et Plugins Manager:

* Plugin URI: http://phpwebgallery.net/ext/extension_view.php?eid=204

Enlever les notice si $page['category'] n'existe pas:
Remplacer

if($page['category']['id'] != "")

Par

if(!empty($page['category']['id']))

Dans le fichier piclens-rss.php, je verrais plutot quelque chose du genre:

if (count($selection) > 0)
{
  $query = '
SELECT *
  FROM '.IMAGES_TABLE.'
  WHERE id IN ('.implode(',', $selection).')
  '.$conf['order_by'].'
;';
  $result = pwg_query($query);
  while ($row = mysql_fetch_assoc($result))
  {
    array_push($pictures, $row);
  }
}

Avant, $page['rank_of'] n'existe pas, donc cela provoque des notices...
Et si celles-ci sont activées, le fichier RSS n'est pas valide.

Dernière modification par P@t (2008-02-01 15:09:30)

Bon, pour ajouter une icone dans la barre d'icone, c'est très simple, dans le fichier main.inc.php:

add_event_handler('loc_end_index', 'add_piclens_icon');
function add_piclens_icon()
{
   global $template;

   $tpl_code = '<li>Code HTML à insérer</li>';

   $template->concat_var('PLUGIN_INDEX_ACTIONS', $tpl_code);
}

Dernière modification par P@t (2008-02-01 16:55:23)

Encore une petite astuce...

Pour que le petit bouton "play" s'affiche sur chaque vignette, il faut ajouter dans le main.inc.php:

add_event_handler('loc_begin_index', 'set_make_full_url_piclens');
function set_make_full_url_piclens()
{
    set_make_full_url();
}

Cool ;-)

Mais encore quelques détails...

Il faut supprimer la ligne:

usort($pictures, 'rank_compare');

Cette ligne est incorrecte et provoque des notices...
Du coup, ca ne fonctionne pas (si les notices sont activées)

En ce qui concerne l'icone (dans la barre d'icones), je pense qu'il faudrait proposer de pouvoir l'activer ou non...
Est-elle vraiment utile?
Tout le monde n'a pas piclens installé... Mais bon, c'est mon avis perso ;-)
Pourquoi ne pas mettre cette icone à coté de l'icone diaporama sur les pages des images?

Pour finir, tu devrais créer un fil dans le forum extensions, pour présenter le plugin, et permettre aux utilisateurs de poser des questions.

Et je terminerai quand meme par un... grand bravo pour ce plugin très sympatique ;-)

Dernière modification par P@t (2008-02-02 00:32:06)

A priori, il n'est pas nécessaire d'installer piclens pour que ca fonctionne sous FF ou IE.
Chez moi, ca fonctionne sans rien installer (surement car ca appelle directement le js de piclens).

Par contre, il faudrait n'afficher l'icône qu'à l'endroit où le diaporama fonctionne (c'est à dire les catégories avec des photos)

tipytch a écrit:

Car chez moi ça fait "planter" le diaporama.

Chez chez toi, ça plante? Chez les autres aussi, je te rassure.
Sincèrement on préfèrerait que ça ne plante que chez toi.

8-)