Le 30 juin 2011 13:06, Franck Paul <carnet.franck.paul(a)gmail.com&gt; a écrit : > Le 30 juin 2011 12:58, Dev Dotclear 2 <trac(a)dotclear.net&gt; a écrit : >> #1240: Formulaire des configurateurs intégrés de thème >> -------------------------+-------------------------------------------------- >>  Reporter:  franck       |        Owner:  franck >>     Type:  enhancement  |       Status:  closed >>  Priority:  normal       |    Milestone:  2.4 >> Component:  module:core  |      Version:  2.3 >>  Severity:  normal       |   Resolution:  fixed >>  Keywords:               | >> -------------------------+-------------------------------------------------- >> >> Comment(by bruno): >> >>  Pas complètement séduit par l'approche : tu appelles un behavior en >>  incluant après cet appel le seul fichier d'administration du thème, à >>  savoir le _config.php. Du coup pour que ça fonctionne, il ne reste comme >>  possibilité que de définir le behavior dans le _define.php du thème, ce >>  que je ne trouve guère séduisant. > > Ça peut aussi être défini dans un _prepend.php, ce que je fais pour Ductile. > >>  De plus, ajouter un behavior uniquement pour définir une valeur booléenne >>  ressemble plus à un contournement qu'autre chose. Pourquoi ne pas proposer >>  plutôt un paramètre additionnel au thème qui permettrait de dire comment >>  gérer la configuration d'un thème ? Je pense notamment au ticket #1223, où >>  on pourrait ajouter un attribut dédié au thème qui le souhaite > > En effet c'est un contournement d'une contrainte imposée par DC pour > la gestion des formulaires. D'autre part, la modification que tu > proposes un peu plus de code que ce simple appel de behaviour et ces > deux tests. > > Maintenant on peut bien évidemment faire autrement, je n'ai pas de > religion toute faite. Je souhaite simplement que la contrainte soit > levée, c'est tout. (Je transfère la conversation sur dev@, après tout tracker@ sera bientôt read-only :) On est complètement d'accord sur le fond. Maintenant sur la forme, il y a plein de choses qu'un thème devrait pouvoir faire, mais qui n'est pas faisable dans la version actuelle de dc. Le _conf.php un peu trop contraint, l'impossibilité de définir des widgets dans le thème, et pourquoi pas par la suite, le choix du moteur de templates, des champs de configuration additionels, ... Je propose juste d'essayer d'aborder cette problématique d'un niveau plus global, plutôt que d'adresser chacune d'entre elles une à une.

> Eh bien dans ce cas je propose de conserver le behaviour que j'ai mis > en place, le temps qu'un développement plus costaud et ambitieux soit > effectué. À moins bien sûr que ce soit possible dès maintenant ? > Auquel cas, no problemo, j'adapterai Ductile, puisqu'il est question > de lui aujourd'hui (mais d'autres pourraient en avoir besoin — Blowup > pourrait peut-être lui aussi en bénéficier). Dans ce cas, il faudrait peut-être commiter sur la branche themes, et non default...