ICEfaces
  1. ICEfaces
  2. ICE-4535

selectInputDate forces validation of the form when open its popup

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Invalid
    • Affects Version/s: 1.5.3
    • Fix Version/s: None
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      ICEFaces 1.5.3
    • Workaround Exists:
      Yes
    • Workaround Description:
      Hide
      A workaround i've found here is enclosing each ice:selectInputDate in a separate ice:form but i find it ugly and i can't align all my labels:inputs correctly with a 2-column panelGrid afterwards :)
      Another workaround would be to validate the manually each field upon submit.
      Show
      A workaround i've found here is enclosing each ice:selectInputDate in a separate ice:form but i find it ugly and i can't align all my labels:inputs correctly with a 2-column panelGrid afterwards :) Another workaround would be to validate the manually each field upon submit.

      Description

      Pushing popup button on selectInputDate performs iceSubmitPartial() and forces form validation.
      Simple example is
      <ice:messages showSummary="false" showDetail="true" layout="table"/>

      <ice:form>
      <ice:panelGrid columns="2">

      <ice:outputLabel value="Date :" for="date"/>
      <ice:selectInputDate id="date" value="#{partial.date}" renderAsPopup="true"/>

      <ice:outputLabel value="Text :" for="text"/>
      <ice:inputText id="text" required="true" value="#{partial.input}"/>
      </ice:panelGrid>
      </ice:form>

      with the managed bean :
      public class PartialTest
      {
      private String input;
      private Date date;

      //add public getters & setters
      }
      So each time you open its popup and select a date, you will have as many validation errors as there are fields yet to validate.
      IMHO this bug makes selectInputDate unusable in real applications.

        Activity

        Hide
        Ken Fyten added a comment -

        Marking as Closed / Invalid as part of legacy ICEfaces 1.x, 2.x JIRA cleanup.

        Note: This issue may be resolved in a newer ICEfaces release, available here: http://www.icesoft.org/java/downloads/icefaces-downloads.jsf

        If the issue persists with the current ICEfaces release, please create a new JIRA for it.

        Show
        Ken Fyten added a comment - Marking as Closed / Invalid as part of legacy ICEfaces 1.x, 2.x JIRA cleanup. Note: This issue may be resolved in a newer ICEfaces release, available here: http://www.icesoft.org/java/downloads/icefaces-downloads.jsf If the issue persists with the current ICEfaces release, please create a new JIRA for it.
        Hide
        Hank Lander added a comment -

        This issue still exists in version 1.7.2.

        <ice:selectInputDate id="completiondate"
        title="Enter Desired Completion Date"
        renderMonthAsDropdown="true"
        renderYearAsDropdown="true"
        renderAsPopup="true"
        value="#

        {createHandler.curHistory.desiredCompletionDt}

        "
        required="true"
        partialSubmit="false">
        </ice:selectInputDate>

        Using firebug, I can see that iceSubmitPartial( document.forms['_id29'], this,event); is still being called for this component. This is very disruptive to the behavior of the form because it forces validation.

        Show
        Hank Lander added a comment - This issue still exists in version 1.7.2. <ice:selectInputDate id="completiondate" title="Enter Desired Completion Date" renderMonthAsDropdown="true" renderYearAsDropdown="true" renderAsPopup="true" value="# {createHandler.curHistory.desiredCompletionDt} " required="true" partialSubmit="false"> </ice:selectInputDate> Using firebug, I can see that iceSubmitPartial( document.forms ['_id29'] , this,event); is still being called for this component. This is very disruptive to the behavior of the form because it forces validation.

          People

          • Assignee:
            Unassigned
            Reporter:
            Hank Lander
          • Votes:
            18 Vote for this issue
            Watchers:
            9 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: