From ccd3a0b432f12e4bba60cf5a519b8045f8fd5e26 Mon Sep 17 00:00:00 2001
From: Eric Kaden <608341+EricKaden@users.noreply.github.com>
Date: Thu, 22 Oct 2026 09:22:36 -0400
Subject: [PATCH] M6: a doc note that numbers_pages tests the script's text

---
 crates/xfa/src/parse.rs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/crates/xfa/src/parse.rs b/crates/xfa/src/parse.rs
index 777ceaf6de..42f93e20f1 100644
--- a/crates/xfa/src/parse.rs
+++ b/crates/xfa/src/parse.rs
@@ -648,7 +648,9 @@ fn parse_node(c: XmlNode, warnings: &mut Vec<String>, depth: usize) -> Option<No
 }
 
 /// Whether a `layout:ready` script uses page numbers (`xfa.layout.page(this)`, `pageCount()`): on a
-/// field, it makes the field one the layout numbers itself.
+/// field, it makes the field one the layout numbers itself. It looks for the names in the
+/// script's text and doesn't parse it, so a script that only mentions one (in a comment or a
+/// string) counts too, and its writes to its field are dropped like a page number's.
 pub fn numbers_pages(script: &str) -> bool {
     script.contains("pageCount") || script.contains("layout.page(") || script.contains("layout.absPage(")
 }