Recherche avancée

Sur d’autres sites (9969)

• Evolution #4548 : Améliorer _request et set_request pour prendre en compte les champs qui sont des...

  10 septembre 2020, par RastaPopoulos ♥

  @marcimat ? "n’a pas beaucoup de sens" ? => c’est la syntaxe officielle normalisée de HTML ! Ça a d’après moi encore plus de sens de gérer cette écriture, que de gérer une écriture personnalisée (à base de / chez nous, de . chez laravel), qui est très bien aussi et qui devrait être fait aussi d’après moi, mais qui mais nous est propre, en interne. Alors que la première écriture ne nous est pas propre : c’est la norme HTML !
  Les deux fonctions request (_request et set_request) sont directement liées aux formulaires HTML, dont la norme de nom de champ est "tableau[index]" et non pas "tableau/index", donc la priorité c’est de prendre en compte l’écriture officielle directement liée.

  Et comme je le dis plus haut, et contrairement à ce qui se discutait dans #4110 : ça n’a aucune spécificité à Saisies. On peut stocker la liste des noms des champs pour d’autres raisons, et on a aussi des squelettes à inclure et des filtres PHP dans le noyau qui génèrent des champs devant leur passer "nom" ou "name" en param (pour id_parent, pour sélectionner rubrique, article…) et qu’on peut parfaitement vouloir poster dans un sous-tableau (ya plein de raisons valables et utiles, comme éditer un objet dans un autre, une géoloc, des infos de commerce, etc, des milliers - si :p - d’autres CMS font ça depuis toujours : patate[gis][lat]… patate[prix][taxe]…).

  Quand ensuite on veut récupérer ou modifier ces valeurs postées en masse automatiquement, car on connait les noms de chacun, sans faire un traitement à la main particulier, bah ce qu’on connait c’est "patate[gis][lat]", donc autant pouvoir l’utiliser directement.

  Après si ça ne prend en compte que "tableau/index" mais que le core fournit aussi la transformation et qu’il faille écrire _request(name2chemin('patate[gis][lat]')) plutôt que directement _request('patate[gis][lat]'), ça serait déjà ça puisque ça permet l’automatisation aussi… (mais dans ce cas pourquoi ne pas l’appeler directement en interne, ça ne coûte rien du tout de plus !)

  Au niveau du code, ya rien de compliqué à gérer dans ce cas, rien de "ça va être chiant", puisque SPIP a déjà tout ce qu’il faut pour chercher une valeur suivant un "chemin". Il y a uniquement 2 choses à faire :
  - avoir une fonction mutualisée qui transforme la norme HTML en chemin de chez nous (avec / donc) et appeler cette fonction au tout début des deux fonctions request
  - dans les deux fonctions chercher table_valeur($tableau, $var) plutôt que directement $_GET[$var], $_POST[$var] etc

  Ainsi dans _request() avant c’était :
  

  if (is_array($c)) 
      return isset($c[$var]) ? $c[$var] : null ;
  
  if (isset($_GET[$var])) 
  
      $a = $_GET[$var] ;
  
   elseif (isset($_POST[$var])) 
  
      $a = $_POST[$var] ;
  
   else 
  
      return null ;
  
  
  

  Et alors ça serait environ :
  

  $chemin = name2chemin($var) ;
  if (is_array($c)) 
  
      return table_valeur($c, $chemin) ;
  
  
  
  if ($valeur=table_valeur($_GET, $chemin) !== null) 
  
      $a = $valeur ;
  
   elseif (…
  

  Quasiment rien qui change !

• Anomalie #4561 : Version php min

  1er octobre 2020

  Oui, j’ai vu et corrigé en version 4.1.1 de spip_loader. Cependant ça ne semble pas suffire pour que spip_loader fonctionne réellement en PHP 5.3 (ce que j’ai testé : j’ai plein d’erreurs sur la récupération du fichier spip_loader_list.json régulièrement et d’autres). Je n’ai pas envie de passer plus de temps que ça dessus, parce que PHP < 5.4 ça devient difficile.
  Mais si quelqu’un veut se pencher sur le problème !

  Un des soucis que j’ai vu c’est que le loader charge spip_loader_list.json à chaque hit (même lorsqu’il décompacte le zip), et c’est pas très efficace. Il faudrait le copier localement au début, et le supprimer à la fin, déjà, par exemple, comme les autres fichiers de langue et pclzip...

• Anomalie #4543 : Accessibilité des chargements ajax (live regions)

  9 septembre 2020, par nicod _

  Au-delà de la recommandation, l’explication précise c’est quoi derrière ? Quand l’attribut est sur le parent, ça fait que le bloc rechargé entier est lu (sans pouvoir être arrêté ?) par le lecteur ?

  C’est ce que j’en ai compris, de mon côté j’ai du mal à tester, je n’ai que VoiceOver sur Safari, il faudrait que j’arrive à installer JAWS.

  Donc pour tout rechargement ajax suite à un clic (liens rechargeant un bloc, ou validation de formulaire), il faudrait retirer l’attribut et "juste" placer le focus au début du bloc. Est-ce déjà le cas aussi ?

  C’est bien ce que je comprends des recos, oui, attribut aria sur le premier résultat et redonner le focus sur cet élément.

  C’est peut être automatisable en JS, si on détecte qu’il y a une pagination avec $(’.pagination).length, pour placer dynamiquement l’attribut et le focus depuis un ajaxLoad().