Je te suis à 200% sauf sur le dernier point.
En base? Pourquoi?
8-)

P@t a écrit:

VDigital a écrit:

Je te suis à 200% sauf sur le dernier point.
En base? Pourquoi?
8-)

Parce que dans la fonction set_filenames, je n'aime pas le $localtpl = './template-extension/' . $file
Cela oblige de metttre un '../' devant un template extension qui ne se trouve pas dans le dossier template-common.
Tu me diras, la fonction set_template_extent pour se charger de ca...

En base, ou en $conf, ou en paramètre de la fonction... (... , $located='./template-extension/' )
=> $localtpl = $located . $file

8-)

Bon, je crois que j'ai bien arrangé tout ca... [Subversion] r2643
Dis-moi ce que tu en penses vincent.

Un petit exemple qui va me servir dans extended description:
Je veux remplacer les tpl pour categories et config, je fais:

$array = array(
  'my_cat_modify.tpl' => 'categories',
  'my_configuration.tpl' => 'config');

$template->set_extents($array, EXTENDED_DESC_PATH . '/template/');

Dernière modification par P@t (2008-10-03 12:04:43)

Certes, les plugins devront utiliser les templates extension avec prudence.
J'ai prévu de toute facon ce cas (avec le paramètre $overwrite)
Un plugin pourra ainsi prévoir d'éccraser ou non un tpl extension déjà présent.

Ce que j'ai commité est de toute facon un peu plus propre que ce qu'il y avait avant.
;-))

Dernière modification par P@t (2008-10-03 15:04:17)

VDigital a écrit:

C'est parfait et je peux le dire P@t, tu codes bien mieux que moi.
Et cela me fait très plaisir sincèrement.
8-)

N'éxagerons rien quand meme!
J'ai eu seulement deux professeurs... rub et toi ;-)

D'ailleurs, je suis en train de migrer Wired For Sound (un de mes tout premier plugin que je n'avais pas retouché depuis bien longtemps), et je constate le chemin parcouru en un peu plus d'un an! Les choses étaient quand meme beaucoup moins claires pour moi à l'époque. Mais elles se sont rapidement éclairées grace à vos conseils, et je ne vous remercierai jamais assez ;-)

Dernière modification par P@t (2008-10-04 01:34:12)

Tu as très bien vu le point.
Et maintenant il ne faut plus faire que des réparations de ce qui cloche dans Piwigo.
On arrête le neuf afin de pouvoir sortir la version 2.0
D'accord?
8-)

P@t a écrit:

Il serait (à  mon avis) necessaire de créer une colonne "template" pour les templates extensions...
Je m'explique... un templatte extensions ppeut très bien fonctionner avec le template yoga, mais ne sera pas bon pour le template gally.
Il serait donc bon de pouvoir (éventuellement mais pas obligatoirement) sélectionner un template pour un template extension donné...

oui oui je suis tout à fait d'accord !
Faire en fait des cases à cocher pour choisir sur quel template appliquer

Ou faire une liste d'exclusion avec template, et thème si possible aussi car certains thèmes peuvent ne pas s'y prêter aussi

P@t a écrit:

A priori, un template extension sera compatible avec tous les templates ou un seul...
Je ne pense pas qu'il puisse être compatible avec 2 templates mais pas un troisième... et si tel était le cas, il faudra faire un fichier pour chaque template.

A posteriori c'est faux ^^

Et pourquoi faire une règle restrictive au lieu d'une règle un poil plus souple . Concernant les thèmes je pensais aux thèmes horizontaux et à largeur fixe, qui sont particuliers, mais vu le nombre de thèmes je pense qu'une restrictions sur les thèmes risque d'être tendu. A moins de faire une liste à rajout : une liste totale en bas contenant tout les thèmes/template et un bouton cochable à coté de chaque template extension pour savoir où rajouter les exclusions.