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

LTI activities' custom icons ("Icon URL" and "Secure icon URL") are not used in the grade book

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Create an LTI activity in a course with a custom icon
      2. Create some other graded activities in the course
      3. Open gradebook and make sure that grader report, user report, singleview report and grade setup page all show correct icons
      Show
      Create an LTI activity in a course with a custom icon Create some other graded activities in the course Open gradebook and make sure that grader report, user report, singleview report and grade setup page all show correct icons
    • Affected Branches:
      MOODLE_29_STABLE, MOODLE_30_STABLE
    • Fixed Branches:
      MOODLE_30_STABLE, MOODLE_31_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-54065-master
    • Pull Master Diff URL:

      Description

      When an LTI activity is created and the user provides an Icon URL/Secure icon URL those icons are used when displaying the link to the activity in a course. However, in the gradebook the "generic" LTI icon (i.e. mod/lti/pix/icon.*) is used.

      One possible solution would be to change grade_structure::get_element_icon to call an optionally implement activity hook to retrieve the icon and then implement that hook in mod/lti. I'm open to other suggestions as well.

      If there is any interest in this, I'd be willing provide a patch.

        Attachments

          Activity

            People

            Assignee:
            marina Marina Glancy
            Reporter:
            sbc24 Sam Chaffee
            Peer reviewer:
            Simey Lameze
            Integrator:
            David Monllaó
            Tester:
            Damyon Wiese
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              11/Jul/16