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

Autolink fails if you try to link content in your own site

    Details

      Description

      The autolink feature in eZOE automatically creates text as link if you paste a link in your xml content and hit enter afterwards, without the need to use the "Insert link" button.

      If you try to use autolink with content from your own site, the feature will generate an invalid link pointing to null.

      Note that, if you use urls to content of your own site through the "Insert link" button, the link will be created without any problems.

      In the query below, the lines are related to:

      1. The link "_http://4-7.ez/ezdemo_site_admin/Getting-Started_" pasted directly in the xml and handled by autolink.
      2. A link to a random site pasted exactly in the same way of the previous line.
      3. The link "http://4-7.ez/ezdemo_site_admin/Getting-Started_" inserted through "_Insert link" button.

      select is_valid, url from ezurl;
      +----------+-------------------------------------------------+
      | is_valid | url                                             |
      +----------+-------------------------------------------------+
      |        1 | null                                            |
      |        1 | https://docs.python.org/2/library/os.path.html  |
      |        1 | http://4-7.ez/ezdemo_site_admin/Getting-Started |
      +----------+-------------------------------------------------+

      Steps to reproduce
      • Created a new article and past in the body any link to any existent ezp content.
      • Hit enter to the link highlight to be activated.
      • Publish the article and check the link in the published version.

        Issue Links

          Activity

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

          @QA you can test it (the patch is for master/5.4 but most likely it will work for each version until 4.7 if you want to test until the customer version)

          Show
          Damien Pobel (Inactive) added a comment - @QA you can test it (the patch is for master/5.4 but most likely it will work for each version until 4.7 if you want to test until the customer version)
          Hide
          Paulo Nunes (Inactive) added a comment -

          PR Verified by QA
          Tested on master/5.4. As suggested, also tried with same patch on 4.7 (had some line adjustments, but the patch was applied without errors).
          Both in master/5.4 as in 4.7, the patch seems to fix the problem.

          I'll send this issue back to dev so that the merge may proceed. Being a customer issue, after the merge, please send the issue back to QA so that we may follow-up the issue to Product Support.

          Show
          Paulo Nunes (Inactive) added a comment - PR Verified by QA Tested on master/5.4. As suggested, also tried with same patch on 4.7 (had some line adjustments, but the patch was applied without errors). Both in master/5.4 as in 4.7, the patch seems to fix the problem. I'll send this issue back to dev so that the merge may proceed. Being a customer issue, after the merge, please send the issue back to QA so that we may follow-up the issue to Product Support.
          Hide
          Paulo Nunes (Inactive) added a comment -

          QA Approved

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

            People

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

              Dates

              • Created:
                Updated:
                Resolved: