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

Problems displaying multiple folder resources inline in the course page

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Resource
    • Labels:
    • Testing Instructions:
      Hide

      Create Folder resources A & B with sub folders with content.

      Case 1:

      • Set both folders settings:
        • "Display foder contents: Inline on a course page"
        • "Show sub-folders expanded: checked"
      • Go to course page and verify that both folder resources display with all their sub-folders expanded

      Case 2:

      • Set folder A settings:
        • "Display foder contents: Inline on a course page"
        • "Show sub-folders expanded: checked"
      • Set folder B settings:
        • "Display foder contents: Inline on a course page"
        • "Show sub-folders expanded: unchecked"
      • Go to course page and verify that folder A resources display with all their sub-folders expanded. Verify that folder B resources display with their sub-folders collapsed.

      Case 3:

      • Set both folders settings:
        • "Display foder contents: Inline on a course page"
        • "Show sub-folders expanded: unchecked"
      • Go to course page and verify that both folder resources display with their sub-folders collapsed.
      Show
      Create Folder resources A & B with sub folders with content. Case 1: Set both folders settings: "Display foder contents: Inline on a course page" "Show sub-folders expanded: checked" Go to course page and verify that both folder resources display with all their sub-folders expanded Case 2: Set folder A settings: "Display foder contents: Inline on a course page" "Show sub-folders expanded: checked" Set folder B settings: "Display foder contents: Inline on a course page" "Show sub-folders expanded: unchecked" Go to course page and verify that folder A resources display with all their sub-folders expanded. Verify that folder B resources display with their sub-folders collapsed. Case 3: Set both folders settings: "Display foder contents: Inline on a course page" "Show sub-folders expanded: unchecked" Go to course page and verify that both folder resources display with their sub-folders collapsed.
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      Hi,

      while playing with the folder resource, I've detected that, if multiple ones are configured to be shown inline... only the first one seems to work, and the rest, are somehow borked (no tree lines, expand/collapse not working...)

      See the attached screenshot.

      Ciao

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            marina Marina Glancy added a comment -

            Hi Eloy, I tested it when I working on MDL-37455 and tried again now - everything works for me. What theme and course format are you using?

            Show
            marina Marina Glancy added a comment - Hi Eloy, I tested it when I working on MDL-37455 and tried again now - everything works for me. What theme and course format are you using?
            Hide
            marina Marina Glancy added a comment -

            ok, I hit it. It is broken on integration master, but ok on master. It seem to conflict with the change MDL-30790

            Show
            marina Marina Glancy added a comment - ok, I hit it. It is broken on integration master, but ok on master. It seem to conflict with the change MDL-30790
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Aha, I was not really sure of which one, MDL-37455 or MDL-30790 was the culprit), thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Aha, I was not really sure of which one, MDL-37455 or MDL-30790 was the culprit), thanks!
            Hide
            marina Marina Glancy added a comment -

            Also it makes other JS on page not working, for example sections drag and drop.

            Show
            marina Marina Glancy added a comment - Also it makes other JS on page not working, for example sections drag and drop.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            yup, same here. confirmed coz i cannot D&D anymore in that course page.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - yup, same here. confirmed coz i cannot D&D anymore in that course page.
            Hide
            rex Rex Lorenzo added a comment -

            Patched problems with folder javascript.

            Show
            rex Rex Lorenzo added a comment - Patched problems with folder javascript.
            Hide
            marina Marina Glancy added a comment -

            Looks good to me. Thanks Rex

            Show
            marina Marina Glancy added a comment - Looks good to me. Thanks Rex
            Hide
            poltawski Dan Poltawski added a comment -

            Integrated to master - thanks Rex!

            Show
            poltawski Dan Poltawski added a comment - Integrated to master - thanks Rex!
            Hide
            phalacee Jason Fowler added a comment -

            All good Rex, thanks for this.

            Show
            phalacee Jason Fowler added a comment - All good Rex, thanks for this.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Your awesome contributions are now part of Moodle, your fav LMS out there.

            Closing this as fixed.

            Many thanks for all the hard work, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Your awesome contributions are now part of Moodle, your fav LMS out there. Closing this as fixed. Many thanks for all the hard work, ciao

              People

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

                Dates

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