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

Portfolio download file page contains opened div

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Step to replicate:

      • Enable file portfolio (Site administration > Advanced features)
      • Enable File Download (Site administration > Plugins > Portfolios > Manage portfolios)
      • Create a course and assignment in it.
      • Submit a file
      • Try to export portfolio and download the file.
      • Page /portfolio/download/file.php?id=ID will be broken in some themes because of not closed <div id="redirect">
      Show
      Step to replicate: Enable file portfolio (Site administration > Advanced features) Enable File Download (Site administration > Plugins > Portfolios > Manage portfolios) Create a course and assignment in it. Submit a file Try to export portfolio and download the file. Page /portfolio/download/file.php?id=ID will be broken in some themes because of not closed <div id="redirect">
    • Affected Branches:
      MOODLE_32_STABLE, MOODLE_33_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE, MOODLE_33_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-58893-master-missing-div

      Description

      The div is not closed on this page. This breaks some themes.

        Attachments

          Activity

            People

            Assignee:
            dmitriim Dmitrii Metelkin
            Reporter:
            dmitriim Dmitrii Metelkin
            Peer reviewer:
            Dan Poltawski
            Integrator:
            Andrew Nicols
            Tester:
            Damyon Wiese
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              10/Jul/17