Propostion changement - Remove - Supprimer - Retirer

Propostion changement - Remove - Supprimer - Retirer

par Nicolas Martignoni,
Nombre de réponses : 2
Avatar Développeurs Avatar Développeurs de plugins Avatar Documentation writers Avatar Moodleurs particulièrement utiles Avatar Testeurs Avatar Traducteurs

Bonjour et merci pour la suggestion bien argumentée et qui me semble pertinente. Cependant, le terme "remove" (et ses dérivés) est utilisé à de nombreux endroits dans Moodle (258 exactement), et il faut s'assurer qu'un tel changement n'ait pas d'autres impacts.

  • Quels sont (tous) les endroits où cette modification devrait être effectuée ?
  • Est-ce que le "supprimer" du bouton (2e screenshot) est utilisé ailleurs ?
En réponse à Nicolas Martignoni

Propostion changement - Remove - Supprimer - Retirer

par Séverin TERRIER,
Avatar Documentation writers Avatar Moodleurs particulièrement utiles Avatar Testeurs Avatar Traducteurs
Bonjour,

Après avoir activé depuis "Administration > Développement > Débogage" le réglage "Afficher l’origine des chaînes de caractères" (debugstringids) et avoir ajouté &strings=1 à l'URL de gestion des membres d'un groupe, j'ai constaté que la chaîne de caractères utilisée porte bien l'identifiant "remove", dans le cœur de Moodle.
 
Du coup, afin d'essayer d'en savoir plus sur les fichiers ou cette chaîne est utilisée (et la ligne concernée), j'ai lancé les commandes suivantes, avec ces résultats (pour Moodle 4.5) :
 
grep -rnie "print_string('remove')" *

admin/roles/admins.php:195:                   title="<?php print_string('remove'); ?>" class="btn btn-secondary"/><br />
admin/roles/assign.php:245:                     title="<?php print_string('remove'); ?>" class="btn btn-secondary"/>
enrol/manual/manage.php:211:                     title="<?php print_string('remove'); ?>" />
grade/edit/outcome/course_form.html:46:                    ' ' . $OUTPUT->rarrow(); ?>" title="<?php print_string('remove'); ?>" />
group/assign.php:168:                   title="<?php print_string('remove'); ?>" />
group/members.php:135:                   title="<?php print_string('remove'); ?>" />
mnet/service/enrol/course.php:156:              <input name="remove" id="remove" type="submit" value="<?php echo get_string('remove').'&nbsp;'.$OUTPUT->rarrow(); ?>" title="<?php print_string('remove'); ?>" />
grep -rnie "get_string('remove')" *

admin/roles/admins.php:194:            <input name="remove" id="remove" type="submit" value="<?php echo get_string('remove').'&nbsp;'.$OUTPUT->rarrow(); ?>"
admin/roles/assign.php:244:              <input name="remove" id="remove" type="submit" value="<?php echo get_string('remove').'&nbsp;'.$OUTPUT->rarrow(); ?>"
cohort/assign.php:124:                     value="<?php echo s(get_string('remove')) . '&nbsp;' . $OUTPUT->rarrow(); ?>"
cohort/assign.php:125:                     title="<?php p(get_string('remove')); ?>" />
course/reset.php:49:$strremove      = get_string('remove');
enrol/manual/manage.php:210:                     value="<?php echo get_string('remove') . '&nbsp;' . $OUTPUT->rarrow(); ?>"
grade/edit/outcome/course_form.html:45:                <input name="remove" class="btn btn-secondary" id="remove" type="submit" value="<?php echo get_string('remove') .
group/assign.php:167:                   value="<?php echo get_string('remove').'&nbsp;'.$OUTPUT->rarrow(); ?>"
group/members.php:134:                   type="submit" value="<?php echo get_string('remove').'&nbsp;'.$OUTPUT->rarrow(); ?>"
mnet/service/enrol/course.php:156:              <input name="remove" id="remove" type="submit" value="<?php echo get_string('remove').'&nbsp;'.$OUTPUT->rarrow(); ?>" title="<?php print_string('remove'); ?>" />
mod/forum/renderer.php:94:        $actioncell->text .= html_writer::empty_tag('input', array('type'=>'submit', 'name'=>'unsubscribe', 'value'=>$this->page->theme->rarrow.' '.get_string('remove'), 'class'=>'actionbutton'));
question/bank/columnsortorder/classes/local/bank/column_action_remove.php:41:        $this->remove = get_string('remove');
webservice/renderer.php:66:                            'value' => $removearrow . ' ' . get_string('remove'),
webservice/renderer.php:67:                            'title' => get_string('remove')));
webservice/renderer.php:162:                                $optionsyes), get_string('remove'));
 
Je ne suis pas certain pour autant que ces commandes permettent de trouver la liste exhaustive des usages de cette chaîne.
 
Séverin
Moyenne des évaluations Utile (1)
En réponse à Séverin TERRIER

Proposition changement - Remove - Supprimer - Retirer

par Nicolas Martignoni,
Avatar Développeurs Avatar Développeurs de plugins Avatar Documentation writers Avatar Moodleurs particulièrement utiles Avatar Testeurs Avatar Traducteurs

J'ai fait de mon côté une revue exhaustive de tout le code où le terme « remove » ou ses dérivés apparaît, soit tout seul, soit dans une chaîne avec d'autres termes. Voici les résultats:

  1. le terme « remove » est, dans les chaînes en anglais, parfois utilisé dans le sens de « retirer », parfois dans le sens de « supprimer », ce qui est bien embêtant, car les développeurs ne font pas toujours la différence ;
  2. dans tous les cas (sauf erreur) où la chaîne avec le terme « Remove » (tout seul) apparaît, on est dans le contexte de l'attribution de rôles/groupes/groupements/cohortes.

Conclusion : la chaîne « Remove » (toute seule) peut être corrigée en « Retirer ». Les autres cas peuvent être facilement résolus (et c'est déjà fait, après relecture et revue des 257 autres chaînes concernées) sourire.

Sauf présentation de contre-exemples, je ferai le changement durant cette semaine.

Moyenne des évaluations Utile (3)