up timide ?

repie38 a écrit:

quelqu'un est-il motiver pour tester la version svn avec la nouvelle option de conf ?
et surtout, un relecteur de code peut-il jeter un oeil sur http://piwigo.org/dev/browser/extension … nf.inc.php ?
j'ai peur que ce soit un peu "freestyle" (de charger la conf a chaque affichage pour savoir quelle icones utiliser ...)

Non, rien de freestyle la dedant!

Par contre, perso, je ferai:

$conf['BlancmontXL'] = unserialize($conf['BlancmontXL']);

Déjà, pas besoin de global $conf (c'est déjà fait dans la méthode $template->load_themeconf)
Ensuite, pas besoin de re-unserializer (oui monsieur, ca vient de sortir) dans ta fonction set_bmxl_header.

Pour résumer:

$conf['BlancmontXL'] = unserialize($conf['BlancmontXL']);

$themeconf = array(
  'parent'      => 'default',
  'local_head'  => 'local_head.tpl',
  'icon_dir'    => $conf['BlancmontXL']['ico'] ? 'themes/montblancxl/icon' : 'themes/blancmontxl/icon',
);

Mieux encore, si ton paramètre $conf['BlancmontXL']['ico'] prend 'montblancxl' ou 'blancmontxl' comme valeur, y'a plus qu'à faire:

'icon_dir' => 'themes/'. $conf['BlancmontXL']['ico'] . '/icon'

Bon, ok, c'est du chipotage...

PS: je n'ai pas testé, mais ca m'a l'air de ressembler à MontblancXL, il n'y a pas de raison que ca ne fonctionne pas ;-)

PS2: perso, je mettrais le pack d'icones de MontblancXL dans ton thème... ca fait éventuellement doublon, mais ca mange pas de pain, et surtout ca évite de devoir installer un autre thème pour en profiter...

voili voilou : [extension by repie38] BlancMont XL
v2.1.c
nouvelle option dans le panneau de config : vous pouvez utiliser le jeu d'icones du theme MontBlanc XL (theme non necessaire)

P@t a écrit:

Beaucoup mieux mais...

global $config;

T'es sur que dans le code de piwigo il n'y a jamais de variable $config? Je pense que non, mais bon, à l'avenir, sait-on jamais.
Sans parler des plugins qui risquent d'écraser ta variable... pas top.
C'est aussi pour ca que je serais resté sur $conf['BlancmontXL'].

exact, je m'etais embrouillé tout seul sur ce que tu m'avais dit ...
j'avais cru que quand tu disais que global $conf n'était pas necessaire, tu parlais aussi de l'appel que j'avais fais dans la fonction set_bmxl_header()

comme quoi 2 paires d'yeux c'est bien, mais coder un plugin a 3h du mat pas trop :D

je commit de suite, mais avant de livrer une nouvelle version ... a ce que j'ai vu des nouvelles fonctionnalitées de piwigo, apparement, les mises a jours auto des themes devraient arriver bientot, non  ;-)   ( <-appel du coude a peine dissimuler )  ?

Dernière modification par repie38 (2010-05-26 16:02:46)

Bonjour,

Je viens juste d'installer ce thème et j'obtiens l'erreur suivante :

Notice: Undefined index: BlancmontXL in /web/emiliemultimedi/www/gallerie/template/yoga/theme/blancmontxl/themeconf.inc.php on line 12
local_head.tpl

J'ai effacé le dossier Blancmontxl et je l'ai rechargé sur le serveur, mais j'ai toujours la même erreur.
Je suis débutante et je ne sais pas du tout ce que cette erreur veut dire. Merci pour votre aide.

Murielle

P@t a écrit:

plg devrait s'occuper de ca prochainement ;-)

[Bugtracker] ticket 1713

Murielle a écrit:

Murielle a écrit:

Bonjour,

Je viens juste d'installer ce thème et j'obtiens l'erreur suivante :

Notice: Undefined index: BlancmontXL in /web/emiliemultimedi/www/gallerie/template/yoga/theme/blancmontxl/themeconf.inc.php on line 12
local_head.tpl

J'ai effacé le dossier Blancmontxl et je l'ai rechargé sur le serveur, mais j'ai toujours la même erreur.
Je suis débutante et je ne sais pas du tout ce que cette erreur veut dire. Merci pour votre aide.

Murielle

Il s'agit de la version 2.0.8 de Piwigo et 2.1.c de Blancmontxl.

Pour Piwigo 2.0.8 il faut utiliser la version 2.0 du thème