Skip to content

Commit b0aa7b5

Browse files
authored
Merge pull request #3 from GSiesto/fix-filename-spaces
Fix filename spaces
2 parents bbcc01f + 68e613c commit b0aa7b5

2 files changed

Lines changed: 6 additions & 5 deletions

File tree

src/components/pdf-processor/ProcessingOptions.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -196,8 +196,7 @@ export default function ProcessingOptions({ mode, options, onOptionsChangeAction
196196
[field]: value,
197197
};
198198
const cleanedEntries = Object.entries(nextMeta)
199-
.map(([key, val]) => [key, typeof val === 'string' ? val.trim() : ''] as const)
200-
.filter(([, val]) => val.length > 0);
199+
.filter(([, val]) => typeof val === 'string' && val.length > 0);
201200

202201
onOptionsChangeAction({
203202
...options,

src/components/pdf-processor/index.tsx

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -294,9 +294,11 @@ export default function PDFProcessor({ initialMode = 'merge' }: { initialMode?:
294294
if (operation === 'merge' && processingOptions.metadata?.title) {
295295
const customTitle = processingOptions.metadata.title.trim();
296296
if (customTitle) {
297-
// Sanitize the custom title to be safe for filenames
298-
const sanitizedTitle = customTitle.replace(/[^a-zA-Z0-9-_ ]/g, '').replace(/\s+/g, '_');
299-
return ensurePdfExtension(`${sanitizedTitle}_pdflince`);
297+
// Sanitize the custom title to be safe for filenames but preserve spaces, accents, &, and ()
298+
const sanitizedTitle = customTitle.replace(/[^\p{L}\p{N}\-_ &()]/gu, '').trim();
299+
if (sanitizedTitle) {
300+
return ensurePdfExtension(sanitizedTitle);
301+
}
300302
}
301303
}
302304

0 commit comments

Comments
 (0)