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

Moving a hidden item to visible section sets the item to visible in 2.3

    Details

    • Testing Instructions:
      Hide

      Run unit tests

      1. Hide an item using eye icon
      2. Move hidden item to a non-hidden section
      3. VERIFY: that the hidden item is not shown.
      4. Add two course modules to one section, set one hidden, one visible. Note which is hidden and which is not.
      5. Hide the entire section using the eye icon
      6. VERIFY: all course modules are hidden
      7. Unhide the section
      8. VERIFY the visibility of course modules is the same as step 3. With one visible and one not.
      Show
      Run unit tests Hide an item using eye icon Move hidden item to a non-hidden section VERIFY: that the hidden item is not shown. Add two course modules to one section, set one hidden, one visible. Note which is hidden and which is not. Hide the entire section using the eye icon VERIFY: all course modules are hidden Unhide the section VERIFY the visibility of course modules is the same as step 3. With one visible and one not.
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE

      Description

      In Moodle 2.3.6

      1. Hide an item
      2. Move hidden item to a non-hidden section

      Item moves to correct location, but is now shown!

      In Moodle 2.3.6, 2.4.3, and Master

      1. Create a new item, setting its visibility to hidden.
      2. Save and return to course.
      3. Move item to a non-hidden section

      Item moves to correct location, but is now shown!

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Votes:
                  10 Vote for this issue
                  Watchers:
                  7 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    13/May/13