Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add xcstrings support #886

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add xcstrings support #886

wants to merge 1 commit into from

Conversation

fthdgn
Copy link

@fthdgn fthdgn commented Mar 8, 2024

Updated version of #865. I couldn't update source branch of previous PR bacause it has references on my own apps.

Difference from #865

  • Rebased to version 7.5.0.
  • Fixed error on keys without values

I did a simple development for xcstrings support.

It is backward compatible for for .xcstrings files converted from .strings files. It means the same R.string codes will be generated.

However, it is not backward compatible for .xcstrings files converted from .stringsdict files, because of named arguments.

About named arguments

My implementation strips names of substitutions from generated arguments. I found using them problematic on some cases.

In my opinion, named arguments does not worth the implementation. Xcode generates substituons only if there are more than one pluralable parameter on new xcstrings files. The most of the string values will not be have any information to generate named arguments.

Problematic case 1

Some key information is lost on the xcstring convertion of stringsdict files.
Original stringsdict content.

	<key>x_users</key>
	<dict>
		<key>NSStringLocalizedFormatKey</key>
		<string>%#@users@</string>
		<key>users</key>
		<dict>
			<key>NSStringFormatSpecTypeKey</key>
			<string>NSStringPluralRuleType</string>
			<key>NSStringFormatValueTypeKey</key>
			<string>d</string>
			<key>one</key>
			<string>%d user</string>
			<key>other</key>
			<string>%d users</string>
		</dict>
	</dict>

Generated xcstrings content

{
  "x_users" : {
    "localizations" : {
      "en" : {
        "variations" : {
          "plural" : {
            "one" : {"stringUnit" : {"value" : "%d user"}},
            "other" : {"stringUnit" : {"value" : "%d users"}
}}}}}}}

R.swift creates .x_users(users: Int) for stringsdict file. However, xcstrings file does not have the argument name information any more.

If string value of the original content is <string>Add %#@users@</string>, generated xcstrings would keep the substitution information.

Problematic case 2

{
  "example": {
    "localizations": {
      "en": {
        "substitutions": {
          "device_iphone": {
            "argNum": 1,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg iPhone"}},
                "other": {"stringUnit": {"value": "%arg iPhones"}}
              }}},
          "device_mac": {
            "argNum": 1,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg Mac"}},
                "other": {"stringUnit": {"value": "%arg Macs"}}
              }}},
          "input_iphone": {
            "argNum": 2,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg touch"}},
                "other": {"stringUnit": {"value": "%arg touches"}}
              }}},
          "input_mac": {
            "argNum": 2,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg key"}},
                "other": {"stringUnit": {"value": "%arg keys"}}
              }}},
        "variations": {
          "device": {
            "iphone": {
              "stringUnit": {
                "value": "%#@device_iphone@ and %#@input_iphone@"
              }},
            "mac": {
              "stringUnit": {
                "value": "%#@device_mac@ and %#@input_mac@"
              }}}}}}}}

This strings value is "1 iPhone and 10 touches" on iPhones and "1 Mac and 10 keys" on Mac.
There are 2 substitutions for each device variation.
Which signature should we use?
.example(device_iphone: Int, input_iphone: Int)
.example(device_mac: Int, input_mac: Int)
.example(_ arg1: Int, _ arg2: Int)
.example(device Int, input: Int) //With some extra coding to detect shared parts of the names.

About algorithm

The alghorithm tries to convert localization of source language to single string with basic format parameters, then it uses FormatPart.formatParts(formatString:)` on this string to extract parameters.

This convertion works like that:

  • If localization has stringUnit, get its value, and replace the substitutions inside if.
  • Else if localization has variations for device or plural, convert each variation value to string with the same algorithm and select the string with most parameters. (To work correctly, all variations should have same parameters, but on some cases such as plural rule one or zero, parameter may not be used)

Substitutions replacement works like that:

  • Substitutions generated by Xcode contain plural variations with %arg parameter. Each substitutions has argNum and formatSpecifier variable to define real properties of this %arg parameter.
  • The value of a substitution is calculated by the convertion algorith above.
  • Then, %arg parameters on the calculated value is replaced according to argNum and formatSpecifier values.

Example 1

  "account": {
    "localizations": {
      "en": {
        "stringUnit": {"value": "Account"}
      }}}
  • Localization has stringUnit, get its value.
  • Localization does not have any substitutions. Nothing replaced.
  • Get paramters of "Account". []

Example 2

  "x_sets": {
    "localizations": {
      "en": {
        "variations": {
          "plural": {
            "one": {"stringUnit": {"value": "a set"}},
            "other": {"stringUnit": {"value": "%d sets"}}
          }}}}}
  • Localization does not have stringUnit.
  • Localization does not have device variations.
  • Localization has plural variations. Get value of each variations with parameter count.
    • one: Value: "a set", parameters []
    • other: Value: "%d sets", parameters [%d]
  • Select the variation with most parameters. "%d sets"
  • Get paramters of "%d sets". [%d]

Example 3

{
  "example": {
    "localizations": {
      "en": {
        "stringUnit": {
          "value": "%#@books@ and %#@pens@"
        },
        "substitutions": {
          "books": {
            "argNum": 1,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg book"}},
                "other": {"stringUnit": {"value": "%arg books"}}
              }}},
          "pens": {
            "argNum": 2,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg pen"}},
                "other": {"stringUnit": {"value": "%arg pens"}}
              }}}}}}}}
  • Localization has stringUnit.
  • Value is %#@books@ and %#@pens@
  • Localization has substitutions.
  • For "books" substitution
  • Get the value like Example 2: %arg book
  • Replace %arg according to argNum and formatSpecifier: %1$lld book
  • Replace substitution on the value: %1$lld book and %#@pens@
  • For "pens" substitution
  • Get the value like Example 2: %arg pen
  • Replace %arg according to argNum and formatSpecifier: %2$lld pen
  • Replace substitution on the value: %1$lld book and%2$lld pen
  • Get paramters of %1$lld book and %2$lld pen. [%lld, %lld]

@fthdgn fthdgn mentioned this pull request Mar 8, 2024
@aiKrice
Copy link

aiKrice commented Mar 10, 2024

Awesome !

@aiKrice
Copy link

aiKrice commented Mar 18, 2024

@mac-cain13 👍

@tlacan
Copy link

tlacan commented Mar 22, 2024

waiting for it ^^

@Flatout73
Copy link

Let's merge it

@deekayd
Copy link

deekayd commented Apr 23, 2024

When are we gonna see that PR in action?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants