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

bugs in download as Microsoft spreadsheet of Statistics quiz report

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.0.3
    • 2.0.6
    • Quiz
    • MOODLE_20_STABLE
    • MOODLE_20_STABLE
    • Hide

      for bug 1 replace with return get_string($question->qtype,'qtype_' . $question->qtype);

      for bug2 remove the line

      sorry to not provide something on github to save your time

      Show
      for bug 1 replace with return get_string($question->qtype,'qtype_' . $question->qtype); for bug2 remove the line sorry to not provide something on github to save your time
    • Hide

      Try to download a statistics quiz report with third party questions types to see bug 1
      For bug 2 it doesn't "always" happend but I would say "frequently". don't exactly know what is allowed in sheet names in xls documents.

      Show
      Try to download a statistics quiz report with third party questions types to see bug 1 For bug 2 it doesn't "always" happend but I would say "frequently". don't exactly know what is allowed in sheet names in xls documents.

    Description

      2 small bugs in download download as Microsoft spreadsheet of Statistics quiz report
      file mod/quiz/report/statistics/statistics_table.php line 143
      return get_string($question->qtype, 'quiz');
      should be
      return get_string($question->qtype,'qtype_' . $question->qtype);
      file mod/quiz/report/statistics/report.php line 242
      $questiontabletitle = "<em>$questiontabletitle</em>";
      is not good for xls download as this is also used for sheet name in the worksheet and will result in broken .xls documents in most cases with Excel saying the document contain invalid sheet names and needs to be repared

      It seems all is corrected in 2.1

      Attachments

        Activity

          People

            timhunt Tim Hunt
            jmvedrine Jean-Michel Vedrine
            Sam Hemelryk Sam Hemelryk
            Ankit Agarwal Ankit Agarwal
            Tim Hunt, Andrew Lyons, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              28/Nov/11