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

          Attachments

            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