Recherche avancée

Sur d’autres sites (2631)

• Anomalie #4114 : paramètre media:joindre_deballer_lister_zip ignoré

  19 mars 2018, par Alexis Z

  Effectivement, j’ai oublier de précisé.
  Spip 3.2.0 révision 23778

  Le 19 mars 2018 à 14:35, <redmine@spip.org> a écrit :

  La demande #4114 a été mise à jour par b b.

  Salut et merci pour le signalement, sur quelle version de SPIP observes-tu
  le problème ?
  ------------------------------
  Anomalie #4114 : paramètre media:joindre_deballer_lister_zip ignoré
  <https://core.spip.net/issues/4114#change-13736>

  - Auteur : Alexis Z
  - Statut : Nouveau
  - Priorité : Normal
  - Assigné à :
  - Catégorie :
  - Version cible :
  - Resolution :
  - Navigateur :

  Bonjour,

  Il me semble avoir trouvé une petite incohérence dans le code du plugin
  media, plus précisément dans la fonction "joindre_deballer_lister_zip"
  ligne 301, de media/inc/joindre_document.php.
  Sauf erreur de ma part, cette fonction a pour but de déballer le contenu
  d’un fichier zip qui lui est passé en paramètre dans un répertoire
  temporaire et de retourner une liste décrivant sont contenus.

  Cette fonction prends deux paramètres $path et $tmp_dir :
  - $path corresponds au chemin du fichier zip à déballer
  - $tmp_dir corresponds au dossier temporaire ou celui-ci sera déballé

  Cette fonction utilise la librairie Pclzip.

  L’incohérence se trouve au niveau du deuximère paramètre, $tmp_dir,
  celui-ci est censé indiquer dans quel répertoire le contenu du zip sera
  déballer or ce chemin n’est pas pris en compte par la fonction
  Pclzip->extraire (ligne 305), et n’est pas non plus pris en compte par la
  fonction callback ’callback_deballe_fichier’ indiqué à la fonction extraire
  de Pclzip.
  En effet dans le code le chemin pris en compte est déclaré dans un define
  "_TMP_DIR" celui-ci déclaré à la ligne 140 de la fonction
  "joindre_trouver_fichier_envoye" (meme fichier php, début ligne 26).
  ($tmp_dir est uniquement utilisé dans la définition du chemin du fichier
  qui est renvoyé par la fonction : ligne 317 : ’tmp_name’ => $tmp_dir . $f)

  Donc le paramètre $tmp_dir quasi non-utilisé induit en erreur car on
  s’attend à se que le contenu ce trouve dans le chemin $tmp_dir de plus si
  on appeler directement la focntion "joindre_deballer_lister_zip" sans
  appeler "joindre_trouver_fichier_envoye" on ne définit pas _TMP_DIR et on
  a une erreur incohmpréensible.

  Du coup, le pire sénario (mon cas) j’appelais
  "joindre_deballer_lister_zip" après un autre appel "joindre_trouver_fichier_envoye"
  indirect, donc la variable _TMP_DIR etait défini et le contenu de mon zip
  déballer à cette endroit alors que je donnais un $tmp_dir completement
  différent, cette destination restait vide et aucun message d’erreur de la
  fonction "joindre_deballer_lister_zip".

  Bref, je suggère de prendre en compte pour l’extraction la variable
  $tmp_dir, et/ou d’ajouter test/définition de la variable _TMP_DIR en début
  de fonction pour prévenir tout exécution "bizarre".
  ------------------------------

  Vous recevez ce mail car vous êtes impliqués sur ce projet.
  Pour changer les préférences d’envoi de mail, allez sur
  http://core.spip.org/my/account

• Anomalie #3988 : meta cache_signature trop restrictive

  4 septembre 2017, par Emmanuel Dreyfus

  cedric - a écrit :

  Le cas qui peut être problématique est celui de la base répliquée
  entre plusieurs serveurs, mais là on entre dans un autre sujet et
  pour qu’il y ait collision cela supposerait que le dossier tmp/cache
  est commun aux serveurs.

  On se heurte au problème dans ce contexte, mais chaque serveur
  a son dossier cache. La base de donnée est partagée, sauf certaines
  tables qui restent locales (jobs ; syndic, urls, transactions, visites, referers).

  Comme chaque machine a son cache, elle tente d’écrire la meta cache_signature avec une valeur qui rentre en conflit avec celle des autres, d’où l’idée d’avoir un nom de meta différente pour chaque cache.

• Evolution #4102 : Ordre des inclures dans cache/charger_plugins_options.php

  26 février 2018, par RastaPopoulos ♥

  TL ;DR : problème résolu, c’est le plugin Albums qui ne fait pas bien les choses + mais un pipeline à l’initialisation est forcément utile et j’en ai déjà eu besoin.

  @azerttyu Mais non, on ne doit surtout pas changer l’ordre des fichiers, il y a des milliers (et peut-être même des milliards :p) de cas dans la nature qui se basent tous sur le fait que l’ordre logique est le même que partout ailleurs, celui de l’ordre des pipelines et celui de l’ordre de toutes les surcharges de fichiers dans SPIP, càd l’ordre du PATH. D’une ce serait incohérent mais surtout c’est pas un nouveau truc ajouté, là on parle des options.php déjà utilisé partout, donc non on ne doit rien changer à l’ordre actuel.

  Si ya un truc qui doit changer c’est sur un ajout, pas sur un truc méga utilisé partout.

  Dans tous les cas, comme dit marcimat, faire de la config avec des define() c’est vraiment pas super, et encore plus définir des define() dans le options.php du plugin qui en a besoin ! Après le problème c’est quand le define() est justement utilisé directement dans ce options.php… Mais s’il ne l’est pas, le define() doit être défini dans le inc/truc, action/truc, etc, ce qui permet bien à d’autres de le définir en amont dans leur options.php.

  D’ailleurs placido a donné des exemples précis, en parlant du plugin Album. Est-ce que celui-ci utilise ces deux define() dans le code de son options.php ? Si ce n’est pas le cas c’est lui qui doit être modifié pour définir ces variables ailleurs.
  La réponse est là :
  https://zone.spip.org/trac/spip-zone/browser/_plugins_/albums/trunk/albums_options.php

  Comme vous le voyez, AUCUNE de ces variables n’est utilisée à cette endroit là. Donc c’est le plugin Albums qui ne fait pas bien les choses. Il faut absolument déplacer l’ensemble de ces variables dans les fichiers où elles sont vraiment utilisés.

  Et du coup c’est fini, problème résolu, c’est le plugin source qui était en problème, et une fois corrigé placido n’a plus de problème à définir ces valeurs avant.

  Les cas où des define() sont définis ET utilisés directement dans le options.php sont méga méga rares, et je crois même qu’il n’y a que Bonux qui fait ça, pour la prévisu temporaire :
  https://zone.spip.org/trac/spip-zone/browser/_plugins_/spip-bonux-3/spip_bonux_options.php#L25
  (et ça m’avait bien saoulé pour le surcharger, j’avais fini par le définir dans les mes_options.php du projet alors que je ne l’utilise jamais et que je voulais le faire dans le plugin de mon projet)

  (Par ailleurs, dans tous les cas ce serait bien qu’il y ait un pipeline/trigger au tout début de SPIP, j’avais déjà eu le besoin pour faire des choses avant la connexion/cookie etc, pour connecter les gens par Facebook ou autre truc extérieur au tout démarrage, et pour le moment j’avais dû le faire dans mon action PHP à moi, au lieu de pouvoir le faire dans un truc générique qui aurait valu pour n’importe quel hit PHP, et du coup j’ai jamais pu en faire un plugin générique. Mais faudrait peut-être faire un ticket dédié pour demander cet ajout.)