Annonce

#151 2008-03-08 12:39:32

rvelices
Équipe Piwigo
2005-12-29
1417

Re: Evolution des templates

P@t a écrit:

rvelices a écrit:

P@t,
Peux-tu t'assurer que sur ton environement de dev, t'as bien dans php.ini
error_reporting  =  E_ALL
? Tu verras qu'il y a qq warnings sur user_list

Oui oui, je suis bien sur E_ALL...
Quels warnings?
EDIT: je viens de tout re-tester à fond le user_list, et je n'ai aucun message d'erreur...

Hmmm ... Plus de warning maintenant ... C'est vrai que j'ai biduouille un peu plusieurs installs ... Probablement Smarty n'a pas recompile le template, croyant qu'il n'a pas besoin ...

Hors ligne

#152 2008-03-08 12:47:53

rvelices
Équipe Piwigo
2005-12-29
1417

Re: Evolution des templates

VDigital a écrit:

Juste histoire de me faire la main...
Je livre l'adaptation smarty d'Admin Advices.
8-)

PS: svn 2266 (Petit coup d'oeil de rvelices) - Merci.

C'est impec :-)

Hors ligne

#153 2008-03-08 12:49:42

P@t
Ex Equipe Piwigo
Nice
2007-06-13
5695

Re: Evolution des templates

D'ailleur, petite question...
J'ai vu que sur certains fichier (index.tpl par exemple), tu as mis:

{$ROOT_URL}{$themeconf.icon_dir}

Alors que sur d'autres, tu n'as pas mis le {$ROOT_URL} (le fichier cat_move.tpl par exemple)

Est-ce que le $ROOT_URL est assigné automatiquement, ou faut-il l'assigner manuellement dans chaque fichier php?
Est-ce qu'il est vraiment necessaire?


P@t

Hors ligne

#154 2008-03-08 13:24:11

P@t
Ex Equipe Piwigo
Nice
2007-06-13
5695

Re: Evolution des templates

VDigital a écrit:

Juste histoire de me faire la main...
Je livre l'adaptation smarty d'Admin Advices.
8-)

PS: svn 2266 (Petit coup d'oeil de rvelices) - Merci.

On peut faire beaucoup plus simple grace à smarty ;-)))
Dans le php:

$row['U_MODIFY'] = get_root_url().'admin.php?page=picture_modify'
            .'&image_id='.$row['id'];
$query = '
SELECT * FROM '.IMAGE_TAG_TABLE.'
WHERE image_id =  ' . $row['id'] .'
;';
$row['NUM_TAGS'] = mysql_num_rows(mysql_query($query));

$template->assign('thumbnail', $row);

Dans le tpl:

{if isset($thumbnail)}
    <a href="{$thumbnail.U_MODIFY}" title="{'link_info_image'|@translate}">
    <img class="thumbnail" src="{$thumbnail|@get_thumbnail_url}"
           alt="{$thumbnail.file}" title="{$thumbnail.name}"></a>
    </td><td style="text-align: left;">
    <img src="{$themeconf.icon_dir}/{if empty($thumbnail.name)}un{/if}check.png"
           alt="{$thumbnail.file}" title="{$thumbnail.name}"> {'Name'|@translate}<br />
    <img src="{$themeconf.icon_dir}/{if empty($thumbnail.comment)}un{/if}check.png"
           alt="{$thumbnail.file}" title="{$thumbnail.name}"> {'Description'|@translate}<br />
    <img src="{$themeconf.icon_dir}/{if empty($thumbnail.author)}un{/if}check.png"
           alt="{$thumbnail.file}" title="{$thumbnail.name}"> {'Author'|@translate}<br />
    <img src="{$themeconf.icon_dir}/{if empty($thumbnail.date_creation)}un{/if}check.png"
           alt="{$thumbnail.file}" title="{$thumbnail.name}"> {'Creation date'|@translate}<br />
    <img src="{$themeconf.icon_dir}/{if empty($thumbnail.date_metadata_update)}un{/if}check.png"
           alt="{$thumbnail.file}" title="{$thumbnail.name}"> {'Metadata'|@translate}<br />
    <img src="{$themeconf.icon_dir}/{if $thumbnail.NUM_TAGS == 0}un{/if}check.png"
           alt="{$thumbnail.file}" title="{$thumbnail.name}"> {'Tags'|@translate} ({$thumbnail.NUM_TAGS})
    {/if}


P@t

Hors ligne

#155 2008-03-08 13:31:15

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Evolution des templates

VDigital a écrit:

Arrêtez de penser comme des développeurs quand vous pensez aux templates.

Quand j'ai écrit, je ne pensais pas comme de développeurs en disant que c'était un simple de faire du smarty.

VDigital a écrit:

2 moteurs, c'est la situation actuelle et rvelices s'en est sorti admirablement. On pouvait les garder pour le temps de la 1.8, d'ailleurs ai-je loupé une marche où l'ancien sera encore présent.

C'est prévu pour être temporaire!

Hors ligne

#156 2008-03-08 13:32:55

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Evolution des templates

VDigital a écrit:

Juste histoire de me faire la main...
Je livre l'adaptation smarty d'Admin Advices.
8-)

PS: svn 2266 (Petit coup d'oeil de rvelices) - Merci.

Et alors, tu en penses quoi?
C'est compliqué, facile? Abordable pour tous?

Hors ligne

#157 2008-03-08 14:00:04

rvelices
Équipe Piwigo
2005-12-29
1417

Re: Evolution des templates

P@t a écrit:

D'ailleur, petite question...
J'ai vu que sur certains fichier (index.tpl par exemple), tu as mis:

{$ROOT_URL}{$themeconf.icon_dir}

Alors que sur d'autres, tu n'as pas mis le {$ROOT_URL} (le fichier cat_move.tpl par exemple)

Est-ce que le $ROOT_URL est assigné automatiquement, ou faut-il l'assigner manuellement dans chaque fichier php?
Est-ce qu'il est vraiment necessaire?

C'est pareil qu'en 1.7... Idealement il faut le mettre partout ; ca marche parce que root_url vaut "." sauf pour les mails, index et picture (si conf[question_mark_in_url]=false) ...

Hors ligne

#158 2008-03-09 13:09:55

mathiasm
Former Piwigo Team
2006-02-06
2692

Re: Evolution des templates

J'ai un souci avec les assign:
A l'heure actuelle, et a priori aussi en smarty, si je veux inclure les liens d'enregistrement dans la page picture, il faut que j'inclue le menubar.inc.php. Je pourrais simplement ajouter les liens, mais je dois savoir si le visiteur est connecté ou non pour lui présenter les bons liens. Avec smarty et un if, j'ai désormais la syntaxe pour tester, mais le test se fait sur $user['username'] et il faut donc que cette valeur soit passée via assign(...).
Plein de personnalisations de ce type vont être contraintes par l'impossibilité d'accéder à la bonne variable.

Je propose donc une réflexion pour charger systématiquement un lot de variables permettant les personnalisations directement dans le template sans passer par un plugin, genre:
- le username (qui permet aussi de savoir si on est connecté)
- la cat physique de l'image

Et de savoir si on code dans le template ou dans le php les liens, à savoir:
actuellement dans menubar.inc.php, on génère U_LOGOUT via get_root_url().'?act=logout'
mais en smarty, on pourrait soit faire la même chose dans le lot de variables évoqué plus haut, soit codé directement dans le template qui s'en sert: {$ROOT_URL}?act=logout
Le problème restant l'accès aux URL dépendantes de l'image et/ou de la catégorie.

Hors ligne

#159 2008-03-09 13:16:35

P@t
Ex Equipe Piwigo
Nice
2007-06-13
5695

Re: Evolution des templates

Très bonne idée...
Mais pourquoi passer uniquement le username?
On pourrait passer directement les variables (complètes) $user, $conf et $page à smarty...


P@t

Hors ligne

#160 2008-03-09 13:44:44

mathiasm
Former Piwigo Team
2006-02-06
2692

Re: Evolution des templates

P@t a écrit:

Très bonne idée...
Mais pourquoi passer uniquement le username?
On pourrait passer directement les variables (complètes) $user, $conf et $page à smarty...

j'osais pas :-)

Hors ligne

#161 2008-03-09 14:31:13

VDigital
Former Piwigo Team
Montpellier (FR)
2005-05-04
15127

Re: Evolution des templates

Bien entendu, il faut que quelque soit l'écran affiché un lot minimal constant soit disponible pour les templates.


Vincent -« Plus vidéaste averti que photographe amateur... »
La galerie - Le blog   

Piwigo est une application libre de gestion de photos en ligne.

Hors ligne

#162 2008-03-09 14:40:32

mathiasm
Former Piwigo Team
2006-02-06
2692

Re: Evolution des templates

VDigital a écrit:

Bien entendu, il faut que quelque soit l'écran affiché un lot minimal constant soit disponible pour les templates.

Voilà. Tout à fait ça.

Hors ligne

#163 2008-03-10 20:14:37

P@t
Ex Equipe Piwigo
Nice
2007-06-13
5695

Re: Evolution des templates

Radu, je veux bien que tu t'occupes de la migration de site_manager...
J'ai l'impression qu'il y a quelques boulettes dans ce fichier...
Et j'ai pas envie d'en rajouter ;-)


P@t

Hors ligne

#164 2008-03-11 02:44:15

rvelices
Équipe Piwigo
2005-12-29
1417

Re: Evolution des templates

P@t a écrit:

Radu, je veux bien que tu t'occupes de la migration de site_manager...

Pas de probleme c'est en cours...

Sinon c'est reparti pour les erreurs de user_list ... J'en ai un paquet. Un exemple est ADVISER_YES qui n'est pas sette dans le template ... (Notice: Undefined index: ADVISER_YES in D:\pwg\bsf\_data\templates_c\%%D1^D1A^D1A29EED%%user_list.tpl.php)

Hors ligne

#165 2008-03-11 06:58:30

rub
Former Piwigo Team
Lille
2005-08-26
5239

Re: Evolution des templates

rvelices a écrit:

P@t a écrit:

Radu, je veux bien que tu t'occupes de la migration de site_manager...

Pas de probleme c'est en cours...

Sinon c'est reparti pour les erreurs de user_list ... J'en ai un paquet. Un exemple est ADVISER_YES qui n'est pas sette dans le template ... (Notice: Undefined index: ADVISER_YES in D:\pwg\bsf\_data\templates_c\%%D1^D1A^D1A29EED%%user_list.tpl.php)

Pour ca, il faut aussi activer le mode debug des templates avec MultiView!

Hors ligne

Pied de page des forums

Propulsé par FluxBB

github twitter newsletter Faire un don Piwigo.org © 2002-2024 · Contact