ICEfaces
  1. ICEfaces
  2. ICE-1040

partialSubmit isRequired management broken

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: 1.5.2
    • Component/s: Bridge
    • Labels:
      None
    • Environment:
      Operating System: Mac OS X 10.0
      Platform: Macintosh

      Description

      partialSubmit determines the active component for isRequired enabling/disabling by retrieving "focus"
      from the request. This functionality is specific to partialSubmit, but has been broken by other focus-
      related changes (relying on setting "focus" to a particular value).

        Activity

        Hide
        Ted Goddard added a comment -

        We need a page with required="true" set on a bunch of the components. I'll bet we don't have a current
        test case because this one slipped through.

        Show
        Ted Goddard added a comment - We need a page with required="true" set on a bunch of the components. I'll bet we don't have a current test case because this one slipped through.
        Hide
        Mircea Toma added a comment -

        Ted, how can I reproduce this bug? I remember that I determined the problem, but now I need to verify
        and I don't remember how.

        Show
        Mircea Toma added a comment - Ted, how can I reproduce this bug? I remember that I determined the problem, but now I need to verify and I don't remember how.
        Hide
        Mircea Toma added a comment -

        1.5 branch fixed in commit #12811.

        Show
        Mircea Toma added a comment - 1.5 branch fixed in commit #12811.
        Hide
        Mircea Toma added a comment -

        Ok, the bug is fixed in the 1.5 branch as well.

        Show
        Mircea Toma added a comment - Ok, the bug is fixed in the 1.5 branch as well.
        Hide
        Mircea Toma added a comment -

        It looks like it was committed just in the HEAD. I think it should.

        Show
        Mircea Toma added a comment - It looks like it was committed just in the HEAD. I think it should.
        Hide
        Ken Fyten added a comment -

        Mircea,

        Was this committed to the 1.5 branch and the head? Should it be in 1.5.2?

        Show
        Ken Fyten added a comment - Mircea, Was this committed to the 1.5 branch and the head? Should it be in 1.5.2?
        Hide
        Mircea Toma added a comment -

        Separated focus state from triggering element state.
        Fixed in commit #12629.

        Show
        Mircea Toma added a comment - Separated focus state from triggering element state. Fixed in commit #12629.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Ted Goddard
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: