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

Report log chart breaks if there's an invalid time

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: 3.2
    • Component/s: Reports
    • Testing Instructions:
      Hide
      1. Visit your profile page.
      2. Click All logs.
      3. Make sure the chart is generated and no Javascript exception were thrown.
      4. Go back to your profile page.
      5. Click Today's logs
      6. Make sure the chart is generated and no Javascript exception were thrown.
      7. Visit other user profile.
      8. Repeat steps above and make sure no exception were thrown.
      Show
      Visit your profile page. Click All logs . Make sure the chart is generated and no Javascript exception were thrown. Go back to your profile page. Click Today's logs Make sure the chart is generated and no Javascript exception were thrown. Visit other user profile. Repeat steps above and make sure no exception were thrown.
    • Affected Branches:
      MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-55367-master
    • Pull Master Diff URL:

      Description

      The report log is throwing an JS exception because the number of series doesn't match the number of labels. 

      require.js:903 Error: Series must match label values.

      Consequently, that breaks the charts and it is not displayed.

        Attachments

          Issue Links

          is a regression caused by

          Task - A task that needs to be completed. Tasks usually refer to work that must be done outside the product. MDL-54998 Convert report_log graphs to the new library

          • Minor - Minor loss of function where workaround is possible
          • Closed

            Activity

              People

              Assignee:
              lameze Simey Lameze
              Reporter:
              lameze Simey Lameze
              Peer reviewer:
              cameron1729
              Integrator:
              Dan Poltawski
              Tester:
              Marina Glancy
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                5/Dec/16