ICEpush
  1. ICEpush
  2. PUSH-67

JSP region tag should evaluate scripts in fetched content

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0-Alpha3
    • Component/s: Integrations
    • Labels:
      None
    • Environment:
      n/a

      Description

      The current implementation of the JSP region tag does not evaluate scripts in fetched content, by way of the bridge get() method not doing so. We should allow scripts to be evaluated in fetched content, and this should be configurable. This will also allow region tags to be nested.

        Activity

        Hide
        Philip Breau added a comment -

        rev 21173

        Show
        Philip Breau added a comment - rev 21173
        Hide
        Ted Goddard added a comment -

        Need to look at createElement() rather than eval()

        Show
        Ted Goddard added a comment - Need to look at createElement() rather than eval()
        Hide
        Mircea Toma added a comment -

        Re-implemented ice.push.searchAndEvaluateScripts function to use document.createElement facility for evaluating dynamicaly inserted code. The advantage over previous implementation is simplified code and the dynamically evaluated code is now managed by the browser cache since the externally referenced code is requested by the browser itself not through an AJAX request.

        Tested in IE6, Safari and FF3.5.

        Show
        Mircea Toma added a comment - Re-implemented ice.push.searchAndEvaluateScripts function to use document.createElement facility for evaluating dynamicaly inserted code. The advantage over previous implementation is simplified code and the dynamically evaluated code is now managed by the browser cache since the externally referenced code is requested by the browser itself not through an AJAX request. Tested in IE6, Safari and FF3.5.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: