[spip-dev] 3 autorisations inutiles dans core/forums ?

Bonjour,

En parcourant les autorisations ce week end, je suis tombé sur celles là :

«autoriser_bouton_forum_dist, autoriser_bouton_forum_admin_dist, autoriser_bouton_controle_forum_dist».

En théorie les boutons créent des autorisations «autoriser_forum_bouton_dist» en appelant autoriser('bouton', $id); et par conséquent «autoriser_bouton_forum_dist» ne serait pas appelé (au moins par cette entrée là).

En pratique, il n'y a aucun bouton de déclaré sous ces noms, aucun appel direct à ces autorisations non plus… Je loggue, et rien ne s'écrit, bref, j'ai bien l'impression que c'est du code mort ou erroné.

http://zone.spip.org/trac/spip-zone/browser/core/plugins/forum/forum_autoriser.php#L45

j’ai du me tromper en nommant la fonction.
Il faut intervetir ‹ bouton › et son nom dans ces 3 fonctions, je crois.
Cédric

C'est ce que je pensais, mais comme je disais, «il n'y a aucun bouton de déclaré sous ces noms»… ni dans le plugin, ni dans SPIP.

Je vois aussi qu'il reste une occurrence dans inc/autoriser http://trac.rezo.net/trac/spip/browser/spip/ecrire/inc/autoriser.php#L658 de «autoriser_forum_admin_suivi_bouton_dist», lui aussi inexistant.

Effectivement, 3 boutons existaient en spip 2.0 nommés «forum», «forum_admin» et «controle_forum». Mais ils disparaissent des écrans ensuite…

Visiblement, les identifiants ont été renommés (mais pas les exec)
«forum» en «forum_interne_suivi»,
«controle_forum» en «forum_reactions».

Reste donc «forum_admin» et «forum_admin_suivi» qui ont totalement disparus. Que deviennent-ils ?

Les 3 autorisations citées sont donc inutiles. Les autorisations des boutons «forum_interne_suivi» et «forum_reactions» étant bien définies.