Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-21432 META: Backup & restore 2.0
  3. MDL-22164

post: clean old backup/restore code

          Details

          • Type: Sub-task
          • Status: Open
          • Priority: Minor
          • Resolution: Unresolved
          • Affects Version/s: 2.0
          • Fix Version/s: STABLE backlog
          • Component/s: Backup
          • Labels:
          • Difficulty:
            Easy
          • Affected Branches:
            MOODLE_20_STABLE

            Description

            Once we have all the logic migrated to new architecture, we'll delete all the old backup/restore 1.x code (core, activities and blocks).

              Gliffy Diagrams

                Attachments

                  Issue Links

                  is blocked by

                  Sub-task - Issues are sometimes broken into multiple sub-tasks. MDL-22153 post: clean old backup/restore files once they aren't necessary anymore

                  • Minor - Minor loss of function where workaround is possible
                  • Closed

                    Activity

                    Ascending order - Click to sort in descending order
                    Hide
                    Permalink
                    mudrd8mz David Mudrák added a comment -

                    There is

                    {modulename}

                    _get_participants() in all activity modules. It seems that only legacy backup/backuplib.php uses this function (I have not searched for contrib, though). We shall consider if this function is still part of standard module API and eventually remove it from NEWMODULE template or mark it as @deprecated

                    Show
                    mudrd8mz David Mudrák added a comment - There is {modulename} _get_participants() in all activity modules. It seems that only legacy backup/backuplib.php uses this function (I have not searched for contrib, though). We shall consider if this function is still part of standard module API and eventually remove it from NEWMODULE template or mark it as @deprecated
                    Hide
                    Permalink
                    stronk7 Eloy Lafuente (stronk7) added a comment -

                    While proceeding with the deletion of old files (MDL-22153), these potential candidates have been also found:

                    • course format: xxxx_backup_format_data() and xxxx_restore_format_data()
                    • activity: xxxx_refresh_events()

                    They need verification (together with xxxx_get_participants()) and be cleaned/deprecated.

                    Ciao

                    Show
                    stronk7 Eloy Lafuente (stronk7) added a comment - While proceeding with the deletion of old files ( MDL-22153 ), these potential candidates have been also found: course format: xxxx_backup_format_data() and xxxx_restore_format_data() activity: xxxx_refresh_events() They need verification (together with xxxx_get_participants()) and be cleaned/deprecated. Ciao
                    Hide
                    Permalink
                    stronk7 Eloy Lafuente (stronk7) added a comment -

                    This issue was assigned to me automatically, however I will not be able to work on this issue in the immediate future. In order to create a truer sense of the state of this issue and to allow other developers to have chance to become involved, I am removing myself as the assignee of this issue.

                    For more information, see http://docs.moodle.org/dev/Changes_to_issue_assignment

                    Show
                    stronk7 Eloy Lafuente (stronk7) added a comment - This issue was assigned to me automatically, however I will not be able to work on this issue in the immediate future. In order to create a truer sense of the state of this issue and to allow other developers to have chance to become involved, I am removing myself as the assignee of this issue. For more information, see http://docs.moodle.org/dev/Changes_to_issue_assignment

                      People

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

                        Dates

                        • Created:
                          Updated: