<c:if test> seems to always evaluate true in JSF2 Facelets

jsf jsf-2 jstl facelets

45,144

Solution 1

The XML namespace is invalid. It should be

xmlns:c="http://java.sun.com/jsp/jstl/core"

Yes, astonishingly with the /jsp part in the URI! The one without /jsp works only in Facelets 1.x for JSF 1.x. If you have checked the generated HTML output in the webbrowser, you should have noticed as well that the <c:if> is left unparsed in the HTML output.

That said, you should prefer JSF components over JSTL tags, unless technically impossible (i.e. when you actually want to control the building of the view, not rendering of the view). The h:panelGroup as you found out yourself is a good candidate, however the ui:fragment is a nicer choice since it has less overhead.

<ui:fragment rendered="#{not empty title}">
    <h1>#{title}</h1>
</ui:fragment>

Note that due to a mistake of the JSF guys in the <ui:fragment> tag definition file of the initial JSF 2.0 version, Netbeans will jerk that the tag doesn't support the rendered attribute, but this is untrue. It certainly supports it. It has been fixed in JSF 2.1 tag definition.

Solution 2

Well,

I used rendered instead, like this:

<h:panelGroup rendered="#{title!=null}">

                <h1>#{title}</h1>

        </h:panelGroup>

45,144

Author by

Dejell

I like traveling around the world. So far I have been to: USA England Italy Slovania Croatia Jordan South Africa Zimbabwe Botswana France Canada Israel Thailand Switzerland Holland Bulgaria I am going to Vietnam soon

Updated on July 18, 2020

Comments

Dejell almost 4 years

I am using JSF2 on Facelets.

I define an <ui:param> in a page:

<ui:composition template="/WEB-INF/templates/ui.xhtml"
  xmlns="http://www.w3.org/1999/xhtml"
  xmlns:f="http://java.sun.com/jsf/core"
  xmlns:h="http://java.sun.com/jsf/html"
  xmlns:ui="http://java.sun.com/jsf/facelets"
>
  <ui:param name="title" value="OnAir WebDemo" />
  ...
</ui:composition>

in the ui.xhtml template I have:

<html
  xmlns="http://www.w3.org/1999/xhtml"
  xmlns:f="http://java.sun.com/jsf/core"
  xmlns:h="http://java.sun.com/jsf/html"
  xmlns:ui="http://java.sun.com/jsf/facelets"
  xmlns:c="http://java.sun.com/jstl/core"
>
  <c:if test="#{not empty title}">
    <h1>#{title}</h1>
  </c:if>
</html>

However, <c:if test> seems to always evaluate true, also if the <ui:param> is not specified. How can I change the code so that <c:if test> will actually evaluate false when the <ui:param> is not specified?

Nick over 11 years

I've been using h:panelGroup just for its rendered attribute for years; I didn't know about ui:fragment! This substantially improves performance and reduces the amount of generated HTML in places where I conditionally render things in ui:repeats over large collections. Thanks! I learn something new with every one of your answers, @BalusC.
BalusC over 11 years

@Nick: you're welcome. Note that <h:panelGroup> already effectively renders nothing when there are no attributes specified which should end up in the HTML, such as id. But indeed, the <ui:fragment> saves some UIComponent encoding overhead (and is IMHO more self-documenting for the sole purpose).
Nick over 11 years

Odd, I could have sworn that I saw <h:panelGroup> rendering an empty <span> tag in the html, even when its only attribute is a rendered that evaluates to false, but I'm not seeing it now...
rlegendi over 10 years

Haven't tried it :-) Was curious about what's the disatvantage or why is it considered a bad practice. Will update if I can try it out.
Dejell over 10 years

I think that it was voted down since !null doesn't check empty.
king almost 10 years

i attempted to add this to the top of my index.xhtml file, but now I receive a java.lang.NullPointerException error with really not much other info. Do you know why this would happen?