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
    • Rank:
      27099

      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.

        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: