includeViewParams=true 将空模型值转换为查询字符串中的空字符串

Question

给定如下<p:selectOneMenu>。

<f:metadata>
    <f:viewParam name="id" value="#{testManagedBean.id}" converter="javax.faces.Long"/>
</f:metadata>
<p:selectOneMenu value="#{localeBean.language}" onchange="changeLanguage();">
    <f:selectItem itemValue="en" itemLabel="English" />
    <f:selectItem itemValue="hi" itemLabel="Hindi" />
</p:selectOneMenu>
<p:remoteCommand action="#{testManagedBean.submitAction}"
                 name="changeLanguage"
                 process="@this"
                 update="@none"/>

对应的托管 bean：

@ManagedBean
@RequestScoped
public final class TestManagedBean {
    private Long id; //Getter and setter.
    public TestManagedBean() {}
    public String submitAction() {
        return FacesContext.getCurrentInstance().getViewRoot().getViewId() + "?faces-redirect=true&includeViewParams=true";
    }
}

<f:viewParam>指示的参数是可选的。例如，使用如下所示的URL访问页面。

https://localhost:8181/Project-war/private_resources/Test.jsf

由于 id 是可选参数，因此 URL 将附加一个空参数(当语言从 <p:selectOneMenu> 更改为 时(，以防未按如下方式提供。

https://localhost:8181/Project-war/private_resources/Test.jsf?id=

这不应该发生。如果未提供空参数，则不应追加空参数，并且 URL 应类似于第一个参数。

有没有办法防止空参数在未传递时附加到 URL？

---

这仅与 <f:viewParam> - javax.faces.Long 指定的转换器相关联。

如果删除此转换器，则不会将参数附加到 URL，以防未提供任何参数。

尽管完全没有必要指定此处演示的转换器，但我有如下所示的转换器，用于将通过 URL 作为查询字符串参数传递的id转换为 JPA 实体。

@ManagedBean
@RequestScoped
public final class ZoneConverter implements Converter {
    @EJB
    private final SharableBeanLocal sharableService = null;
    @Override
    public Object getAsObject(FacesContext context, UIComponent component, String value) {
        try {
            long parsedValue = Long.parseLong(value);
            if (parsedValue <= 0) {
                throw new ConverterException(new FacesMessage(FacesMessage.SEVERITY_ERROR, "Message Summary", "Message"));
            }
            ZoneTable entity = sharableService.findZoneById(parsedValue);
            if (entity == null) {
                throw new ConverterException(new FacesMessage(FacesMessage.SEVERITY_WARN, "Message Summary", "Message"));
            }
            return entity;
        } catch (NumberFormatException e) {
            throw new ConverterException(new FacesMessage(FacesMessage.SEVERITY_ERROR, "Message Summary", "Message"), e);
        }
    }
    @Override
    public String getAsString(FacesContext context, UIComponent component, Object value) {
        return value instanceof ZoneTable ? ((ZoneTable) value).getZoneId().toString() : "";
    }
}

现在需要显式指定此转换器，如下所示<f:viewParam>。

<f:viewParam name="id" 
             value="#{testManagedBean.id}"
             converter="#{zoneConverter}"
             rendered="#{not empty param.id}"/>

并且需要按如下方式更改关联的受管 Bean。

@ManagedBean
@RequestScoped
public final class TestManagedBean {
    private ZoneTable id;  //Getter and setter.
    public TestManagedBean() {}
    public String submitAction() {
        return FacesContext.getCurrentInstance().getViewRoot().getViewId() + "?faces-redirect=true&includeViewParams=true";
    }
}

Answer 1

这可能是

Mojarra默认实现UIViewParameter#getStringValueFromModel()的疏忽，其来源是复制粘贴的参考：

384    public String getStringValueFromModel(FacesContext context)
385        throws ConverterException {
386        ValueExpression ve = getValueExpression("value");
387        if (ve == null) {
388            return null;
389        }
390
391        Object currentValue = ve.getValue(context.getELContext());
392
393        // If there is a converter attribute, use it to to ask application
394        // instance for a converter with this identifer.
395        Converter c = getConverter();
396
397        if (c == null) {
398            // if value is null and no converter attribute is specified, then
399            // return null (null has meaning for a view parameters; it means remove it).
400            if (currentValue == null) {
401                return null;
402            }
403            // Do not look for "by-type" converters for Strings
404            if (currentValue instanceof String) {
405                return (String) currentValue;
406            }
407
408            // if converter attribute set, try to acquire a converter
409            // using its class type.
410
411            Class converterType = currentValue.getClass();
412            c = context.getApplication().createConverter(converterType);
413
414            // if there is no default converter available for this identifier,
415            // assume the model type to be String.
416            if (c == null) {
417                return currentValue.toString();
418            }
419        }
420
421        return c.getAsString(context, this, currentValue);
422    }

在为includeViewParams=true生成查询字符串期间，为每个UIViewParameter(<f:viewParam>后面的 UI 组件(调用此方法。我们在源代码中看到，无论currentValue是否null，它都会调用转换器。换句话说，即使模型值是null，它仍然用它调用转换器。

根据 javadoc

的 javadoc Converter#getAsString()转换器在值null时需要返回零长度字符串：

getAsString

。

返回：如果值为null则为零长度字符串，否则为转换结果

因此，转换器实际上应该永远不会在getAsString()上返回null。然后他们返回一个空字符串。在查询字符串中的视图参数的情况下，这是非常不可取的。空字符串值与查询字符串中完全不存在之间的差异非常重要。

我已将其作为问题 3288 报告给 Mojarra 伙计们。然后，他们应该按如下方式解决此问题：

391        Object currentValue = ve.getValue(context.getELContext());
392
393        if (currentValue == null) {
394            return null;
395        }

与此同时，我已经为OmniFaces提供了一个解决方案。此修复程序扩展了<o:viewParam>。根据今天的 1.8 快照，它可用。

<f:metadata>
    <o:viewParam name="id" value="#{testManagedBean.id}" converter="javax.faces.Long"/>
</f:metadata>

<小时 />

更新：他们决定不修复它。无论如何，有OmniFaces。