[MYFACES-3554] REGRESSION: 2.0.12/2.1.6 (and above) fail on state saving

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.12, 2.0.13, 2.1.6, 2.1.7
• Fix Version/s: 2.0.14, 2.1.8
• Environment: Tomcat 7.0.25
• Assignee: Leonardo Uribe
• Reporter: Kennard Consulting
• Created: 28/May/12 23:44
• Updated: 13/Jun/12 13:51
• Resolved: 29/May/12 10:42

Description

Hi guys,

Thanks again for all the great work you do on MyFaces!

There appears to have been an identical regression between MyFaces 2.0.11 and 2.0.12, and between MyFaces 2.1.5 and 2.1.6. My apologies for not picking this up earlier. This regression is likely related to a suite of unit tests I gave you in MYFACES-2935, though unfortunately I guess my suite didn't cover this particular bug?

I attach 4 versions of the same sample application. You'll see it works for the 2.0.11/2.1.5 versions, but not the 2.0.12/2.1.6 versions. To reproduce:

1. Run the app
2. On the opening page click on contact 'Homer Simpson'
3. Click Edit

In the regressed versions you will see:

java.lang.IllegalStateException: component with duplicate id "form:j_id_b" found
org.apache.myfaces.view.facelets.compiler.CheckDuplicateIdFaceletUtils.checkIdsStatefulComponents(CheckDuplicateIdFaceletUtils.java:54)
org.apache.myfaces.view.facelets.compiler.CheckDuplicateIdFaceletUtils.checkIdsStatefulComponents(CheckDuplicateIdFaceletUtils.java:75)
org.apache.myfaces.view.facelets.compiler.CheckDuplicateIdFaceletUtils.checkIdsStatefulComponents(CheckDuplicateIdFaceletUtils.java:75)
org.apache.myfaces.view.facelets.compiler.CheckDuplicateIdFaceletUtils.checkIdsStatefulComponents(CheckDuplicateIdFaceletUtils.java:75)
org.apache.myfaces.view.facelets.compiler.CheckDuplicateIdFaceletUtils.checkIdsStatefulComponents(CheckDuplicateIdFaceletUtils.java:75)
org.apache.myfaces.view.facelets.compiler.CheckDuplicateIdFaceletUtils.checkIdsStatefulComponents(CheckDuplicateIdFaceletUtils.java:35)
org.apache.myfaces.view.facelets.DefaultFaceletsStateManagementStrategy.saveView(DefaultFaceletsStateManagementStrategy.java:488)
org.apache.myfaces.application.StateManagerImpl.saveView(StateManagerImpl.java:166)
org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguage.renderView(FaceletViewDeclarationLanguage.java:1619)
org.apache.myfaces.application.ViewHandlerImpl.renderView(ViewHandlerImpl.java:264)
org.apache.myfaces.lifecycle.RenderResponseExecutor.execute(RenderResponseExecutor.java:115)
org.apache.myfaces.lifecycle.LifecycleImpl.render(LifecycleImpl.java:239)
javax.faces.webapp.FacesServlet.service(FacesServlet.java:191)

I'm assuming this is on your end, but if it's a case of you tightening up spec compliance and exposing a bug in my code, please let know!

Regards,

Richard.