mirror of
https://github.com/storytold/pdfcraft.git
synced 2026-11-14 06:42:20 +00:00
* i18n: let the layers below hand their sentences over, so they can be translated The accessibility findings, the PDF/A problems, the signature details, the page-range mistakes and the edit labels were English in all ten languages, because matching their wording from the UI with a tl! literal would break silently whenever the engine changed it. So each layer now hands the wording over instead, apart from the document's own words: a WORDINGS list of every sentence it can say, with holes in it, and said(&words), which looks the wording up through the closure it is given and fills the holes in a single pass. in_english changes nothing, for the CLI and the tests; the UI passes &i18n::words, a catalog lookup. A wording with no entry comes back as it stands, which is the English, so the nine catalogs this does not touch read exactly as before. a11y::report_html takes a language tag and a words closure, so a saved report reads the way the panel read, down to its lang attribute. print::PrintError gained a variant per thing a person can be wrong about, with the numbers apart from the words; faults rather than mistakes keep their own text. Spanish translates all of it (107 entries). spanish_covers_what_the_engine_says requires an entry per wording in the eight lists and spanish_keeps_the_holes_in_what_the_engine_says requires the holes to survive the translation, so the catalog cannot fall behind the engine unnoticed. No new tl! literal, so the Chinese, French and Russian UI-literal gates are unaffected. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * Sign tests from #1099: compare against the &'static str kinds * Camera tests: undo returns a Label now --------- Co-authored-by: Fernando Areán <soporte@codebit.es> Co-authored-by: Claude Opus 5 <noreply@anthropic.com> Co-authored-by: Brandon Thomas <echelon@gmail.com>
217 lines
11 KiB
Rust
217 lines
11 KiB
Rust
use super::*;
|
|
|
|
fn build(objs: &[&str], trailer_extra: &str) -> Document {
|
|
let mut out = b"%PDF-1.7\n".to_vec();
|
|
let mut offsets = Vec::new();
|
|
for (i, o) in objs.iter().enumerate() {
|
|
offsets.push(out.len());
|
|
out.extend_from_slice(format!("{} 0 obj\n{o}\nendobj\n", i + 1).as_bytes());
|
|
}
|
|
let xref = out.len();
|
|
out.extend_from_slice(format!("xref\n0 {}\n0000000000 65535 f \n", objs.len() + 1).as_bytes());
|
|
for o in offsets {
|
|
out.extend_from_slice(format!("{o:010} 00000 n \n").as_bytes());
|
|
}
|
|
out.extend_from_slice(format!("trailer\n<< /Size {} /Root 1 0 R {trailer_extra} >>\nstartxref\n{xref}\n%%EOF\n", objs.len() + 1).as_bytes());
|
|
Document::open(std::sync::Arc::new(out)).unwrap()
|
|
}
|
|
|
|
/// A page with embedded-font-free text, a note without the print flag, a link with a
|
|
/// JavaScript action, an interpolated image, and a document JavaScript.
|
|
fn messy() -> Document {
|
|
build(
|
|
&[
|
|
"<< /Type /Catalog /Pages 2 0 R /Names << /JavaScript << /Names [(x) 9 0 R] >> >> /AA << /WC 9 0 R >> >>",
|
|
"<< /Type /Pages /Kids [3 0 R] /Count 1 /MediaBox [0 0 200 200] >>",
|
|
"<< /Type /Page /Parent 2 0 R /Contents 4 0 R /Resources << /Font << /F1 5 0 R >> /XObject << /Im 8 0 R >> >> /Annots [6 0 R 7 0 R] >>",
|
|
"<< /Length 44 >>\nstream\n1 0 0 rg BT /F1 12 Tf 10 10 Td (Hi) Tj ET /Im Do\nendstream",
|
|
"<< /Type /Font /Subtype /Type1 /BaseFont /Helvetica >>",
|
|
"<< /Type /Annot /Subtype /Text /Rect [10 10 30 30] /P 3 0 R /AP << /N 10 0 R >> >>",
|
|
"<< /Type /Annot /Subtype /Link /Rect [40 10 80 30] /P 3 0 R /F 4 /A 9 0 R >>",
|
|
"<< /Type /XObject /Subtype /Image /Width 1 /Height 1 /ColorSpace /DeviceGray /BitsPerComponent 8 /Interpolate true /Length 1 >>\nstream\n\u{0}\nendstream",
|
|
"<< /Type /Action /S /JavaScript /JS (app.alert(1)) >>",
|
|
"<< /Type /XObject /Subtype /Form /BBox [0 0 20 20] /Length 0 >>\nstream\n\nendstream",
|
|
],
|
|
"/Info << /Title (Report) /Producer (Test) /CreationDate (D:20240105093000Z) >>",
|
|
)
|
|
}
|
|
|
|
fn clauses(issues: &[Issue]) -> Vec<&'static str> {
|
|
let mut c: Vec<&str> = issues.iter().map(|i| i.clause).collect();
|
|
c.dedup();
|
|
c
|
|
}
|
|
|
|
#[test]
|
|
fn verify_reports_what_breaks_pdfa() {
|
|
let issues = verify(&messy(), Level::A2b);
|
|
let msgs: Vec<&str> = issues.iter().map(|i| i.message.as_str()).collect();
|
|
for want in [
|
|
"The document has no XMP metadata",
|
|
"Device colour is used, but there is no PDF/A output intent",
|
|
"The document has additional actions (/AA)",
|
|
"The document has document-level JavaScript",
|
|
"A JavaScript action isn't allowed",
|
|
"A Text annotation isn't set to print, or is hidden",
|
|
"The font Helvetica isn't embedded",
|
|
"An image asks to be interpolated",
|
|
] {
|
|
assert!(msgs.contains(&want), "missing {want:?} in {msgs:#?}");
|
|
}
|
|
assert!(issues.iter().find(|i| i.message.starts_with("The font")).is_some_and(|i| !i.fixable));
|
|
assert!(clauses(&issues).contains(&"6.6.2.1"));
|
|
// Each problem is also worded separately from what the document calls things, so an interface
|
|
// can say it in another language; in English the two come to the same sentence.
|
|
for i in &issues {
|
|
assert_eq!(i.said(&in_english), i.message);
|
|
assert!(WORDINGS.contains(&i.pattern), "{:?} is not in WORDINGS", i.pattern);
|
|
}
|
|
let font = issues.iter().find(|i| i.pattern == "The font {} isn't embedded").unwrap();
|
|
assert_eq!(font.said(&|w: &str| format!("«{w}»")), "«The font {} isn't embedded»".replace("{}", "Helvetica"));
|
|
}
|
|
|
|
/// Every problem the rules can report is listed in `WORDINGS`, so an interface showing them in
|
|
/// another language has something to translate. The wording is `verify`'s second argument, which is
|
|
/// what this reads off the file; the sentences `convert` reports as fixed are not wordings.
|
|
#[test]
|
|
fn wordings_are_complete() {
|
|
let source = include_str!("lib.rs");
|
|
let mut said = Vec::new();
|
|
let mut rest = source;
|
|
while let Some(at) = rest.find("issue(\"") {
|
|
rest = &rest[at + 7..];
|
|
// Step over the clause, then take the wording that follows it.
|
|
let Some(end) = rest.find('"') else { break };
|
|
rest = &rest[end + 1..];
|
|
let Some(text) = rest.strip_prefix(", \"") else { continue };
|
|
let Some(end) = text.find('"') else { break };
|
|
said.push(&text[..end]);
|
|
rest = text;
|
|
}
|
|
assert!(said.len() >= 24, "the wordings are not being read off the file: {said:#?}");
|
|
let missing: Vec<&&str> = said.iter().filter(|word| !WORDINGS.contains(word)).collect();
|
|
assert!(missing.is_empty(), "add these to WORDINGS, and a translation of each to every interface: {missing:#?}");
|
|
let unsaid: Vec<&&str> = WORDINGS.iter().filter(|word| !said.contains(word)).collect();
|
|
assert!(unsaid.is_empty(), "WORDINGS lists problems the rules no longer report: {unsaid:#?}");
|
|
}
|
|
|
|
#[test]
|
|
fn convert_fixes_all_but_fonts() {
|
|
let mut doc = messy();
|
|
let r = convert(&mut doc, Level::A2b).unwrap();
|
|
assert!(r.fixed.iter().any(|f| f.contains("output intent")), "{:?}", r.fixed);
|
|
let left: Vec<&str> = r.remaining.iter().map(|i| i.message.as_str()).collect();
|
|
assert_eq!(left, ["The font Helvetica isn't embedded"], "{:#?}", r.remaining);
|
|
let d = declared(&doc);
|
|
assert_eq!(d.pdfa, Some((2, "B".into())));
|
|
assert_eq!(d.output_intents, ["sRGB IEC61966-2.1"]);
|
|
// The metadata carries the document information.
|
|
let x = metadata(&doc).unwrap();
|
|
assert_eq!(xmp::value(&x, "dc:title").as_deref(), Some("Report"));
|
|
assert_eq!(xmp::value(&x, "xmp:CreateDate").as_deref(), Some("2024-01-05T09:30:00Z"));
|
|
// It survives a save.
|
|
let bytes = pdfcraft_cos::write_full(&doc, &Default::default()).unwrap();
|
|
let back = Document::open(std::sync::Arc::new(bytes)).unwrap();
|
|
assert_eq!(verify(&back, Level::A2b).len(), 1);
|
|
}
|
|
|
|
#[test]
|
|
fn embedded_files_differ_between_parts() {
|
|
let doc = build(
|
|
&[
|
|
"<< /Type /Catalog /Pages 2 0 R /Names << /EmbeddedFiles << /Names [(a.txt) 4 0 R] >> >> >>",
|
|
"<< /Type /Pages /Kids [3 0 R] /Count 1 /MediaBox [0 0 200 200] >>",
|
|
"<< /Type /Page /Parent 2 0 R >>",
|
|
"<< /Type /Filespec /F (a.txt) /EF << /F 5 0 R >> >>",
|
|
"<< /Type /EmbeddedFile /Length 2 >>\nstream\nhi\nendstream",
|
|
],
|
|
"",
|
|
);
|
|
assert!(verify(&doc, Level::A2b).iter().any(|i| i.clause == "6.8" && !i.fixable));
|
|
let mut d3 = doc.clone();
|
|
assert!(verify(&d3, Level::A3b).iter().any(|i| i.message.contains("AFRelationship")));
|
|
let r = convert(&mut d3, Level::A3b).unwrap();
|
|
assert!(r.remaining.is_empty(), "{:#?}", r.remaining);
|
|
}
|
|
|
|
/// A page that paints a DeviceCMYK image (as `doc_create` makes from a CMYK JPEG), with
|
|
/// `intent` as the catalog's extra entries and `extra` as more objects (from object 6 on).
|
|
fn cmyk_page(intent: &str, content: &str, extra: &[&str]) -> Document {
|
|
let catalog = format!("<< /Type /Catalog /Pages 2 0 R {intent} >>");
|
|
let stream = format!("<< /Length {} >>\nstream\n{content}\nendstream", content.len());
|
|
let mut objs = vec![
|
|
catalog.as_str(),
|
|
"<< /Type /Pages /Kids [3 0 R] /Count 1 /MediaBox [0 0 200 200] >>",
|
|
"<< /Type /Page /Parent 2 0 R /Contents 4 0 R /Resources << /XObject << /Im0 5 0 R >> >> >>",
|
|
stream.as_str(),
|
|
"<< /Type /XObject /Subtype /Image /Width 1 /Height 1 /ColorSpace /DeviceCMYK /BitsPerComponent 8 /Length 4 >>\nstream\n\u{0}\u{0}\u{0}\u{0}\nendstream",
|
|
];
|
|
objs.extend_from_slice(extra);
|
|
build(&objs, "")
|
|
}
|
|
|
|
fn messages(issues: &[Issue]) -> Vec<&str> {
|
|
issues.iter().map(|i| i.message.as_str()).collect()
|
|
}
|
|
|
|
/// Issue #667: converting a CMYK-only document doesn't add an sRGB output intent (it would
|
|
/// describe a colour space the document doesn't use); the CMYK output intent it needs is
|
|
/// reported as not fixable instead.
|
|
#[test]
|
|
fn convert_does_not_give_cmyk_documents_an_srgb_output_intent() {
|
|
let mut doc = cmyk_page("", "q 200 0 0 200 0 0 cm /Im0 Do Q", &[]);
|
|
let r = convert(&mut doc, Level::A2b).unwrap();
|
|
assert!(!r.fixed.iter().any(|f| f.contains("output intent")), "{:?}", r.fixed);
|
|
assert!(declared(&doc).output_intents.is_empty());
|
|
let left = messages(&r.remaining);
|
|
assert!(left.contains(&"Device colour is used, but there is no PDF/A output intent"), "{left:#?}");
|
|
assert!(left.contains(&"DeviceCMYK is used: it needs a CMYK output intent (not added automatically)"), "{left:#?}");
|
|
assert!(r.remaining.iter().all(|i| !i.fixable));
|
|
}
|
|
|
|
/// Issue #667: verify checks the output intent's colour space against the device colour the
|
|
/// pages use, not only that an output intent exists.
|
|
#[test]
|
|
fn verify_reports_an_output_intent_that_does_not_match_device_colour() {
|
|
let intent = "/OutputIntents [<< /Type /OutputIntent /S /GTS_PDFA1 /OutputConditionIdentifier (X) /DestOutputProfile 6 0 R >>]";
|
|
let profile = |n: u8| format!("<< /N {n} /Length 0 >>\nstream\n\nendstream");
|
|
let (rgb, cmyk) = (profile(3), profile(4));
|
|
let cmyk_msg = "DeviceCMYK is used, but the PDF/A output intent isn't a CMYK profile";
|
|
let rgb_msg = "DeviceRGB is used, but the PDF/A output intent isn't an RGB profile";
|
|
|
|
// CMYK content under an sRGB output intent (what convert used to write).
|
|
let issues = verify(&cmyk_page(intent, "/Im0 Do", &[&rgb]), Level::A2b);
|
|
assert!(messages(&issues).contains(&cmyk_msg), "{issues:#?}");
|
|
assert!(issues.iter().find(|i| i.message == cmyk_msg).is_some_and(|i| i.clause == "6.2.4.3" && !i.fixable));
|
|
|
|
// CMYK content under a CMYK output intent is fine; RGB and gray content beside it is not
|
|
// and is, respectively.
|
|
assert!(!messages(&verify(&cmyk_page(intent, "/Im0 Do 0 g", &[&cmyk]), Level::A2b)).iter().any(|m| m.contains("output intent")));
|
|
let issues = verify(&cmyk_page(intent, "/Im0 Do 1 0 0 rg", &[&cmyk]), Level::A2b);
|
|
assert!(messages(&issues).contains(&rgb_msg) && !messages(&issues).contains(&cmyk_msg), "{issues:#?}");
|
|
|
|
// DefaultCMYK in the page's resources draws DeviceCMYK in that space, which PDF/A accepts
|
|
// under any output intent.
|
|
let catalog = format!("<< /Type /Catalog /Pages 2 0 R {intent} >>");
|
|
let doc = build(
|
|
&[
|
|
catalog.as_str(),
|
|
"<< /Type /Pages /Kids [3 0 R] /Count 1 /MediaBox [0 0 200 200] >>",
|
|
"<< /Type /Page /Parent 2 0 R /Contents 4 0 R /Resources << /ColorSpace << /DefaultCMYK [/ICCBased 7 0 R] >> /XObject << /Im0 5 0 R >> >> >>",
|
|
"<< /Length 7 >>\nstream\n/Im0 Do\nendstream",
|
|
"<< /Type /XObject /Subtype /Image /Width 1 /Height 1 /ColorSpace /DeviceCMYK /BitsPerComponent 8 /Length 4 >>\nstream\n\u{0}\u{0}\u{0}\u{0}\nendstream",
|
|
rgb.as_str(),
|
|
cmyk.as_str(),
|
|
],
|
|
"",
|
|
);
|
|
assert!(!messages(&verify(&doc, Level::A2b)).contains(&cmyk_msg), "DefaultCMYK covers DeviceCMYK");
|
|
|
|
// Without /N, the ICC header's colour space decides.
|
|
let mut header = vec![b' '; 128];
|
|
header[16..20].copy_from_slice(b"CMYK");
|
|
let header = String::from_utf8(header).unwrap();
|
|
let headed = format!("<< /Length 128 >>\nstream\n{header}\nendstream");
|
|
assert!(!messages(&verify(&cmyk_page(intent, "/Im0 Do", &[&headed]), Level::A2b)).contains(&cmyk_msg));
|
|
}
|