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

unused variable table->head in scorm reports

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: SCORM
    • Labels:
    • Testing Instructions:
      Hide

      I can not really think of anything that this change can break. If you want to be extra sure just browse around SCORM reports and make sure there are no regressions

      Show
      I can not really think of anything that this change can break. If you want to be extra sure just browse around SCORM reports and make sure there are no regressions
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:
      MDL-30965-master

      Description

      $table->head is never used in scorm reports

      $table->head[]= format_string($sco->title,'',$formattextoptions);

      and in interactions

      $table->head[]= format_string($sco->title);

        Attachments

          Activity

            People

            Assignee:
            ankit_frenz Ankit Agarwal
            Reporter:
            ankit_frenz Ankit Agarwal
            Peer reviewer:
            Dan Marsden
            Integrator:
            Sam Hemelryk
            Tester:
            Adrian Greeve
            Participants:
            Component watchers:
            Damyon Wiese, Dan Marsden, Matteo Scaramuccia, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              12/Mar/12