Uploaded image for project: 'PUBLIC - Liferay Portal Community Edition'
  1. PUBLIC - Liferay Portal Community Edition
  2. LPS-42110

Space is removed among BBCode tags when switching to Source mode in Message Boards portlet

    Details

    • Branch Version/s:
      6.2.x, 6.1.x
    • Backported to Branch:
      Committed
    • Epic/Theme:
    • Story Points:
      2
    • Fix Priority:
      4

      Description

      Reproduction Steps

      1. Add Message Boards portlet to a page
      2. Type "text text"
      3. Add hyperlink to both "text"
      4. Switch to "Source" view (and back to WYSIWYG if you want)

      Result: The space between the two words disappeared.

        Activity

        Hide
        istvan.sajtos Istvan Sajtos added a comment -

        The issue comes from bbcode_data_processor.js. The spaces are removed, because _isIgnorable function say that those nodes are not necessary. We should clarify which nodes can be indeed ignored and why.

        creol_data_processor.js is also affected by the issue.

        Show
        istvan.sajtos Istvan Sajtos added a comment - The issue comes from bbcode_data_processor.js . The spaces are removed, because _isIgnorable function say that those nodes are not necessary. We should clarify which nodes can be indeed ignored and why. creol_data_processor.js is also affected by the issue.
        Hide
        iliyan.peychev Iliyan Peychev (Inactive) added a comment -

        Spot fix will be provided to the client and I will work on this issue.

        Show
        iliyan.peychev Iliyan Peychev (Inactive) added a comment - Spot fix will be provided to the client and I will work on this issue.
        Hide
        felix.zhang Felix Zhang added a comment -

        PASSED Manual Testing following the steps in the description.

        Reproduced on:
        Tomcat 7.0.42 + MySQL 5.5.21. Portal master GIT ID: b9f1d4e55addfd9596fbc0ce660e606810261977.

        The space between the two words disappeared.

        Fixed on:
        Tomcat 7.0.42 + MySQL 5.5.21. Portal ee-6.2.x EE GIT ID: ef1825f1968cd14c7cfc3aef41cac85f8b8c183b.
        Tomcat 7.0.42 + MySQL 5.5.21. Portal master GIT ID: 9cc4997fe5920c162862c06686b88dd9549af70f.

        The space between the two words still exist.

        Show
        felix.zhang Felix Zhang added a comment - PASSED Manual Testing following the steps in the description. Reproduced on: Tomcat 7.0.42 + MySQL 5.5.21. Portal master GIT ID: b9f1d4e55addfd9596fbc0ce660e606810261977. The space between the two words disappeared. Fixed on: Tomcat 7.0.42 + MySQL 5.5.21. Portal ee-6.2.x EE GIT ID: ef1825f1968cd14c7cfc3aef41cac85f8b8c183b. Tomcat 7.0.42 + MySQL 5.5.21. Portal master GIT ID: 9cc4997fe5920c162862c06686b88dd9549af70f. The space between the two words still exist.
        Hide
        felix.zhang Felix Zhang added a comment -

        FAILED Manual Testing following the steps in the description.

        Failed on:
        Tomcat 7.0.40 + MySQL 5.6.10. Portal 6.1.x EE GIT ID: 4da838ebd63788313b99601d5d7e1cec0e7189d9.

        The space between the two words disappeared.

        Show
        felix.zhang Felix Zhang added a comment - FAILED Manual Testing following the steps in the description. Failed on: Tomcat 7.0.40 + MySQL 5.6.10. Portal 6.1.x EE GIT ID: 4da838ebd63788313b99601d5d7e1cec0e7189d9. The space between the two words disappeared.
        Hide
        felix.zhang Felix Zhang added a comment -

        Confirmed with Dustin by email, it can be passed when set editor.ckeditor.version=latest in portal-ext.properties.

        Show
        felix.zhang Felix Zhang added a comment - Confirmed with Dustin by email, it can be passed when set editor.ckeditor.version=latest in portal-ext.properties.
        Hide
        felix.zhang Felix Zhang added a comment -

        PASSED Manual Testing using the following steps:

        1. Set editor.ckeditor.version=latest in portal-ext.properties.
        2. Add Message Boards portlet to a page.
        3. Type "text text".
        4. Add hyperlink to both "text".
        5. Switch to "Source" view (and back to WYSIWYG if you want).

        Fixed on:
        Tomcat 7.0.40 + MySQL 5.6.10. Portal 6.1.x EE GIT ID: 4da838ebd63788313b99601d5d7e1cec0e7189d9.

        The space between the two words still exist.

        Show
        felix.zhang Felix Zhang added a comment - PASSED Manual Testing using the following steps: Set editor.ckeditor.version=latest in portal-ext.properties. Add Message Boards portlet to a page. Type "text text". Add hyperlink to both "text". Switch to "Source" view (and back to WYSIWYG if you want). Fixed on: Tomcat 7.0.40 + MySQL 5.6.10. Portal 6.1.x EE GIT ID: 4da838ebd63788313b99601d5d7e1cec0e7189d9. The space between the two words still exist.

          People

          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:
              Days since last comment:
              39 weeks, 1 day ago

              Development

                Subcomponents