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

Added "Continue" button under Error notification message (IMS CP)

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Add a new IMS-CP resource to a course, using the "gitta_ims_bad-manifest.zip" attached package.
      2. Click to view the package.
      3. When error page is displayed, click the "Continue" button and make sure you are redirected to the course main page.
      Show
      Add a new IMS-CP resource to a course, using the "gitta_ims_bad-manifest.zip" attached package. Click to view the package. When error page is displayed, click the "Continue" button and make sure you are redirected to the course main page.
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_29_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-32285_master

      Description

      After uploading a problematic (bad or no manifest.xml file) IMS CP file (package)
      I am getting an error message with no continue button.
      It seems like a minor usability issue.

      Adding a "Continue" button with a link to the course's main page could be very helpful to novice teachers.
      Attached, is a proposed fix.

      EDIT:
      For testing purposes, I am attaching two IMS CP packages, One with bad (missing) manifest file and one which is perfectly OK.

        Attachments

        1. gitta_ims_bad-manifest.zip
          2.18 MB
        2. gitta_ims.zip
          2.18 MB
        3. imscp-error.png
          imscp-error.png
          29 kB

          Activity

            People

            Assignee:
            nadavkav Nadav Kavalerchik
            Reporter:
            nadavkav Nadav Kavalerchik
            Peer reviewer:
            Marina Glancy
            Integrator:
            Eloy Lafuente (stronk7)
            Tester:
            Mark Nelson
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              11/May/15