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
              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
              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
              Damien Pobel (Inactive) made changes -
              Link This issue relates to EZP-26096 [ 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
              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
              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.
              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 ]
              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
              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
              Permalink
              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
              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
              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
              Permalink
              André Rømcke added a comment -

              Peter Keung fixed

              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: