Revisions : SPIP

Les articles publiés sur le site

• Anomalie #4849 (Nouveau) : Lisibilité du tableau des statistiques

  12 juillet 2021, par JLuc -

  #2693 a été fermé suite à la correction du code, mais la présentation du tableau de résultat n'a pas évolué et pour le comprendre, il a fallu que j'étudie le code. Du coup j'ai compris qu'il y a un petit problème de conception que je signale ici, ainsi que le manque de clarté dans un énoncé.

  La colonne de gauche présente 2 listes à la suite.
  Ces 2 listes sont séparées par un intertitre centré "..." qui indique qu'il y a une forme d'interruption dans la continuité d'une même liste.
  Or ce n'est pas le cas puisque l'analyse du code révèle que :
  - la première liste présente les 30 articles les plus populaires
  - la 2eme liste présente la popularité des 10 articles les plus récents (classés du plus récent au plus ancien)
  L'ordonnancement étant totalement différent, il s'agit bien de 2 listes qui devraient être perçues comme totalement différentes...
  MAIS
  - il y a quand même une interférence entre ces 2 listes puisque les articles présents dans la première (les articles populaires) sont exclus de la seconde.
  - il n'y a qu'un seul titre pour ces 2 listes, qui reflète l'ambigüité du mélange (malheureusement sans l'éclairer) : « Afficher les visites pour les articles les plus populaires et pour les derniers articles publiés : « Afficher les visites pour les articles les plus populaires et pour les derniers articles publiés : »
  Mais ce n'est pas possible, de même qu'il n'est pas possible de lister dans une même liste des voitures triées par vitesse maximale et des pommes triées par teneur en sucre.

  Donc il faudrait
  - avoir un titre différent pour chacune de ces listes
  - à mon avis aussi retirer le {doublon} qui rend difficile la lecture de la 2eme liste car autant il est simple de dire "c'est la popularité des articles les plus récents" (et intéressant d'avoir une telle liste), autant il est complexe d'expliquer et de comprendre "C'est la popularité des articles les plus récents sauf s'ils font partie des 30 plus populaires du site" (et difficile à utiliser une telle information circonluée).
  - ne pas avoir de séparateur "[...]" entre ces 2 listes. Ce ne sera pas utile avec le 2eme intertitre.

  Par ailleurs le texte explicatif "Comment lire ce tableau" en bas de la colonne de droite contient une explication de ce qu'est la "popularité de l’article : (une estimation du nombre de visites quotidiennes qu’il recevra si le rythme actuel de consultation se maintient)". L'emploi du futur + l'hypothèse "si le rythme actuel de consultation se maintient" donnent des pistes incomplètes et ne permettent pas de comprendre ce que c'est.
  - Spontanément j'avais retenu le futur et compris que c'était le nombre total final de visite de cet article, à supposer que sa fréquentation s'épuise mais sur quelle durée ? ça ne voulait rien dire.
  - Maintenant je comprend que c'est une estimation du nombre de visite quotidienne à partir d'une "moyenne instantannée" de fréquentation. Mais sur quelle durée se fait cette moyenne instantanée ? La fréquentation estimée se fait elle aussi sur la base d'une moyenne instantanée pour les articles ayant plus d'une journée d'âge ou bien c'est la fréquentation de la veille qui est prise dans ce cas ? Ça ne veut pas dire grand chose encore...
  En bref : ça en dit trop ou pas assez. On peut en dire moins avec simplement "une estimation du nombre de visites quotidiennes". Mais pour en dire plus et être plus précis... faudrait donner les infos manquantes.

• Documentation #4847 : Corriger le phpdoc de medias_trouver_modele_emb

  12 juillet 2021, par JLuc -

  Ya une PR : https://git.spip.net/spip/medias/pulls/27

• Nomenclature #4846 : Commentaire de code contenant SPIP 3.3 et _LEGACY_MODE_IMAGE_DOCUMENT

  11 juillet 2021

  b b a écrit :

  Tu payes ta PR ?

  Ben justement, je peine vraiment à faire des PR sur le code de SPIP parce qu'il faut faire un fork.
  Et que j'ai toujours pas compris comment gérer plusieurs PR/fork en même temps.

  Bref, c'est une ligne de remarque, je doute que ça mérite une PR

• Documentation #4848 (Nouveau) : Documenter le choix du modele appelé

  11 juillet 2021, par JLuc -

  Expliquer comment est cherché le modèle appelé.

  - Cf marcimat pour un fichier svg : https://discuter.spip.net/t/re-spip-dev-spip-3-3-0-dev-pb-avec-certains-fichiers-svg/154565/6
  - C'est géré par la fonction medias_trouver_modele_emb (https://git.spip.net/spip/medias/src/branch/master/medias_fonctions.php#L370)
  (dont le phpdoc peut être amélioré cf #4847) et utilisé dans modeles/image_emb (https://git.spip.net/spip/medias/src/branch/master/modeles/image_emb.html), audio_emb, video_emb et file_emb.

  Encore un truc à préciser : tout passe par le modele xxxtype_emb ?

• Documentation #4847 (Nouveau) : Corriger le phpdoc de medias_trouver_modele_emb

  11 juillet 2021, par JLuc -

  Le phpDoc indique que ça retourne prioritairement :
  

  * modeles/{modele_base}_emb_html.html si il existe
  * modeles/{modele_base}_text_html.html si il existe,
  * modeles/{modele_base}_text.html si il existe,
  * modeles/{modele_base}.html sinon
  

  MAIS c'est pas clair ce que représente "_html" et "_text" ici. Au vu du code, ça retourne, par ordre de priorité :
  

  * modeles/{modele_base}_emb_{extension}.html si il existe
  * modeles/{modele_base}_emb_{mimetype}.html si il existe, dans lequel les caractères non alphanumériques du mimetype (typiquement '/') ont été remplacés par '_' (par exemple "text/css" devient "text_css")
  * modeles/{modele_base}_emb_{mimetypeprincipal}.html si il existe, dans lequel mimetypeprincipal est la partie du mimetype avant le / (par exemple text/css devient text)
  * modeles/{modele_base} sinon
  

  Le phpDoc actuel correspond donc au cas d'un fichier .html (dont le type mime est text/html). Ce serait plus parlant avec un fichier .jpg, dont l'extension est différente du 'jpeg' du mimetype : 'image/jpeg'. Je propose de remplacer le phdoc avec la version cidessus et l'exemple .jpg.