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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fbt.plural manifest is wrong in --react-native-mode #401

Open
bumbu opened this issue Jun 14, 2023 · 0 comments
Open

fbt.plural manifest is wrong in --react-native-mode #401

bumbu opened this issue Jun 14, 2023 · 0 comments

Comments

@bumbu
Copy link

bumbu commented Jun 14, 2023

馃悰 Bug Report

Only reproes with:

  • fbt.plural with showCount: 'no', other plurals will work
  • --react-native-mode enabled

Running yarn translate-fbts will throw with

yarn run v1.22.19
$ fbt-translate --translations translations/*.json --jenkins > src/translatedFbts.json
/Users/bumbu/Development/fbt/node_modules/nullthrows/nullthrows.js:9
  throw error;
  ^

Error: Expect `token` to not be null as the metadata has variation mask.
    at nullthrows (/Users/bumbu/Development/fbt/node_modules/nullthrows/nullthrows.js:7:15)
    at new TranslationBuilder (/Users/bumbu/Development/fbt/packages/babel-plugin-fbt/dist/translate/TranslationBuilder.js:109:23)
    at /Users/bumbu/Development/fbt/packages/babel-plugin-fbt/dist/bin/translateUtils.js:113:53
    at Array.map (<anonymous>)
    at processTranslations (/Users/bumbu/Development/fbt/packages/babel-plugin-fbt/dist/bin/translateUtils.js:113:38)
    at /Users/bumbu/Development/fbt/packages/babel-plugin-fbt/dist/bin/translateUtils.js:46:12
    at Array.map (<anonymous>)
    at processFiles (/Users/bumbu/Development/fbt/packages/babel-plugin-fbt/dist/bin/translateUtils.js:44:45)
    at Object.<anonymous> (/Users/bumbu/Development/fbt/packages/babel-plugin-fbt/dist/bin/translate.js:147:15)
    at Module._compile (node:internal/modules/cjs/loader:1254:14) {
  framesToPop: 1
}

Node.js v18.16.0

To Reproduce

Steps to reproduce the behavior:

  • In demo-app/package.json add --react-native-mode to collect-fbts script
  • In Example.react.js add fbt(fbt.plural('has', 1, {many: 'have',showCount: 'no'}),'example')
  • Run
    • yarn install in main folder
    • yarn collect-fbts in demo-app
    • yarn translate-fbts in demo-app

Expected behavior

Should work

Issue

Running yarn collect-fbts will output following json:

  {
   "hashToLeaf": {
    "7VzaqByBO7Pb/+HbPfOSwg==": {
     "text": "have",
     "desc": "example"
    },
    "AdZhIiwu5YM4eTswNtofDQ==": {
     "text": "has",
     "desc": "example"
    }
   },
   "filepath": "src/example/Example.react.js",
   "line_beg": 332,
   "col_beg": 19,
   "line_end": 338,
   "col_end": 7,
   "project": "fbt-demo-project",
   "jsfbt": {
    "t": {
     "*": {
      "desc": "example",
      "text": "have"
     },
     "_1": {
      "desc": "example",
      "text": "has"
     }
    },
    "m": [
     {
      "type": 2  // <<<<<< this is the issue
     }
    ]
   }
 }

And then inside of TranslationBuilder we expect that if manifest has a type, is should also have the token. Imho in this case the type shouldn't be added at all , so the issue is in the yarn collect-fbts step.

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

No branches or pull requests

1 participant