[SPIP Zone] r118048 - in _plugins_/mesfavoris/trunk

Oubli :
ce commit n'actualise pas paquet.xml

Et je suis (un peu) perplexe car il modifie (un peu)
le html de présentation inclure/mesfavoris.html
https://zone.spip.net/trac/spip-zone/changeset/118048/spip-zone#file7
donc si pas de chance ça bouscule une page.
C'est la vie ?
JL

Le 07/10/2019 à 19:15, spip-zone-commit@rezo.net a écrit :

Author: gildas.cotomale@gmail.com
Date: 2019-10-07 17:15:31 +0000 (Mon, 07 Oct 2019)
New Revision: 118048
Added:
    _plugins_/mesfavoris/trunk/action/categoriser_favori.php
    _plugins_/mesfavoris/trunk/formulaires/editer_favori_categorie.html
    _plugins_/mesfavoris/trunk/formulaires/editer_favori_categorie.php
    _plugins_/mesfavoris/trunk/images/favoris-edit-16.png
    _plugins_/mesfavoris/trunk/inclure/mes_fav_cat_c.html
    _plugins_/mesfavoris/trunk/inclure/mes_fav_cat_g.html
Modified:
    _plugins_/mesfavoris/trunk/inc/mesfavoris.php
    _plugins_/mesfavoris/trunk/inclure/mesfavoris.html
    _plugins_/mesfavoris/trunk/lang/mesfavoris_fr.php
Log:
prise en compte de la categorie
(au passage je note un bogue en cours d investiguation)

Details: Connexion · GitLab

Le lun. 7 oct. 2019 22:45, JLuc a écrit :

Oubli :
ce commit n’actualise pas paquet.xml

Oui, je veux éviter de régénérer un zip tant que je ne suis pas sûr d’avoir laissé de coquille…

Et je suis (un peu) perplexe car il modifie (un peu)
le html de présentation inclure/mesfavoris.html
https://zone.spip.net/trac/spip-zone/changeset/118048/spip-zone#file7
donc si pas de chance ça bouscule une page.
C’est la vie ?

Un oubli de ma part ; ce n’est l’objet de ce commit, mais un bug à corriger quand même… C’était un des premiers fichiers que j’ai modifié parce-que ayant eu la bonne idée de faire trois inclusions dans une même page et me suis retrouvé avec autant de fois cet id (plus une classe de ce nom trop générique.)

JL

Le 07/10/2019 à 23:27, Gildas Cotomale a écrit :

    Et je suis (un peu) perplexe car il modifie (un peu)
    le html de présentation inclure/mesfavoris.html
    https://zone.spip.net/trac/spip-zone/changeset/118048/spip-zone#file7
    donc si pas de chance ça bouscule une page.
Un oubli de ma part ; ce n'est l'objet de ce commit, mais un bug à corriger quand même... C'était un des premiers fichiers que j'ai modifié parce-que ayant eu la bonne idée de faire trois inclusions dans une même page et me suis retrouvé avec autant de fois cet id (plus une classe de ce nom trop générique.)

Ah Ok.
J'ai aussi dû surcharger cette noisette ainsi que le fichier mesfavoris-32.css bizarrement à la racine.
J'ai eu l'impression que c'était une base sur laquelle construire plutôt qu'une soluce clé en main.

J'ai

Le mar. 8 oct. 2019 00:16, JLuc a écrit :

Le 07/10/2019 à 23:27, Gildas Cotomale a écrit :

Et je suis (un peu) perplexe car il modifie (un peu)

le html de présentation inclure/mesfavoris.html
https://zone.spip.net/trac/spip-zone/changeset/118048/spip-zone#file7
donc si pas de chance ça bouscule une page.
Un oubli de ma part ; ce n’est l’objet de ce commit, mais un bug à corriger quand même… C’était un des premiers
fichiers que j’ai modifié parce-que ayant eu la bonne idée de faire trois inclusions dans une même page et me suis
retrouvé avec autant de fois cet id (plus une classe de ce nom trop générique.)

Ah Ok.
J’ai aussi dû surcharger cette noisette ainsi que le fichier mesfavoris-32.css bizarrement à la racine.
J’ai eu l’impression que c’était une base sur laquelle construire plutôt qu’une soluce clé en main.

Je me doute que c’est une base de travail, mais la nature humaine est telle que beaucoup feront juste l’inclusion indiquée sur la page Contrib au lieu de surcharger (j’en fais partie, j’ai opté pour la facilité d’abord…) Il vaut donc mieux que je revert ce fichier en attendant, pour ne pas casser les sites en prod (le plugin étant marqué stable, on leur en voudra pas de mettre à jour sans test sauf si la version majeure change)

J’ai


spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

Le 08/10/2019 à 00:45, Gildas Cotomale a écrit :

Je me doute que c'est une base de travail, mais la nature humaine est telle que beaucoup feront juste l'inclusion indiquée sur la page Contrib au lieu de surcharger (j'en fais partie, j'ai opté pour la facilité d'abord...) Il vaut donc mieux que je revert ce fichier en attendant, pour ne pas casser les sites en prod (le plugin étant marqué stable, on leur en voudra pas de mettre à jour sans test sauf si la version majeure change)

Peut être serait il possible d'en mettre une version améliorée dans un sous répertoire ?
Et aussi y mettre mesfavoris-32.css et autres éléments de présentation ?
Mais je crois me souvenir que ça nécessite de changer (un peu) plus largement le code...
et c'est aussi ce qui, dans mon souvenir, me l'a rendu peu pratique à personnaliser, ce plugin.

Alors une nouvelle branche du plugin ?
Surtout si tu envisageais des améliorations plus radicales qui pourraient être bienvenues.

JL

Le mar. 8 oct. 2019 08:38, JLuc a écrit :

Peut être serait il possible d’en mettre une version améliorée dans un sous répertoire ?
Et aussi y mettre mesfavoris-32.css et autres éléments de présentation ?

Pas une mauvaise idée, une branche pour de nouvelles noisettes et un changement de la présentation (carrément une branche et non juste un dossier qui porterait un mauvais nom…)

Par contre je ne me suis pas encore penché sur l’habillage (et je trouve qu’effectivement il y a de petites choses à faire.)
Mon urgence était de permettre un petit contrôle des catégories (pour des choses plus poussées il faudra un vrai plugin annexe comme le dit la doc, mais déjà pouvoir éditer les catégories existantes me semblait un minimum que j’ai voulu rajouter.)

Mais je crois me souvenir que ça nécessite de changer (un peu) plus largement le code…
et c’est aussi ce qui, dans mon souvenir, me l’a rendu peu pratique à personnaliser, ce plugin.

Il y a donc de l’intérêt pour une nouvelle branche/version :slight_smile: Dans l’immédiat, comme je suis focus sur mon projet j’ai peur de ne pas avoir assez de recul et de me retrouver à imposer mes goût au lieu de proposer une base qui conviennent à une majorité :-/ (déjà pour l’édition de catégories je ne suis pas certain d’avoir fait au mieux)

Le 08/10/2019 à 09:40, Cerdic a écrit :

Il renomme juste l’ID pour éviter les collisions si on a plusieurs listes dans la même page et deplace l’ancien id dans
les classes.
Ça semble donc légitime, et il faut juste modifier tes selecteurs CSS.
Mais comme ça casse un peu la présentation/compat, ça mériterait un incrément du y a minima

Je vois tardivement la réponse ; j’ai entre temps « revert » ce fichier et fait un z+1

J’attends quand même des avis de testeurs sur mon « poc » :slight_smile:

Le 08/10/2019 à 20:39, Gildas Cotomale a écrit :

Mon urgence était de permettre un petit contrôle des catégories (pour
des choses plus poussées il faudra un vrai plugin annexe comme le dit la
doc, mais déjà pouvoir éditer les catégories existantes me semblait un
minimum que j'ai voulu rajouter.)

Yo, je sais pas trop vraiment à quoi ça sert les "catégories", mais on
avait fait un vrai plugin extension permettant à chacun de classer ses
favoris en plusieurs listes différentes : Collections de favoris
https://websvn.spip.net/listing.php?repname=Zone&path=%2F_plugins_%2Fmesfavoris_collections%2Ftrunk%2F&#a5542cb27dce2eb5a52629c102d89594f

Ça marche très bien, et ça fournit tous les formulaires nécessaires à
donner aux gens. Et c'est carré, des vrais id sql, de noms humains qui
servent pas d'identifiant, une vraie structure de données faite pour ça
quoi. :slight_smile:

Et oui : c'est pas documenté alors que c'est en prod depuis trois ans. :frowning:

--
RastaPopoulos

Le mar. 8 oct. 2019 21:44, RastaPopoulos a écrit :

Le 08/10/2019 à 20:39, Gildas Cotomale a écrit :

Mon urgence était de permettre un petit contrôle des catégories (pour
des choses plus poussées il faudra un vrai plugin annexe comme le dit la
doc, mais déjà pouvoir éditer les catégories existantes me semblait un
minimum que j’ai voulu rajouter.)

Yo, je sais pas trop vraiment à quoi ça sert les « catégories »,

Alors, qu’est-ce donc ? D’après la doc <https://contrib.spip.net/Mes-Favoris-2-0#Developpement-categories-de-favoris> : « Depuis 2013 et la version 2.4.2, encore considérée comme « en test », les favoris peuvent être classés en catégories. » Techniquement, en regardant les sources, c’est le typage de la liaison (mais le champ s’appelle bien « categorie » et non « type » comme d’habitude.)

La même section de la doc poursuit par : « Le plugin ne fournit cependant pas l’interface utilisateur pour cela. »
Or la section précédente <https://contrib.spip.net/Mes-Favoris-2-0#Liste-des-favoris> indique que « Le squelette inclure/mesfavoris.html peut être utilisé pour afficher la liste des favoris d’un auteur. » et que « La liste affichée comporte sur chaque élément un bouton permettant de le retirer des favoris. »
Constatant donc une mini gestion de la liste de favoris, je me suis fait la remarque qu’elle pourrait inclure un chouïa de gestion de catégories via une liste déroulante (définie ailleurs par les webmestres ou un autre plugin par exemple) ou un champ texte libre. Après avoir fait mumuse, j’ai voulu partager sur la zone pour que ça profite à la communauté (j’ai vu en plus qu’il y a quelques questions dans le forum) et soit amélioré si besoin.

Voilà pour la petite histoire. ^^
Msis comme dit la doc <https://contrib.spip.net/Mes-Favoris-2-0#Developpement-categories-de-favoris> ; il faut une vraie extension dédiée que je ne prétends pas remplacer/introduire ici ! « C’est un autre plugin qui doit coder une interface adaptée aux besoins du site, afin de permettre à l’utilisateur de définir et/ou utiliser ces catégories. » Peut-être est-ce ce plugin attendu/espéré ? (dans ce cas, il ne faut pas hésiter à le zip’er et le référencer sur plugins.spip.net, surtout que c’est éprouvé depuis plus/près de 36 mois)

mais on
avait fait un vrai plugin extension permettant à chacun de classer ses
favoris en plusieurs listes différentes : Collections de favoris
https://websvn.spip.net/listing.php?repname=Zone&path=%2F_plugins_%2Fmesfavoris_collections%2Ftrunk%2F&#a5542cb27dce2eb5a52629c102d89594f

Ça marche très bien, et ça fournit tous les formulaires nécessaires à
donner aux gens. Et c’est carré, des vrais id sql, de noms humains qui
servent pas d’identifiant, une vraie structure de données faite pour ça
quoi. :slight_smile:

Et oui : c’est pas documenté alors que c’est en prod depuis trois ans. :frowning:

Je vais tester ça. Merci pour la découverte ; je suis passé à côté à cause de l’absence de zip.

Le 09/10/2019 à 00:40, Gildas Cotomale a écrit :

Techniquement, en regardant les sources, c'est le typage de la liaison
(mais le champ s'appelle bien "categorie" et non "type" comme d'habitude.)

Oui ok donc c'est plus comme ça que je le voyais, une sorte de
typage/tagage avec donc des types définis en amont avant par les admins,
surtout que c'est juste un mini varchar, et que ça peut que être un
identifiant, et que du coup t'as même pas de correspondance avec un vrai
label humain.

Bref, donc c'est pas vraiment la même fonction que les collections, qui
sert vraiment à ce que chaque personne puisse ranger ses favoris en
plusieurs listes, plusieurs collections, au lieu d'une seule.

--
RastaPopoulos

Le 09/10/2019 à 10:27, RastaPopoulos a écrit :

Le 09/10/2019 à 00:40, Gildas Cotomale a écrit :

Techniquement, en regardant les sources, c'est le typage de la liaison
(mais le champ s'appelle bien "categorie" et non "type" comme d'habitude.)

Oui ok donc c'est plus comme ça que je le voyais, une sorte de
typage/tagage avec donc des types définis en amont avant par les admins,
surtout que c'est juste un mini varchar, et que ça peut que être un
identifiant, et que du coup t'as même pas de correspondance avec un vrai
label humain.

Bref, donc c'est pas vraiment la même fonction que les collections, qui
sert vraiment à ce que chaque personne puisse ranger ses favoris en
plusieurs listes, plusieurs collections, au lieu d'une seule.

Du coup on a
mesfavoris qui propose un champ "categorie" en varchar 25
mais sans interface pour la gérer

Et mesfavoris_collection qui propose une gestion complète de catégories
définies par chaque utilisateur.

C'est cela ?

Du coup hier on évoquait la possibilité de supprimer le champ categorie
du plugin de base, ou de le transformer en MEDIUMTEXT
et le renommer "notesperso" ou "descriptif" ou "texte"...
pour permettre à un utilisateur de qualifier ses favoris.
Ainsi, par exemple dans un site d'offres d'emploi,
une personne pourrait gérer l'état et le suivi de sa candidature...
Sur un site de vente d'imprimantes,
une personne pourrait noter ses synthèses d'analyse des tests lus dans la presse...
avant de se décider.

Utile donc sur un site web ou le LIKE n'est pas qu'une action réflexe
ne mobilisant que le cerveau reptilien de l'internaute.

JL

Le mer. 9 oct. 2019 10:27, RastaPopoulos a écrit :

Le 09/10/2019 à 00:40, Gildas Cotomale a écrit :

Techniquement, en regardant les sources, c’est le typage de la liaison
(mais le champ s’appelle bien « categorie » et non « type » comme d’habitude.)

Oui ok donc c’est plus comme ça que je le voyais, une sorte de
typage/tagage avec donc des types définis en amont avant par les admins,

C’est une des possibilités que devrait proposer un plugin de gestion dédié (certains sites voudront des types inflexibles définis par eux, mais gérable facilement depuis le Privé par les admins …ou alors une constante définie par les webmestres ?) Mais pas que celle là (d’autres sites voudront que chaque utilisateur puisse gérer --dans le Privé ou mieux le Public-- ses propres listes) ; et mon commit initial allait dans ce sens (mon besoin actuel mais surchargeable.)

surtout que c’est juste un mini varchar, et que ça peut que être un
identifiant, et que du coup t’as même pas de correspondance avec un vrai
label humain.

Après effectivement ça paraît court et j’ai augmenté un peu (à 99 carrément) même si je doute un peu du besoin (ou plutôt je n’en ai pas encore l’utilité et ne connais personne qui fasse des « tag » de plus de quinze caractères)

Il n’y a pas de restriction de type de caractères ou autre (sauf effectivement la taille) donc on peut avoir des trucs humains (nonCryptique ou-autre_truc si c’est ce que tu voulais dire)

Bref, donc c’est pas vraiment la même fonction que les collections, qui
sert vraiment à ce que chaque personne puisse ranger ses favoris en
plusieurs listes, plusieurs collections, au lieu d’une seule.

On peut avoir plusieurs listes (enfin tags/catégories) et le même objet peut être en favori dans plusieurs : pas vu cette limitation (mais comme c’était pas géré ça s’est ressenti sur les formulaires/boutons par défaut --d’où mon second commit)

Je viens de récupérer favoris_collections (mais pas eu le temps de pousser le commit préparé pour aujourd’hui) Soirée studieuse en vue ^^

Le mer. 9 oct. 2019 19:52, JLuc a écrit :

Le 09/10/2019 à 10:27, RastaPopoulos a écrit :

Le 09/10/2019 à 00:40, Gildas Cotomale a écrit :

Techniquement, en regardant les sources, c’est le typage de la liaison
(mais le champ s’appelle bien « categorie » et non « type » comme d’habitude.)

Oui ok donc c’est plus comme ça que je le voyais, une sorte de
typage/tagage […]
surtout que c’est juste un mini varchar, et que ça peut que être un
identifiant, et que du coup t’as même pas de correspondance avec un vrai
label humain.

Bref, donc c’est pas vraiment la même fonction que les collections, qui
sert vraiment à ce que chaque personne puisse ranger ses favoris en
plusieurs listes, plusieurs collections, au lieu d’une seule.

Du coup on a
mesfavoris qui propose un champ « categorie » en varchar 25
mais sans interface pour la gérer

Exact.

Et mesfavoris_collection qui propose une gestion complète de catégories
définies par chaque utilisateur.

C’est cela ?

Exactement ce que je comprends (et vais découvrir ce soir si pas d’imprévu) Et ça étend mesfavoris (d’après le plugin.xml) pour offrir la gestion sous forme de « collections » par utilisateur(e).

Du coup hier on évoquait la possibilité de supprimer le champ categorie
du plugin de base, ou de le transformer en MEDIUMTEXT
et le renommer « notesperso » ou « descriptif » ou « texte »…
pour permettre à un utilisateur de qualifier ses favoris.
Ainsi, par exemple dans un site d’offres d’emploi,
une personne pourrait gérer l’état et le suivi de sa candidature…
Sur un site de vente d’imprimantes,
une personne pourrait noter ses synthèses d’analyse des tests lus dans la presse…
avant de se décider.

Une dimension à laquelle je n’avais pas pensé et qui du coup justifie la remarque de RastaPopoulos sur la petitesse d’un champ de 25c.

Utile donc sur un site web ou le LIKE n’est pas qu’une action réflexe

Nous avons la même problématique ^^
Encore que, pour du simple j’aime je préfère utiliser le plugin Notations :wink: