Recherche avancée

Sur d’autres sites (8402)

• Anomalie #2307 : md5_boutons_plugins()

  21 juillet 2014, par marcimat ☺☮☯♫

  Et les traces pour voir plus clairement les 2 créations du même define (_UPDATED_boutons_plugins) :

  Premier passage (constante non définie)¶

  array (size=5)
    0 => 
      array (size=4)
        ’file’ => string ’[...]spip/ecrire/inc/plugin.php’ (length=57)
        ’line’ => int 636
        ’function’ => string ’plugin_ongletbouton’ (length=19)
    1 => 
      array (size=4)
        ’file’ => string ’[...]spip/ecrire/inc/plugin.php’ (length=57)
        ’line’ => int 530
        ’function’ => string ’plugins_precompile_xxxtions’ (length=27)
    2 => 
      array (size=4)
        ’file’ => string ’[...]spip/ecrire/inc/plugin.php’ (length=57)
        ’line’ => int 431
        ’function’ => string ’ecrire_plugin_actifs’ (length=20)
    3 => 
      array (size=4)
        ’file’ => string ’[...]spip/ecrire/inc_version.php’ (length=58)
        ’line’ => int 391
        ’function’ => string ’actualise_plugins_actifs’ (length=24)
    4 => 
      array (size=4)
        ’file’ => string ’[...]spip/ecrire/index.php’ (length=52)
        ’line’ => int 21
        ’function’ => string ’include’ (length=7)
  

  Second passage (constante déjà définie)¶

  array (size=5)
    0 => 
      array (size=4)
        ’file’ => string ’[...]spip/ecrire/inc/plugin.php’ (length=57)
        ’line’ => int 636
        ’function’ => string ’plugin_ongletbouton’ (length=19)
    1 => 
      array (size=4)
        ’file’ => string ’[...]spip/ecrire/inc/plugin.php’ (length=57)
        ’line’ => int 530
        ’function’ => string ’plugins_precompile_xxxtions’ (length=27)
    2 => 
      array (size=4)
        ’file’ => string ’[...]spip/ecrire/inc/plugin.php’ (length=57)
        ’line’ => int 431
        ’function’ => string ’ecrire_plugin_actifs’ (length=20)
    3 => 
      array (size=4)
        ’file’ => string ’[...]spip/plugins-dist/svp/exec/admin_plugin.php’ (length=74)
        ’line’ => int 52
        ’function’ => string ’actualise_plugins_actifs’ (length=24)
    4 => 
      array (size=4)
        ’file’ => string ’[...]spip/ecrire/index.php’ (length=52)
        ’line’ => int 151
        ’function’ => string ’exec_admin_plugin_dist’ (length=22)

• Anomalie #3799 : appliquer_filtre ne s’applique pas aux filtres image_

  10 février 2017

  L’intention que tu cites n’était peut être pas forcément le but de la fonction appliquer_filtre() à l’origine.

  Bon, j’ai un problème. Comme je disais en #note-7 il y a un bug sur appliquer filtre et l’option qui avait été ajoutée en r71781.

  En plein dans le bug¶

  Il y a un cas intéressant sur la zone dans SarkaSPIP, parmi d’autres :

  [(#ENVid_rubrique|appliquer_filtreaccesrestreint_rubrique_restreinte,0|oui) #SETretour_logout, #URL_PAGEsommaire]
  function accesrestreint_rubrique_restreinte($id_rubrique, $id_auteur = null) 
  
  

  Le zéro passé en second paramètre, on ne sait pas s’il était
  - à destination de appliquer_filtre() pour éviter absolument qu’il retourne la valeur id_rubrique SI accès restreint est absent, ou s’il était
  - à destination de accesrestreint_rubrique_restreinte() pour indiquer l’auteur 0 (ce qui est peu probable)
  Et du coup, ça casse le fonctionnement de accesrestreint_rubrique_restreinte() car ça ne prend pas en compte le visiteur courant car $id_auteur ne vaut plus null

  À d’autres endroits au contraire, par exemple dans Emballe Medias, on trouve :
  

  [(#INCLUREjavascript/jquery.shiftcheckbox.js|appliquer_filtreminifier,js)]
  

  Ici, la présence de l’option ’js’ fait que appliquer_filtre va comprendre qu’il doit toujours retourner le contenu d’origine même en absence du filtre.
  C’est d’ailleurs très certainement ce qui est souhaité ici. Sauf que ce n’est pas voulu explicitement a priori.

  On trouve aussi un usage avec |sinon :
  

  ["geometry" : (#GEOMETRY|appliquer_filtrewkt_to_json|sinon"type" : "Point", "coordinates" : [#LON, #LAT]),]
  

  Corrections de appliquer_filtre en créant une seconde fonction ?¶

  Ces 2 derniers exemples montrent que quelque soit l’option choisie pour corriger on va casser des usages (en créant une seconde fonction pour faire l’inverse), ie :

  Option Historique)
  - T|appliquer_filtre{F} retourne ’’ si F introuvable
  - T|appliquer_filtre_ou_continuer retourne T si F introuvable

  Option Chaînage)
  - T|appliquer_filtre{F} retourne T si F introuvable
  - T|appliquer_filtre_sinon_rien{F} retourne ’’ si F introuvable

  Ou déprécier appliquer_filtre()¶

  Une autre solution est de créer 2 nouvelles fonctions pour remplacer appliquer_filtre et son fonctionnement incertain.
  Contrainte : elles ne doivent pas commencer par filtre_.
  - T|utiliser_filtre{F} retourne T si F introuvable (chaînage)
  - T|utiliser_filtre_sinon_rien{F} retourne ’’ si F introuvable

  C’est peut être une solution acceptable. On aurait du être plus vigilent en introduisant une nouvelle fonction plutôt qu’un paramètre en plus à appliquer_filtre() alors que celle ci utilise func_get_args…

• Evolution #4487 : Une option sur image_recadre_mini pour vraiment recadrer au mini (et sans garder...

  15 février 2021, par cedric -

  bon j’ai relu tout le code à tête reposé et donc il y a pas de bug, ou plutot la doc est pas claire : la fonction recadre uniquement de manière symétrique haut vs bas et gauche vs droite, pour garder le centre de l’image au centre.
  Donc si tu as un pixel non transparent en 1ère ligne de l’image, le bas ne sera pas recadré non plus.

  Il faudrait donc ajouter une option pour recadrer effectivement au plus juste, indépendamment du centre, mais du coup c’est une évolution et c’est pour plus tard
  (sauf si tu veux proposer une PR prête à l’intégration)