Moodle
  1. Moodle
  2. MDL-21509

create backup and restore for 'timecreated' field in user table

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Backup
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Rank:
      26558

      Description

      Recently we are adding a new field called 'timecreated' to user table. (MDL-21226).

      As suggested by Eloy, the additional field needs to be also handled by backup and restore.

      It should achieve:
      1) On backup, save the timecreated field to XML
      2) On restore, load the timecreated field from XML
      3) On restore, if new user is being created, set it to time()

        Issue Links

          Activity

          Hide
          Rossiani Wijaya added a comment -

          Eloy,

          Could you take a look the patch for backup and restore. I'm not entirely sure this is the correct way to do it.
          Please let me know if i missed any steps.

          Thanks

          Show
          Rossiani Wijaya added a comment - Eloy, Could you take a look the patch for backup and restore. I'm not entirely sure this is the correct way to do it. Please let me know if i missed any steps. Thanks
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Looks perfect Rosie, +1

          (note that the whole backup/restore is changing soon, but better to have the "old" (current) one, having all the information so we won't miss anything in the conversion).

          Cia-o

          Show
          Eloy Lafuente (stronk7) added a comment - Looks perfect Rosie, +1 (note that the whole backup/restore is changing soon, but better to have the "old" (current) one, having all the information so we won't miss anything in the conversion). Cia-o
          Hide
          Rossiani Wijaya added a comment -

          Thanks Eloy.

          commited to head.

          Show
          Rossiani Wijaya added a comment - Thanks Eloy. commited to head.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: