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

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

              Dates

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