Ticket #1223 (closed defect: fixed)
Métadonnées dans la déclaration des plugins/thèmes
Reported by: | bruno | Owned by: | xave |
---|---|---|---|
Priority: | normal | Milestone: | 2.4 |
Component: | module:core | Version: | 2.2 |
Severity: | normal | Keywords: | |
Cc: |
Description
Si on veut étendre les fonctionnalités des thèmes, on n'a aujourd'hui pas vraiment d'autre choix que d'ajouter un attribut à dcModule::registerModule(). Cela force à saisir tous les paramètres optionnels avant ce nouveau paramètre, ce qui est moche.
Je propose de passer un tableau de paramètres plutôt que d'ajouter sans cesse des paramètres, afin d'avoir quelque chose de plus évolutif.
Ancienne méthode :
public function registerModule($name,$desc,$author,$version,$permissions=null,$priority=1000)
Nouvelle méthode :
public function registerModule($name,$desc,$author,$version,$permissions=null,$priority=1000,$properties=array())
Idéalement, je mettrais bien $permissions et $priority dans $properties, mais il faut voir comment gérer les modules déjà existants... peut-être en jouant du func_get_args...
Change History
comment:2 Changed 14 years ago by bruno
(j'aime les monologues :)
C'est donc réalisable de manière plus propre.
Futur prototype :
public function registerModule($name,$desc,$author,$version,$properties=array())
si properties n'est pas un tableau, on parse les arguments "comme avant", sinon on récupère$priority et $permissions directement depuis $propertiespriority? et $propertiespermissions?.
comment:3 Changed 14 years ago by Dsls <dsls@…>
- Status changed from new to closed
- Resolution set to fixed
(In [e6b2db1d228a]) * New way of registering modules/themes, more extensible (backward compatible with previous method)
- small fix on attachments plugin template tags
Closes #1223.
pour approfondir, voici les propriétés qui pourraient voir le jour :