ICEfaces
  1. ICEfaces
  2. ICE-1159

Error Messaging: Nested forms are not throwing an exeption.

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.5.1
    • Fix Version/s: 1.6DR#2, 1.6
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Operating System: Windows XP
      Platform: PC

      Activity

      Hide
      Philip Breau added a comment -

      I would actually recommend raising the priority of this as it's one of the more important error messaging improvements. Nesting forms can result in indeterminant behaviour that can be very hard to diagnose on the surface.

      Show
      Philip Breau added a comment - I would actually recommend raising the priority of this as it's one of the more important error messaging improvements. Nesting forms can result in indeterminant behaviour that can be very hard to diagnose on the surface.
      Hide
      Adnan Durrani added a comment -

      The validation of nesting form has been added to the FormRenderer under the core branch, it will ensure that the Form action element can never be nested on a JSF page.

      Adding it inside the core FormRenderer would prohabite the nesting of any combination of h and ice form. If forms were found nested we throw a facesException, which force developers to fix the problem before proceeding ahead.

      revision # 13362

      Show
      Adnan Durrani added a comment - The validation of nesting form has been added to the FormRenderer under the core branch, it will ensure that the Form action element can never be nested on a JSF page. Adding it inside the core FormRenderer would prohabite the nesting of any combination of h and ice form. If forms were found nested we throw a facesException, which force developers to fix the problem before proceeding ahead. revision # 13362

        People

        • Assignee:
          Unassigned
          Reporter:
          Philip Breau
        • Votes:
          0 Vote for this issue
          Watchers:
          0 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved: