Merge pull request #21459 from Snuffleupagus/PDFEditor-ternary-Dict-set

Use ternary expressions to shorten code in `src/core/editor/pdf_editor.js`
This commit is contained in:
Tim van der Meij 2026-06-16 20:50:05 +02:00 committed by GitHub
commit 8bc13d502b
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -674,21 +674,13 @@ class PDFEditor {
const classNames = node.get("C");
if (classNames instanceof Name) {
const newClassName = dedupClasses.get(classNames.name);
if (newClassName) {
newNode.set("C", Name.get(newClassName));
} else {
newNode.set("C", classNames);
}
newNode.set("C", newClassName ? Name.get(newClassName) : classNames);
} else if (Array.isArray(classNames)) {
const newClassNames = [];
for (const className of classNames) {
if (className instanceof Name) {
const newClassName = dedupClasses.get(className.name);
if (newClassName) {
newClassNames.push(Name.get(newClassName));
} else {
newClassNames.push(className);
}
newClassNames.push(newClassName ? Name.get(newClassName) : className);
}
}
newNode.set("C", newClassNames);
@ -698,11 +690,7 @@ class PDFEditor {
const roleName = node.get("S");
if (roleName instanceof Name) {
const newRoleName = dedupRoles.get(roleName.name);
if (newRoleName) {
newNode.set("S", Name.get(newRoleName));
} else {
newNode.set("S", roleName);
}
newNode.set("S", newRoleName ? Name.get(newRoleName) : roleName);
}
// Fix the ID.
@ -710,11 +698,7 @@ class PDFEditor {
if (typeof id === "string") {
const stringId = stringToPDFString(id, /* keepEscapeSequence = */ false);
const newId = dedupIDs.get(stringId);
if (newId) {
newNode.set("ID", stringToAsciiOrUTF16BE(newId));
} else {
newNode.set("ID", id);
}
newNode.set("ID", newId ? stringToAsciiOrUTF16BE(newId) : id);
}
// Table headers may contain IDs that need to be deduplicated.