Moodle
  1. Moodle
  2. MDL-21255 Mnet overhaul for Moodle 2.0
  3. MDL-23600

Warning: Invalid argument supplied for foreach() in /home1/jesuits1/public_html/moodle/mnet/lib.php on line 638

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: MNet
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Rank:
      33633

      Description

      Penny,

      I received the following error on one of my sites that I was experimenting updating to 2.0. The errors seems to be caused because we are assuming that a userid of 1 will exist. I am not a fan of having a hard coded id listed in the code. Might it be better to use something like $USER->id instead? Although looking at the code, I'm not sure it is worth adding in the global $USER to the function. Perhaps there might be a better way of grabbing a user such as deleted=0.

      Peace - Anthony

        Activity

        Hide
        Anthony Borrow added a comment -

        Here's a screen shot from my site which did not have a user id of 1

        Show
        Anthony Borrow added a comment - Here's a screen shot from my site which did not have a user id of 1
        Hide
        Anthony Borrow added a comment -

        Re-assigning to Penny since she committed the code, feel free to re-assign as desired. Peace - Anthony

        Show
        Anthony Borrow added a comment - Re-assigning to Penny since she committed the code, feel free to re-assign as desired. Peace - Anthony
        Hide
        David Mudrak added a comment -

        Thanks for reporting this Anthony. I just committed a fix for this so it just requires that there is at least some record in the user table. Can you please re-test the situation?

        Show
        David Mudrak added a comment - Thanks for reporting this Anthony. I just committed a fix for this so it just requires that there is at least some record in the user table. Can you please re-test the situation?
        Hide
        Anthony Borrow added a comment -

        David - I saw the commit and it looks good. I tested it on the site that was causing the problem and it looks good. I will go ahead and close. Peace - Anthony

        Show
        Anthony Borrow added a comment - David - I saw the commit and it looks good. I tested it on the site that was causing the problem and it looks good. I will go ahead and close. Peace - Anthony
        Hide
        Anthony Borrow added a comment -

        Tested committed fix and it resolved the issue. Peace - Anthony

        Show
        Anthony Borrow added a comment - Tested committed fix and it resolved the issue. Peace - Anthony

          People

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

            Dates

            • Created:
              Updated:
              Resolved: