changer de fournisseur de vignette ou les supprimer #4124

Closed
opened 5 years ago by b_b · 10 comments
b_b commented 5 years ago
Owner

apercite.fr est mort et on l'utilise ici : https://zone.spip.org/trac/spip-zone/browser/core/plugins/statistiques/prive/squelettes/contenu/stats_referers_fonctions.php#L15

Il faudrait basculer sur un autre service ou simplement supprimer ces vignettes (perso je penche pour la seconde option, c'est jamais bon de dépendre d'un service externe). Piste possible, easy-thumb, cf https://zone.spip.org/trac/spip-zone/changeset/109815

apercite.fr est mort et on l'utilise ici : https://zone.spip.org/trac/spip-zone/browser/_core_/plugins/statistiques/prive/squelettes/contenu/stats_referers_fonctions.php#L15 Il faudrait basculer sur un autre service ou simplement supprimer ces vignettes (perso je penche pour la seconde option, c'est jamais bon de dépendre d'un service externe). Piste possible, easy-thumb, cf https://zone.spip.org/trac/spip-zone/changeset/109815
Owner

Salut

C'est le but de thumbsite https://contrib.spip.net/Thumbsites
Le plugin avait été fait dans l'optique de retirer à terme la
fonctionnalité du core et d'éviter ce genre de cas de figure.

Salut C'est le but de thumbsite https://contrib.spip.net/Thumbsites Le plugin avait été fait dans l'optique de retirer à terme la fonctionnalité du core et d'éviter ce genre de cas de figure.
b_b commented 5 years ago
Poster
Owner

Donc tu serais aussi d'avis de retirer ces vignettes de la page ?

À propos de thumbsite, je vois qu'il utiliser aussi apercite et robothumb, ce qui ne règle pas la question de la dépendance à un service externe.

Donc tu serais aussi d'avis de retirer ces vignettes de la page ? À propos de thumbsite, je vois qu'il utiliser aussi apercite et robothumb, ce qui ne règle pas la question de la dépendance à un service externe.

Ce que je comprends c'est que si on retire du core, le plugin pourrait venir remplacer la fonctionnalité perdue ?
Ça déplace le problème dans le plugin et c'est certainement mieux. Ça reste au plugin d'évoluer...

Ce que je comprends c'est que si on retire du core, le plugin pourrait venir remplacer la fonctionnalité perdue ? Ça déplace le problème dans le plugin et c'est certainement mieux. Ça reste au plugin d'évoluer...
b_b commented 5 years ago
Poster
Owner

Statut changé à En cours

**Statut changé à En cours**

Dans tous les cas ce service, à moins de monter à part un serveur exprès avec un logiciel libre qui ferait ça, c'est forcément avec un service externe. Mais le plugin propose plusieurs fournisseur et peut évoluer et être mis à jour plus vite quand il y a des changements.

Par contre si on l'enlève du noyau enfin plugins-dist, il faut que le plugin Thumb remette bien les vignettes au même endroit dans les stats aussi (je ne sais plus ce qu'il fait, s'il fournit juste des API ou quoi).

Et donc oui aussi pour l'enlever de plugins-dist.

Dans tous les cas ce service, à moins de monter à part un serveur exprès avec un logiciel libre qui ferait ça, c'est forcément avec un service externe. Mais le plugin propose plusieurs fournisseur et peut évoluer et être mis à jour plus vite quand il y a des changements. Par contre si on l'enlève du noyau enfin plugins-dist, il faut que le plugin Thumb remette bien les vignettes au même endroit dans les stats aussi (je ne sais plus ce qu'il fait, s'il fournit juste des API ou quoi). Et donc oui aussi pour l'enlever de plugins-dist.
Owner

Salut

Par défaut je ne pense que la fonctionnalité des aperçus doit être
gérée dans SPIP même. Comme noté on a entre autres raisons la
dépendance à service tiers non maîtrisés.
Je ne pense pas non plus qu'on doive fournir un service spip sur ce
sujet, comme on peut le voir avec la danse des services existants
c'est une plaie à maintenir.

Le fait de gérer ceci via un plugin simplifie la mise à jours des
serveurs de vignette. Thumbsite a été codé pour permettre l'ajout et
le retrait simple d'un générateur d'aperçu. Donc si easy-thumb est
manquant c'est facile à faire.

Pour ma part par défaut, on devrait avoir l'option 2 : retirer la
gestion des vigenttes. On laisserait 2 plugins faire mieux si besoin
(un pour les aperçus sans présupposer de leur usages et un pour
surcharger statistiques si besoin)

Salut Par défaut je ne pense que la fonctionnalité des aperçus doit être gérée dans SPIP même. Comme noté on a entre autres raisons la dépendance à service tiers non maîtrisés. Je ne pense pas non plus qu'on doive fournir un service spip sur ce sujet, comme on peut le voir avec la danse des services existants c'est une plaie à maintenir. Le fait de gérer ceci via un plugin simplifie la mise à jours des serveurs de vignette. Thumbsite a été codé pour permettre l'ajout et le retrait simple d'un générateur d'aperçu. Donc si easy-thumb est manquant c'est facile à faire. Pour ma part par défaut, on devrait avoir l'option 2 : retirer la gestion des vigenttes. On laisserait 2 plugins faire mieux si besoin (un pour les aperçus sans présupposer de leur usages et un pour surcharger statistiques si besoin)
b_b commented 5 years ago
Poster
Owner

Voilà un patch qui délègue la génération des vignettes au plugin thumbsite (PS : la doc du plugin indique que le filtre thumbshot renvoie le chemin de la vignette, mais c'est faux, c'est une balise img complète qui est renvoyée). Pour être complet, le patch doit aussi supprimer le fichier https://zone.spip.org/trac/spip-zone/browser/core/plugins/statistiques/prive/squelettes/contenu/stats_referers_fonctions.php

Reste un autre points à traiter : le bloc de configuration des stats affiche une mention "Lorsque vous consultez les statistiques, vous pouvez visualiser des aperçus des sites d’origine des visites". Que fait-on pour cette partie de la config, on vire ce blog où on le remplace par une mention indiquant qu'il faut utiliser le plugin thumbsites avec un lien vers https://plugins.spip.net/thumbsites.html (ce qui implique la création d'une chaîne de langue et du travail de traduction). Autre piste, on garde le bloc tel quel et on modifie son explication pour indiquer qu'il faut utiliser le plugin thumbsites (car on peut souhaiter utiliser thumbsite et ne pas avoir les vignettes des referers).

Voilà un patch qui délègue la génération des vignettes au plugin thumbsite (PS : la doc du plugin indique que le filtre thumbshot renvoie le chemin de la vignette, mais c'est faux, c'est une balise img complète qui est renvoyée). Pour être complet, le patch doit aussi supprimer le fichier https://zone.spip.org/trac/spip-zone/browser/_core_/plugins/statistiques/prive/squelettes/contenu/stats_referers_fonctions.php Reste un autre points à traiter : le bloc de configuration des stats affiche une mention "Lorsque vous consultez les statistiques, vous pouvez visualiser des aperçus des sites d’origine des visites". Que fait-on pour cette partie de la config, on vire ce blog où on le remplace par une mention indiquant qu'il faut utiliser le plugin thumbsites avec un lien vers https://plugins.spip.net/thumbsites.html (ce qui implique la création d'une chaîne de langue et du travail de traduction). Autre piste, on garde le bloc tel quel et on modifie son explication pour indiquer qu'il faut utiliser le plugin thumbsites (car on peut souhaiter utiliser thumbsite et ne pas avoir les vignettes des referers).
b_b commented 5 years ago
Poster
Owner

Patch mis à jour en incluant la proposition du commentaire précédent : ajout d'une mention et un lien vers la page du plugin Thumbsites & attribut disabled + checked à false sur la checkbox si le plugin n'est pas actif.

Patch mis à jour en incluant la proposition du commentaire précédent : ajout d'une mention et un lien vers la page du plugin Thumbsites & attribut disabled + checked à false sur la checkbox si le plugin n'est pas actif.

Bonsoir,

J'ai très bien les vignettes sur mes sites, elles ont changer de couleurs pour celle inconnu.
Si on pouvais passer les liens en relatif ça serait top pour les sites en https ;)

Mes 2 noisettes.

Bonsoir, J'ai très bien les vignettes sur mes sites, elles ont changer de couleurs pour celle inconnu. Si on pouvais passer les liens en relatif ça serait top pour les sites en https ;) Mes 2 noisettes.
b_b commented 5 years ago
Poster
Owner

Corrigé dans le trunk, amha ça n'est pas nécessaire de reporter dans la branche stable puisque apercite aurait refait surface entre temps... Je ferme, mais n'hésitez pas à commenter si vous penser qu'il faut tout de même reporter en 3.2.
Statut changé à Fermé

Corrigé dans le trunk, amha ça n'est pas nécessaire de reporter dans la branche stable puisque apercite aurait refait surface entre temps... Je ferme, mais n'hésitez pas à commenter si vous penser qu'il faut tout de même reporter en 3.2. **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No Assignees
5 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.