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

Problem with hidden sections on weeks course format

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0.2
    • Fix Version/s: 2.0.4
    • Component/s: Course
    • Environment:
      Linux; PHP 5.2;

      Description

      When some sections are hidden, moodle totally confuses the weeknumbers and when hidding some sections, some labels get scrambled.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              robin Robin de Vries added a comment - - edited

              The situation is worse than I though, see new screenshot.

              Excuse me: I've uploaded one picture double, can somebody delete it for me?

              Show
              robin Robin de Vries added a comment - - edited The situation is worse than I though, see new screenshot. Excuse me: I've uploaded one picture double, can somebody delete it for me?
              Hide
              nikhilgupta0901 Nikhil Gupta added a comment -

              Hi Robin,
              Yes you are right,it seems to be very confusing,if the section is hidden,then the weeknumbers are totally jumbled..i am trying to make a patch for it..

              thnx
              nikhil

              Show
              nikhilgupta0901 Nikhil Gupta added a comment - Hi Robin, Yes you are right,it seems to be very confusing,if the section is hidden,then the weeknumbers are totally jumbled..i am trying to make a patch for it.. thnx nikhil
              Hide
              robin Robin de Vries added a comment -

              I think there is a relationship with MDL-25852

              Show
              robin Robin de Vries added a comment - I think there is a relationship with MDL-25852
              Hide
              nikhilgupta0901 Nikhil Gupta added a comment -

              Hello,
              I have fixed the bug and prepared patch file for it,i am attaching it.

              Thnx
              Nikhil

              Show
              nikhilgupta0901 Nikhil Gupta added a comment - Hello, I have fixed the bug and prepared patch file for it,i am attaching it. Thnx Nikhil
              Hide
              robin Robin de Vries added a comment -

              Hi,

              Patch worked fine for me

              Thanks,
              Robin

              Show
              robin Robin de Vries added a comment - Hi, Patch worked fine for me Thanks, Robin
              Hide
              robin Robin de Vries added a comment -

              Can someone apply the patch on the current moodle versions?

              Show
              robin Robin de Vries added a comment - Can someone apply the patch on the current moodle versions?
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              The change seems correct, thanks!

              I've transformed the patch into (better) git branches, and I'm sending this for integration tomorrow. If everything goes well, it will be available in next weekly build next Thursday.

              Thanks Nikhil Gupta and Robin de Vries!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - The change seems correct, thanks! I've transformed the patch into (better) git branches, and I'm sending this for integration tomorrow. If everything goes well, it will be available in next weekly build next Thursday. Thanks Nikhil Gupta and Robin de Vries!
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Thanks guys this has been integrated now.

              I also tested during integration so this will pass testing immediately as well

              Cheers
              Sam

              Show
              samhemelryk Sam Hemelryk added a comment - Thanks guys this has been integrated now. I also tested during integration so this will pass testing immediately as well Cheers Sam
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Passed

              Show
              samhemelryk Sam Hemelryk added a comment - Passed
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Closing, because this has been sent upstream some minutes ago.

              Many thanks for your collaboration!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Closing, because this has been sent upstream some minutes ago. Many thanks for your collaboration!

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    1/Aug/11