Recherche avancée

Sur d’autres sites (9009)

• Revision 90346 : on peut demander maintenant depuis le tableau que le fieldset soit encadré ...

  19 juin 2015, par toutati@… — Log

  on peut demander maintenant depuis le tableau que le fieldset soit encadré d’un div au lieu du ul quand SPIP &lt ;3.1

• Anomalie #3055 (En cours) : Autorisation sur les listes d’objets

  5 mai 2015, par Eric Lupinacci

  En fait, c’est à mon avis un problème plus général, de rigueur dans l’appel des autorisations en entrée des pages mais aussi de stratégie de définition des autorisations qui sont malheureusement assez rarement arborescentes.

  Si on prend les pages objet comme article, rubrique et auteur par exemple, elle commencent toutes par une autorisation :
  

  (#AUTORISERvoir,article,#ID_ARTICLE ou
  (#AUTORISERvoir,rubrique,#ID_RUBRIQUE ou
  (#AUTORISERvoir,auteur,#ID_AUTEUR
  

  
  Cependant d’ailleurs seul l’autorisation article_voir est définie, les autres renvoyant sur l’autorisation par défaut. Mais au moins une autorisation est appelée systématiquement quelque soit le moyen avec lequel on arrive sur cette page.
  A partir de là si il existe un bouton qui envoie sur cette page il faut le conditionner par cette autorisation (code HTML) et si c’est un menu créé automatiquement alors là il faut créer une autorisation de menu qui appelle l’autorisation de voir et pas le contraire à mon avis.

  Donc si on étend ça aux pages articles, auteurs et rubriques je suis d’accord qu’il faudrait une autorisation voir_xxxx (ce qui n’est pas le cas actuellement) qui serait appelée en entrée de la page comme l’exemple ci-dessous pour la page articles :
  

  (#AUTORISERvoir,_articles
  

  
  et il faudrait alors coder l’autorisation du menu Articles de cette façon :
  

  function autoriser_articles_menu_dist($faire, $type, $id, $qui, $opt) 
      return autoriser(’voir’, ’_articles’) ;
  
  

  
  En effet, la décision d’afficher ou pas le menu est indépendante du fait de voir ou pas la page articles mais doit s’appuyer dessus si on décide de ne pas présenter un lien qui mène vers une page interdite.
  Et il faudrait systématiser cela pour tous les plugins objet du core à mon avis.

  En second lieu il faudrait faire une passe sur le code des autorisations actuelles pour imbriquer un peu plus les autorisations de façon arborescente ce qui leur donnerait plus de lisibilité car parfois elles sont identiques sans qu’on s’en rende compte car le code est dupliqué.

• Anomalie #3429 : Incohérence du 2è paramètre du critère `{pagination}`

  4 mai 2015, par Eric Lupinacci

  Oui, je confirme le problème remonté par Marcimat.
  Je pense que la proposition de mettre une virgule est pertinente pagination 10, nom
  Le code pour tenir compte de cette modification en gardant la compatibilité avec l’ancienne est le suivant :

      // Calcul du nommage de la pagination si il existe.
      // La nouvelle syntaxe pagination 20, nom est prise en compte et privilégiée mais on reste
      // compatible avec l’ancienne car certains cas fonctionnent correctement
      if (!isset($crit->param[0][1])) 
          $type = !isset($crit->param[1][0]) ? "’$idb’" 
   : calculer_liste(array($crit->param[1][0]), array(), $boucles, $boucle->id_parent) ;
          
      else 
          $type = calculer_liste(array($crit->param[0][1]), array(), $boucles, $boucle->id_parent) ;
      
  

  en lieu et place de :

      $type = !isset($crit->param[0][1]) ? "’$idb’" 
   : calculer_liste(array($crit->param[0][1]), array(), $boucles, $boucle->id_parent) ;
  

  Si on est d’accord avec la proposition de Marcimat je peux commiter.