P@t a écrit:

rvelices a écrit:

- la fonction check_server_plugins -> t'aurais pu passer le fs_plugins comme variable en reference a la fonction a la place d'utiliser une variable globale dans la fonction ?

Oui... mais la fonction check_server_plugins modifie au passage fs_plugins (pour ajouter le numéro d'extension à chaque plugin)
Et vu qu'elle renvoit les données du serveur... on est obligé de la mettre en global.
Est-ce que c'est mieux de la mettre aussi en variable de référence?

si tu passes le parametre par reference, tu n'est pas oblige de la mettre en global... C'est mieux juste parce que c'est plus propre (je n'ai pas a me soucier dans ma fonction si la variable globale existe/change de nom etc...), mais ce que t'as fait marche parfaitement. je ferais cette modif si tu veux (c'est super simple)...

rvelices a écrit:

si tu passes le parametre par reference, tu n'est pas oblige de la mettre en global... C'est mieux juste parce que c'est plus propre (je n'ai pas a me soucier dans ma fonction si la variable globale existe/change de nom etc...), mais ce que t'as fait marche parfaitement. je ferais cette modif si tu veux (c'est super simple)...

Oui oui! Mais je répète que le tableau fs_plugins a besoin d'etre modifié pour le script principal: la fonction check_server_plugin ajoute l'extension à chaque plugin:

if (!empty($fs_plugin['uri']) and strpos($fs_plugin['uri'] , 'extension_view.php?eid='))
    {
      list( , $extension) = explode('extension_view.php?eid=', $fs_plugin['uri']);
      if (!is_numeric($extension)) continue;
      $plugins_to_check[] = $extension;
      $fs_plugins[$plugin_id]['extension'] = $extension;
    }

C'est pour ca que je l'ai mis en global.

Sinon, pour le l10n plugins_action, un oubli de ma part...
A remplacer dans le fichier plugins_update.tpl par 'Actions'

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

rvelices a écrit:

Avec le passage par reference ca doit marcher. Si tu modifie le parametre passe par reference dans ta fonction, alors la modif est repercute sur la variable dans le script appelant. J'ai commite - ca devrait marcher.

Ok, ca fonctionne, merci radu ;-)
En fait, je ne connaissais pas le passage d'arguments par référence (le &).
Je viens de lire le manuel...
Maintenant, c'est ok ;-)

PS: quel éditeur utilises-tu? j'ai vu que tout les espaces en fin de lignes sont supprimés...

Dernière modification par P@t (2008-03-03 16:14:19)

Bonjour a tous,

Et mille excuses pour les accents car le clavier est US naturellement.
Bien, je ne vais pas prendre le temps de lire tous les posts, mais...

J ai lu la doc de smarty.

1 - Je trouve que la boucle foreach rend les templates complexes.
2 - Le is odd n arrange rien.
3 - Je voudrai avoir des tests efficacite du cache sur les acces a la page d index.

Sincerement yoga en 1.8 devrait rester avec les anciens templates.
L admin devrait migrer dans un nouveau template utilisant smarty.
Il serait alors plus simple d avoir une variable dans les plugins pour indiquer si smarty doit etre utilise ou non.

=>
yoga : plus d admin + les anciens tpl
smrt : admim smarty + les nouveaux tpl

+ Plugin set $conf smarty = true / false

Bonnes reflexions a tous...
8-)

VDigital a écrit:

1 - Je trouve que la boucle foreach rend les templates complexes.

Après lu la doc de smarty, migré quelques templates et plugins, intégrer Plugins Manager en dur... je peux dire que la boucle foreach est très simple d'utilisation et simplifie énormément les templates!
Sans compter les possiblités supplémentaires qui nous sont offertes!
Que dire aussi des html_options, html_radios, la fonction get_template_vars, etc, etc, etc....
Extremement pratique!

VDigital a écrit:

- Le is odd n arrange rien.

Idem, je trouve ca plus pratique d'avoir ca dans le template.
Et meme plus logique

VDigital a écrit:

3 - Je voudrai avoir des tests efficacite du cache sur les acces a la page d index.

On verra bien quand tout sera migré ;-)

VDigital a écrit:

Sincerement yoga en 1.8 devrait rester avec les anciens templates.

Complètement PAS D'ACCORD!
La migration est presque terminée (merci radu!)
Et en plus j'ai déjà oublié comment coder avec l'ancien template!
Essaye smarty, et en 1heure, t'es déjà accros...

Dernière modification par P@t (2008-03-03 20:48:42)

P@t a écrit:

rvelices a écrit:

Merci de te lancer dans la migration... Ca aide beaucoup d'avoir plusieurs mains ...

Je comptais faire en prochain le element_set_global et element_set_unit. Donc si tu veux, tu peux te pencher sur user_list / site_manager / site_update ?

Merci à toi de faire le plus gros du boulot! ;-)
Je viens migrer register, profile et install...
Je ferai user_list, site_manager et site_update ce soir.... ou après demain... (demain, c'est ski!)

Vous êtes tous les 2 très actifs sur la migration, je ne sais pas ou vous trouvez le temps... mais, c'est vrai que vous bossez vite...
Je garde mes options sur les mails et la NBM mais à la vitesse où ca va, je vais les perdre... Je voudrais les faire pour bien appréhender smarty!

rub a écrit:

rub a écrit:

Par contre, il manque une traduction "plugin_action".
Et j'ai "[l10n] language key "plugins_action" is not defined
plugins_action" en local chez moi et plugins_action sur la démo.
Ce qui m'embête, c'est le "[l10n] language key", est-ce à cause de smarty?

Par contre, je reviens sur ca.
C'est bien un problème avec smarty?