Moodle
  1. Moodle
  2. MDL-37540

Assignment upgrade does not preserve setting "Hide description before available date"

        Details

        • Testing Instructions:
          Hide
          1. Create 2 advanced upload of files assignments
            1. A) Hide description before duedate = No
            2. B) Hide description before duedate = Yes
          2. Upgrade both assignments using the assignment upgrade tool
          3. Check the settings for the upgraded assignments
            1. A) Always show description = Yes
            2. B) Always show description = No
          Show
          Create 2 advanced upload of files assignments A) Hide description before duedate = No B) Hide description before duedate = Yes Upgrade both assignments using the assignment upgrade tool Check the settings for the upgraded assignments A) Always show description = Yes B) Always show description = No
        • Affected Branches:
          MOODLE_23_STABLE, MOODLE_24_STABLE
        • Fixed Branches:
          MOODLE_23_STABLE
        • Pull from Repository:
          git@github.com:damyon/moodle.git
        • Pull Master Branch:
          MDL-37540-master
        • Pull Master Diff URL:

          Description

          For advanced upload of files, this setting is not preserved during the upgrade.

            Gliffy Diagrams

              Issue Links

              is blocked by

              Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-37543 Assignment upgrade does not preserve setting "Enable send for marking button".

              • Minor - Minor loss of function where workaround is possible
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                Damyon Wiese added a comment -

                Setting blocker links to avoid conflicts.

                Show
                Damyon Wiese added a comment - Setting blocker links to avoid conflicts.
                Hide
                Permalink
                Dan Poltawski added a comment -

                The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

                TIA and ciao

                Show
                Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
                Hide
                Permalink
                Aparup Banerjee added a comment - - edited

                this looks good. waiting on MDL-37543 which blocks.. (and same question there can apply here)

                Show
                Aparup Banerjee added a comment - - edited this looks good. waiting on MDL-37543 which blocks.. (and same question there can apply here)
                Hide
                Permalink
                Aparup Banerjee added a comment -

                thanks, integrated into 23, 24 and master.

                Show
                Aparup Banerjee added a comment - thanks, integrated into 23, 24 and master.
                Hide
                Permalink
                Mark Nelson added a comment -

                Works as expected, passing.

                Show
                Mark Nelson added a comment - Works as expected, passing.
                Hide
                Permalink
                Eloy Lafuente (stronk7) added a comment -

                Surely you will be happy to know that your code is now part of Moodle upstream. Thanks, thanks!

                Closing as fixed, ciao

                Show
                Eloy Lafuente (stronk7) added a comment - Surely you will be happy to know that your code is now part of Moodle upstream. Thanks, thanks! Closing as fixed, ciao

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: