Moodle

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

Details

  • Type: Bug Bug
  • Status: Closed 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

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

has been marked as being related by

Improvement - An enhancement to an existing Moodle feature. MDL-21226 Add a 'timecreated' column to the mdl_user table

  • Minor - Minor loss of function where workaround is possible
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
Permalink
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
Permalink
Rossiani Wijaya added a comment -

Thanks Eloy.

commited to head.

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

People

Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: