Revisions : SPIP

Les articles publiés sur le site

• Anomalie #4480 (Fermé) : Modèle : ajouter un attribut alt à la balise même si l’image...

  18 mars 2021, par cedric -

  et corrigé par https://git.spip.net/spip/medias/pulls/14 plutot

• Evolution #4615 (Fermé) : Les documents (au moins image) devraient avoir un champ "alt" différents...

  18 mars 2021, par cedric -

  C'est intégré

• Anomalie #4697 (Nouveau) : Mauvaise redirection d’article virtuel si [] dans l’url

  17 mars 2021

  Bonjour,

  Utilisant le plugin critère {mots}, j'ai des url du type : ?rubrique21&mots[]=124

  J'ai eu besoin de faire une redirection via un article virtuel sur cette url.
  Mais elle est interprétée dasn l'admin en :

  • à l'affichage : ?rubrique21&mots[=124]
  • en tant que code html :

    ?rubrique21&mots[=124]

    donc :
    • lien : ?rubrique21&mots[
    • suivi de =124]

  Mais, curieusement, si je clique non pas sur ce lien (qui donne une 404), mais sur [Voir en ligne], j'arrive bien au bon endroit.

  C'est contusionnant.

  Testé sur SPIP 3.2.9 (avec des plugins) et 3.3 (sans aucun plugin)

• Anomalie #3946 : Notice php

  16 mars 2021

  https://git.spip.net/spip/petitions/commit/ec83efda

  Du coup. Avec une note sur le `$p->nom_champ`

• Anomalie #3946 : Notice php

  16 mars 2021

  Plusieurs pistes pour corriger :

  A) Casser la compat ascendente pour dire : le formulaire de signature #FORMULAIRE_SIGNATURES doit être dans une boucle (ARTICLES, pour le coup)
  B) Lorsque balise_PETITION est appellé depuis #FORMULAIRE_SIGNATURE, $p->nom_champ vaut 'FORMULAIRE_SIGNATURES' (Je trouve cela étrange ceci — quand c'est appelé depuis collecter_balise_dynamique()). On pourrait donc le tester dans balise_PETITION en exeption
  C) Remettre comme c'était avant (pas de tests hors boucle)

  Voilà un patch pour B)

  diff --git a/petitions_fonctions.php b/petitions_fonctions.php
  index 8ded5a6..7623752 100644
  --- a/petitions_fonctions.php
  +++ b/petitions_fonctions.php
  @@ -33,7 +33,10 @@ if (!defined('_ECRIRE_INC_VERSION')) {
    */
   function balise_PETITION_dist($p) {
          $nom = index_boucle($p);
  -       if ($nom === '' or empty($p->boucles[$nom])) {
  +       if (
  +               ($nom === '' or empty($p->boucles[$nom]))
  +               and $p->nom_champ !== 'FORMULAIRE_SIGNATURE'
  +       ) {
                  $msg = array(
                          'zbug_champ_hors_boucle',
                          array('champ' => '#PETITION')
  @@ -41,15 +44,22 @@ function balise_PETITION_dist($p) {
                  erreur_squelette($msg, $p);
          }
          else {
  -               $p->code = "quete_petitions(" .
  -                       champ_sql('id_article', $p) .
  -                       ",'" .
  -                       $p->boucles[$nom]->type_requete .
  -                       "','" .
  -                       $nom .
  -                       "','" .
  -                       $p->boucles[$nom]->sql_serveur .
  -                       "', \$Cache)";
  +               if (!empty($p->boucles[$nom])) {
  +                       $p->code = "quete_petitions(" .
  +                               champ_sql('id_article', $p)
  +                               . ",'" .$p->boucles[$nom]->type_requete . "'" 
  +                               . ",'$nom'," 
  +                               . "'" . $p->boucles[$nom]->sql_serveur . "'" 
  +                               . ", \$Cache)";
  +               } else {
  +                       $p->code = "quete_petitions(" .
  +                               champ_sql('id_article', $p)
  +                               . ",''" 
  +                               . ",'$nom'," 
  +                               . "\$connect" 
  +                               . ", \$Cache)";
  +               }
  +
                  $p->interdire_scripts = false;
          }