Jetspeed 2
  1. Jetspeed 2
  2. JS2-1109

ContentFragment separator may interfere with namespace

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.2.1
    • Fix Version/s: 2.2.1
    • Component/s: Layout
    • Labels:
      None
    • Environment:
      Jetspeed 2.2.1-SNAPSHOT trunk

      Description

      Use of '.' to separate fully qualified ContentFragment and PortletWindow ids may interfere with use of same id for javascript namespace.

      Fix for this may involve using a different, (or no separator), or postprocessing the id to ensure it is safe for javascript use.

        Activity

        Randy Watler created issue -
        Hide
        Randy Watler added a comment -

        Change '.' content fragment id separator to '$' which is safe for javascript identifiers.

        SVN commit 911946

        Show
        Randy Watler added a comment - Change '.' content fragment id separator to '$' which is safe for javascript identifiers. SVN commit 911946
        Hide
        Randy Watler added a comment -

        Switched to '__' instead of '$' which confused CSS selectors.

        SVN commits: 911966 and 911972

        Show
        Randy Watler added a comment - Switched to '__' instead of '$' which confused CSS selectors. SVN commits: 911966 and 911972
        Randy Watler made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ate Douma made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        2h 37m 1 Randy Watler 19/Feb/10 20:50
        Resolved Resolved Closed Closed
        592d 14m 1 Ate Douma 04/Oct/11 22:04

          People

          • Assignee:
            Randy Watler
            Reporter:
            Randy Watler
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development