Handle dollar signs properly when generating localizations #125514

thkim1011 · 2023-04-25T21:13:33Z

Currently, the code doesn't properly handle strings which contain dollar signs. The return expression for the generated localization function is computed by generateReturnExpr which concatenates several strings, which are either interpolated placeholders, interpolated function calls, or normal strings, but we didn't properly escape dollar signs before sending normal strings to generateReturnExpr.

Fixes #125461.

christopherfujino

LGTM thanks for the quick fix!

HansMuller · 2023-04-25T21:20:16Z

packages/flutter_tools/lib/src/localizations/localizations_utils.dart

  return value;
 }

 /// Given a list of strings, placeholders, or helper function calls, concatenate


Can expressions really include "helper function calls"?

Ah you're right, I think that was an outdated comment from when I used to directly embed helper functions to generate some expressions.

packages/flutter_tools/lib/src/localizations/localizations_utils.dart

HansMuller · 2023-04-25T21:26:08Z

packages/flutter_tools/lib/src/localizations/localizations_utils.dart

+///
+/// The following rules describe all the cases which needs to be handled:
+/// 1. String with only a single interpolated variable is converted to the variable itself e.g. ["'$expr'"] -> "expr".
+/// 2. If one string ends in an interpolation and the next begins with an alphanumeric character, then the former should


Shouldn't the rules mention an empty string and the reverse of #2:

["'another string'", "'$expr1'"] -> "'another string'${expr1}".

It is acceptable to not put braces in the reverse case since "'another string$expr1'" is a valid Dart string. But I'll add a mention of what happens when expressions is empty.

…ls.dart Co-authored-by: Hans Muller <hansmuller@google.com>

HansMuller

LGTM

…lutter/flutter#125514)

Roll Flutter from c9004ff to 66fa4c5 (68 revisions) flutter/flutter@c9004ff...66fa4c5 2023-04-26 zanderso@users.noreply.github.com Remove web compile benchmarks that specify an attached device (flutter/flutter#125559) 2023-04-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from f75908135e10 to f125a54f5a57 (1 revision) (flutter/flutter#125560) 2023-04-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from c9db1587f207 to f75908135e10 (1 revision) (flutter/flutter#125558) 2023-04-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from 58a5a1e3f2c1 to c9db1587f207 (1 revision) (flutter/flutter#125555) 2023-04-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from 321f8015b9c2 to 58a5a1e3f2c1 (1 revision) (flutter/flutter#125552) 2023-04-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from 87f5f4e939cf to 321f8015b9c2 (1 revision) (flutter/flutter#125549) 2023-04-26 goderbauer@google.com Update prefer_final_parameters comment (flutter/flutter#125465) 2023-04-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from 706c023df44b to 87f5f4e939cf (1 revision) (flutter/flutter#125539) 2023-04-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from 7b385abd18a2 to 706c023df44b (1 revision) (flutter/flutter#125536) 2023-04-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from 610c57781b47 to 7b385abd18a2 (1 revision) (flutter/flutter#125533) 2023-04-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from b3cbf0678487 to 610c57781b47 (1 revision) (flutter/flutter#125531) 2023-04-26 zanderso@users.noreply.github.com Add hostonly web compile size benchmarks (flutter/flutter#125518) 2023-04-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from 34ece7a4abac to b3cbf0678487 (6 revisions) (flutter/flutter#125529) 2023-04-25 thkim1011@users.noreply.github.com Handle dollar signs properly when generating localizations (flutter/flutter#125514) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 6d798393a15a to 34ece7a4abac (1 revision) (flutter/flutter#125516) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 469e5b409686 to 6d798393a15a (2 revisions) (flutter/flutter#125513) 2023-04-25 47866232+chunhtai@users.noreply.github.com Adds selected semantics flag to radio button for Apple devices (flutter/flutter#125499) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 0e236982086b to 469e5b409686 (1 revision) (flutter/flutter#125510) 2023-04-25 58529443+srujzs@users.noreply.github.com Relabel JSFunction as JSExportedDartFunction (flutter/flutter#125453) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 60c4e61416cd to 0e236982086b (2 revisions) (flutter/flutter#125504) 2023-04-25 jmccandless@google.com iOS TextField spell check style (flutter/flutter#125432) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 308bce499087 to 60c4e61416cd (1 revision) (flutter/flutter#125490) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 54ddef68b124 to 308bce499087 (1 revision) (flutter/flutter#125485) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 307416f43b5a to 54ddef68b124 (1 revision) (flutter/flutter#125481) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 232810b36671 to 307416f43b5a (1 revision) (flutter/flutter#125475) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 1c4b6c0cb862 to 232810b36671 (2 revisions) (flutter/flutter#125472) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from a26a479e86b3 to 1c4b6c0cb862 (1 revision) (flutter/flutter#125463) 2023-04-25 tessertaha@gmail.com Fix `SliverAppBar.medium` & `SliverAppBar.large` text scale (flutter/flutter#125038) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 3f4116c225e8 to a26a479e86b3 (1 revision) (flutter/flutter#125459) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 64882f4b7371 to 3f4116c225e8 (2 revisions) (flutter/flutter#125458) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 8fe8e94c02fc to 64882f4b7371 (1 revision) (flutter/flutter#125454) 2023-04-25 engine-flutter-autoroll@skia.org Roll Flutter Engine from 5fbde6c0fc57 to 8fe8e94c02fc (10 revisions) (flutter/flutter#125452) 2023-04-24 tessertaha@gmail.com Add Sliders to `macrobenchmarks` (flutter/flutter#125296) 2023-04-24 pateltirth454@gmail.com [date_picker] [date_range_picker] add properties to change switch-to icons (flutter/flutter#124881) 2023-04-24 thkim1011@users.noreply.github.com Sliver Constrained Cross Axis (flutter/flutter#125239) 2023-04-24 kevmoo@users.noreply.github.com tool: Move cdKey to CustomDimensionsEnum (flutter/flutter#125335) 2023-04-24 bernaferrari2@gmail.com Add `borderRadius` to LinearProgressIndicator (flutter/flutter#123517) 2023-04-24 58529443+srujzs@users.noreply.github.com Fix JS types in _test_http_request.dart (flutter/flutter#125220) 2023-04-24 tessertaha@gmail.com Fix `OutlinedButton`, `TextButton`, and `IconButton` throw exception when passing only one cursor to `styleFrom` (flutter/flutter#125204) 2023-04-24 magder@google.com Swap some iOS TESTOWNERS (flutter/flutter#125340) 2023-04-24 rmolivares@renzo-olivares.dev Fix iOS touch drag behavior (flutter/flutter#125169) 2023-04-24 54558023+keyonghan@users.noreply.github.com Mark module_test_ios flaky (flutter/flutter#125426) 2023-04-24 47866232+chunhtai@users.noreply.github.com Implement url support for RouteInformation and didPushRouteInformation (flutter/flutter#119968) 2023-04-24 kevmoo@users.noreply.github.com [tool,web] Include more info URI on Wasm init failure (flutter/flutter#125312) 2023-04-24 magder@google.com Do not run `windows_home_scroll_perf__timeline_summary` benchmark in presubmit (flutter/flutter#125343) 2023-04-24 5236035+fzyzcjy@users.noreply.github.com Tiny fix doc pointing to incorrect widget confusing the readers (flutter/flutter#125394) ...

tomassasovsky · 2023-08-26T19:04:40Z

This broke our use case:

    "onboardingDescription": "{count, plural, =0{$onboardingFirstDescription} =1{$onboardingSecondDescription} =2{$onboardingThirdDescription} other{ERROR: onboardingDescription for this index not found}}",
    "@onboardingDescription": {
        "placeholders": {
            "count": {
                "type": "int"
            }
        }
    },
    "onboardingFirstDescription": "View your upcoming and past games. Select a game and record it by pressing the red button.",
    "onboardingSecondDescription": "To record a game follow the set-up instructions. Once finished recording, save and upload later to get it analyzed.",
    "onboardingThirdDescription": "Upload your game's videos in My Recordings.",

Used to generate:

  @override
  String onboardingDescription(int count) {
    String _temp0 = intl.Intl.pluralLogic(
      count,
      locale: localeName,
      other: 'ERROR: onboardingDescription for this index not found',
      two: '$onboardingThirdDescription',
      one: '$onboardingSecondDescription',
      zero: '$onboardingFirstDescription',
    );
    return '$_temp0';
  }

And now generates:

  @override
String onboardingDescription(int count) {
  String _temp0 = intl.Intl.pluralLogic(
    count,
    locale: localeName,
    other: 'ERROR: onboardingDescription for this index not found',
    two: '\$onboardingThirdDescription',
    one: '\$onboardingSecondDescription',
    zero: '\$onboardingFirstDescription',
  );
  return '$_temp0';
}

init

8fba51a

flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Apr 25, 2023

thkim1011 requested review from HansMuller and christopherfujino April 25, 2023 21:14

christopherfujino approved these changes Apr 25, 2023

View reviewed changes

thkim1011 added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 25, 2023

HansMuller reviewed Apr 25, 2023

View reviewed changes

thkim1011 removed the autosubmit Merge PR when tree becomes green via auto submit App label Apr 25, 2023

thkim1011 and others added 2 commits April 25, 2023 14:28

Update packages/flutter_tools/lib/src/localizations/localizations_uti…

885b011

…ls.dart Co-authored-by: Hans Muller <hansmuller@google.com>

address comments

1e22cfc

thkim1011 added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 25, 2023

HansMuller approved these changes Apr 25, 2023

View reviewed changes

auto-submit bot merged commit 8be335f into flutter:master Apr 25, 2023

engine-flutter-autoroll mentioned this pull request Apr 26, 2023

Roll Flutter from c9004ff822cf to b0ae2e81bab5 (66 revisions) flutter/packages#3827

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 26, 2023

8be335f Handle dollar signs properly when generating localizations (f…

f694762

…lutter/flutter#125514)

engine-flutter-autoroll mentioned this pull request Apr 26, 2023

Roll Flutter from c9004ff822cf to c9d7bed4216b (67 revisions) flutter/packages#3829

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 26, 2023

8be335f Handle dollar signs properly when generating localizations (f…

ee58f44

…lutter/flutter#125514)

engine-flutter-autoroll mentioned this pull request Apr 26, 2023

Roll Flutter from c9004ff822cf to 66fa4c5d301c (68 revisions) flutter/packages#3830

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 26, 2023

8be335f Handle dollar signs properly when generating localizations (f…

075acee

…lutter/flutter#125514)

HansMuller mentioned this pull request Apr 28, 2023

Flutter Framework Team: Notable Commits #121415

Closed

tomassasovsky mentioned this pull request Sep 2, 2023

[gen-l10n] nested localizations broken #133919

Open

2 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Handle dollar signs properly when generating localizations #125514

Handle dollar signs properly when generating localizations #125514

Uh oh!

thkim1011 commented Apr 25, 2023 •

edited

Loading

Uh oh!

christopherfujino left a comment

Uh oh!

HansMuller Apr 25, 2023

Uh oh!

thkim1011 Apr 25, 2023

Uh oh!

Uh oh!

HansMuller Apr 25, 2023

Uh oh!

thkim1011 Apr 25, 2023

Uh oh!

HansMuller left a comment

Uh oh!

tomassasovsky commented Aug 26, 2023 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Handle dollar signs properly when generating localizations #125514

Handle dollar signs properly when generating localizations #125514

Uh oh!

Conversation

thkim1011 commented Apr 25, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

christopherfujino left a comment

Choose a reason for hiding this comment

Uh oh!

HansMuller Apr 25, 2023

Choose a reason for hiding this comment

Uh oh!

thkim1011 Apr 25, 2023

Choose a reason for hiding this comment

Uh oh!

Uh oh!

HansMuller Apr 25, 2023

Choose a reason for hiding this comment

Uh oh!

thkim1011 Apr 25, 2023

Choose a reason for hiding this comment

Uh oh!

HansMuller left a comment

Choose a reason for hiding this comment

Uh oh!

tomassasovsky commented Aug 26, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

thkim1011 commented Apr 25, 2023 •

edited

Loading

tomassasovsky commented Aug 26, 2023 •

edited

Loading