Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

<o:messages> does not show the error validations for inputs inside a table when for attribute is used #32

Closed
yersan opened this issue May 20, 2014 · 3 comments

Comments

@yersan
Copy link

yersan commented May 20, 2014

Hi,

I don't know if the o:message included this behavior but error messages for inputs in a table are not displayed using the for attribute. I have tested it with mojarra 2.2.5.

For example suppose the next page:

 <h:form prependId="false">
           <o:messages id="msg" for="panel1" />

           <h:panelGroup id="panel1">
                <h:dataTable value="#{of:createIntegerArray(1, 2)}" var="row">
                    <h:column>
                        <h:inputText required="true" value="#{row}" />
                    </h:column>
                </h:dataTable>
           </h:panelGroup>

            <o:messages id="msg2" for="panel2" />

            <h:panelGroup id="panel2">
                    <h:inputText required="true" value="1" />              
                    <h:inputText required="true" value="2" />
            </h:panelGroup>


            <h:commandButton value="TEST">
               <f:ajax execute="@form" render="@form" />
           </h:commandButton>
        </h:form>

If I click on command button with all inputs in blank I only can see the messages generated in panel2. I expected to see the messages of panel1 also.

@BalusC
Copy link
Member

BalusC commented May 20, 2014

Should be already fixed a week ago: 409d2fa

Please try current 1.8 snapshot and confirm: https://oss.sonatype.org/content/repositories/snapshots/org/omnifaces/omnifaces/1.8-SNAPSHOT/

@yersan
Copy link
Author

yersan commented May 20, 2014

Confirmed, works on 1.8-SNAPSHOT

Thanks !!

@BalusC
Copy link
Member

BalusC commented May 21, 2014

Thank you for reporting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants