Moodle
  1. Moodle
  2. MDL-33527

Unable to export database entries

    Details

    • Testing Instructions:
      Hide

      This test requires the following:

      • Portfolios and the file download portfolio plugin to be enabled for the site
      • A database activity containing entries from at least two students
      1. Login as teacher, edit the single template for the database activity and add an ##export## tag.
      2. Login as a student, access the database activity and try exporting one of your entries, selecting file download as the portfolio.

      Expected result:

      The student can export the file.

      Show
      This test requires the following: Portfolios and the file download portfolio plugin to be enabled for the site A database activity containing entries from at least two students Login as teacher, edit the single template for the database activity and add an ##export## tag. Login as a student, access the database activity and try exporting one of your entries, selecting file download as the portfolio. Expected result: The student can export the file.
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-MDL-33527-master
    • Rank:
      41449

      Description

      When a student tries to export a database entry, the Moodle serves a blank screen.

      The original error reported by Bas Brands in MDLQA-1972 was: Fatal error: Class 'file_info' not found in /var/www/latest.sunstreammedia.com/moodle/mod/data/locallib.php on line 410

      To Recreate:
      This test requires the following:

      • Portfolios and the file download portfolio plugin to be enabled for the site
      • A database activity containing entries from at least two students

      1. Login as teacher, edit the single template for the database activity and add an ##export## tag.
      2. Login as a student, access the database activity and try exporting one of your entries, selecting file download as the portfolio.

      Expected result:

      • The student can export the file.

      Actual result:

      • Moodle serves a blank page.

        Issue Links

          Activity

          Hide
          Jason Fowler added a comment - - edited

          the actual error is:

          Fatal error: Class 'file_info' not found in /var/www/repos/imaster/moodle/mod/data/locallib.php on line 410

          it is only shown if your server is set to display errors

          Show
          Jason Fowler added a comment - - edited the actual error is: Fatal error: Class 'file_info' not found in /var/www/repos/imaster/moodle/mod/data/locallib.php on line 410 it is only shown if your server is set to display errors
          Hide
          Aparup Banerjee added a comment -

          Thanks , thats integrated into master.

          Show
          Aparup Banerjee added a comment - Thanks , thats integrated into master.
          Hide
          Andrew Davis added a comment -

          Raised MDL-33552 which I ran into while testing.

          Show
          Andrew Davis added a comment - Raised MDL-33552 which I ran into while testing.
          Hide
          Andrew Davis added a comment -
          Show
          Andrew Davis added a comment - and MDL-33553
          Hide
          Andrew Davis added a comment -

          Passing.

          Show
          Andrew Davis added a comment - Passing.
          Hide
          Dan Poltawski added a comment -

          Hi,

          While looking at MDL-33594 I noticed this change. I don't think its quite correct. I think we should be including filelib.php instead of this internal part of filelib.

          Show
          Dan Poltawski added a comment - Hi, While looking at MDL-33594 I noticed this change. I don't think its quite correct. I think we should be including filelib.php instead of this internal part of filelib.
          Hide
          Jason Fowler added a comment -

          I have made the change as part of the solution for MDL-33594, once that is integrated, this should be fine

          Show
          Jason Fowler added a comment - I have made the change as part of the solution for MDL-33594 , once that is integrated, this should be fine
          Hide
          Eloy Lafuente (stronk7) added a comment -

          We could celebrate it today... but better if we perform a bigger party after releasing Moodle 2.3.

          Print this message and come to Perth that day, it's valid for one beer, wine, coke or... water, as you wish.

          Many thanks for your collaboration!

          Ciao

          Show
          Eloy Lafuente (stronk7) added a comment - We could celebrate it today... but better if we perform a bigger party after releasing Moodle 2.3. Print this message and come to Perth that day, it's valid for one beer, wine, coke or... water, as you wish. Many thanks for your collaboration! Ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: