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

          Attachments

            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