[SPIP Zone] Plugin Archives : coquille dans http://zone.spip.org/trac/spip-zone/changeset/91146 ?

Bonjour,

Je n'utilise pas ce plugin, je regardais le code simplement pour exemple, et il me semble que ce commit ajoute un bug : on obtient ce code (la partie else est ajoutée) :

  /**
   * Si on souhaite que l'archivage ne dépublie pas les articles
   */
  include_spip('inc/config');
  if(lire_config('archive/archiver_publier','non') == 'oui'){
    $tables['spip_articles']['statut'][0]['publie'] .= ',archive';
  }
  else{
    $tables['spip_auteurs']['statut'][0]['publie'] .= ',archive';
    $tables['spip_auteurs']['statut'][0]['previsu'] .= ',archive';
  }

Or du coup, ça fait que 'archive' est ajouté comme statut sur les éléments publiés, quelque soit la configuration indiquée dans la configuration.

Est-ce que c'était pas, peut être quelque chose comme ça qui était souhaité (ajouter toujours la prévisu des statut archive) et selon la conf donc sur les publiés ?

  /**
   * Si on souhaite que l'archivage ne dépublie pas les articles
   */
  include_spip('inc/config');
  if (lire_config('archive/archiver_publier','non') == 'oui'){
    $tables['spip_articles']['statut'][0]['publie'] .= ',archive';
  }
  $tables['spip_auteurs']['statut'][0]['previsu'] .= ',archive';

J'en sais rien en fait (c'est une supposition).
Si quelqu'un sait :slight_smile:

MM.

Le 28/04/2016 18:27, Matthieu Marcillaud a écrit :

Bonjour,

Je n'utilise pas ce plugin, je regardais le code simplement pour
exemple, et il me semble que ce commit ajoute un bug : on obtient ce
code (la partie else est ajoutée) :

     /**
      * Si on souhaite que l'archivage ne dépublie pas les articles
      */
     include_spip('inc/config');
     if(lire_config('archive/archiver_publier','non') == 'oui'){
         $tables['spip_articles']['statut'][0]['publie'] .= ',archive';
     }
     else{
         $tables['spip_auteurs']['statut'][0]['publie'] .= ',archive';
         $tables['spip_auteurs']['statut'][0]['previsu'] .= ',archive';
     }

Ah mais c'est même encore plus bizarre : d'un côté spip_articles, de l'autre spip_auteurs ?

MM.