ICEfaces
  1. ICEfaces
  2. ICE-5670

Javascript insertion should use startElement to create DOM structures

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Invalid
          • Affects Version/s: 2.0-Alpha2
          • Fix Version/s: 2.0-Alpha3, 2.0.0
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            ICEFaces 2,0

            Description

            The currently approved method for inserting javascript into the response is to create an instance of UIOutput that starts a script element, modifies attributes, writes the javascript as a text child, and closes the Element. The current method creates an UIOutput object to write a textNode which consists of the complete script String. This makes it very difficult to appropriately escape the <script> tags in the response writer

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Ted Goddard added a comment -

              In the basic demo, it looks like a <span> tag is being produced rather than a <script> tag.

              Show
              Ted Goddard added a comment - In the basic demo, it looks like a <span> tag is being produced rather than a <script> tag.
              Hide
              Permalink
              Deryk Sinotte added a comment -

              Please update this case with any relevant info and resolve.

              Show
              Deryk Sinotte added a comment - Please update this case with any relevant info and resolve.
              Hide
              Permalink
              Greg Dick added a comment -

              We decided to leave this as is.

              Show
              Greg Dick added a comment - We decided to leave this as is.

                People

                • Assignee:
                  Greg Dick
                  Reporter:
                  Greg Dick
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: