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

Prevent the grader report from storing serialized data in the user_preferences table

    XMLWordPrintable

Details

    • Any
    • MOODLE_22_STABLE, MOODLE_27_STABLE
    • MOODLE_26_STABLE, MOODLE_27_STABLE
    • wip-MDL-30668-master
    • Hide

      BEFORE UPGRADE (or just revert the commit before testing since there is no upgrade step):

      1. Create several courses with grade categories in them
      2. View the grader report, collapse/aggregate some of categories. Remember the state (for example leave it open and continue in other browser tabs)
      3. UPGRADE CODE
      4. Open the grader report in the same courses and make sure it looks the same.
      5. Try collapsing categories, refresh page, make sure it remembers settings.
      Show
      BEFORE UPGRADE (or just revert the commit before testing since there is no upgrade step): Create several courses with grade categories in them View the grader report, collapse/aggregate some of categories. Remember the state (for example leave it open and continue in other browser tabs) UPGRADE CODE Open the grader report in the same courses and make sure it looks the same. Try collapsing categories, refresh page, make sure it remembers settings.
    • 20
    • BACKEND Sprint 15

    Description

      See MDL-20245. When categories are collapsed on the grader report it appears than an array is serialized and stored in the user_preferences table. This table is really for simple name value pairs.

      It would be good if we could avoid storing serialized data and instead only stored human readable data.

      Attachments

        Issue Links

          Activity

            People

              marina Marina Glancy
              andyjdavis Andrew Davis
              Damyon Wiese Damyon Wiese
              Dan Poltawski Dan Poltawski
              Andrew Davis Andrew Davis
              Adrian Greeve, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie
              Votes:
              14 Vote for this issue
              Watchers:
              24 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                8/Sep/14