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

Autolinking Filter: Problem autolinking if resource name contains use of double quote (i.e. " )

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.8.7, 1.9.3, 2.0
    • Fix Version/s: 1.8.8, 1.9.4
    • Component/s: Filters
    • Labels:
      None
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE

      Description

      To demonstrate the bug, create a resource called Article: "Testing" and then create an offline assignment that instructs the student to Read Article: "Testing". I would expect autolinking to work (especially if I copy and paste the resource name into the assignment's description. I've not looked at the code to see what the problem is but I do notice that the description field stores the double quote as " A workaround is to use two single quotes which gives the appearance of a double quote but this is not intuitive to users. This issue is important as teachers who wish to demonstrate good formatting of resource names require the use of quotations (this is how I came across this issue). So getting this patched up would be very helpful. A similar issue occurs when italics are used in the assignment description. Since italics are not allowed in the resource name (they get stripped out if the html is added manually), then I think the text string should also be stripped using the same function prior to doing the compare. Peace - Anthony

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    28/Jan/09