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

          Activity

          Nuno Oliveira (Inactive) created issue -
          Nuno Oliveira (Inactive) made changes -
          Field Original Value New Value
          Link This issue relates to CS-5961 [ CS-5961 ]
          Nuno Oliveira (Inactive) made changes -
          Status Open [ 1 ] Confirmed [ 10037 ]
          Nuno Oliveira (Inactive) made changes -
          Fix Version/s Customer request [ 11018 ]
          Nuno Oliveira (Inactive) made changes -
          Status Confirmed [ 10037 ] InputQ [ 10001 ]
          Yannick Roger (Inactive) made changes -
          Assignee Damien Pobel [ damien.pobel@ez.no ]
          Damien Pobel (Inactive) made changes -
          Status InputQ [ 10001 ] Development [ 3 ]
          Damien Pobel (Inactive) made changes -
          Summary eZOE: disabling/enabling editor causes error on links with "?" or "&" Enabling editor causes error on links with "&"
          Damien Pobel (Inactive) made changes -
          Description h4. Steps to reproduce:

          * Edit a content object in the admin interface, such as an article;
          * Type some text e.g. "This is a test link." on a XML block field, such as "Body";
          * Highlight the text and click "Insert/edit link" button;
          * Type a link with a get parameter like http://google.com/?this=that&something=nothing
          * Save the link (popup close);
          * Click the "Disable editor" button;
          * View the raw HTML;
          * Click the "Enable editor" button;
          * The following error is generated:

          h5. The draft could not be stored.
          Required data is either missing or is invalid:
          Body: Wrong opening tag
          h4. 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:

          h5. The draft could not be stored.
          Required data is either missing or is invalid:
          Body: Wrong opening tag
          Hide
          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
          Damien Pobel (Inactive) made changes -
          Link This issue relates to EZP-26096 [ EZP-26096 ]
          Show
          Damien Pobel (Inactive) added a comment - PR: https://github.com/ezsystems/ezpublish-legacy/pull/1278
          Damien Pobel (Inactive) made changes -
          Remote Link This issue links to "PR (Web Link)" [ 17400 ]
          Damien Pobel (Inactive) made changes -
          Status Development [ 3 ] Development Review [ 10006 ]
          Damien Pobel (Inactive) made changes -
          Status Development Review [ 10006 ] Documentation Review done [ 10011 ]
          Assignee Damien Pobel [ damien.pobel@ez.no ]
          Rui Silva (Inactive) made changes -
          Status Documentation Review done [ 10011 ] QA [ 10008 ]
          Rui Silva (Inactive) made changes -
          Flagged Impediment [ 10000 ]
          Hide
          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.
          Rui Silva (Inactive) made changes -
          Status QA [ 10008 ] InputQ [ 10001 ]
          Assignee Rui Silva [ rui.silva@ez.no ]
          Damien Pobel (Inactive) made changes -
          Flagged Impediment [ 10000 ]
          Damien Pobel (Inactive) made changes -
          Status InputQ [ 10001 ] Development [ 3 ]
          Assignee Damien Pobel [ damien.pobel@ez.no ]
          Show
          Damien Pobel (Inactive) added a comment - Fixed in ezpublish-legacy master in https://github.com/ezsystems/ezpublish-legacy/commit/863c3e4b3a2d19cec08f0641697c26f707de86a5
          Damien Pobel (Inactive) made changes -
          Status Development [ 3 ] Development Review [ 10006 ]
          Damien Pobel (Inactive) made changes -
          Status Development Review [ 10006 ] Documentation Review done [ 10011 ]
          Assignee Damien Pobel [ damien.pobel@ez.no ]
          Hide
          Rui Silva (Inactive) added a comment -

          Approved by QA.

          Show
          Rui Silva (Inactive) added a comment - Approved by QA.
          Rui Silva (Inactive) made changes -
          Status Documentation Review done [ 10011 ] QA [ 10008 ]
          Rui Silva (Inactive) made changes -
          Assignee Rui Silva [ rui.silva@ez.no ]
          Status QA [ 10008 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Hide
          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
          André Rømcke made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          André Rømcke made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Fix Version/s 5.4.10 [ 14640 ]
          Resolution Fixed [ 1 ]
          Hide
          André Rømcke added a comment -
          Show
          André Rømcke added a comment - Peter Keung fixed
          Alex Schuster made changes -
          Workflow EZ* Development Workflow [ 102261 ] EZEE Development Workflow [ 126239 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Confirmed Confirmed
          2m 36s 1 nuno.oliveira@ez.no 05/Jan/17 12:58 AM
          Confirmed Confirmed InputQ InputQ
          36s 1 nuno.oliveira@ez.no 05/Jan/17 12:59 AM
          QA QA InputQ InputQ
          2h 21m 1 rui.silva@ez.no 03/Feb/17 2:45 PM
          InputQ InputQ Development Development
          20d 11h 27m 2 damien.pobel@ez.no 03/Feb/17 4:05 PM
          Development Development Development Review Development Review
          5h 43m 2 damien.pobel@ez.no 03/Feb/17 4:09 PM
          Development Review Development Review Documentation Review done Documentation Review done
          8d 17h 59m 2 damien.pobel@ez.no 03/Feb/17 4:09 PM
          Documentation Review done Documentation Review done QA QA
          1h 43m 2 rui.silva@ez.no 03/Feb/17 4:15 PM
          QA QA Closed Closed
          10s 1 rui.silva@ez.no 03/Feb/17 4:15 PM
          Closed Closed Reopened Reopened
          174d 16h 1 André Rømcke 28/Jul/17 9:16 AM
          Reopened Reopened Closed Closed
          18s 1 André Rømcke 28/Jul/17 9:16 AM

            People

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

              Dates

              • Created:
                Updated:
                Resolved: