Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-26096

Custom tags' text line parameters not saved correctly

    Details

      Description

      Steps to reproduce:

      1. Create a new article;
      2. On an an XML block field (used "body"), insert a custom tag of type "factbox";
      3. On the custom tag popup window's "title" field, entered HTML code such as "<a href="http://ez.no">Test link</a>";
      4. Click "OK", effectively inserting the new custom tag into the "body" field;
      5. Click on the "Disable editor" button;
      6. The resulting HTML code is this:

      <custom name="factbox">Test linkattribute_separationalign|right&quot;&gt;
       
       
      factbox</custom>
      

        Issue Links

          Activity

          Show
          Damien Pobel (Inactive) added a comment - Additional PR: https://github.com/ezsystems/ezpublish-legacy/pull/1256
          Hide
          Damien Pobel (Inactive) added a comment - - edited

          Second PR ready to be tested.
          Specific steps to reproduce:

          1. In Online Editor add a factbox
          2. As the title set 'Test > test'
          3. Add a paragraph with some text after the factbox
          4. Add a second factbox
          5. As the title set 'Test > test'
          6. Store the draft

          the factbox should be kept intact, without the patch the second one is broken

          Show
          Damien Pobel (Inactive) added a comment - - edited Second PR ready to be tested. Specific steps to reproduce: In Online Editor add a factbox As the title set 'Test > test' Add a paragraph with some text after the factbox Add a second factbox As the title set 'Test > test' Store the draft the factbox should be kept intact, without the patch the second one is broken
          Hide
          Rui Silva (Inactive) added a comment -

          Tested and approved second PR for 5.4.
          Sending back so that merge can be done.

          Show
          Rui Silva (Inactive) added a comment - Tested and approved second PR for 5.4. Sending back so that merge can be done.
          Show
          Damien Pobel (Inactive) added a comment - Additional fix merged in https://github.com/ezsystems/ezpublish-legacy/commit/e4ac9a071b7d31d37492ea4249396183ae532f01
          Hide
          Paulo Nunes (Inactive) added a comment -

          QA Approved

          Show
          Paulo Nunes (Inactive) added a comment - QA Approved

            People

            • Assignee:
              Unassigned
              Reporter:
              Nuno Oliveira (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: