Repère les termes du glossaire entre parenthèses ou avant un point final (#116)

[clementpolito]

Corrige #116 et #119.

Problème

Un terme du glossaire n'était pas signalé quand il était :

• entre parenthèses, ex. « le dossier de téléversement (uploads) » ;
• suivi d'un point, ex. « …dans le dossier uploads. ».

Cause (getGlossaryRegex, entrypoints/spte.content.js) : la regex joignait les termes avec leurs bornes, si bien que le premier terme n'avait pas de lookbehind et que seul le dernier acceptait un point final. Les parenthèses ne faisaient pas partie des bornes.

Correction

• Le lookbehind et le lookahead entourent maintenant tout le groupe des termes.
• ( est accepté avant le terme ; ), !, ? et » sont acceptés après.
• Le point n'est une borne que s'il est suivi d'un espace ou de la fin du texte, pour ne pas signaler widget.php.

Tests

Bloc « bornes du terme » dans tests/spte.content.test.js : 7 cas repérés (parenthèses, point final sur le premier terme et sur un terme du milieu, !, début de chaîne, guillemets droits) et 3 cas non repérés (widget.php, widgetisé, « widget »). Sans le correctif, 5 de ces tests échouent.

Cas vérifiés avec l'ancienne et la nouvelle regex (aucun n'était détecté avant, tous le sont maintenant) :

1. …dans le dossier de téléversement (uploads). Les fichiers qui n’ont pas été ajoutés… : terme entre parenthèses, suivi d'un point et d'une autre phrase ;
2. …dans le dossier de téléversement uploads. Les fichiers qui n’ont pas été ajoutés… : terme suivi d'un point et d'une autre phrase ;
3. …dans le dossier de téléversement uploads. : terme suivi d'un point en fin de chaîne.

Vérifié aussi sur de vraies chaînes de translate.wordpress.org (plugin offair), dans Firefox avec le build de la branche : « (uploads) » et « uploads. » sont surlignés. Pour « uploads. », le script de contenu des builds avec et sans correctif a aussi été exécuté dans jsdom sur la page de la traduction ci-dessous : surligné avec le correctif, pas sans.

Pour vérifier

Pages de translate.wordpress.org avec SPTE chargé :

• uploads (sans guillemets, doit être surligné, avec ou sans correctif) :
  UpdraftPlus, traductions contenant « uploads » (lignes « old » : « répertoires uploads ou plugins », « le dossier uploads, »)
• « uploads » (entre guillemets français, ne doit pas être surligné : c'est une citation) :
  même page UpdraftPlus, ligne courante « répertoires « uploads » ou « plugins » »
• (uploads) (entre parenthèses, doit être surligné avec ce correctif, ne l'était pas avant) :
  offair, traductions contenant « uploads » (lignes courantes « …dossier de téléversement (uploads)… »)
• uploads. (point final, doit être surligné avec ce correctif, ne l'était pas avant) :
  offair, chaîne « …le dossier de contenu dans uploads. Les fichiers… » (traduction rejetée : voir ci-dessous)
• Chaîne de l'issue : offair, « The content of the sites of the network could not… » : la traduction courante « (uploads) » est surlignée.

Tester sur les chaînes rejetées

SPTE ne traite pas les traductions rejetées (checkTranslation, depuis la 3.0.0). Or la chaîne « uploads. » ci-dessus est rejetée. Pour y voir le surlignage, il faut modifier temporairement l'extension, sans commiter, et afficher les erreurs sur les chaînes rejetées.

Dans entrypoints/spte.content.js, ligne 73 de checkTranslation, remplacer :

if (!preview || (preview.classList.contains('status-rejected') && newStatus !== 'rejected')) { return; }

par :

if (!preview) { return; }

Puis npm run build:firefox (ou npm run build) et recharger l'extension. Les chaînes rejetées sont alors analysées comme les autres, et leurs avertissements comptés. Sans le correctif, « uploads. » n'est pas surligné sur cette chaîne ; avec le correctif, il l'est.

Comportement inchangé

Un terme entre guillemets français (« uploads », espaces insécables comprises) n'est toujours pas signalé : c'est une citation, comme pour les autres mots du glossaire.

Proposition de ligne pour le CHANGELOG

Fixed

• Les termes du glossaire entre parenthèses ou suivis d'un point sont maintenant signalés (ex. (uploads), dossier uploads.).