Recherche avancée

Sur d’autres sites (4954)

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

  19 mars 2018, par Alexis Z

  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".

• Revision 112925 : les afficher_si fonctionnent avec eval. Du coup on pourrait s’en ...

  19 décembre 2018, par maieul@… — Log

  les afficher_si fonctionnent avec eval.
  Du coup on pourrait s’en servir pour executer du code PHP et pas
  simplement faire des tests conditionnels.
  On limite les dégats en empêchant l’emploi de $ et de ; dans un
  afficher_si, sauf si entre guillemets.
  Début de tests unitaires pour saisies, parce que ce sont vraiment des
  emplois complexes.

• Anomalie #4189 : extraire_multi mélange un /li /ul final avec le de langue ajouté par code_...

  17 novembre 2018, par jluc -

  Ce bug révèle peut être un pb conceptuel dans la manière qu’a extraire_multi de parfois ajouter des div en début et fin de texte lors de ses traitements intermédiaires. Ç’a peut être été conçu à l’origine avec l’idée qu’il n’y aurait que des éléments inlines dans une chaîne traitée, et pas de balises html, alors que maintenant il y a des blocs et des balises. Et l’astuce initiale, qu’on peut qualifier de hack, se trouve devenue foireuse dans certains cas.