ICEfaces
  1. ICEfaces
  2. ICE-932

Facelets: Duplicate component id's are not caught

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: 1.5.3
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      Operating System: Windows XP
      Platform: PC

      Description

      If you have duplicate id's in a Facelets application you'll get rendering
      problems, but no error or warning, or exceptions. If there's any way we can
      catch this, it would be good.

        Activity

        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Ken Fyten made changes -
        Affects Version/s 1.5 [ 10027 ]
        Affects Version/s 1.0.1 [ 10025 ]
        Icefaces Administrator made changes -
        Field Original Value New Value
        issue.field.bugzillaimportkey 951 12201
        Hide
        Mark Collette added a comment -

        Verified with Win XP, Firefox 1.5, JBoss 4.0.4 GA.

        Show
        Mark Collette added a comment - Verified with Win XP, Firefox 1.5, JBoss 4.0.4 GA.
        Hide
        Mark Collette added a comment -

        When we build the component tree, it now recurses down through the tree,
        checking for null or redundant IDs.

        1.5 Branch
        Subversion 13020
        icefaces\core\src\com\icesoft\faces\facelets\D2DFaceletViewHandler.java

        HEAD
        Subversion 13021
        icefaces\core\src\com\icesoft\faces\facelets\D2DFaceletViewHandler.java

        Show
        Mark Collette added a comment - When we build the component tree, it now recurses down through the tree, checking for null or redundant IDs. 1.5 Branch Subversion 13020 icefaces\core\src\com\icesoft\faces\facelets\D2DFaceletViewHandler.java HEAD Subversion 13021 icefaces\core\src\com\icesoft\faces\facelets\D2DFaceletViewHandler.java
        Hide
        Ken Fyten added a comment -

        Mark, please respond if this is feasible to be implemented in the next day or two.

        Show
        Ken Fyten added a comment - Mark, please respond if this is feasible to be implemented in the next day or two.
        Hide
        Philip Breau added a comment -

        I'd like to raise the priority of this as it keeps coming up with support clients.

        Supportilla 3844, 3849.

        Ken, can we push this to 1.5.3?

        Show
        Philip Breau added a comment - I'd like to raise the priority of this as it keeps coming up with support clients. Supportilla 3844, 3849. Ken, can we push this to 1.5.3?
        Hide
        Mark Collette added a comment -

        We could traverse the component tree and make sure that all IDs are unique.

        Show
        Mark Collette added a comment - We could traverse the component tree and make sure that all IDs are unique.
        Hide
        Ted Goddard added a comment -


        Assigning to Mark for comment (likely cannot investigate or fix at this time).

        Show
        Ted Goddard added a comment - Assigning to Mark for comment (likely cannot investigate or fix at this time).
        Philip Breau created issue -

          People

          • Assignee:
            Mark Collette
            Reporter:
            Philip Breau
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: