Moodle
  1. Moodle
  2. MDL-29102

Auto-creating a group throws an error

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.2
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: Groups
    • Labels:
    • Testing Instructions:
      Hide

      1. Log in as admin
      2. Create "auto create group" (Settings -> Users -> Groups and click "Auto-create groups" )
      3. Enter required data and select "Create in grouping" as "New Group"
      4. Click Submit and you should not be able to see above mentioned error.

      Show
      1. Log in as admin 2. Create "auto create group" (Settings -> Users -> Groups and click "Auto-create groups" ) 3. Enter required data and select "Create in grouping" as "New Group" 4. Click Submit and you should not be able to see above mentioned error.
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w03_MDL-29102_m23_autogrouping
    • Rank:
      18626

      Description

      Creating a new "auto-create group" throw error

      Notice: Undefined property: stdClass::$groupingname in /usr/local/www/moodle/group/autogroup.php on line 175 Call Stack: 0.0035 779720 1. {main}() /usr/local/www/moodle/group/autogroup.php:0 

        Activity

        Hide
        Michael de Raadt added a comment -

        You seem to be finding a lot of bugs today. I think we might have to send you home

        Show
        Michael de Raadt added a comment - You seem to be finding a lot of bugs today. I think we might have to send you home
        Hide
        Henning Bostelmann added a comment -

        I wasn't able to reproduce the problem on the latest master today - the problem may have been fixed meanwhile.

        Show
        Henning Bostelmann added a comment - I wasn't able to reproduce the problem on the latest master today - the problem may have been fixed meanwhile.
        Hide
        Petr Škoda added a comment -

        thansk for the report.

        Show
        Petr Škoda added a comment - thansk for the report.
        Hide
        Sam Hemelryk added a comment -

        Collisions are fun...

        Show
        Sam Hemelryk added a comment - Collisions are fun...
        Hide
        Jason Fowler added a comment -

        Bam! and the error is gone!

        Show
        Jason Fowler added a comment - Bam! and the error is gone!
        Hide
        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
        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: