Steven, thanks for attaching the template - being able to see the fix is genuinely useful.
A question rather than a suggestion, since I don't write Lua and may be missing something. The guard matches specific literal values:
if value == "<div></div>" or value == "untitled" thenGiven that the "untitled" clause looks like an earlier sentinel from Adobe, and that this change by Adobe isn't easy to find, is there a case for also passing linkURL through the escape_xml function the template already uses for the gallery-level attributes? Something like:
linkURL="<%= escape_xml(clean_url(...)) %>"The idea would be that a slightly different future value would then produce a bad link rather than an unreadable file. But you will know whether that has downsides I cannot see.
Separately, just as an observation: empty Titles and Captions still come through as <div></div> with the new template. They are inside CDATA so nothing breaks - only mentioning it in case it was meant to be covered too.
Thanks again (Jos and Steven) for turning this around so quickly.