Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-13192 stats cleanup - meta
  3. MDL-13313

Stats graphs break due to relationship to roleid

    XMLWordPrintable

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9
    • Fix Version/s: 1.9
    • Component/s: Administration
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE

      Description

      In our environement, roleids aren't just 1, 2, 3,4 etc and this breaks most of the pretty graphs:

      Notice: Undefined offset: 41 in /var/www/moodle/course/report/stats/graph.php on line 122
      Notice: Undefined index: in /var/www/moodle/lib/graphlib.php on line 447
      Notice: Undefined index: in /var/www/moodle/lib/graphlib.php on line 1677

      This is because the colours trying to set the roles as colours on the graph (which only go up to 30). We should remove the colouring of role by roleid, and instead just use some sort of incremental counter. (And accept we can't show more than 30 roles at once).

        Attachments

          Activity

            People

            Assignee:
            skodak Petr Skoda
            Reporter:
            poltawski Dan Poltawski
            Participants:
            Component watchers:
            Andrew Nicols, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              3/Mar/08