lien_objet() devrait être renommé objet_lien() #3201

Closed
opened 9 years ago by rastapopoulos · 8 comments
Owner

Par cohérence avec l'API des objets, toutes les fonctions qui travaillent sur un objet, devraient commencer par le préfixe "objet_".

On peut garder l'ancienne pour compat pendant un temps, avec mapping sur la nouvelle. Puis un jour la mettre au grenier.

Par cohérence avec l'API des objets, toutes les fonctions qui travaillent sur un objet, devraient commencer par le préfixe "objet_". On peut garder l'ancienne pour compat pendant un temps, avec mapping sur la nouvelle. Puis un jour la mettre au grenier.
Poster
Owner
There is no content yet.
Owner

De quelle fonction lien_objet parles-tu ? Je ne comprends pas ce ticket...

De quelle fonction `lien_objet` parles-tu ? Je ne comprends pas ce ticket...
Owner

Assigné à b_b

**Assigné à b_b**
Owner

Assigné à cedric

**Assigné à cedric**
Poster
Owner

Héhé, j'ai dû rechercher moi aussi, je ne me souvenais plus. Je ne sais plus à quel occasion j'en ai eu besoin, mais il s'agit d'une fonction générique MAIS placée dans le plugin Médias ! Je ne savais même pas que le code était placé là, je l'avais vu utilisé dans des squelettes, et j'avais copié ça dans des squelettes à moi.
http://code.spip.net/`lien_objet

Donc en fait il faudrait :

  1. La déplacer dans le core, dans les filtres accessibles en squelettes
  2. La renommer comme expliqué ici
  3. Pointer l'ancienne de Médias vers la nouvelle du core

Ça peut donc se faire sans rien casser.

Héhé, j'ai dû rechercher moi aussi, je ne me souvenais plus. Je ne sais plus à quel occasion j'en ai eu besoin, mais il s'agit d'une fonction générique MAIS placée dans le plugin Médias ! Je ne savais même pas que le code était placé là, je l'avais vu utilisé dans des squelettes, et j'avais copié ça dans des squelettes à moi. http://code.spip.net/`lien_objet Donc en fait il faudrait : 1) La déplacer dans le core, dans les filtres accessibles en squelettes 2) La renommer comme expliqué ici 3) Pointer l'ancienne de Médias vers la nouvelle du core Ça peut donc se faire sans rien casser.
Owner

Bon donc en fait non : les filtres objet_xx concernent les fonctions qui portent sur la famille d'objet (recuperer les propriétés génériques de ces objets).
Pour appliquer un traitement a un objet precis et generer un texte c'est plutot la famille generer_url_entite, generer_info_entite et je propose donc generer_lien_entite

Bon donc en fait non : les filtres `objet_xx` concernent les fonctions qui portent sur la famille d'objet (recuperer les propriétés génériques de ces objets). Pour appliquer un traitement a un objet precis et generer un texte c'est plutot la famille `generer_url_entite`, `generer_info_entite` et je propose donc `generer_lien_entite`
Owner
Corrigé par http://core.spip.org/projects/spip/repository/revisions/21668 et http://zone.spip.org/trac/spip-zone/changeset/85478 **Statut changé à Fermé**
Poster
Owner

Pas forcément sur la famille non, il y en a qui portent sur un objet précis, mais justement en y réfléchissant ce sont des fonctions qui modifient l'objet lui-même (objet_inserer, objet_modifier, etc). Donc ça n'allait pas non plus. Donc oui super pour la famille generer_truc : qui extrait de l'information à partir d'un objet.

Un jour prochain, je proposerais bien d'uniformiser ces noms avec le même préfixe :

  • generer_objet_url
  • generer_objet_info
  • generer_objet_lien, etc

Là aussi en mappant on pourra ne rien casser, et ça rend alors le truc plus consistant, plus facile à apprendre et retenir (et quand on liste les choses alphabétiquement elles sont ensemble).

Pas forcément sur la famille non, il y en a qui portent sur un objet précis, mais justement en y réfléchissant ce sont des fonctions qui modifient l'objet lui-même (objet_inserer, objet_modifier, etc). Donc ça n'allait pas non plus. Donc oui super pour la famille generer_truc : qui extrait de l'information à partir d'un objet. Un jour prochain, je proposerais bien d'uniformiser ces noms avec le même préfixe : - generer_objet_url - generer_objet_info - generer_objet_lien, etc Là aussi en mappant on pourra ne rien casser, et ça rend alors le truc plus consistant, plus facile à apprendre et retenir (et quand on liste les choses alphabétiquement elles sont ensemble).
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.