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

      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 

        Gliffy Diagrams

          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 Skoda added a comment -

          thansk for the report.

          Show
          Petr Skoda 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: