3:24 p.m.
Re,
Je viens de faire quelques tests, et ça marche. La proposition est :
changer la déclaration dans le _define.php d'un thème/plugin de :
$this->registerModule(
/* Name */ "Pages",
/* Description*/ "Serve entries as simple web pages",
/* Author */ "Olivier Meunier",
/* Version */ '1.1.1',
/* Permissions */ 'contentadmin,pages',
999
);
en :
$this->registerModule(
"Pages",
"Serve entries as simple web pages",
"Olivier Meunier",
'1.1.1',
array(
'permissions' => 'contentadmin,pages',
'priority' => 999
)
);
Donc en gros, remplacer tout ce qui suit la version par un tableau de
clefs/valeurs. Cela permettra d'ajouter de nouveaux paramètres de
description des thèmes/plugins, sans casser à chaque fois la signature
de l'appel à registerModule, et sans avoir à se palucher des
paramètres optionnels qu'on ne souhaite pas.
La rétrocompatibilité est assurée en testant le 5e argument. Si c'est
un tableau => c'est le nouveau modèle, sinon on prend l'ancien.
Z'en dites ?
--
Bruno
3:28 p.m.
Le 30 juin 2011 15:24, Dsls <dsls(a)morefnu.org> a écrit :
Donc en gros, remplacer tout ce qui suit la version par un tableau
de
clefs/valeurs. Cela permettra d'ajouter de nouveaux paramètres de
description des thèmes/plugins,
Cela permettrait donc d’internationaliser la description ?
--
Philippe
3:46 p.m.
Cela permettrait donc d’internationaliser la description ?
Pas forcément, je pensais surtout à des choses plus techniques, du
style ajouter des dépendances entre plugins.
Pourquoi pas forcément ? Parce que le _define.php est lu tout le
temps, il faut essayer de le conserver assez minimaliste.
Par contre on peut tout à fait envisager un autre fichier pour
répondre au besoin d'i18n. C'était l'idée de proposition d'un retour
du desc.xml dans les plugins/thèmes
--
Bruno
3:32 p.m.
Je viens de relire le ticket pour comprendre concrètement ce que cela pouvait apporter et
effectivement, c'est assez top pour les thèmes. En tout cas, la modif me paraît bien
et propre donc oui pour moi.
-----Original Message-----
From: Dsls <dsls(a)morefnu.org>
Sender: dev-bounces(a)list.dotclear.org
Date: Thu, 30 Jun 2011 15:24:21
To: <dev(a)list.dotclear.org>
Reply-To: dev(a)list.dotclear.org
Subject: [Dotclear Dev] Ticket 1223 -- fichiers define.php
Re,
Je viens de faire quelques tests, et ça marche. La proposition est :
changer la déclaration dans le_define.php d'un thème/plugin de :
$this->registerModule(
/* Name */ "Pages",
/* Description*/ "Serve entries as simple web pages",
/* Author */ "Olivier Meunier",
/* Version */ '1.1.1',
/* Permissions */ 'contentadmin,pages',
999
);
en :
$this->registerModule(
"Pages",
"Serve entries as simple web pages",
"Olivier Meunier",
'1.1.1',
array(
'permissions' => 'contentadmin,pages',
'priority' => 999
)
);
Donc en gros, remplacer tout ce qui suit la version par un tableau de
clefs/valeurs. Cela permettra d'ajouter de nouveaux paramètres de
description des thèmes/plugins, sans casser à chaque fois la signature
de l'appel à registerModule, et sans avoir à se palucher des
paramètres optionnels qu'on ne souhaite pas.
La rétrocompatibilité est assurée en testant le 5e argument. Si c'est
un tableau => c'est le nouveau modèle, sinon on prend l'ancien.
Z'en dites ?
--
Bruno
_______________________________________________
Dev mailing list
Dev(a)list.dotclear.org
http://ml.dotclear.org/listinfo/dev
4:17 p.m.
Le 30 juin 2011 15:24, Dsls <dsls(a)morefnu.org> a écrit :
Re,
Je viens de faire quelques tests, et ça marche. La proposition est :
changer la déclaration dans le _define.php d'un thème/plugin de :
$this->registerModule(
/* Name */ "Pages",
/* Description*/ "Serve entries as simple web pages",
/* Author */ "Olivier Meunier",
/* Version */ '1.1.1',
/* Permissions */ 'contentadmin,pages',
999
);
en :
$this->registerModule(
"Pages",
"Serve entries as simple web pages",
"Olivier Meunier",
'1.1.1',
array(
'permissions' => 'contentadmin,pages',
'priority' => 999
)
);
Donc en gros, remplacer tout ce qui suit la version par un tableau de
clefs/valeurs. Cela permettra d'ajouter de nouveaux paramètres de
description des thèmes/plugins, sans casser à chaque fois la signature
de l'appel à registerModule, et sans avoir à se palucher des
paramètres optionnels qu'on ne souhaite pas.
La rétrocompatibilité est assurée en testant le 5e argument. Si c'est
un tableau => c'est le nouveau modèle, sinon on prend l'ancien.
Z'en dites ?
Et donc on pourra tester les paramètres via un
$core->themes->moduleInfo($id_themes,'param_key') kivabien, si j'ai
bien compris ?
4:34 p.m.
Le 30 juin 2011 16:27, Dsls <dsls(a)morefnu.org> a écrit :
> Et donc on pourra tester les paramètres via un
> $core->themes->moduleInfo($id_themes,'param_key') kivabien, si
j'ai
> bien compris ?
Exactement m'sieur :)
Ok j'ai plus qu'à trouver un nom parlant au paramètre qui va remplacer
le behaviour que j'ai mis en place ce matin. Z'avez des idées ou des
recommandations ?
4:43 p.m.
Le 30 juin 2011 16:34, Franck Paul <carnet.franck.paul(a)gmail.com> a écrit :
Le 30 juin 2011 16:27, Dsls <dsls(a)morefnu.org> a écrit :
>> Et donc on pourra tester les paramètres via un
>> $core->themes->moduleInfo($id_themes,'param_key') kivabien, si
j'ai
>> bien compris ?
>
> Exactement m'sieur :)
Ok j'ai plus qu'à trouver un nom parlant au paramètre qui va remplacer
le behaviour que j'ai mis en place ce matin. Z'avez des idées ou des
recommandations ?
'standalone_config' ?
4:58 p.m.
Le 30 juin 2011 16:43, Dsls <dsls(a)morefnu.org> a écrit :
Le 30 juin 2011 16:34, Franck Paul
<carnet.franck.paul(a)gmail.com> a écrit :
> Le 30 juin 2011 16:27, Dsls <dsls(a)morefnu.org> a écrit :
>>> Et donc on pourra tester les paramètres via un
>>> $core->themes->moduleInfo($id_themes,'param_key') kivabien, si
j'ai
>>> bien compris ?
>>
>> Exactement m'sieur :)
>
> Ok j'ai plus qu'à trouver un nom parlant au paramètre qui va remplacer
> le behaviour que j'ai mis en place ce matin. Z'avez des idées ou des
> recommandations ?
'standalone_config' ?
En fait j'hésitais surtout sur la forme de la clé :
minuscule+underscore ou camelCase
'standalone_config' conviendra parfaitement, merci.
6:40 a.m.
> 'standalone_config' ?
En fait j'hésitais surtout sur la forme de la clé :
minuscule+underscore ou camelCase
'standalone_config' conviendra parfaitement, merci.
Ok. Je commite quelque chose dans ce sens dans la journée. A bien y
réfléchir, je ne vois pas d'inconvénients à cette approche :)
--
Bruno
1:55 p.m.
Salut Kevin !
Posons la question essentielle pour l'avenir de cette liste : les
réponses, dessus ou dessous ?
Anne/Kozlika
Le 1 juillet 2011 13:27, kévin Lepeltier <kevin(a)lepeltier.info> a écrit :
On m'a dit que ça parlerais klingon, mais je vois que l'on se
pose ici
de vrai question existentiel.
> En fait j'hésitais surtout sur la forme de la clé :
> minuscule+underscore ou camelCase
Cette question qui ne se l'ais jamais posé :)
Bonjour, ravi de vous suivre.
3:25 p.m.
Dedans !
Quel que soit la situation je préfère toujours être dedans.
Enfin je veut dire la mettre dedans.
Je parle bien de citation hein
--
kévin - lipki
4689
days inactive
4690
days old
12 comments
6 participants
participants (6)
-
Dsls -
Franck Paul -
Kozlika -
kévin Lepeltier -
Philippe -
tbouron＠gmail.com