Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-22390

Auto-linking does not like brackets.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.8, 2.2.1, 2.3.7, 2.4.4, 2.5
    • Fix Version/s: 2.3.8, 2.4.5, 2.5.1
    • Component/s: Filters
    • Labels:
    • Testing Instructions:
      Hide

      1. Ensure you have the URL auto-linking filter enabled.

      2. Try posting some text with some tricky URLs in, such as in https://moodle.org/mod/forum/discuss.php?d=227432.

      Verify that all reasonable URLs are linked, and that stray punctuation immediately after the link is not made into part of the link.

      Show
      1. Ensure you have the URL auto-linking filter enabled. 2. Try posting some text with some tricky URLs in, such as in https://moodle.org/mod/forum/discuss.php?d=227432 . Verify that all reasonable URLs are linked, and that stray punctuation immediately after the link is not made into part of the link.
    • Affected Branches:
      MOODLE_19_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      This came up on the Moodle developers course.

      There were some questions like

      According to Patrick Lauke (http://www.ariadne.ac.uk/issue44/lauke/) which ...

      (There were created without using the HTML editor, so using FORMAT_MOODLE, not FORMAT_HTML.)

      What was happening was that the bit that automatically converts URLs to links was making a link out of

      http://www.ariadne.ac.uk/issue44/lauke/)

      which does not work. I think we need to adjust this bit of code so that it does not include the ) character in this case.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Votes:
                  1 Vote for this issue
                  Watchers:
                  5 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    8/Jul/13