Revoir $core->meta->getMeta

[bruno]

La signature de la méthode getMeta de la classe Meta est la suivante :

	public function getMeta($type=null,$limit=null,$meta_id=null,$post_id=null)

Il y a certainement plus "élégant" comme approche, et surtout plus extensible sans casser la compatibilité.

Je propose une harmonisation comme c'est fait pour les billets, ie.

	public function getMeta($params=array(), $count_only=false) {

A noter : l'ajout de $count_only, qui manque actuellement. Dans un but transitoire, on peut bien sûr ajouter une nouvelle méthode plutôt que de renommer l'ancienne...

[bruno]

Accessoirement, le code situé sous l'appel $this->con->select (appel à toStatic compris) me semble bien spécifique aux tags, il faudrait peut-être envisager de le déplacer dans le plugin tags...

[bruno]

(In [3140]) Proposal for $core->meta->getMeta enhancement, see #1001. Compatibility is kept with older getMeta method, now defines a new method "getMetas".

[xave]

Replying to bruno:

Accessoirement, le code situé sous l'appel $this->con->select (appel à toStatic compris) me semble bien spécifique aux tags, il faudrait peut-être envisager de le déplacer dans le plugin tags...

Pas d'accord. Si tu parles bien des lignes vers 200~210, je n'y vois rien de spécifique au tags. C'est un traitement qui n'est utilisé que par le plugin tag, mais qui peut éventuellement servir ailleurs.

[bruno]

Replying to xave:

Pas d'accord. Si tu parles bien des lignes vers 200~210, je n'y vois rien de spécifique au tags. C'est un traitement qui n'est utilisé que par le plugin tag, mais qui peut éventuellement servir ailleurs.

Je ne suis pas contre le laisser dans la classe Meta, mais il faudrait bien distinguer ce traitement du reste (c'est le sens de ma proposition de patch), il reste quand même réservé à certains cas bien spécifiques.

[xave]

Oui. Moi, ce qui m'emmerde, c'est d'avoir deux fonctions avec un nom aussi proche. C'est vrai que dans l'absolu, getMetas devrait prendre la place de getMeta _{quitte à ajouter un getMetaPercents ou un truc du genre, mais jusqu'où on peut aller dans cette direction là sans casser l'existant ? (je pense surtout à l'appel à getMeta(s), le calcul des pourcentages, je ne suis pas persuadé qu'il soit utilisé ailleurs que dans le plugin Tags, on peut sans doute sans trop de risques ajouter un appel dans celui-ci.)}

[bruno]

J'ai choisi getMetas comme nom, mais on peut opter pour getMetadata, en précisant que getMeta sera deprecated...

J'ai cherché sur le lab, aucun plugin n'exploite les percent...

[xave]

Oui mais il y en a qui exploitent les roundpercent.

[xave]

Sinon, getMetadata, j'aime mieux. Tu me fais un getMetaStats pour qu'on puisse vraiment passer getMeta en deprecated sans perdre de fonctionnalité ? :)

[bruno]

(In [3146]) Introduced computeMetaStats method, renamed getMetas to getMetadata, documented the code... see #1001

[bruno]

(In [3147]) Updated services.php and tags plugins according to [3146], see #1001

[xave]

(In [3148]) Cleanup in Metadata class, updates here & there to acknowledge the changes. by DSLS. Closes #1001.