ICEfaces
  1. ICEfaces
  2. ICE-1883

Partial submit disables previous validation messages

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6
    • Fix Version/s: 1.7DR#3, 1.7
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      All

      Description

      When using standard JSF validation on various required fields, the form is submitted and validation functions correctly. The required fields are flagged with ICEfaces messages and the form is not submitted. When partial submit is enabled, a user will enter data/click out of an input field and the form is partially submitted. The problem is that ICEfaces disables all required statuses in order to submit the page and when the partial submit returns, it does not retain the original validation messages for required inputs.

        Activity

        Tyler Johnson created issue -
        Hide
        Philip Breau added a comment -

        This is a large usability more than a bug itself. We may want to have a default retainAfterPartialSubmit attribute on the ice:message and messages components, where they retain previous messages until a full submit.

        Show
        Philip Breau added a comment - This is a large usability more than a bug itself. We may want to have a default retainAfterPartialSubmit attribute on the ice:message and messages components, where they retain previous messages until a full submit.
        Philip Breau made changes -
        Field Original Value New Value
        Support Case References https://www.icesoft.ca:4443/supportilla/show_bug.cgi?id=4171
        Support Customer Ref. #s 713, 724
        Ken Fyten made changes -
        Fix Version/s 1.7 [ 10080 ]
        Assignee Ted Goddard [ ted.goddard ]
        Hide
        Ted Goddard added a comment -

        A special message component variant is an interesting idea. Are we losing all validation error messages or just the "required" error messages?

        Another option may be to create a custom validator that returns an error until it is cleared – the validator would not test for an empty value until the component was "required", but once "required" would continue to test for an empty value.

        Show
        Ted Goddard added a comment - A special message component variant is an interesting idea. Are we losing all validation error messages or just the "required" error messages? Another option may be to create a custom validator that returns an error until it is cleared – the validator would not test for an empty value until the component was "required", but once "required" would continue to test for an empty value.
        Hide
        Ted Goddard added a comment -

        Assigning to Philip to comment on which messages specifically are being lost (with input from Tyler from his observations).

        Show
        Ted Goddard added a comment - Assigning to Philip to comment on which messages specifically are being lost (with input from Tyler from his observations).
        Ted Goddard made changes -
        Assignee Ted Goddard [ ted.goddard ] Philip Breau [ philip.breau ]
        Hide
        Philip Breau added a comment -

        This involves any validation message, whether arising from a conversion error, validation error, or missing required error. The proposed new behaviour would be quite simple. A new attribute, such as 'retainOnPartialSubmit' would cause the lifespan of a message to span partialSubmits and only be cleared on a full submit or a partialSubmit originating from the corresponding component of the message itself. This is the desired behaviour requested from both clients.

        Show
        Philip Breau added a comment - This involves any validation message, whether arising from a conversion error, validation error, or missing required error. The proposed new behaviour would be quite simple. A new attribute, such as 'retainOnPartialSubmit' would cause the lifespan of a message to span partialSubmits and only be cleared on a full submit or a partialSubmit originating from the corresponding component of the message itself. This is the desired behaviour requested from both clients.
        Philip Breau made changes -
        Assignee Philip Breau [ philip.breau ] Ted Goddard [ ted.goddard ]
        Ken Fyten made changes -
        Assignee Ted Goddard [ ted.goddard ] Mark Collette [ mark.collette ]
        Ken Fyten made changes -
        Assignee Mark Collette [ mark.collette ] Adnan Durrani [ adnan.durrani ]
        Adnan Durrani made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15209 Tue Nov 20 14:58:21 MST 2007 adnan.durrani Fix for ICE-1883 (Partial submit disables previous validation messages)
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectOneListbox.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectManyMenu.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlInputTextarea.java
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/core/ReceiveSendUpdates.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlInputText.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectOneRadio.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectManyListbox.java
        Ken Fyten made changes -
        Fix Version/s 1.7DR#3 [ 10112 ]
        Fix Version/s 1.7 [ 10080 ]
        Hide
        Adnan Durrani added a comment -

        Revision 15209

        Show
        Adnan Durrani added a comment - Revision 15209
        Adnan Durrani made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15220 Wed Nov 21 16:30:21 MST 2007 adnan.durrani Fix for ICE-1883 added to the two remaining components (Partial submit disables previous validation messages)
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectOneMenu.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectManyCheckbox.java
        Hide
        Adnan Durrani added a comment -

        revision 15220

        Show
        Adnan Durrani added a comment - revision 15220
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15238 Thu Nov 22 16:54:06 MST 2007 adnan.durrani isValid() method used (Mark's suggestion) instead of an additional state. Related to ICE-1883
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectOneListbox.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectManyMenu.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlInputTextarea.java
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/webapp/http/core/ReceiveSendUpdates.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlInputText.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectOneRadio.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectOneMenu.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectManyListbox.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlSelectManyCheckbox.java
        Hide
        Adnan Durrani added a comment -

        revision 15238

        Show
        Adnan Durrani added a comment - revision 15238
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15289 Tue Dec 04 10:58:08 MST 2007 adnan.durrani Related to ICE-1883 (messages fixed for the dataTable)
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/panelseries/UISeries.java
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/util/CoreUtils.java
        Hide
        Adnan Durrani added a comment -

        revision: 15289

        Show
        Adnan Durrani added a comment - revision: 15289
        Ken Fyten made changes -
        Issue Type Bug [ 1 ] Improvement [ 4 ]
        Hide
        Ken Fyten added a comment -

        The implemented enhancement is to retain the display of error messages for any components that:

        • Were in an invalid state prior to the partialSubmit
        • Are not the component that is the subject of the partialSubmit

        This supports the use-cases where user's are entering data in a partial-submit form, enter an invalid value in field 1, see the error msg., then move to field2 and enter a value there. Prior to this change the original error msg. from their invalid entry into field1 would disappear when field2 was partialSubmitted. Now error msgs for all invalid state components will continue to be displayed until they are in a valid state.

        Show
        Ken Fyten added a comment - The implemented enhancement is to retain the display of error messages for any components that: Were in an invalid state prior to the partialSubmit Are not the component that is the subject of the partialSubmit This supports the use-cases where user's are entering data in a partial-submit form, enter an invalid value in field 1, see the error msg., then move to field2 and enter a value there. Prior to this change the original error msg. from their invalid entry into field1 would disappear when field2 was partialSubmitted. Now error msgs for all invalid state components will continue to be displayed until they are in a valid state.
        Ken Fyten made changes -
        Fix Version/s 1.7 [ 10080 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Adnan Durrani [ adnan.durrani ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Tyler Johnson
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: