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

Auto-linking does not like brackets.

    XMLWordPrintable

    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.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              timhunt Tim Hunt
              Reporter:
              timhunt Tim Hunt
              Peer reviewer:
              Dan Poltawski
              Integrator:
              Sam Hemelryk
              Tester:
              Petr Skoda
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

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