Improve file name validator to support -, (, ), comma and space

LinwoodDev · Oct 13, 2023 · c84bcb6 · c84bcb6
1 parent 4543de1
commit c84bcb6
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/app/lib/api/file_system/file_system.dart b/app/lib/api/file_system/file_system.dart
@@ -19,7 +19,7 @@ abstract class GeneralFileSystem {
   RemoteStorage? get remote => null;
 
   String convertNameToFile(String name) {
-    return '${name.replaceAll(RegExp(r'[^a-zA-Z0-9]'), '_')}.bfly';
+    return name.replaceAll(RegExp(r'[^a-zA-Z0-9\-_() ,]'), '_');
   }
 
   Future<String> _findAvailableName(
@@ -180,7 +180,8 @@ abstract class DocumentFileSystem extends GeneralFileSystem {
     if (path.endsWith('/')) {
       path = path.substring(0, path.length - 1);
     }
-    return createFile('$path/${document.name}.bfly', document.save());
+    return createFile('$path/${convertNameToFile(document.name ?? '')}.bfly',
+        document.save());
   }
 }
 

diff --git a/app/lib/views/files.dart b/app/lib/views/files.dart
@@ -295,10 +295,7 @@ class _FilesViewState extends State<FilesView> {
                       if (asset == null) return;
                       final path = _locationController.text;
                       var newPath =
-                          '$path/${_fileSystem.convertNameToFile(name!)}';
-                      if (!newPath.endsWith('.bfly')) {
-                        newPath += '.bfly';
-                      }
+                          '$path/${_fileSystem.convertNameToFile(name!)}.bfly';
                       await _fileSystem.updateDocument(
                           newPath,
                           asset.createDocument(

diff --git a/fastlane/metadata/android/en-US/changelogs/74.txt b/fastlane/metadata/android/en-US/changelogs/74.txt
@@ -12,6 +12,7 @@
 * Fix tool reordering
 * Improve default pattern templates values
 * Improve documentation
+* Improve file name validator to support -, (, ), comma and space
 * Switch back to flutter beta
 
 View all changes in the blog: https://linwood.dev/butterfly/2.0.0-beta.10