Moodle
  1. Moodle
  2. MDL-23681

IMS CC with no title won't import

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.9
    • Fix Version/s: 1.9.10
    • Labels:
      None
    • Environment:
      Angel exports it that way...
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE

      Description

      I have taken an Angel IMS CC export and corrected the errors within it:
      Change all \ to // in imsmanifest
      Remove _about.html pages (or one could add them to the manifest)

      This cartridge now validates with IMS CC Desktop tester 'cctesterV1p0p8'

      Moodle refuses to import it, because it has no course title.

      A sample cartridge of a silly course is attached.

      IMHO, Moodle should give the course an "Untitled" title, or dig deeper into IMS specs to pull out the course title from wherever it is hiding.

        Gliffy Diagrams

          Activity

          Hide
          Eloy Lafuente (stronk7) added a comment -

          Assigning to Darko...

          Show
          Eloy Lafuente (stronk7) added a comment - Assigning to Darko...
          Hide
          Richard Lynch added a comment -

          My buddy grep and I don't think Angel even bothers to dump out the course title anywhere, at least not in plain-text we can find.
          Lovely.

          Show
          Richard Lynch added a comment - My buddy grep and I don't think Angel even bothers to dump out the course title anywhere, at least not in plain-text we can find. Lovely.
          Hide
          Richard Lynch added a comment -

          This one-liner patch appears to work fine so far...

          Show
          Richard Lynch added a comment - This one-liner patch appears to work fine so far...
          Hide
          Darko Miletic added a comment -

          That patch is not valid.
          Here is the proper patch

          Also you should not replace \ with // . It should be / .

          Show
          Darko Miletic added a comment - That patch is not valid. Here is the proper patch Also you should not replace \ with // . It should be / .
          Hide
          Darko Miletic added a comment -

          Valid patch for this issue

          Show
          Darko Miletic added a comment - Valid patch for this issue
          Hide
          Darko Miletic added a comment -

          ccpatch.patch contains valid fix for this issue. Your turn now Eloy.

          Show
          Darko Miletic added a comment - ccpatch.patch contains valid fix for this issue. Your turn now Eloy.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: