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

enrol_plugin::update_user_enrol does not update timemodified upon execution

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.0.6, 3.1.5, 3.2.2
    • 3.1.6, 3.2.3
    • Enrolments
    • Any
    • MOODLE_30_STABLE, MOODLE_31_STABLE, MOODLE_32_STABLE
    • MOODLE_31_STABLE, MOODLE_32_STABLE
    • MDL-58079_master
    • Easy
    • Hide
      • Create a new course
      • Create new user
      • Enrol user into course as student
      • Check the value of timemodified field in user_enrolments table
      • Wait few moments
      • Modify user enrollment and set it to disabled
      • Check the value of timemodified field in user_enrolments table

       

      Before the fix:

      timemodified field will remain unchanged

      After the fix

      timemodified field will be updated

      Show
      Create a new course Create new user Enrol user into course as student Check the value of timemodified field in user_enrolments table Wait few moments Modify user enrollment and set it to disabled Check the value of timemodified field in user_enrolments table   Before the fix: timemodified field will remain unchanged After the fix timemodified field will be updated

    Description

      Strange behavior I noticed.

      When modifying user enrollment in a course after save timemodified field remains unchanged in user_enrolments table.

      That does not feel right.

      Attachments

        Activity

          People

            darko.miletic Darko Miletic
            darko.miletic Darko Miletic
            Ankit Agarwal Ankit Agarwal
            David Monllaó David Monllaó
            Mathieu Petit-Clair Mathieu Petit-Clair
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
            Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              8/May/17