Skip to content

fix: pour `formulaire_instituer_objet, mettre en cohérence les tests dans...

Maïeul a demandé de fusionner spip_3408_incoherence vers 2.x

Cette PR vise à résoudre spip#3408 (closed) autrement que par spip!5890 (comment 193664) et !5 (closed).

Au lieu de chercher à améliorer la fonction autoriser_instituer pour tenir compte du statut (ce qui serait le plus cohérent, mais complexe en raison des remarques soulevés par @cerdic dans la précédente PR), elle se contente de mettre en cohérence

  • les tests effectués dans les 2 fonctions _charger() et _verifier() de formulaire_instituer_objet
  • les appels à formulaires_editer_objet_charger() pour ne pas passer de parent (et donc pas de parent à 0)
  • le fait que désormais cette dernière fonction renvoie id_parent et pas id_rubrique (mais quelle idée!)

Reste une limite encore sur les parents : ca va chercher autoriser_publier_rubrique SANS tenir compte de l'API parent. Mais c'est un autre problème, et je propopse d'ouvrir un autre ticket.

Finalement c'est corrigé via 7273484f

En attendant ca me semble valable en 4.3, avant de chercher plus loin dans la mutualisation et dans l'amélioration de autoriser_instituer

Modification effectuée par Maïeul

Rapports de requête de fusion

Chargement en cours