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.

        Issue Links

          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:
                1 year, 1 week, 4 days ago

                Development

                  Subcomponents