Use a better regex for removing invalid characters from filenames

Fixes atlas exports remove international characters from filenames Fixes #18291
qgis · Mar 3, 2018 · 8543a91264154dea6a7d4916f603975cbef25405 · 8543a91
1 parent 847683b
commit 8543a91264154dea6a7d4916f603975cbef25405
Unified Split

Showing with 3 additions and 3 deletions.

+1 −1 src/core/qgsfileutils.cpp

+2 −2 tests/src/python/test_qgsfileutils.py
diff --git a/src/core/qgsfileutils.cpp b/src/core/qgsfileutils.cpp
@@ -86,7 +86,7 @@ QString QgsFileUtils::addExtensionFromFilter( const QString &fileName, const QSt
 
 QString QgsFileUtils::stringToSafeFilename( const QString &string )
 {
-  QRegularExpression rx( "[^\\w\\-. ]" );
+  QRegularExpression rx( "[/\\\\\\?%\\*\\:\\|\"<>]" );
   QString s = string;
   s.replace( rx, QStringLiteral( "_" ) );
   return s;

diff --git a/tests/src/python/test_qgsfileutils.py b/tests/src/python/test_qgsfileutils.py
@@ -58,8 +58,8 @@ def testAddExtensionFromFilter(self):
     def testStringToSafeFilename(self):
         self.assertEqual(QgsFileUtils.stringToSafeFilename('my FiLe v2.0_new.tif'), 'my FiLe v2.0_new.tif')
         self.assertEqual(
-            QgsFileUtils.stringToSafeFilename('rendered map_final? rev (12-03-1017)_real@#$&*#%&*$!!@$%^&(*(.tif'),
-            'rendered map_final_ rev _12-03-1017__real____________________.tif')
+            QgsFileUtils.stringToSafeFilename('rendered map_final? rev (12-03-1017)_real/\\?%*:|"<>.tif'),
+            'rendered map_final_ rev (12-03-1017)_real__________.tif')
 
 
 if __name__ == '__main__':