fix: suggested changes from code review

tolgee · Sep 20, 2024 · 04f5f81 · 04f5f81
1 parent e22a391
commit 04f5f81
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/...n/kotlin/io/tolgee/formats/genericStructuredFile/in/GenericSuffixedPluralsPreprocessor.kt b/...n/kotlin/io/tolgee/formats/genericStructuredFile/in/GenericSuffixedPluralsPreprocessor.kt
@@ -69,10 +69,9 @@ class GenericSuffixedPluralsPreprocessor(
   private fun Map<*, *>.preprocessMap(): Map<*, *> {
     return this.groupByPlurals(pluralsViaSuffixesParser).flatMap { (commonKey, values) ->
       if (commonKey == null || values.size < 2) {
-        values.useOriginalKey()
-      } else {
-        values.usePluralsKey(commonKey)
+        return@flatMap values.useOriginalKey()
       }
+      return@flatMap values.usePluralsKey(commonKey)
     }.toMap()
   }
 }
diff --git a/.../src/main/kotlin/io/tolgee/formats/paramConvertors/in/I18nextToIcuPlaceholderConvertor.kt b/.../src/main/kotlin/io/tolgee/formats/paramConvertors/in/I18nextToIcuPlaceholderConvertor.kt
@@ -20,7 +20,7 @@ class I18nextToIcuPlaceholderConvertor : ToIcuPlaceholderConvertor {
     val parsed = parser.parse(matchResult) ?: return matchResult.value.escapeIcu(isInPlural)
 
     if (parsed.nestedKey != null) {
-      // TODO: nested keys are not yet supported
+      // Nested keys are not supported
       return matchResult.value.escapeIcu(isInPlural)
     }