test case for tomcat 6.0 with ICEfaces 1.7.2 SP1

Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\panelseries\UISeries.java
Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSet.java
Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\util\CustomComponentUtils.java
Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\panelseries\UISeries.java
Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSet.java
Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\util\CustomComponentUtils.java
Completed: At revision: 18366

Fix backported to 1.7 branch:

Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\panelseries\UISeries.java
Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSet.java
Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\util\CustomComponentUtils.java
Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\panelseries\UISeries.java
Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSet.java
Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\util\CustomComponentUtils.java
Completed: At revision: 18426

QA tested it successfully on ICEfaces-1.8.0 build 3 + tomcat6 for IE6, FF2, Opera9.27. Both ice:panelTabSet and ice:dataTable passed the test.

Follow-up regression created by this fix is noted in ICE-4253.

verified on 1.7.2-SP2 build 3 with success (tomcat 6 + IE6 and FF2)

The problem has been fixed only partially. As I see in the code:

    public Object getValue() {
        try {
            return super.getValue();
        } catch (Exception e) {
            //ICE-4066
            if (CustomComponentUtils.isAncestorRendered(this)) {
                throw new FacesException(e);
            }
        }
        return null;
    }

the method isAncestorRenderer finds the first ancestor and evaluates its rendered attribute. But what if ancestor rendered attribute is a value binding, and that ancestor should not be rendered at all ? Value binding is correct only for this panelGroup which is not rendered too. Below is a sample code:

<ice:panelSeries value="#{bean.list}" var="item">

    <ice:panelGroup rendered="false">

        <ice:panelPopup rendered="#{bean.visible}">

            <ice:panelTabSet id="icePnlTbSet" var="tab" value="#{bean.listqq}" title="title">
            <ice:panelTab label="#{tab}" title="#{tab}">
                <ice:panelGroup style="height:50px;">
                <ice:outputText value="#{tab}" />
                </ice:panelGroup>
             </ice:panelTab>
             </ice:panelTabSet>

        </ice:panelPopup>

    </ice:panelGroup>

</ice:panelSeries>