Skip to content

mes_favoris bridé et catégories inabouties

Il me semble que la gestion des catégories n'est pas au point, notamment à cause d'un manque dans mesfavoris_trouver

Dans https://git.spip.net/spip-contrib-extensions/mesfavoris/-/blob/master/inc/mesfavoris.php?ref_type=heads, certaines fonctions

  • utilisent les catégories si on les spécifie (mesfavoris_supprimer, mesfavoris_ajouter),
  • d'autres les ignorent : mesfavoris_trouver car « sinon le bouton d'ajout ne voit pas que ca existe » ; et par conséquent également mesfavoris_categoriser qui utilise mesfavoris_trouver.

De ce fait, quand des catégories sont utilisées, mesfavoris_trouver ne renvoie qu'un seul favori alors qu'il peut y en avoir plusieurs répondant aux critères mais avec des catégories différentes, et donc mesfavoris_categoriser fait un peu n'importe quoi dans ce cas là.

L'ignorance par mise en commentaire du critère portant sur la catégorie dans mesfavoris_trouver a été faite par e2f1fd7b dont le log parle d'un bug mais sans le référencer.

Dès lors qu'il y a des catégories mais qu'elles ne sont pas prises en compte pour trouver un favori... ya clairement un problème.

Si le problème est relatif au « bouton d'ajout » qui ne « voit pas que ça existe » c'est peut être plutôt au niveau de ce bouton qu'il faut intervenir ?

À part ça, il serait peut être pas inutile nécessaire de pouvoir paramétrer la nécessité d'une catégorie : faut-il absolument une catégorie (non vide donc) à chaque favori ? / ou bien il n'y a pas de catégories définies ? / ou bien il y a des catégories mais elles sont facultatives ? Car ça influe peut influencer sur le sens d'une opération quand on ne précise pas un favori.