Advanced search

On other websites (5094)

• Anomalie #3408 (En cours): Ergo : incoherence entre l’accès au formulaire instituer et si on a vr...

  9 February 2021, by b b

  J’aillais envoyer le patch suivant, mais...

  < b_b‎ >  je voulais tester avant d’envoyer maieul 
  < b_b‎ >  mais je ne comprends pas le bug
  < b_b‎ >  je dois faire quoi pour le reproduire ?
  < b_b‎ >  en tant que rédacteur j’ai :
  < b_b‎ >  en cours de rédac + proposé + à la poubelle
  < b_b‎ >  j’envoie rien tant que je ne peux pas reproduire le bug
  

  * d702e8779a - (HEAD -> issue_3408) tester aussi qu’on peut publier dans la rubrique courante au chargement du formulaire (b_b il y a 5 minutes)
  | 
  | diff —git a/prive/formulaires/instituer_objet.php b/prive/formulaires/instituer_objet.php
  | index b5ae508405..cfa5f710e4 100644
  | --- a/prive/formulaires/instituer_objet.php
  | +++ b/prive/formulaires/instituer_objet.php
  | @@ -93,6 +93,11 @@ function formulaires_instituer_objet_charger_dist($objet, $id_objet, $retour = ’
  |              
  |          
  |      
  | +
  | +    if (isset($v[’id_rubrique’]) and !autoriser(’publierdans’, ’rubrique’, $v[’id_rubrique’])) 
  | +        $publiable = false;
  | +    
  | +
  |      $statuts = lister_statuts_proposes($desc, $editable ? $publiable : true);
  |      if (count($statuts) == 1 and isset($statuts[$v[’statut’]])) 
  |          $editable = false;
  

• Anomalie #3408: Ergo : incoherence entre l’accès au formulaire instituer et si on a vraiment le d...

  9 February 2021, by Maïeul Rouquette

  Pour reproduire le bug.

  Ajouter dans mes_options.php les lignes suivantes

  $GLOBALS[’spip_pipeline’][’autoriser’] .= "|mes_autorisations";
  function mes_autorisations()
      function autoriser_article_instituer($faire, $type, $id, $qui, $opt) 
          return true;
      
  
  

  Se connecter en tant que simple auteur/autrice

  On voit qu’on peut publier un article dans le formulaire de changement de statut. Mais lorsqu’on choisit de le faire, le statut n’est pas modifié !

  A noter que par rapport au code de b_b, mais aussi à mon analyse initiale, $v[’id_rubrique’] est systématiquement null.
  Voici mes pistes :
  - `formulaire_editer_objet_charger` devrait renvoyer un id_rubrique (+ corriger le id_parent qui est null dans mon cas !) https://git.spip.net/SPIP/spip/src/branch/master/ecrire/inc/editer.php#L200
  - il y a 2 fois le test de si on a le droit de publier : dans objet_instituer https://git.spip.net/SPIP/spip/src/branch/master/ecrire/action/editer_objet.php#L332 + dans la fonction verifier du formulaire https://git.spip.net/spip/spip/src/branch/master/prive/formulaires/instituer_objet.php#L146 > c’est ceinture et bretelle. je sais pas si c’est pertinent de garder les deux, a priori dans objet_instituer cela suffit non ?

• Anomalie #3408: Ergo : incoherence entre l’accès au formulaire instituer et si on a vraiment le d...

  5 February 2021, by RastaPopoulos ♥

  Si j’ai bien compris, il n’est pas dit que c’est avoir les deux autorisations qui est incohérent, ça à la limite pourquoi pas… (même s’il me semble qu’il ne devrait y avoir qu’une seule autorisation dédiée appelée, donc "instituer" et DANS celle-ci ça devrait tester "publierdans" du parent non ?)
  C’est que si on autoriser "instituer" pour un objet et bien ça affiche le choix possible de le faire dans l’interface, alors qu’en fait la personne n’a pas le droit si pas aussi "publierdans".

  Et donc la résolution ça serait plutôt que pour l’affichage aussi (pour charger) il faut tester les deux, s’il faut les deux pour verifier, non ?

  Ce n’est pas logique, et donc bien un bug ergo, si on laisse à la personne la possibilité d’utiliser le formulaire alors qu’elle n’en a pas le droit en vrai.

  Donc je vois pas trop pourquoi on fermerait, même si on corrige pas tout de suite…