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

Enrolment data discarded during upgrade

    Details

    • Affected Branches:
      MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE

      Description

      When upgrading from 1.6.x, new role assignments are given default values rather than retaining those stored in user_students and user_teachers. This appears to be the only place Internal Enrolment keeps a user's enrolment date and method.

      Attached is a patch which retains enrolment dates and methods during upgrade.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            cbandy Chris Bandy added a comment -

            Updated patch against HEAD (accesslib.php 1.441)

            Show
            cbandy Chris Bandy added a comment - Updated patch against HEAD (accesslib.php 1.441)
            Hide
            cbandy Chris Bandy added a comment -

            Updating affected versions.

            Show
            cbandy Chris Bandy added a comment - Updating affected versions.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Assigning to Petr, for your consideration.

            Thanks for the report Chris (I've deleted the requested subtask).

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Assigning to Petr, for your consideration. Thanks for the report Chris (I've deleted the requested subtask). Ciao
            Hide
            skodak Petr Skoda added a comment -

            MD was working on this, right?

            Show
            skodak Petr Skoda added a comment - MD was working on this, right?
            Hide
            hatshepsut N Hansen added a comment -

            Wish I had had this yesterday when I upgraded and lost all this info!!!!

            Show
            hatshepsut N Hansen added a comment - Wish I had had this yesterday when I upgraded and lost all this info!!!!
            Hide
            hatshepsut N Hansen added a comment -

            May I ask why such a serious bug with a patch offered wasn't taken care of months ago?

            Show
            hatshepsut N Hansen added a comment - May I ask why such a serious bug with a patch offered wasn't taken care of months ago?
            Hide
            dougiamas Martin Dougiamas added a comment - - edited

            Somehow I never saw this one before(it was flagged as minor). Thanks, Chris, I'll patch everything today.

            Show
            dougiamas Martin Dougiamas added a comment - - edited Somehow I never saw this one before(it was flagged as minor). Thanks, Chris, I'll patch everything today.
            Hide
            dougiamas Martin Dougiamas added a comment -

            Patched in CVS

            Show
            dougiamas Martin Dougiamas added a comment - Patched in CVS
            Hide
            skodak Petr Skoda added a comment -

            reviewed, closing

            Show
            skodak Petr Skoda added a comment - reviewed, closing

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  15/May/08