fix: Remove unnecessary ending comma on Share or View on the web (#…

…4887)
openfoodfacts · Dec 10, 2023 · 5ffdca3 · 5ffdca3
1 parent 05a1328
commit 5ffdca3
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/packages/smooth_app/lib/helpers/temp_product_list_share_helper.dart b/packages/smooth_app/lib/helpers/temp_product_list_share_helper.dart
@@ -3,15 +3,11 @@ import 'package:smooth_app/query/product_query.dart';
 
 // TODO(m123): Move this to off-dart
 Uri shareProductList(List<String> barcodes) {
-  final StringBuffer buffer = StringBuffer();
-
-  for (final String i in barcodes) {
-    buffer.write('$i,');
-  }
+  final String barcodesString = barcodes.join(',');
 
   return UriHelper.replaceSubdomain(
     ProductQuery.uriProductHelper.getUri(
-      path: 'products/$buffer',
+      path: 'products/$barcodesString',
       addUserAgentParameters: false,
     ),
     language: OpenFoodAPIConfiguration.globalLanguages?.first,