Moodle
  1. Moodle
  2. MDL-36855

Undefined offset errors in Scorm interactions Report

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: None
    • Component/s: SCORM
    • Labels:
    • Rank:
      46381

      Description

      Notice the following errors while testing MDLQA-4683:

      Notice: Undefined offset: 23 in /integration/master/lib/tablelib.php on line 983
      
      Notice: Undefined index: in /integration/master/lib/tablelib.php on line 986
      
      Notice: Undefined index: in /integration/master/lib/tablelib.php on line 996
      
      Notice: Undefined index: in /integration/master/lib/tablelib.php on line 997
      

      Steps to replicate:

      1. Create a scorm activity with the attached scorm package.
      2. Select 'interactions report' from the reports tab.

      Notice the above error.

        Issue Links

          Activity

          Hide
          Dan Marsden added a comment -

          Thanks - adding Ankit here - this looks like it's coming from your interactions report - do you want to take a look - if not leave it here and I'll get to it at some point.

          Show
          Dan Marsden added a comment - Thanks - adding Ankit here - this looks like it's coming from your interactions report - do you want to take a look - if not leave it here and I'll get to it at some point.
          Hide
          Ankit Agarwal added a comment -

          Thanks Dan,
          Since this not a QA issue, I will look into it during the next sprint.
          Thanks

          Rosie,
          Can you please attach the scorm pack you used and let me know if you did anything besides making normal attempts. Since am not able to replicate this.

          Thanks

          Show
          Ankit Agarwal added a comment - Thanks Dan, Since this not a QA issue, I will look into it during the next sprint. Thanks Rosie, Can you please attach the scorm pack you used and let me know if you did anything besides making normal attempts. Since am not able to replicate this. Thanks
          Hide
          Rossiani Wijaya added a comment -

          Hi Ankit,

          The changes look great.

          [y] Syntax
          [y] Output
          [y] Whitespace
          [-] Language
          [-] Databases
          [y] Testing (master only)
          [-] Security
          [-] Documentation
          [y] Git
          [y] Sanity check

          +1 for integration.

          Show
          Rossiani Wijaya added a comment - Hi Ankit, The changes look great. [y] Syntax [y] Output [y] Whitespace [-] Language [-] Databases [y] Testing (master only) [-] Security [-] Documentation [y] Git [y] Sanity check +1 for integration.
          Hide
          Ankit Agarwal added a comment -

          Thanks for the review Rosie,
          Sending for integration
          Thanks

          Show
          Ankit Agarwal added a comment - Thanks for the review Rosie, Sending for integration Thanks
          Hide
          Sam Hemelryk added a comment -

          Thanks Ankit this has been integrated now.

          Show
          Sam Hemelryk added a comment - Thanks Ankit this has been integrated now.
          Hide
          David Monllaó added a comment -

          It passes, tested in 23 and master with the provided SCORM package

          Show
          David Monllaó added a comment - It passes, tested in 23 and master with the provided SCORM package
          Hide
          Eloy Lafuente (stronk7) added a comment -

          And your fantastic code has met core, hope they become good friends for a long period.

          Closing, thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - And your fantastic code has met core, hope they become good friends for a long period. Closing, thanks!

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: