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

eZOE link popup doesn't urlencode search string

    Details

      Description

      When inserting a link for a content in eZ OE, and using the search, the search text is not urlencoded, so if your search text contains "&", it is considered as a URL GET separator.

      Steps to reproduce :
      • Create a content (article) named "new&article".
      • In a separate content, select a text in eZOE, and click on the "insert link" button.
      • in the popup window, click on the search (loupe) to open the search form.
      • Search for a text which contains "&" character: "new&article"
      Expected results:
      • solr should return results for "new&article"
      Actual results:
      • The SearchStr param has been split, so search results are returned only for "new".
      Further notes:

      Opening the browser's network debugger and looking at the ajax call, we can see the problem query with split parameters:
      ...EncodingFetchChildrenCount=1&SearchStr=new&articlee&ezjscServer_function_arguments...

        Activity

        Show
        Joao Inacio added a comment - PR: https://github.com/ezsystems/ezpublish-legacy/pull/1264
        Hide
        Rui Silva added a comment -

        PR approved by QA. Sending back to dev so that merge can be done.
        After merge is done, please send back to QA again, so that QA can properly report approval status on affected CS issue.

        Show
        Rui Silva added a comment - PR approved by QA. Sending back to dev so that merge can be done. After merge is done, please send back to QA again, so that QA can properly report approval status on affected CS issue.
        Show
        Joao Inacio added a comment - Merged in ezpublish-legacy: https://github.com/ezsystems/ezpublish-legacy/commit/2dfb5cba9575293584c0cbfeff6c1070afc2c4a1
        Hide
        Rui Silva added a comment -

        Tested and approved by QA for 4.7, 5.3 and 5.4.

        Show
        Rui Silva added a comment - Tested and approved by QA for 4.7, 5.3 and 5.4.

          People

          • Assignee:
            Unassigned
            Reporter:
            Joao Inacio
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: