Просмотр исходного кода

Allow digits in PDF filenames (#13122)

GitOrigin-RevId: 58c93d299c3bfe79dd4c223b8b060411bad078c7
Alf Eaton 3 лет назад
Родитель
Сommit
4670c698ee

+ 1 - 1
services/web/app/src/Features/Compile/CompileController.js

@@ -371,7 +371,7 @@ module.exports = CompileController = {
   },
 
   _getSafeProjectName(project) {
-    return project.name.replace(/\P{L}/gu, '_')
+    return project.name.replace(/[^\p{L}\p{Nd}]/gu, '_')
   },
 
   deleteAuxFiles(req, res, next) {

+ 2 - 2
services/web/test/unit/src/Compile/CompileControllerTests.js

@@ -356,7 +356,7 @@ describe('CompileController', function () {
       this.req.params = { Project_id: this.projectId }
 
       this.req.query = { pdfng: true }
-      this.project = { name: 'test namè;' }
+      this.project = { name: 'test namè; 1' }
       this.ProjectGetter.getProject = sinon
         .stub()
         .callsArgWith(2, null, this.project)
@@ -380,7 +380,7 @@ describe('CompileController', function () {
 
       it('should set the content-disposition header with a safe version of the project name', function () {
         this.res.setContentDisposition.should.be.calledWith('inline', {
-          filename: 'test_namè_.pdf',
+          filename: 'test_namè__1.pdf',
         })
       })