thswlsqls opened a new pull request, #8552:
URL: https://github.com/apache/paimon/pull/8552

   ### Purpose
   
   fix #8551
   
   - `Utils.escapeCharacters` escaped `<` and `>` but not `&`, so an `&` in a 
config option key/default/type/enum column was left unescaped in the generated 
HTML.
   - Add a `&` -> `&amp;` replacement first (before `<`/`>`) so the `&` in the 
produced `&lt;`/`&gt;` is not double-escaped.
   - Matches the sibling `HtmlFormatter.escapeCharacters` (paimon-api), which 
renders the Description column of the same table and already escapes `&`, `<`, 
`>`.
   
   ### Tests
   
   - Added `UtilsTest` covering `&` escaping, order correctness (`<a&b>` -> 
`&lt;a&amp;b&gt;`), `<wbr>` preservation, and unchanged `<`/`>` escaping.
   - `mvn -pl paimon-docs -DfailIfNoTests=false clean install` — 13 unit tests 
passed. Generated-docs diff is 0 (no existing option contains `&`), so 
`ConfigOptionsDocsCompletenessITCase` is unaffected and passed.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to