Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-1427

SCORM: Error downloading SCORM if package is updated and course contents view is cached

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.9
    • Fix Version/s: 2.9
    • Component/s: SCORM add-on
    • Labels:
    • Testing Instructions:
      Hide
      1. In the app, go to a course and download a SCORM package. Don't leave the course contents view.
      2. In Moodle, update the package and use a ZIP with a different name.
      3. In the app, enter the SCORM, do a Pull To Refresh and go back to course contents view. Check that you see the refresh icon.
      4. Click the refresh icon. Check that the file is downloaded.
      Show
      In the app, go to a course and download a SCORM package. Don't leave the course contents view. In Moodle, update the package and use a ZIP with a different name. In the app, enter the SCORM, do a Pull To Refresh and go back to course contents view. Check that you see the refresh icon. Click the refresh icon. Check that the file is downloaded.
    • Affected Branches:
      MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_29_STABLE

      Description

      Steps to reproduce:

      1. In the app, go to a course and download a SCORM package. Don't leave the course contents view.
      2. In Moodle, update the package and use a ZIP with a different name.
      3. In the app, enter the SCORM, do a Pull To Refresh and go back to course contents view. Check that you see the refresh icon.
      4. Click the refresh icon. Check that an error is shown.

      This is because the SCORM course content handler calls getScorm when it's loaded, not when the download button is clicked. This causes that the URL used when the refresh button is clicked is the old one, so the download fails because the package isn't found. If you leave the view and load it again everything works fine.

        Attachments

          Activity

            People

            Assignee:
            dpalou Dani Palou
            Reporter:
            dpalou Dani Palou
            Peer reviewer:
            Juan Leyva
            Integrator:
            Juan Leyva
            Participants:
            Component watchers:
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              29/Feb/16