OFBiz向Groovy脚本传参并通过FreeMarker展示数据的问题排查

核心问题定位 #

现有代码共有3处会导致功能异常的问题，按影响优先级排序：

• screen配置的set标签写法错误
  原代码中<set field="carId" from-field = "parameters"/>逻辑是将整个请求参数Map对象赋值给carId字段，而非提取请求参数中的carId值，属于字段取值逻辑错误。
• FTL模板存放路径不符合OFBiz安全规范
  原代码将FreeMarker模板list.ftl存放在groovyScripts目录下，OFBiz默认安全配置中该目录仅允许执行Groovy脚本，会拦截该目录下的模板资源访问，直接导致模板加载失败、页面报错。
• 公共装饰器路径依赖请求参数存在风险
  原screen中装饰器路径使用${parameters.mainDecoratorLocation}取值，自定义跳转链接默认不会携带该参数，会导致找不到公共装饰器screen，渲染失败。
• 非阻断类问题：FreeMarker模板中两行表头重复使用${uiLabelMap.CarId}，属于内容冗余，不影响运行但不符合展示逻辑。

修复方案 #

1. 修正screen中set标签的取值逻辑，直接提取parameters中的carId参数；将装饰器路径修改为组件内固定路径；将FTL模板移动到规范的模板存放目录。
2. 调整Groovy脚本，增加carId非空判断，避免空参数执行无效查询。
3. 修正FTL模板的冗余表头，按需补充车辆其他字段展示，增加无数据时的提示。
4. 检查前端跳转链接，确保触发点击时请求地址携带正确的carId参数，格式为目标screen地址?carId=实际车辆ID值。

修复后代码参考 #

1. screen定义代码 #

<screen name="carDetail">
    <section>
        <actions>
            <!-- 修正from-field取值，直接取参数中的carId -->
            <set field="carId" from-field="parameters.carId"/>
            <script location="component://ParcAutomobile/groovyScripts/find.groovy"/>
        </actions>
        <widgets>
            <!-- 替换为组件下装饰器的实际固定路径，避免依赖参数传值 -->
            <decorator-screen name="ParcAutomobileCommonDecorator" location="component://ParcAutomobile/widget/CommonScreens.xml">
                <decorator-section name="body">
                    <platform-specific>
                        <html>
                            <!-- 调整FTL到规范的模板存放目录，避免被安全规则拦截 -->
                            <html-template location="component://ParcAutomobile/template/list.ftl"></html-template>
                        </html>
                    </platform-specific>
                </decorator-section>
            </decorator-screen> 
        </widgets>
    </section>
</screen>

2. Groovy查询脚本代码 #

carId = parameters.carId
context.carId = carId
// 增加非空判断，避免无效查询
if (carId) {
    car = from("Car").where("carId", carId).queryOne()
    context.car = car
}

3. FreeMarker模板代码 #

<div>
<#if car?has_content>
<table border="1" cellpadding="5" cellspacing="0">
        <tr>
            <th>${uiLabelMap.CarId}</th>
            <td>${car.carId}</td>
        </tr>
        <!-- 第二行替换为实际需要展示的车辆字段，示例为车牌号码字段plateNumber，按自身实体字段调整 -->
        <tr>
            <th>${uiLabelMap.PlateNumber}</th>
            <td>${car.plateNumber!''}</td>
        </tr>
</table>
<#else>
<p>未查询到对应车辆信息</p>
</#if>
</div>

额外校验点 #

如果修复后仍无法正常展示，检查以下两点：

• 确认Car实体已经正确定义，且数据库中存在carId对应的数据记录
• 确认访问carDetail screen的请求地址正确携带了carId参数，无参数名拼写错误

内容的提问来源于stack exchange，提问作者badjo_dibe