Fix fr translations

Author: simonprev

lib/langue/langue.ex

@@ -43,6 +43,7 @@ defmodule Langue do
   def placeholder_regex do
     @format_modules
     |> Enum.map(& &1.placeholder_regex())
+    |> Enum.uniq()
     |> Enum.reject(&(&1 === :not_supported))
   end
 end

lib/lint/checks/apostrophe_as_single_quote.ex

@@ -11,7 +11,9 @@ defmodule Accent.Lint.Checks.ApostropheAsSingleQuote do
   def enabled?, do: true
 
   @impl true
-  def applicable(%{language_slug: slug}, _), do: slug in @applicable_languages
+  def applicable(entry, _) do
+    entry.language_slug in @applicable_languages and not String.match?(entry.value, ~r/MMMM?|YYYY?|HH|AA/i)
+  end
 
   @impl true
   def check(entry, _) do

lib/lint/checks/first_letter_case.ex

@@ -54,7 +54,7 @@ defmodule Accent.Lint.Checks.FirstLetterCase do
   end
 
   defp starts_with_letter?(text) do
-    Regex.match?(~r/^[\w]/i, text)
+    Regex.match?(~r/^\w/iu, text)
   end
 
   defp starts_with_capitalized_letter?(""), do: false

lib/lint/checks/placeholder_count.ex

@@ -10,7 +10,9 @@ defmodule Accent.Lint.Checks.PlaceholderCount do
   def enabled?, do: true
 
   @impl true
-  def applicable(entry, _), do: is_binary(entry.value) and not entry.is_master and is_binary(entry.master_value)
+  def applicable(entry, _) do
+    is_binary(entry.value) and not entry.is_master and is_binary(entry.master_value)
+  end
 
   @impl true
   def check(entry, _) do

lib/lint/checks/spelling.ex

@@ -49,22 +49,37 @@ defmodule Accent.Lint.Checks.Spelling do
   defp reject_match?(match, entry, config) do
     error_term = String.slice(entry.value, match["offset"], match["length"])
 
-    Enum.any?(config.lint_entries, fn lint_entry ->
-      cond do
-        "spelling" in lint_entry.check_ids and lint_entry.ignore and lint_entry.type === :term and
-            String.downcase(error_term) === String.downcase(lint_entry.value) ->
-          true
-
-        "spelling" in lint_entry.check_ids and lint_entry.ignore and
-          lint_entry.type === :language_tool_rule_id and
-            match["rule"]["id"] === lint_entry.value ->
-          true
-
-        true ->
-          false
-      end
-    end) or String.match?(error_term, ~r/^,/) or
-      (match["offset"] === 0 and String.starts_with?(entry.value, "{"))
+    match_project_lint_entries = fn ->
+      Enum.any?(config.lint_entries, fn lint_entry ->
+        cond do
+          "spelling" in lint_entry.check_ids and lint_entry.ignore and lint_entry.type === :term and
+              String.downcase(error_term) === String.downcase(lint_entry.value) ->
+            true
+
+          "spelling" in lint_entry.check_ids and lint_entry.ignore and
+            lint_entry.type === :language_tool_rule_id and
+              match["rule"]["id"] === lint_entry.value ->
+            true
+
+          true ->
+            false
+        end
+      end)
+    end
+
+    error_term_starts_with_comma = fn -> String.match?(error_term, ~r/^,/) end
+    match_starts_with_brace = fn -> match["offset"] === 0 and String.starts_with?(entry.value, "{") end
+    error_term_is_icu_match = fn -> error_term === "other" and String.at(entry.value, match["offset"] - 2) === "}" end
+
+    Enum.any?(
+      [
+        match_project_lint_entries,
+        error_term_starts_with_comma,
+        match_starts_with_brace,
+        error_term_is_icu_match
+      ],
+      & &1.()
+    )
   end
 
   defp find_replacement(match, entry) do

test/lint/lint_test.exs

@@ -16,10 +16,25 @@ defmodule AccentTest.Lint do
     end
 
     test "lint placeholder simple" do
-      entry = %Entry{key: "a", value: "nothing", is_master: false, master_value: "{{bar}}", value_type: "string"}
+      entry = %Entry{key: "a", value: "Nothing", is_master: false, master_value: "Test {{bar}}", value_type: "string"}
       [{_, messages}] = Lint.lint([entry])
 
-      assert messages === [%Accent.Lint.Message{check: :placeholder_count, replacement: nil, text: "nothing"}]
+      assert messages === [%Accent.Lint.Message{check: :placeholder_count, replacement: nil, text: "Nothing"}]
+    end
+
+    @tag :skip
+    test "lint placeholder icu" do
+      entry = %Entry{
+        key: "a",
+        value: "Nothing",
+        value_type: "string",
+        is_master: false,
+        master_value: "{count, plural, =0 {No items} =1 {1 item} other {# items}}"
+      }
+
+      [{_, messages}] = Lint.lint([entry])
+
+      assert messages === []
     end
 
     test "lint placeholder many placeholders" do

test/test_helper.exs

@@ -51,7 +51,7 @@ defmodule Langue.Expectation.Case do
   end
 end
 
-ExUnit.start()
+ExUnit.start(exclude: [:skip])
 
 Sandbox.checkout(Accent.Repo)
 Accent.Factory.bootstrap()

webapp/app/components/lint-translations-page/add-lint-entry/component.ts

@@ -22,7 +22,8 @@ export default class LintTranslationsPageAddLintEntry extends Component<Args> {
   }
 
   get spellingTermValue() {
-    if (!this.args.message.offset || !this.args.message.length) return;
+    if (this.args.message.offset == null || this.args.message.length == null)
+      return;
 
     return this.args.message.text.substring(
       this.args.message.offset,

webapp/app/locales/en-us.json

@@ -772,7 +772,7 @@
     "lint_translations_page": {
       "empty_text": "No checks warnings or errors. You’re all good!",
       "add_lint_entry": {
-        "ignore_checks_for_term": "Ignore all checks for term",
+        "ignore_checks_for_term": "Accept the term in this project",
         "ignore_spellcheck_rule": "Ignore spellcheck rule",
         "ignore_checks_for_key": "Ignore all checks for key",
         "ignore_check_for_project": "Ignore check for project"

webapp/app/locales/fr-ca.json

@@ -57,7 +57,7 @@
     },
     "revision_selector": {
       "languages_count": "{count, plural, =1 {1 autre langue} other {# autres langues}}",
-      "master": "Principale"
+      "master": "principale"
     },
     "translation_comments_subscriptions": {
       "title": "Notifier les nouveaux messages"
@@ -178,7 +178,7 @@
       "translations_version_notice": "Vous consultez les textes à réviser de la version",
       "correct_all_button": "Marquer toutes les chaînes comme révisées pour cette langue",
       "no_translations": "Aucune chaîne à examiner pour : {query}",
-      "all_reviewed_title": "Tous revus",
+      "all_reviewed_title": "Tous révisés",
       "all_reviewed_subtitle": "Toutes les chaînes ont été marquées comme révisées"
     },
     "dashboard_master_revision": {
@@ -208,7 +208,7 @@
       "last_synced_at_label": "Dernière synchronisation :",
       "master_language_label": "La langue principale est :",
       "never_synced": "Synchroniser le projet pour la première fois",
-      "reviewed": "revu"
+      "reviewed": "révisé"
     },
     "dashboard_navigation": {
       "collaborators_link_title": "Collaborateurs",
@@ -226,8 +226,8 @@
       "slaves": "Traductions",
       "sync": "Synchroniser",
       "strings": "chaînes",
-      "all_reviewed": "Tous revus !",
-      "reviewed": "revu",
+      "all_reviewed": "Tous révisés!",
+      "reviewed": "révisé",
       "activities_title": "Dernières activités",
       "item": {
         "rtl_badge": "RTL",
@@ -243,7 +243,7 @@
       "humanized_date_title_format": "EEEE, MMMM do yyyy, H:mm a"
     },
     "versions_list": {
-      "empty_text": "Le versionning vous permet de réviser/exporter/synchroniser les traductions indépendamment de votre projet principal. Cela signifie que la prise en charge des versions précédentes de votre application sera aussi simple que l’ajout de nouvelles fonctionnalités !",
+      "empty_text": "Le versionning vous permet de réviser/exporter/synchroniser les traductions indépendamment de votre projet principal. Cela signifie que la prise en charge des versions précédentes de votre application sera aussi simple que l’ajout de nouvelles fonctionnalités!",
       "export": "Exportation",
       "update": "Éditer"
     },
@@ -292,7 +292,7 @@
     "project_activity": {
       "stats_label": "Activités réalisées :",
       "overview_label": "Aperçu:",
-      "review_label": "A été revu ?",
+      "review_label": "A été révisé?",
       "reviewed_yes": "Oui",
       "reviewed_no": "Non",
       "last_synced_text_label": "Dernier texte synchronisé :",
@@ -420,7 +420,7 @@
         "badges_text": "Aperçu rapide des statistiques du projet, intégrées dans les images",
         "api_token": "Jetons d’API",
         "api_token_text": "Intégrez le flux de travail Accent avec les requêtes CLI ou API simples",
-        "service_integrations": "Service & intégrations",
+        "service_integrations": "Service et intégrations",
         "service_integrations_text": "Notifier et interagir avec des systèmes externes en fonction des événements Accent",
         "manage_languages": "Gérer les langues",
         "manage_languages_text": "Ajouter/supprimer des langues traduites dans le projet",
@@ -443,7 +443,7 @@
         "delete_project_title": "Supprimer ce projet",
         "delete_project_text": "Une fois que vous avez supprimé un projet, il n’y a plus de retour en arrière.",
         "delete_project_button": "Supprimer ce projet",
-        "delete_project_confirm": "Voulez-vous vraiment supprimer ce projet ? Cette action ne peut pas être annulée."
+        "delete_project_confirm": "Voulez-vous vraiment supprimer ce projet? Cette action ne peut pas être annulée."
       },
       "form": {
         "update_button": "Projet de mise à jour",
@@ -471,7 +471,7 @@
         "title": "Jetons d’API",
         "text_1": "Avec ce jeton, vous pouvez effectuer des appels authentifiés à l’API d’Accent. Tous les tokens ont les permissions ’bot’ : synchroniser et ajouter des traductions. Vous pouvez spécifier des autorisations personnalisées pour avoir des jetons plus précis dans votre projet. Définissez le jeton sur <code>ACCENT_API_KEY</code> dans vos workflows d’actions GitHub à authentifier à l’aide d’accent-cli.",
         "revoke_button": "Révoquer",
-        "revoke_confirm": "Voulez-vous vraiment révoquer le jeton ? Cette opération ne peut pas être annulée.",
+        "revoke_confirm": "Voulez-vous vraiment révoquer le jeton? Cette opération ne peut pas être annulée.",
         "inserted_at": "Créé:",
         "create_title": "Ajouter un nouveau jeton d’API",
         "create_button": "Créer",
@@ -517,7 +517,7 @@
         "uninvite_button": "Supprimer l’invitation"
       },
       "integrations": {
-        "title": "Service & intégrations",
+        "title": "Service et intégrations",
         "help": "Les services sont des intégrations prédéfinies qui effectuent certaines actions lorsque des événements se produisent sur Accent. <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.accent.reviews/guides/service-integrations.html\">Plus d’infos -></a>",
         "save": "Sauver",
         "cancel": "Annuler",
@@ -527,7 +527,7 @@
         "data": {
           "azure_storage_container_sas": "SAS URL",
           "aws_s3_bucket": "Nom du bucket",
-          "aws_s3_path_prefix": "Préfix",
+          "aws_s3_path_prefix": "Préfixe",
           "aws_s3_region": "Région",
           "aws_s3_access_key_id": "Access Key ID",
           "aws_s3_secret_access_key": "Secret Access Key",
@@ -552,13 +552,13 @@
             "text": "Publiez les notifications de votre projet dans Slack"
           }
         },
-        "webhook_url_how": "Comment obtenir une URL de webhook ?",
+        "webhook_url_how": "Comment obtenir une URL de webhook?",
         "events": {
-          "title": "Quels événements souhaitez-vous déclencher ce webhook ?",
+          "title": "Quels événements souhaitez-vous déclencher ce webhook?",
           "options": {
             "sync": "Synchroniser avec n’importe quelles modifications",
             "new_conflicts": "Nouvelles chaîne à réviser",
-            "complete_review": "Le projet est revu à 100%",
+            "complete_review": "Le projet est révisé à 100%",
             "integration_execute_azure_storage_container": "Fichiers téléversés sur Azure Storage Container"
           }
         },
@@ -569,7 +569,7 @@
             "error": "Une erreur s’est produite lors de la publication sur Azure. Veuillez vérifier vos informations d’identification et réessayer.",
             "push_button": "Publier",
             "bucket": "Nom du bucket",
-            "path_prefix": "Préfix",
+            "path_prefix": "Préfixe",
             "submit_confirm": "La publication de fichiers vers AWS S3 peut remplacer les fichiers existants si vous publiez la dernière version ou une version déjà publiée.",
             "target_version": {
               "label": "Version Cible",
@@ -713,7 +713,7 @@
       "title": "Nouvelle version",
       "text": "La création d’une version crée un instantané de toutes les chaînes actives (révisées ou non) pour être visible avec la certitude qu’il restera intact. Cela peut être utile lors de la maintenance de plusieurs versions de la même application.",
       "error": "Version invalide",
-      "copy_on_update_translation_help": "Chaque fois qu'une traduction dans cette version est mise à jour, le même texte sera appliqué à la traduction source (dernière version). Cela garantira qu'un correctif dans la version sera également présent dans la dernière version sans aucune action requise de la part du développeur ou le traducteur.",
+      "copy_on_update_translation_help": "Chaque fois qu’une traduction dans cette version est mise à jour, le même texte sera appliqué à la traduction source (dernière version). Cela garantira qu’un correctif dans la version sera également présent dans la dernière version sans aucune action requise de la part du développeur ou le traducteur.",
       "copy_on_update_translation_label": "Copie sur mise à jour de la traduction",
       "cancel_button": "Annuler",
       "name_label": "Nom:",
@@ -723,7 +723,7 @@
     "version_update_form": {
       "title": "Version de mise à jour",
       "error": "Version invalide",
-      "copy_on_update_translation_help": "Chaque fois qu'une traduction dans cette version est mise à jour, le même texte sera appliqué à la traduction source (dernière version). Cela garantira qu'un correctif dans la version sera également présent dans la dernière version sans aucune action requise de la part du développeur ou le traducteur.",
+      "copy_on_update_translation_help": "Chaque fois qu’une traduction dans cette version est mise à jour, le même texte sera appliqué à la traduction source (dernière version). Cela garantira qu’un correctif dans la version sera également présent dans la dernière version sans aucune action requise de la part du développeur ou le traducteur.",
       "copy_on_update_translation_label": "Copie sur mise à jour de la traduction",
       "cancel_button": "Annuler",
       "name_label": "Nom:",
@@ -770,9 +770,9 @@
       "versions_link_title": "Versions"
     },
     "lint_translations_page": {
-      "empty_text": "Aucune avertissement ou erreur. Tout est bon!",
+      "empty_text": "Aucun avertissement ou erreur. Tout est bon!",
       "add_lint_entry": {
-        "ignore_checks_for_term": "Ignorer toutes les vérifications pour les mots",
+        "ignore_checks_for_term": "Accepter les mots dans ce projet",
         "ignore_spellcheck_rule": "Ignorer la règle de vérification orthographique",
         "ignore_checks_for_key": "Ignorer toutes les vérifications pour la clé",
         "ignore_check_for_project": "Ignorer la vérification pour le projet"
@@ -828,9 +828,9 @@
       "revision_inserted_at_label": "Créé",
       "revision_deleted_label": "La langue est en cours de suppression de votre projet. Elle disparaîtra automatiquement une fois toutes les opérations propagées dans le système.",
       "master_badge": "principale",
-      "delete_revision_confirm": "Voulez-vous vraiment supprimer cette langue de votre projet ? Cette action ne peut pas être annulée.",
+      "delete_revision_confirm": "Voulez-vous vraiment supprimer cette langue de votre projet? Cette action ne peut pas être annulée.",
       "delete_revision_button": "Supprimer cette langue",
-      "promote_revision_master_confirm": "Voulez-vous vraiment utiliser cette langue comme langue principale de votre projet ?",
+      "promote_revision_master_confirm": "Voulez-vous vraiment utiliser cette langue comme langue principale de votre projet?",
       "promote_revision_master_button": "Utiliser comme principale"
     },
     "revision_export_options": {
@@ -944,9 +944,9 @@
           "LEADING_SPACES": "La chaîne contient des espaces de début",
           "DOUBLE_SPACES": "La chaîne contient des espaces doubles",
           "APOSTROPHE_AS_SINGLE_QUOTE": "Un guillemet simple a été utilisé à la place d’une apostrophe",
-          "FIRST_LETTER_CASE": "La première lettre de traduction ne correspond pas à la casse de la langue principal",
+          "FIRST_LETTER_CASE": "La première lettre de traduction ne correspond pas à la casse de la langue principale",
           "THREE_DOTS_ELLIPSIS": "La chaîne contient trois points au lieu de points de suspension",
-          "SAME_TRAILING_CHARACTER": "La chaîne ne correspond pas au caractère de fin de la langue principal",
+          "SAME_TRAILING_CHARACTER": "La chaîne ne correspond pas au caractère de fin de la langue principale",
           "SPELLING": "La chaîne contient une faute d’orthographe",
           "URL_COUNT": "Le nombre d’URL ne correspond pas à la chaîne principale"
         }
@@ -1141,7 +1141,7 @@
           "translate_error": {
             "unsupported_target": "La chaîne n’a pas pu être traduite : Langue non prise en charge {target}",
             "unsupported_source": "La chaîne n’a pas pu être traduite : Langue non prise en charge {source}",
-            "unsupported_source_and_target": "La chaîne n’a pas pu être traduite : Langues non prise en charge {source} et {target}"
+            "unsupported_source_and_target": "La chaîne n’a pas pu être traduite : Langues non prises en charge {source} et {target}"
           }
         }
       },

webapp/app/styles/components/lint-translations-page/add-lint-entry.scss

@@ -44,13 +44,16 @@
 
 .title {
   display: flex;
-  align-items: center;
+  align-items: flex-start;
+  line-height: 1.2;
   gap: 10px;
   font-size: 20px;
   margin-bottom: 14px;
 }
 
 .title-icon {
+  position: relative;
+  top: 5px;
   width: 13px;
   height: 13px;
   opacity: 0.6;
@@ -72,6 +75,14 @@
   color: var(--color-primary);
   opacity: 0.8;
   font-weight: normal;
+
+  &::before {
+    content: '”';
+  }
+
+  &::after {
+    content: '“';
+  }
 }
 
 .menu-button {