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

bugs in download as Microsoft spreadsheet of Statistics quiz report

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.3
    • Fix Version/s: 2.0.6
    • Component/s: Quiz
    • Labels:
    • Testing Instructions:
      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.
    • Workaround:
      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
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Pull from Repository:

      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

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            timhunt Tim Hunt added a comment -

            Thank you for the detailed bug report.

            Show
            timhunt Tim Hunt added a comment - Thank you for the detailed bug report.
            Hide
            timhunt Tim Hunt added a comment -

            I had forgotten about this, but just found it again, and your suggestions are correct, so I'm submitting the fix for integration now. Thanks Jean-Michel.

            Show
            timhunt Tim Hunt added a comment - I had forgotten about this, but just found it again, and your suggestions are correct, so I'm submitting the fix for integration now. Thanks Jean-Michel.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Tim - this has been integrated now

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Tim - this has been integrated now
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            test passed!
            Thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - test passed! Thanks
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Done, your delicious hacks have been sent upstream, many thanks!

            Closing as fixed, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Done, your delicious hacks have been sent upstream, many thanks! Closing as fixed, ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  28/Nov/11