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

User input used as regexp flag in database

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Add a new database activity
      2. Add a text field to the database named 'Test/Woo'
      3. Go to the templates > Add template
      4. Submit the template
      5. Make sure you don't see any error (see description)
      Show
      Add a new database activity Add a text field to the database named 'Test/Woo' Go to the templates > Add template Submit the template Make sure you don't see any error (see description)
    • Affected Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE, MOODLE_28_STABLE

      Description

      Steps to reproduce:

      1. Add a new database activity
      2. Add a text field to the database named 'Test/Woo'
      3. Go to the templates > Add template
      4. Submit the template

      Expected result:
      No warnings

      Actual result:
      Warning: preg_match_all(): Unknown modifier 'w' in /Users/danp/moodles/im/moodle/mod/data/lib.php on line 889

        Attachments

        1. MDL-50128-26.mdk.patch
          0.9 kB
        2. MDL-50128-27.mdk.patch
          0.9 kB
        3. MDL-50128-28.mdk.patch
          0.9 kB
        4. MDL-50128-master.mdk.patch
          0.9 kB

          Activity

            People

            Assignee:
            fred Frédéric Massart
            Reporter:
            poltawski Dan Poltawski
            Peer reviewer:
            Dan Poltawski
            Integrator:
            Andrew Nicols
            Tester:
            Damyon Wiese
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              11/May/15