Eviter des confusions sur le sens de "utilise" #151

Merged
marcimat merged 2 commits from maieul/spip:message_plus_claire into master 3 days ago
maieul commented 2 weeks ago

SPIP distingue pour un plugin X :

  • nécessite une dépendance Y (on a absolument besoin)
  • utilise une dépendance Y (si on a, tire profit)

Mais dans le cadre d'utilise une dépendance, on vérifie quand même que
cette dépendance respecte certaines condition de version.

Sauf que le message d'erreur, en employant "utilise" laisse croire que
le plugin X a besoin de la dépendance Y, alors qu'en réalité, on
souhaite juste indiquer que la dépendance Y n'a pas la bonne version.

Ex de mécompréhension :
https://www.mail-archive.com/spip@rezo.net/msg81070.html

Proposition de reformulation pour éviter cela.

SPIP distingue pour un plugin X : - nécessite une dépendance Y (on a absolument besoin) - utilise une dépendance Y (si on a, tire profit) Mais dans le cadre d'utilise une dépendance, on vérifie quand même que cette dépendance respecte certaines condition de version. Sauf que le message d'erreur, en employant "utilise" laisse croire que le plugin X a besoin de la dépendance Y, alors qu'en réalité, on souhaite juste indiquer que la dépendance Y n'a pas la bonne version. Ex de mécompréhension : https://www.mail-archive.com/spip@rezo.net/msg81070.html Proposition de reformulation pour éviter cela.
maieul added 1 commit 2 weeks ago
9d71acf428 SPIP distingue pour un plugin X :
Poster

(Si accepté idéalement à reporter en 3.2)

(Si accepté idéalement à reporter en 3.2)
maieul added 1 commit 2 weeks ago
Poster

En complément, un second commit : ne pas envoyer de message d'erreur si le plugin "utilise" un plugin pas actif. Il faut le faire que si le plugin "nécessite" un plugin pas actif.

Capture d'écran avant/après ci-joint.

En complément, un second commit : ne pas envoyer de message d'erreur si le plugin "utilise" un plugin pas actif. Il faut le faire que si le plugin "nécessite" un plugin pas actif. Capture d'écran avant/après ci-joint.

+1 on comprend jamais rien à ces erreurs massives, alors que dans le lot il n'y en a généralement qu'une seule qui est pertinente. Parfois ça affiche genre 15 lignes de trucs en cascade juste pour un seul plugin pas à la bonne version

+1 on comprend jamais rien à ces erreurs massives, alors que dans le lot il n'y en a généralement qu'une seule qui est pertinente. Parfois ça affiche genre 15 lignes de trucs en cascade juste pour un seul plugin pas à la bonne version

Et ça fait quoi si dans la liste en plus de celui vraiment en erreur, les autres sont bien actifs mais dans la bonne version, par ex là sur ta capture si t'avais Yaml ou Vérifier dans une version ok avec le utilise ?

Et ça fait quoi si dans la liste en plus de celui vraiment en erreur, les autres sont bien actifs mais dans la bonne version, par ex là sur ta capture si t'avais Yaml ou Vérifier dans une version ok avec le utilise ?
Poster

Et ça fait quoi si dans la liste en plus de celui vraiment en erreur, les autres sont bien actifs mais dans la bonne version, par ex là sur ta capture si t'avais Yaml ou Vérifier dans une version ok avec le utilise ?

Non je n'avais pas, mais je viens de tester et ca affiche correctement les infos. Cf nvle capture d'écran. (et j'ai également vérifier si j'ai 2 plugins actifs mais pas en bonne version, ca génére bien les erreurs)

> Et ça fait quoi si dans la liste en plus de celui vraiment en erreur, les autres sont bien actifs mais dans la bonne version, par ex là sur ta capture si t'avais Yaml ou Vérifier dans une version ok avec le utilise ? Non je n'avais pas, mais je viens de tester et ca affiche correctement les infos. Cf nvle capture d'écran. (et j'ai également vérifier si j'ai 2 plugins actifs mais pas en bonne version, ca génére bien les erreurs)
b_b commented 2 weeks ago
Owner

(Si accepté idéalement à reporter en 3.2)

Je crois que c'est pas gagné pour le report, car j'ai souvenir qu'on ne reporte pas les chaines de langue de master vers une branche stable (ou alors on a abandonné cette "politique").

> (Si accepté idéalement à reporter en 3.2) Je crois que c'est pas gagné pour le report, car j'ai souvenir qu'on ne reporte pas les chaines de langue de master vers une branche stable (ou alors on a abandonné cette "politique").
Owner

Ok. Par contre essaie d'utiliser des opérateurs stricts (=== plutôt que ==).

Ok. Par contre essaie d'utiliser des opérateurs stricts (=== plutôt que ==).
maieul force-pushed message_plus_claire from 2b76929a6b to 073b55843a 2 weeks ago
Poster

Je crois que c'est pas gagné pour le report, car j'ai souvenir qu'on ne reporte pas les chaines de langue de master vers une branche stable (ou alors on a abandonné cette "politique").

ok, bah tant pis :)

Ok. Par contre essaie d'utiliser des opérateurs stricts (=== plutôt que ==).

corrigé en force

> Je crois que c'est pas gagné pour le report, car j'ai souvenir qu'on ne reporte pas les chaines de langue de master vers une branche stable (ou alors on a abandonné cette "politique"). ok, bah tant pis :) > Ok. Par contre essaie d'utiliser des opérateurs stricts (=== plutôt que ==). corrigé en force

Hello :)

Pour:
'plugin_utilise_extension_php' => 'L’extension PHP @plugin@ doit être en version @version@.',

Cela ne serait pas mieux de mettre ?
'plugin_utilise_extension_php' => 'L’extension PHP du plugin @plugin@ doit être en version @version@.',

ou alors ?
'plugin_utilise_extension_php' => 'L’extension PHP de @plugin@ doit être en version @version@.',

Franck

Hello :) Pour: 'plugin_utilise_extension_php' => 'L’extension PHP @plugin@ doit être en version @version@.', Cela ne serait pas mieux de mettre ? 'plugin_utilise_extension_php' => 'L’extension PHP du plugin @plugin@ doit être en version @version@.', ou alors ? 'plugin_utilise_extension_php' => 'L’extension PHP de @plugin@ doit être en version @version@.', Franck
Poster

heu non... @plugin@ désige dan s ce contexte le nom de l'extension PHP (je sais c'est un peu tordu, mais ca simplifie le code)

heu non... @plugin@ désige dan s ce contexte le nom de l'extension PHP (je sais c'est un peu tordu, mais ca simplifie le code)

Ha, ok :D

Ha, ok :D
marcimat merged commit 8bbbc843a7 into master 3 days ago
maieul deleted branch message_plus_claire 2 days ago
The pull request has been merged as 8bbbc843a7.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
5 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.