fix: 修复水印配置缺失时 xlsx 预览白页

[yuhaibohotmail]

问题

在未提供外部 config/application.properties 的部署上，预览任何 xlsx 都失败（5.0.2 与当前 master 都有）。

页面从 <html> 开始正常输出、断在半截 JavaScript 里、永远没有 </html>，浏览器上表现为白页或一直转圈。
HTTP 状态码是 200，服务端日志里只有一句「响应已提交、错误页渲染不了」——它长得像转换超时，不像模板错，
所以很容易一路去查 LibreOffice 和转换链路。

真正的报错是：

FreeMarker template error:
The following has evaluated to null or missing:
==> watermarkTxt  [in template "officeweb.ftl" at line 25, column 31]

原因

两处叠在一起，缺一个都不会出事：

1. 十个 watermark* 请求属性可能整族不存在。

WatermarkConfigConstants 没有任何类级注解（对比 ConfigConstants 上的 @Component），
所以它不是 Spring bean，那十个 @Value("${watermark.xxx:默认值}") 的 setter 从来不会被调用。
唯一会填充那十个静态字段的是 ConfigRefreshComponent#loadConfig()，而它在配置文件不存在时直接 return：

if (!Files.exists(configPath)) {
    LOGGER.warn("配置文件不存在: {}", configFilePath);
    return;
}

此时十个 getter 全部返回 null，而 AttributeSetFilter 的 request.setAttribute(name, null)
按 Servlet 规范等同于删除属性——不是「值是 null」，是「这个属性不存在」。

2. officeweb.ftl 是唯一没有 classic_compatible 兜底的预览模板。

commonHeader.ftl 第 1 行是 <#setting classic_compatible=true>，而这个设置作用于整个渲染环境、
覆盖到 include 它的那份模板后续的引用（已单独用 FreeMarker 验过）。于是 csv / markdown / pdf / picture 等
所有 include 它的模板在属性缺失时把变量渲染成空串、照常出页面；
officeweb.ftl 既不 include 它、自己也没有这个设置，所以只有 xlsx 预览会硬失败。

修复

给 officeweb.ftl 里那十个变量补上 FreeMarker 缺省值，取的就是 WatermarkConfigConstants.DEFAULT_*
那一组（10 / 10 / 微软雅黑 / 18px / black / 0.2 / 240 / 80 / 10），不另编一套。

watermarkTxt 取空串——空串会让模板下一行的 if (watermarkTxt !== '') 跳过 watermark.init()，
不会给所有预览凭空加上水印。其余九个只在水印真的开着时才用得上，但那个 if 是 JavaScript 的、
不是模板的，FreeMarker 两条分支都会渲染，所以十个都要给（只补 watermarkTxt 的话，页面会改为断在
下一个变量 watermarkXSpace 上，症状一模一样）。

六个数值缺省值写成字符串（${watermarkAlpha!'0.2'}，不是 ${watermarkAlpha!0.2}），与 Java 侧
DEFAULT_* 的类型一致。数值缺省值会走 FreeMarker 按 locale 的数字格式：请求带 Accept-Language: de-DE
或 fr-FR 时 0.2 输出成 0,2，ar-EG 时数字变成阿拉伯-印度数字，内联脚本解析失败，预览照样起不来
——而此时响应是 200、页面也有 </html>（感谢 review 指出并复现）。已配置的值本来就是字符串，不受影响。

验证

新增 server/src/test/java/cn/keking/web/OfficeWebWatermarkDefaultsTests.java，直接用 FreeMarker
渲染这个模板——不需要把服务跑起来，也不需要 LibreOffice。模板按请求的 locale 取用，与 Spring
FreeMarkerView 一致：

• shouldRenderXlsxPreviewPageWhenWatermarkAttributesAreMissing：数据模型里一个 watermark* 都不给
  （那正是缺陷发生时的情形），按 zh-CN / en-US / de-DE / fr-FR / ar-EG 各渲染一次，断言渲染得出
  </html>、watermarkTxt 落到空串，并逐个断言其余九个值的输出文本。
  ⚠ 只查 </html> 或 HTTP 200 都不够：模板报错时响应头早已发出，也是 200；数字被本地化时页面完整、
  但脚本解析不了。所以判据落在每个值实际输出的那一行。
• shouldKeepConfiguredWatermarkSettingsWhenAttributesArePresent：十个值全给上，按 zh-CN / de-DE / ar-EG
  各跑一次，断言逐个原样渲染，保证缺省值不会盖掉真实配置。

改模板之前先跑过：没有缺省值时，缺属性那条报 InvalidReferenceException（第 25 行 watermarkTxt）；
数值缺省值时，de-DE、fr-FR 红在 watermark_alpha: 0,2，ar-EG 红在 watermark_x_space: ١٠，其余通过
——说明红的确实是被测的那个条件。改完之后：

mvn -B -pl server -Dtest=OfficeWebWatermarkDefaultsTests test
Tests run: 8, Failures: 0, Errors: 0, Skipped: 0
BUILD SUCCESS

没有一并改的

• commonHeader.ftl 与 pdf.ftl 里也是裸 ${watermark*}，今天靠 classic_compatible=true 兜着，
  没跟着改——改动越小越好审。
• 更彻底的修法是让那十个值不可能为 null：给 WatermarkConfigConstants 加 @Component
  （让它的 @Value 缺省值真正生效），或让 getter 回落到 DEFAULT_*。那会动到配置装载的语义，
  超出这个 PR 的范围；如果维护者更倾向那条路，我可以照着改成那一版。
• 顺带一个不影响本 PR 的不一致，供参考：server/src/main/config/application.properties 里
  watermark.width 默认是 180，而 WatermarkConfigConstants.DEFAULT_WATERMARK_WIDTH 是 240。
  本 PR 取的是后者（Java 侧声明的默认值）。

[klboke]

已核对当前提交 c4e1929 的模板、配置加载和请求属性链路。水印配置缺失导致 FreeMarker 渲染中断的根因成立，采用模板缺省值作为局部修复也合理。不过还有一个已复现的 locale 问题需要在合并前修复，详见行内评论。

验证情况：

• 本地运行 OfficeWebWatermarkDefaultsTests，2 条测试通过；以 -DargLine="-Duser.language=de -Duser.country=DE" 再运行也通过，说明现有断言未覆盖默认透明度的错误输出。
• 使用项目实际依赖中的 Spring FreeMarkerView、模拟请求/响应及 AcceptHeaderLocaleResolver 渲染当前模板，并对生成的内联脚本运行 node --check：缺少全部水印属性时，zh-CN 通过，de-DE、fr-FR、ar-EG 失败；提供完整水印配置时均通过。
• 在临时模板中将六个数值缺省值改为字符串后，上述四种语言的缺失配置和已有配置场景均通过 JavaScript 语法检查，已有配置值保持原样。

这次复测验证的是模板渲染和生成脚本的语法，不依赖 LibreOffice 或完整服务启动。

[klboke]

Re-reviewed head 45d7e34. The six numeric defaults are now string literals, so FreeMarker emits valid JavaScript numbers independently of the request locale. The empty watermark text keeps watermark rendering disabled when no text is configured, and supplied settings retain their original values.

The earlier locale blocker is fixed and the review thread is resolved. Validation against current master:

• All 24 related regression tests pass, including the 8 new watermark cases.
• Actual Spring FreeMarkerView rendering with AcceptHeaderLocaleResolver passes for missing and configured attributes across zh-CN, en-US, de-DE, fr-FR, and ar-EG: 10 complete responses and 20 inline scripts passing node --check.
• Linux, Windows, and macOS packaging and PR E2E are green on this head. The Actions checkout log confirms those runs used historical merge snapshot 9ec677f based on cd127fd, rather than current master. The local integration checks above use current master 5e3c013; I am synchronizing master into the PR and running fresh CI before merging.

No remaining blocking findings.

[klboke]

Reviewed the latest head 567c8b9, including synchronization with current master. The locale-sensitive defaults reported in the earlier review are fixed: all six numeric fallbacks are strings, missing watermark text disables watermark initialization, and configured settings are preserved. The original review thread is resolved.

Validation:

• All 24 related regression tests pass on the exact same file tree as this head, including the 8 watermark default/configuration cases.
• Actual Spring FreeMarkerView rendering with AcceptHeaderLocaleResolver passes for missing/configured attributes across zh-CN, en-US, de-DE, fr-FR, and ar-EG: 10 complete responses and 20 inline scripts passing node --check.
• Fresh Linux, Windows, and macOS packaging and PR E2E are all green for this head, which contains master 5e3c013 and the recent preview fixes.

No remaining blocking findings.