Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-13761 Issues with add_to_log URL field
  3. MDL-13915

Make add_to_log handle long strings without causing database errors

XMLWordPrintable

    • Icon: Sub-task Sub-task
    • Resolution: Fixed
    • Icon: Minor Minor
    • 1.9.1
    • 1.9
    • Libraries
    • None
    • MOODLE_19_STABLE
    • MOODLE_19_STABLE

      This is the changes in MDL-13761 that were agreed with Eloy for probable inclusion in 1.9 branch:

      a) Check length is under the limited for 'url' and 'info' parameters (only ones which are likely to potentially depend on user input) and trim to limited length, with '...', if needed.

      b) If this trim happens, show debugging() info so that developers might notice they are logging something that can get too long.

      c) In log view, if url field ends with ..., don't treat it as a link (as it won't work!).

            quen Sam Marshall
            quen Sam Marshall
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.