[spip-dev] Fonction afficher_contenu_objet

Salut,

Je trouve la fonction afficher_contenu_objet un peu limitative. Voici son code:

function inc_afficher_contenu_objet_dist($type, $id, $id_rubrique){
  include_spip('public/assembler');
  $contexte = array('id'=>$id,'id_rubrique'=>$id_rubrique);
  
  // permettre aux plugin de faire des modifs ou des ajouts
  $contenu_objet = pipeline('afficher_contenu_objet',
    array(
      'args'=>array(
        'type'=>$type,
        'id_objet'=>$id,
        'contexte'=>$contexte
      ),
      'data'=> recuperer_fond("prive/contenu/$type",$contexte)
    )
  );
  
  return "<div id='wysiwyg'>$contenu_objet</div>";
}

=> Pourquoi on calcule le $contexte à l'intérieur de la fonction en prenant $id et $id_rubrique, alors qu'on pourrait carrément passer $contexte en variable, par exemple de cette façon:
function inc_afficher_contenu_objet_dist($type, $id, $contexte){...}

Est-ce que ça ne rendrait pas la fonction un peu plus «souple».

=> Pourquoi on ajoute un id='wysiwyg' en retour? Ca limite l'utilisation de la fonction à un seul appel par page.

ARNO*