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

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

          Details

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

              Gliffy Diagrams

                Attachments

                  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

                    Activity

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

                    Thanks Eloy.

                    commited to head.

                    Show
                    rwijaya 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:
                          Fix Release Date:
                          24/Nov/10