fix #890: change separator for splitting the path

Change the pattern from ", " to ", alias " when splitting the output
of osascript. Although this is not a perfect solution, it allows to pick
filenames that contain a comma followed by a blank (", ").

Author: philenius

lib/src/file_picker_macos.dart

@@ -158,21 +158,32 @@ class FilePickerMacOS extends FilePicker {
       .replaceAll('\n', '\\\n');
 
   /// Transforms the result string (stdout) of `osascript` into a [List] of
-  /// file paths.
+  /// POSIX file paths.
   List<String> resultStringToFilePaths(String fileSelectionResult) {
     if (fileSelectionResult.trim().isEmpty) {
       return [];
     }
-    return fileSelectionResult
+
+    final paths = fileSelectionResult
         .trim()
-        .split(', ')
+        .split(', alias ')
         .map((String path) => path.trim())
         .where((String path) => path.isNotEmpty)
-        .map((String path) {
-      final pathElements = path.split(':').where((e) => e.isNotEmpty).toList();
+        .toList();
+
+    if (paths.length == 1 && paths.first.startsWith('file ')) {
+      // The first token of the first path is "file" in case of the save file
+      // dialog
+      paths[0] = paths[0].substring(5);
+    } else if (paths.isNotEmpty && paths.first.startsWith('alias ')) {
+      // The first token of the first path is "alias" in case of the
+      // file/directory picker dialog
+      paths[0] = paths[0].substring(6);
+    }
 
-      // First token is either "alias" or "file" (in case of saveFile dialog)
-      final volumeName = pathElements[0].split(' ').sublist(1).join(' ');
+    return paths.map((String path) {
+      final pathElements = path.split(':').where((e) => e.isNotEmpty).toList();
+      final volumeName = pathElements[0];
       return ['/Volumes', volumeName, ...pathElements.sublist(1)].join('/');
     }).toList();
   }

test/file_picker_macos_test.dart

@@ -189,6 +189,39 @@ void main() {
         equals('/Volumes/TAILS 4.20 - 202/EFI/debian/grub/x86_64-efi'),
       );
     });
+
+    test('should interpret the result of picking filenames that contain blanks and commas',
+        () {
+      final picker = FilePickerMacOS();
+
+      final filePaths = picker.resultStringToFilePaths(
+        'alias Macintosh:Users:JohnDoe:test, test.csv, alias macOS Base System:bin:unicorn , generator.sh',
+      );
+
+      expect(filePaths.length, equals(2));
+      expect(
+        filePaths[0],
+        equals('/Volumes/Macintosh/Users/JohnDoe/test, test.csv'),
+      );
+      expect(
+        filePaths[1],
+        equals('/Volumes/macOS Base System/bin/unicorn , generator.sh'),
+      );
+    });
+
+    test('should interpret the result of the save file dialog', () {
+      final picker = FilePickerMacOS();
+
+      final filePaths = picker.resultStringToFilePaths(
+        'file macOS:Users:JohnDoe:Desktop:bill.pdf',
+      );
+
+      expect(filePaths.length, equals(1));
+      expect(
+        filePaths[0],
+        equals('/Volumes/macOS/Users/JohnDoe/Desktop/bill.pdf'),
+      );
+    });
   });
 
   group('generateCommandLineArguments()', () {