Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7.1
    • Fix Version/s: 1.8DR#1, 1.8
    • Component/s: None
    • Labels:
      None
    • Environment:
      ICEfaces

      Description


      UIXhtmlComponent instances have ids of the form "icesoftXhtmlID0". These ids are certainly longer than necessary and may be entirely unnecessary since they are never transmitted to the client.

        Activity

        Hide
        Ted Goddard added a comment -

        Mark, please comment on whether these ids are necessary. The idea is that elimination of unnecessary Strings will reduce server memory consumption.

        Show
        Ted Goddard added a comment - Mark, please comment on whether these ids are necessary. The idea is that elimination of unnecessary Strings will reduce server memory consumption.
        Hide
        Mark Collette added a comment -

        When I went to investigate the ids we generate, I found that the framework was already assigning them generated ids, which we were stomping over with our own custom generated id scheme. I removed the code for our custom generated ids, leaving the components with the UIViewRoot generated ids, and the Tags with null ids. This appears to work fine with JSF 1.1 and JSF 1.2.

        TRUNK
        Subversion 17278
        ICEfaces 1.7 branch
        Subversion 17279
        icefaces\core\src\com\icesoft\faces\webapp\parser\IceOutputTextTag.java
        icefaces\core\src\com\icesoft\faces\webapp\parser\XhtmlTag.java

        Show
        Mark Collette added a comment - When I went to investigate the ids we generate, I found that the framework was already assigning them generated ids, which we were stomping over with our own custom generated id scheme. I removed the code for our custom generated ids, leaving the components with the UIViewRoot generated ids, and the Tags with null ids. This appears to work fine with JSF 1.1 and JSF 1.2. TRUNK Subversion 17278 ICEfaces 1.7 branch Subversion 17279 icefaces\core\src\com\icesoft\faces\webapp\parser\IceOutputTextTag.java icefaces\core\src\com\icesoft\faces\webapp\parser\XhtmlTag.java

          People

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

            Dates

            • Created:
              Updated:
              Resolved: