Moodle
  1. Moodle
  2. MDL-21446

Excel export does not work with >31 character worksheet names

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.9.7, 1.9.11, 2.0.2
    • Fix Version/s: 1.9.12, 2.0.3
    • Component/s: Libraries
    • Labels:
    • Affected Branches:
      MOODLE_19_STABLE, MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE, MOODLE_20_STABLE

      Description

      In latin 1 mode, the PEAR library throws an error when a sheet name is longer than 31 chars.

      In Unicode mode, the PEAR libarary thinks that longer sheet names will work, but we are getting errors when we try to open those files in Excel.

      I propose to truncate all worksheet names to 31 chars max in excellib.

      I am about to check this change into OU Moodle. please review and decide if you want it in 1.9 and or HEAD.

        Gliffy Diagrams

          Activity

          Hide
          Tim Hunt added a comment -

          Grrr! this has been just sitting here for a year. I'll put it into git, and see if that make anyone more inclined to comment.

          Show
          Tim Hunt added a comment - Grrr! this has been just sitting here for a year. I'll put it into git, and see if that make anyone more inclined to comment.
          Hide
          Tim Hunt added a comment -
          Show
          Tim Hunt added a comment - OK. https://github.com/timhunt/moodle/compare/master...MDL-21446 - for comment.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Doh, my fault Tim, apologizes

          I had this completely lost in all my lists. Thanks for pinging!

          I've left this open in my browser so will look to it tomorrow!

          Ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Doh, my fault Tim, apologizes I had this completely lost in all my lists. Thanks for pinging! I've left this open in my browser so will look to it tomorrow! Ciao
          Hide
          Eloy Lafuente (stronk7) added a comment -

          +1

          Assigning to Tim as commented @ HQ. Thanks!!!

          Show
          Eloy Lafuente (stronk7) added a comment - +1 Assigning to Tim as commented @ HQ. Thanks!!!
          Hide
          Tim Hunt added a comment -

          Pull requests done.

          One way to test this is to make a quiz with long question names. Then do several attempts, then go the the quiz statistics report, and choose "Download entire report as Excel". You should find you get a spreadsheet with worksheet names truncated to 31 chars.

          Show
          Tim Hunt added a comment - Pull requests done. One way to test this is to make a quiz with long question names. Then do several attempts, then go the the quiz statistics report, and choose "Download entire report as Excel". You should find you get a spreadsheet with worksheet names truncated to 31 chars.
          Hide
          Helen Foster added a comment -

          Thanks for fixing Tim.

          Show
          Helen Foster added a comment - Thanks for fixing Tim.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: