Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.1, 2.3
    • Fix Version/s: 2.2.2
    • Component/s: Roles / Access
    • Labels:
    • Environment:
      Moodle 2.2.1 (Build: 20120109) PHP Version 5.3.6, MySQL 5.1.57

      Description

      I am the administrator of my moodle. I go into my course, click on "switch role to student", and I get a code error (see attachment). I have "debug" turned on, which is why I am seeing this problem.

      I don't think it causes any errors or problems, but I wanted to let you know.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for reporting that, Rick.

            I was able to replicate the problem also. It doesn't seem to matter what role I switch to. It appears to happen only once after I log in. Switching roles a second time does not produce the error. I also see the same error on the course page immediately after I switch, but only until the page reloads. If I log out and log in again I am able to reproduce the error.

            I was not able to reproduce the error in 2.1, so this must be a recent regression. Yep, I found the issue: MDL-29602.

            Show
            salvetore Michael de Raadt added a comment - Thanks for reporting that, Rick. I was able to replicate the problem also. It doesn't seem to matter what role I switch to. It appears to happen only once after I log in. Switching roles a second time does not produce the error. I also see the same error on the course page immediately after I switch, but only until the page reloads. If I log out and log in again I am able to reproduce the error. I was not able to reproduce the error in 2.1, so this must be a recent regression. Yep, I found the issue: MDL-29602 .
            Hide
            skodak Petr Skoda added a comment - - edited

            Hello,

            I am not able to reproduce this problem, I have reviewed the code that initialises this array and did not find any problems. Did you install any contrib plugins or modify the code?

            Thanks for the report.

            Petr

            Show
            skodak Petr Skoda added a comment - - edited Hello, I am not able to reproduce this problem, I have reviewed the code that initialises this array and did not find any problems. Did you install any contrib plugins or modify the code? Thanks for the report. Petr
            Hide
            skodak Petr Skoda added a comment -

            oops, I have managed to replicate it now...

            Show
            skodak Petr Skoda added a comment - oops, I have managed to replicate it now...
            Hide
            skodak Petr Skoda added a comment -

            Thanks for the report!

            Show
            skodak Petr Skoda added a comment - Thanks for the report!
            Hide
            rjerz Rick Jerz added a comment -

            Petr,

            Thanks for noticing this problem. I feel good that I reported it. Sometimes I wonder if I should report these (minor) problems. But then I know that you folks really do care about fixing these issues. I am not trying to be bothersome. I really like Moodle and I like the support for it!

            Show
            rjerz Rick Jerz added a comment - Petr, Thanks for noticing this problem. I feel good that I reported it. Sometimes I wonder if I should report these (minor) problems. But then I know that you folks really do care about fixing these issues. I am not trying to be bothersome. I really like Moodle and I like the support for it!
            Hide
            skodak Petr Skoda added a comment -

            Rick, it is great when you report issues like this especially in the latest stable branch soon after release. Thanks again and keep reporting. Ciao

            Show
            skodak Petr Skoda added a comment - Rick, it is great when you report issues like this especially in the latest stable branch soon after release. Thanks again and keep reporting. Ciao
            Hide
            skodak Petr Skoda added a comment -

            ah, I just discovered your other open reported issue, I might find some time later today to fix at least some easy ones, reciao

            Show
            skodak Petr Skoda added a comment - ah, I just discovered your other open reported issue, I might find some time later today to fix at least some easy ones, reciao
            Hide
            rjerz Rick Jerz added a comment -

            I just added one more. See MDL-31127.

            Show
            rjerz Rick Jerz added a comment - I just added one more. See MDL-31127 .
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated (22 and master), thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (22 and master), thanks!
            Hide
            phalacee Jason Fowler added a comment -

            The notice is gone now. Nice work

            Show
            phalacee Jason Fowler added a comment - The notice is gone now. Nice work
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            This is now available in the git and cvs repositories.

            Consider the responsibility of your fingerprints engraved there for future generations!

            Thanks for the work, closing, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - This is now available in the git and cvs repositories. Consider the responsibility of your fingerprints engraved there for future generations! Thanks for the work, closing, ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  12/Mar/12