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

Enabling editor causes error on links with "&"

        Details

          Description

          Steps to reproduce:

          • Edit a content object in the admin interface, such as an article;
          • Click the "Disable editor" button;
          • Type <link href="http://google.com/?this=that&something=nothing">link</link> in the textarea
          • Click the "Enable editor" button;
          • The following error is generated:
          The draft could not be stored.

          Required data is either missing or is invalid:
          Body: Wrong opening tag

            Issue Links

            relates to

            Bug - A problem which impairs or prevents the functions of the product. EZP-26096 Custom tags' text line parameters not saved correctly

            • High - Major loss of functionality.
            • Closed
            links to

            Web Link PR

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Damien Pobel (Inactive) added a comment -

              Most likely a regression caused by the fix for EZP-26096

              Show
              Damien Pobel (Inactive) added a comment - Most likely a regression caused by the fix for EZP-26096
              Hide
              Permalink
              Damien Pobel (Inactive) added a comment -

              PR: https://github.com/ezsystems/ezpublish-legacy/pull/1278

              Show
              Damien Pobel (Inactive) added a comment - PR: https://github.com/ezsystems/ezpublish-legacy/pull/1278
              Hide
              Permalink
              Rui Silva (Inactive) added a comment -

              PR tested and approved by QA for 5.4.
              Sending back to dev-needed so that PR can be merged.
              Please send again to QA afterwards so that QA can inform Support Team.

              Show
              Rui Silva (Inactive) added a comment - PR tested and approved by QA for 5.4. Sending back to dev-needed so that PR can be merged. Please send again to QA afterwards so that QA can inform Support Team.
              Hide
              Permalink
              Damien Pobel (Inactive) added a comment -

              Fixed in ezpublish-legacy master in https://github.com/ezsystems/ezpublish-legacy/commit/863c3e4b3a2d19cec08f0641697c26f707de86a5

              Show
              Damien Pobel (Inactive) added a comment - Fixed in ezpublish-legacy master in https://github.com/ezsystems/ezpublish-legacy/commit/863c3e4b3a2d19cec08f0641697c26f707de86a5
              Hide
              Permalink
              Rui Silva (Inactive) added a comment -

              Approved by QA.

              Show
              Rui Silva (Inactive) added a comment - Approved by QA.
              Hide
              Permalink
              Peter Keung added a comment -

              This needs to be added to the 5.4.10 release notes: https://doc.ez.no/display/EZP/5.4.10+Release+Notes

              Show
              Peter Keung added a comment - This needs to be added to the 5.4.10 release notes: https://doc.ez.no/display/EZP/5.4.10+Release+Notes
              Hide
              Permalink
              André Rømcke added a comment -

              Peter Keung fixed

              Show
              André Rømcke added a comment - Peter Keung fixed

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: