From ce8f4dd8bf7a723aadbebdee5cd80aad91c73e17 Mon Sep 17 00:00:00 2001
From: aaronlaib <89222728+aaronlaib@users.noreply.github.com>
Date: Mon, 6 May 2024 19:44:46 +0000
Subject: [PATCH 1/2] Download the latest strings from OneSky

---
 app/src/main/res/values-ar/strings_dashboard.xml | 2 +-
 app/src/main/res/values-ar/strings_languages.xml | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/src/main/res/values-ar/strings_dashboard.xml b/app/src/main/res/values-ar/strings_dashboard.xml
index 06f8dff5e2..cb21e7885b 100644
--- a/app/src/main/res/values-ar/strings_dashboard.xml
+++ b/app/src/main/res/values-ar/strings_dashboard.xml
@@ -50,7 +50,7 @@ https://godtoolsapp.com \n\n
         <item quantity="other">%1$d أدوات متاحة</item>
         <item quantity="few">%1$d أدوات متوفرة</item>
         <item quantity="many">%1$d أداة متوفرة</item>
-        <item quantity="two">d$1% أدواتين متوفرة</item>
+        <item quantity="two">%1$d أدواتين متوفرة</item>
         <item quantity="zero">لا بوجد أدوات متوفرة</item>
     </plurals>
     <string name="dashboard_tools_section_filter_available_tools_all">جميع الأدوات المتاحة</string>
diff --git a/app/src/main/res/values-ar/strings_languages.xml b/app/src/main/res/values-ar/strings_languages.xml
index 7b18c60487..2af9efd5ee 100644
--- a/app/src/main/res/values-ar/strings_languages.xml
+++ b/app/src/main/res/values-ar/strings_languages.xml
@@ -2,7 +2,7 @@
 <resources>
     <!-- App Language UI -->
     <string name="language_settings_app_language_title">لغة التطبيق</string>
-    <string name="language_settings_app_language_dialog_message">لقد حددت %1$s  كلغة تطبيقك. سيؤدي هذا إلى تغيير لغة تطبيق GodTools بالكامل. هل أنت متأكد؟</string>
+    <string name="language_settings_app_language_dialog_message">لقد حددت %1$s كلغة تطبيقك. سيؤدي هذا إلى تغيير لغة تطبيق GodTools بالكامل. هل أنت متأكد؟</string>
     <string name="language_settings_app_language_dialog_confirm">تغيير اللغة</string>
     <string name="language_settings_app_language_dialog_dismiss">لا تهتم</string>
     <!-- Language Settings UI -->
@@ -11,9 +11,9 @@
     <string name="language_settings_section_app_language_heading">لغة واجهة التطبيق</string>
     <plurals name="language_settings_section_app_language_available">
         <item quantity="one">%1$d لغة متوفرة</item>
-        <item quantity="other">%1$d  لغات متوفرة</item>
+        <item quantity="other">%1$d لغات متوفرة</item>
         <item quantity="few">%1$d عض اللغات متوفرة</item>
-        <item quantity="many">%1$d  كثير من  اللغات متوفرة</item>
+        <item quantity="many">%1$d كثير من اللغات متوفرة</item>
         <item quantity="two">%1$d لغات متوفرة</item>
         <item quantity="zero">لا يوجد لغات متوفرة</item>
     </plurals>
@@ -28,7 +28,7 @@
         <item quantity="other">%1$d أدوات متاحة</item>
         <item quantity="few">%1$d أدوات متوفرة</item>
         <item quantity="many">%1$d أداة متوفرة</item>
-        <item quantity="two">d$1% أدواتين متوفرة</item>
+        <item quantity="two">%1$d أدواتين متوفرة</item>
         <item quantity="zero">لا بوجد أدوات متوفرة</item>
     </plurals>
 </resources>

From 39e65af19d84362503762c4b58c4dbaf6877fb31 Mon Sep 17 00:00:00 2001
From: Daniel Frett <daniel.frett@cru.org>
Date: Mon, 6 May 2024 14:15:42 -0600
Subject: [PATCH 2/2] add a test to make sure string format placeholders are
 the correct syntax

---
 .../godtools/ui/languages/I18nStringsTest.kt  | 28 +++++++++++++++++++
 1 file changed, 28 insertions(+)
 create mode 100644 app/src/test/kotlin/org/cru/godtools/ui/languages/I18nStringsTest.kt

diff --git a/app/src/test/kotlin/org/cru/godtools/ui/languages/I18nStringsTest.kt b/app/src/test/kotlin/org/cru/godtools/ui/languages/I18nStringsTest.kt
new file mode 100644
index 0000000000..b546ce1b55
--- /dev/null
+++ b/app/src/test/kotlin/org/cru/godtools/ui/languages/I18nStringsTest.kt
@@ -0,0 +1,28 @@
+package org.cru.godtools.ui.languages
+
+import android.app.Application
+import android.content.Context
+import androidx.test.core.app.ApplicationProvider
+import androidx.test.ext.junit.runners.AndroidJUnit4
+import kotlin.test.Test
+import org.cru.godtools.R
+import org.junit.runner.RunWith
+import org.robolectric.annotation.Config
+
+@RunWith(AndroidJUnit4::class)
+@Config(application = Application::class)
+class I18nStringsTest {
+    private val context: Context get() = ApplicationProvider.getApplicationContext()
+    private val resources get() = context.resources
+
+    @Test
+    @Config(qualifiers = "ar")
+    fun `Languages - Arabic - Valid format placeholders`() {
+        // plurals
+        repeat(200) {
+            resources.getQuantityString(R.plurals.dashboard_tools_section_filter_available_tools, it, it)
+            resources.getQuantityString(R.plurals.language_settings_downloadable_languages_available_tools, it, it)
+            resources.getQuantityString(R.plurals.language_settings_section_app_language_available, it, it)
+        }
+    }
+}